Ricardo Ribalda has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48631 )
Change subject: mb/google/poppy: Fix race condition in acpi camera_pmic ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/48631/2/src/mainboard/google/poppy/... File src/mainboard/google/poppy/variants/baseboard/include/baseboard/acpi/camera_pmic.asl:
https://review.coreboot.org/c/coreboot/+/48631/2/src/mainboard/google/poppy/... PS2, Line 403: 0
Do you need to set the resourceorder for OVCM and OVTH to guarantee a specific order?
I need the OVCM to be set before the other PR, and clear after the others are ready. I thought that was enforced by this order:
Name (_PR0, Package () {^^I2C2.PMIC.OVCM, ^^I2C2.PMIC.OVFI})
But reading: 7.2 https://www.uefi.org/sites/default/files/resources/ACPI%206_2_A_Sept29.pdf probes that I am worng. Thanks!!