Attention is currently required from: Nico Huber, Angel Pons. szarpaj@grubelek.pl has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51952 )
Change subject: Docs/mb/lenovo/t420: Added information what's tested and works. ......................................................................
Patch Set 3:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/51952/comment/aa6fa2ab_21245493 PS1, Line 7: Added
nit: `Add`
Not sure what that means.
File Documentation/mainboard/lenovo/t420.md:
https://review.coreboot.org/c/coreboot/+/51952/comment/4ac8ac0c_a70f445c PS1, Line 21: ## Tested
Note: other boards have three sections: Working, Untested, Not working (or Broken) […]
Information added.
https://review.coreboot.org/c/coreboot/+/51952/comment/149142f2_e8a97e1e PS1, Line 23: CPU: Sandy Bridge i5-2520M, i7-2670QM
Should be formatted as a list, prefix with ` * ` (space, asterisk, space)
Done.
https://review.coreboot.org/c/coreboot/+/51952/comment/061a9f87_c33ede98 PS1, Line 27: internal
I'd rather say `integrated`
Fixed that typo. ;)