Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41290 )
Change subject: Documentation: Add GbE and test results disabling it ......................................................................
Patch Set 1:
(6 comments)
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... File Documentation/soc/intel/gbe.md:
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... PS1, Line 4: withing within
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... PS1, Line 6: regiong region
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... PS1, Line 15: in a no boot In the OS, the firmware?
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... PS1, Line 19: Overwriting How?
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... PS1, Line 20: up on GNU/Linux. The e1000e fails to load with 'error -3'. On the Lenovo X230, for example:
e1000e: probe of 0000:00:19:0 failed with error -3
https://review.coreboot.org/c/coreboot/+/41290/1/Documentation/soc/intel/gbe... PS1, Line 22: Remove one blank line.