Hello build bot (Jenkins), David Hendricks, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/41399
to look at the new patch set (#2).
Change subject: spi: Allow cached ID bytes to be cleared ......................................................................
spi: Allow cached ID bytes to be cleared
This does two things: 1. Removes the static variables from various ID functions and moves them to a single struct. 2. Adds a function to clear the struct.
The idea of the original caching mechanism introduced years ago was simply to speed up probe_flash() by not issuing the same read ID commands dozens or hundreds of times for a single chip.
That implementation assumed that we would only call probe_flash() once. However, there are cases when we want to call probe_flash() multiple times, for example, if using an external programmer and using different voltages (for probing the chip).
This patch is extremely similar to the original works of: `commit 57b7524b1448189d3630d6c4735e60dbbdf14d51`. Author: David Hendricks dhendrix@chromium.org && `commit 7f7c711ee1a41649607f81f1533e5135fc1361fc`. Author: David Hendricks dhendrix@chromium.org
BUG=b:15656443 BRANCH=none TEST=none
Change-Id: I879cb08dbe66db9ab0c3b8a7f93b04fe1c5980f4 Signed-off-by: Edward O'Callaghan quasisec@google.com --- M spi.h M spi25.c 2 files changed, 51 insertions(+), 28 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/99/41399/2