[flashrom-gerrit] Change in flashrom[master]: ichspi.c: Implement ich_hwseq_get_flash_id()

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/48133 )

Change subject: ichspi.c: Implement ich_hwseq_get_flash_id()
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/flashrom/+/48133/1/ichspi.c 
File ichspi.c:

https://review.coreboot.org/c/flashrom/+/48133/1/ichspi.c@1375 
PS1, Line 1375: ich_read_data((uint8_t *)&data, len, ICH9_REG_FDATA0);
> This seems to return 0 to data on my ich9 device here, which reports "Chip identified: KB9012 (EDI)" […]

Hey Arthur, that's very weird. Could you pastebin somewhere the full verbose output of a run so we can to try to inspect what is happening?