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

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34359 )

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

Patch Set 3: Code-Review+1

(2 comments)

please limit pictures to 800px in width and try to reduce it's size by using 70% JPEG compression.

https://review.coreboot.org/c/coreboot/+/34359/3//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34359/3//COMMIT_MSG@27 
PS3, Line 27: - Mainboard serial number is invalid
does the mainboard have at24rf80 i2c eeprom?
I noticed that the t480 doesn't have one.

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

https://review.coreboot.org/c/coreboot/+/34359/3/src/mainboard/lenovo/t440p/... 
PS3, Line 1: chip northbridge/intel/haswell # FIXME: check gfx.ndid and gfx.did
remove FIXME