[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/skyrim/var/frostflow: Set Package Power Parameters

[Tim Van Patten (Code Review)]

Attention is currently required from: Frank Wu, John Su, Jason Nien, Matt DeVillier, Chris Wang, Martin Roth.

Tim Van Patten has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69565 )

Change subject: mb/google/skyrim/var/frostflow: Set Package Power Parameters
......................................................................

Patch Set 4:

(1 comment)

File src/mainboard/google/skyrim/variants/frostflow/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/69565/comment/bf2aa1e8_817cbac8 
PS4, Line 10: 	register "stapm_time_constant_s" = "200"
> Hi Tim, […]

Ah, right, that's what I was missing.

I wonder if it would make more sense to move the values from `skyrim/overridetree.cb` into `baseboard/devicetree.cb`, since they all need them defined anyway. That would avoid the duplication.

CL is available here:
https://review.coreboot.org/c/coreboot/+/69904