[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/i82801gx: Add a mb hook to override the SPI opmenu

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30888 )

Change subject: sb/intel/i82801gx: Add a mb hook to override the SPI opmenu
......................................................................

Patch Set 1:
> Patch Set 1:
> > It might be better to have flashrom deal with this properly.
>
> How does coreboot handle it? AFAICS, the chips in question would
> either have to be programmed byte-wise or need us to patch the
> opmenu on the fly. I assume coreboot does the latter?

Coreboot indeed does the latter. This is not very mainboard specific, but more intel+SPI-flash specific, so IMO it does not really belong here, but could good-enough for now...