[coreboot-gerrit] Change in coreboot[master]: mainboard: Add Lenovo ThinkPad T440p

[Iru Cai (vimacs) (Code Review)]

Iru Cai (vimacs) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34359 )

Change subject: mainboard: Add Lenovo ThinkPad T440p
......................................................................

Patch Set 12:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34359/12/Documentation/mainboard/le... 
File Documentation/mainboard/lenovo/t440p.md:

https://review.coreboot.org/c/coreboot/+/34359/12/Documentation/mainboard/le... 
PS12, Line 38: - Cannot get the mainboard serial number
> Are there any eeproms on smbus?

As I said at https://review.coreboot.org/c/coreboot/+/34359/12#message-984b10347baf70fd8e..., Lenovo has something they call it in UEFI a emulated eeprom, but I don't know how to initialize it to make it readable.
Now I can set DRIVER_LENOVO_SERIALS=n as a work around.

https://review.coreboot.org/c/coreboot/+/34359/12/src/mainboard/lenovo/t440p... 
File src/mainboard/lenovo/t440p/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/34359/12/src/mainboard/lenovo/t440p... 
PS12, Line 39: register "gen3_dec" = "0x00000000"
> remove

Done