Attention is currently required from: Felix Singer, Paul Menzel, Michael Büchler. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56838 )
Change subject: mb/acer/g43t-am3: Add documentation ......................................................................
Patch Set 3: Code-Review+1
(1 comment)
File Documentation/mainboard/acer/g43t-am3.md:
https://review.coreboot.org/c/coreboot/+/56838/comment/77cb8b7d_a82c4745 PS2, Line 108: flashrom -p internal -r backup.rom
I'm pretty sure I haven't, but I found the board in a dumpster so it might very well be that somebody before me did. The relevant fields in the Flash Descriptor are all 0xff.. suspicious?!
I'll rewrite that section to warn about this..
Actually I think flashrom always gave a note that `The Flash Descriptor Override Strap-Pin is set`. Should I leave it like this?
Hmmm, this explains why the IFD restrictions are ignored. I'm sure it's controlled with the red jumper labelled "ME DISABLE", between the ICH10 southbridge and the SATA ports: https://imgur.com/N98fJsR.png
Would be good to test if changing this jumper results in the IFD restrictions being enforced (and the flashrom note disappearing), and documenting the jumper if so.
-
Angel Pons (Code Review)