Edward O'Callaghan has uploaded this change for review.

View Change

raiden_debug_spi.c:  Add a delay following AP/EC flash enable

Add a delay following the AP and EC flash enable requests. This allows
any power rails enabled by these signals to settle and to meet the power
on to first SPI write timing requirements.

Forward ports the downstream commit:
https://chromium-review.googlesource.com/c/chromiumos/third_party/flashrom/+/2036738

Change-Id: I4c1777777ee67580605c6e6f4c0c228cccc392c7
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
---
M raiden_debug_spi.c
1 file changed, 9 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/12/39312/1
diff --git a/raiden_debug_spi.c b/raiden_debug_spi.c
index fa42268..6914455 100644
--- a/raiden_debug_spi.c
+++ b/raiden_debug_spi.c
@@ -562,6 +562,15 @@
return ret;
}

+ /*
+ * Allow for power to settle on the AP and EC flash devices.
+ * Load switches can have a 1-3 ms turn on time, and SPI flash devices
+ * can require up to 10 ms from power on to the first write.
+ */
+ if ((request_enable == RAIDEN_DEBUG_SPI_REQ_ENABLE_AP) ||
+ (request_enable == RAIDEN_DEBUG_SPI_REQ_ENABLE_EC))
+ usleep(50 * 1000);
+
register_spi_master(&spi_master_raiden_debug);
register_shutdown(shutdown, NULL);


To view, visit change 39312. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I4c1777777ee67580605c6e6f4c0c228cccc392c7
Gerrit-Change-Number: 39312
Gerrit-PatchSet: 1
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-MessageType: newchange