[coreboot-gerrit] Change in coreboot[master]: Add ThinkPad R60 iGPU support.

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35293 )

Change subject: Add ThinkPad R60 iGPU support.
......................................................................

Patch Set 31:

(3 comments)

Why can’t this be a variant?

https://review.coreboot.org/c/coreboot/+/35293/31//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35293/31//COMMIT_MSG@2 
PS31, Line 2: Maccraft123
Please use your real name or a pseudonym.

https://review.coreboot.org/c/coreboot/+/35293/31//COMMIT_MSG@7 
PS31, Line 7: Add ThinkPad R60 iGPU support.
No dot/period at the end of the commit message summary please, and please use a prefix. Maybe:
> mb/lenovo: Add ThinkPad R60 iGPU support

https://review.coreboot.org/c/coreboot/+/35293/31//COMMIT_MSG@9 
PS31, Line 9: It runs without any blobs in firmware
            : Ethernet controller is detected by Linux
Please make this a list, and add a period/dot to the end of a sentence.