Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38781 )
Change subject: mb/lenovo: Remove thermal.h header ......................................................................
Patch Set 1: Code-Review+1
(3 comments)
https://review.coreboot.org/c/coreboot/+/38781/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38781/1//COMMIT_MSG@10 PS1, Line 10: autoport Since everyone who used autoport is using tcrt = 100 and tpsv = 90, how about making them the default values somehow?
https://review.coreboot.org/c/coreboot/+/38781/1/src/mainboard/lenovo/t430/a... File src/mainboard/lenovo/t430/acpi_tables.c:
https://review.coreboot.org/c/coreboot/+/38781/1/src/mainboard/lenovo/t430/a... PS1, Line 28: gnvs->tmax = 105; Why does nobody else configure this? Shouldn't it be default anyway?
https://review.coreboot.org/c/coreboot/+/38781/1/src/mainboard/lenovo/t430/t... File src/mainboard/lenovo/t430/thermal.h:
PS1: I'd keep the comments from this file