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

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Furquan Shaikh, Sumeet R Pawnikar, Patrick Rudolph, Karthik Ramasubramanian, Felix Held.
Tim Wawrzynczak 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/8b15dbef_7791e0c2 
PS4, Line 356: write_pkgc_method();
> This is required only for READ methods, not required for WRITE. So, I kept inside if(... […]

But this will write out 5 copies of the PKGC Method won't it? Don't you see something in `dmesg` about conflicting definitions? Have you double checked the SSDT output?

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

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

I had thought the above `Return 0` was working. Is this a change in the ESIF shell that it requires a package now?