Attention is currently required from: Felix Singer, Nico Huber, Edward O'Callaghan, Anastasia Klimchuk, Nikolai Artemiev. Hello build bot (Jenkins), Nico Huber, Edward O'Callaghan, Angel Pons, Nikolai Artemiev,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/58735
to look at the new patch set (#2).
Change subject: ichspi: Split very long init function into two ......................................................................
ichspi: Split very long init function into two
ich_init_spi is very long, but logically it can be split. Init function detects the chipset and then the rest of operations depends on the chipset.
Init function is more readable now, it consists of only a switch.
Initialisation of hwseq and swseq that used to happen in the beginning of init function now moved to init_ich_default, because hwseq and swseq are only used for chipsets served by init_ich_default.
TEST ME ON DEVICE
BUG=b:204488958
Change-Id: I6789bc456a4878e6555831ae0b80ecbdbf62938b Signed-off-by: Anastasia Klimchuk aklm@chromium.org --- M ichspi.c 1 file changed, 269 insertions(+), 255 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/35/58735/2