Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38926 )
Change subject: Documentation: Add Montevina ThinkPads common page ......................................................................
Patch Set 10:
(3 comments)
https://review.coreboot.org/c/coreboot/+/38926/10/Documentation/mainboard/le... File Documentation/mainboard/lenovo/montevina_series.md:
https://review.coreboot.org/c/coreboot/+/38926/10/Documentation/mainboard/le... PS10, Line 53: 001fffff 01ffffff
https://review.coreboot.org/c/coreboot/+/38926/10/Documentation/mainboard/le... PS10, Line 106: $ ./bincfg ifd-x200.spec ifd-x200.set ifd.bin There were concerns in the past that this .set file might not contain correct settings for other models. Did somebody check that? should we warn here?
https://review.coreboot.org/c/coreboot/+/38926/10/Documentation/mainboard/le... PS10, Line 117: 0x1fd000 0x1ffd000