Ravishankar Sarawadi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43980 )
Change subject: soc/intel/tigerlake: Configure TCSS D3Cold for pre-QS and QS platforms ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/43980/2/src/soc/intel/tigerlake/fsp... File src/soc/intel/tigerlake/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/43980/2/src/soc/intel/tigerlake/fsp... PS2, Line 119: cpu_id = cpu_get_cpuid(); : if (cpu_id == CPUID_TIGERLAKE_A0) : params->D3ColdEnable = 0; : else Shouldn't it be handled in DT as done in https://review.coreboot.org/c/coreboot/+/43621