[coreboot-gerrit] Change in coreboot[master]: mb/ocp/deltalake: Enable PCH thermal sensor via FSP

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44075 )

Change subject: mb/ocp/deltalake: Enable PCH thermal sensor via FSP
......................................................................

Patch Set 3:

(2 comments)

https://review.coreboot.org/c/coreboot/+/44075/2/src/mainboard/ocp/deltalake... 
File src/mainboard/ocp/deltalake/romstage.c:

https://review.coreboot.org/c/coreboot/+/44075/2/src/mainboard/ocp/deltalake... 
PS2, Line 53: 	mupd->FspmConfig.ThermalDeviceEnable = 1;
> what is the address of this device (bus/device/function)?

I was thinking of doing something like CB:42557

https://review.coreboot.org/c/coreboot/+/44075/3/src/mainboard/ocp/deltalake... 
File src/mainboard/ocp/deltalake/romstage.c:

https://review.coreboot.org/c/coreboot/+/44075/3/src/mainboard/ocp/deltalake... 
PS3, Line 15: static const config_t *config = NULL;
> Why do we need this global variable? […]

This should not be needed if doing something like CB:42557