Hello Daniel Kurtz, build bot (Jenkins), Nico Huber, Martin Roth, Kangheui Won, Angel Pons, Chris McDonald, Kangheui Won,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/44073
to look at the new patch set (#10).
Change subject: sb600spi.c: Add support for 0x790b rev 0x61 (AMD Zen) ......................................................................
sb600spi.c: Add support for 0x790b rev 0x61 (AMD Zen)
Adds support for rev 0x59 || 0x61 of did 0x790b.
This is quite confusing however it turns out FCH chipsets called 'Promontory' contain the so-called SPI100 ip core that uses memory mapping and not a ring buffer for transactions. Typically this is found on both Stoney Ridge and Zen platforms. In light of this, separate out the promontory path into its own callback struct state tracker so that it's implementation does not interfere with previous generations that predate the SPI100 controller.
Since there is some life-time state required to track the mapping during between the first attempted read and the final tear-down of the spi master we take the opportunity to avoid static locals and instead implement the functionality in a re-entrant way for follow up clean ups.
BUG=none BRANCH=none TEST= Zork => 'Promontory (rev 0x61) detected.' && Grunt => 'Promontory (rev 0x4b) detected.'
Change-Id: I5ce63b5de863aed0442cb4ffeff981e9b2fa445b Signed-off-by: Edward O'Callaghan quasisec@google.com --- M sb600spi.c 1 file changed, 64 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/73/44073/10