[coreboot-gerrit] Change in coreboot[master]: drivers/intel/dptf: Add support for PCH methods

[Sumeet R Pawnikar (Code Review)]

Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Patrick Rudolph, Karthik Ramasubramanian, Felix Held.
Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57925 )

Change subject: drivers/intel/dptf: Add support for PCH methods
......................................................................

Patch Set 5:

(2 comments)

File src/drivers/intel/dptf/dptf.c:

https://review.coreboot.org/c/coreboot/+/57925/comment/c1d07e12_0b57b376 
PS4, Line 356: write_pkgc_method();
> But this will write out 5 copies of the PKGC Method won't it? Don't you see something in `dmesg` abo […]

Yes, you are correct. We do not need to do this inside for loop. Uploaded SSDT output on bug for your reference.

File src/drivers/intel/dptf/dptf.c:

https://review.coreboot.org/c/coreboot/+/57925/comment/a7568d5d_f5564823 
PS5, Line 230: acpigen_write_package(1);
             : 	acpigen_write_zero();
> Sorry I mean this seems like a separate change.  […]

As per fsp code reference, we need this change for ESIF shell support.