Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35864 )
Change subject: mb/lenovo/{t60,r60}: Add ThinkPad R60 support as variant board ......................................................................
Patch Set 1:
(3 comments)
I guess you messed up with git, but here are some comments about the gpio.c file.
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... File src/mainboard/lenovo/t60/variants/r60/gpio.c:
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... PS1, Line 30: .gpio21 = GPIO_MODE_GPIO, /* LCD_PRESENCE */ Do you have the "LISA-7 FOR SVT" schematics? This is not true in there.
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... PS1, Line 82: * SMB_3B_EN */ Also not true. GPIO34 is NC
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... PS1, Line 102: .gpio33 = GPIO_LEVEL_LOW, : .gpio35 = GPIO_LEVEL_HIGH, This is meaningless as those are not configured as GPIO's