Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/47655 )
Change subject: it85spi.c: Refactor singleton states into reentrant pattern ......................................................................
Patch Set 5:
(2 comments)
https://review.coreboot.org/c/flashrom/+/47655/1/it85spi.c File it85spi.c:
https://review.coreboot.org/c/flashrom/+/47655/1/it85spi.c@238 PS1, Line 238: _rom(drv_data);
After looking at this a bit more, I think you may as well create a patch that removes some of these […]
As discussed, this can go into a later patch.
https://review.coreboot.org/c/flashrom/+/47655/1/it85spi.c@309 PS1, Line 309: msg_pdbg("FWH: %s():%d ret=%d\n", __func__, __LINE__, ret); : if (!ret) {
Part 2/2) Inline it85xx_spi_common_init() implementation here.
As discussed, this can go into a later patch.