Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44264 )
Change subject: mb/google/zork: add dptc interface support ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44264/2/src/mainboard/google/zork/v... File src/mainboard/google/zork/variants/baseboard/include/baseboard/acpi/dptc.asl:
https://review.coreboot.org/c/coreboot/+/44264/2/src/mainboard/google/zork/v... PS2, Line 24: return These methods don't do anything? What is the purpose of these methods? And why is this file in mainboard? Do we expect the same logic across designs? If so, we should move it to soc?