diff options
| author | Raul E Rangel <rrangel@chromium.org> | 2021-03-30 14:00:24 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2021-03-31 21:14:13 +0000 |
| commit | 6ef0e80bf42909bf0a6609041953b8b23f68e7f3 (patch) | |
| tree | dd411c69770bcee7058b3e389456c0b9b85553dd | |
| parent | a7659ebe4e1101a7a2cbe1f156943e83e9f59249 (diff) | |
| download | coreboot-6ef0e80bf42909bf0a6609041953b8b23f68e7f3.tar.xz | |
mb/google/guybrush: Switch eSPI to 16 MHz
It looks like we are having SI issues on eSPI at 33 MHz. Switching to 16
MHz makes everything a lot more stable.
BUG=b:183524609
TEST=Boot to OS and run `ectool version` 1000 times and see no problems.
Before with 33 MHz there was an error every few cycles.
declare -i i=0; while ectool version; do i+=1; echo "$i"; sleep .11; done; echo "Finished: $i"
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: I6ab515629703a157c1d1ac6adcf5cf379e80f8ee
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51953
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-by: Mathew King <mathewk@chromium.org>
Reviewed-by: Rob Barnes <robbarnes@google.com>
| -rw-r--r-- | src/mainboard/google/guybrush/variants/baseboard/devicetree.cb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb b/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb index 4d57b52810..b90ea0bafc 100644 --- a/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb @@ -27,7 +27,7 @@ chip soc/amd/cezanne }, .io_mode = ESPI_IO_MODE_QUAD, - .op_freq_mhz = ESPI_OP_FREQ_33_MHZ, + .op_freq_mhz = ESPI_OP_FREQ_16_MHZ, .crc_check_enable = 1, .dedicated_alert_pin = 1, .periph_ch_en = 1, |