[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch/var/kindred: Update DPTF parameters and TDP PL1/PL2

[Peter Ou (Code Review)]

Peter Ou has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35131 )

Change subject: mb/google/hatch/var/kindred: Update DPTF parameters and TDP PL1/PL2
......................................................................

Patch Set 2:

(2 comments)
> Patch Set 2:
>
> (2 comments)

https://review.coreboot.org/c/coreboot/+/35131/2/src/mainboard/google/hatch/... 
File src/mainboard/google/hatch/variants/kindred/include/variant/acpi/dptf.asl:

https://review.coreboot.org/c/coreboot/+/35131/2/src/mainboard/google/hatch/... 
PS2, Line 83: 80, 70, 60, 50, 40, 30
> Do we really need these? You have defined only DPTF_CPU_ACTIVE_AC0 entry above.

Hi Sumeet
At the moment these numbers don't work, but they might be used in the next stage, so I put them in the table first.

https://review.coreboot.org/c/coreboot/+/35131/2/src/mainboard/google/hatch/... 
PS2, Line 87: 100, 80, 70, 60, 50, 40, 30
> same as above, do we need these? There is no entry above for DPTF_TSR0_ACTIVE_AC*.

This setting is same as the CPU, and we will verify it in the next stage.