[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Add ACPI support for camera in WDee

[Tomasz Figa (Code Review)]

Tomasz Figa has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42205 )

Change subject: mb/google/dedede: Add ACPI support for camera in WDee
......................................................................

Patch Set 22:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42205/19/src/mainboard/google/deded... 
File src/mainboard/google/dedede/variants/waddledee/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/42205/19/src/mainboard/google/deded... 
PS19, Line 197: 				register "pr0" = ""\_SB.PCI0.I2C3.CAM1.PRIC""
> if physically they share the same power source how can they be powered on/off separately? can you pl […]

I believe they don't share all of the power sequencing. An EEPROM usually requires only a single voltage rail and no poweron/reset signals. Please consult with respective schematics or hw engineers.