[coreboot-gerrit] Change in coreboot[master]: mb/lenovo: DELETEME: Add ThinkPad R60 iGPU support (don't merge worki...

[Maccraft123 (Code Review)]

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

Change subject: mb/lenovo: DELETEME: Add ThinkPad R60 iGPU support (don't merge working on making it variant board)
......................................................................

Patch Set 50:

(2 comments)
> Patch Set 49:
>
> Should be abandoned in favor of  Ibd9208a5eafd228f8eedbc8fb4f4eb9ed1932a14

Well it now _is_ abandoned

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

https://review.coreboot.org/c/coreboot/+/35293/36//COMMIT_MSG@15 
PS36, Line 15: -Sometimes it takes 20s to boot to payload.
> take a look at timestamps with cbmem -t

Will test when i'm back home.

https://review.coreboot.org/c/coreboot/+/35293/20/src/mainboard/lenovo/r60/s... 
File src/mainboard/lenovo/r60/smihandler.c:

https://review.coreboot.org/c/coreboot/+/35293/20/src/mainboard/lenovo/r60/s... 
PS20, Line 43: 		printk(BIOS_DEBUG,
> fix code style

I don't really know what to do