[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 19:

(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""
> Attaching the SSDT CAM related entries from WDoo, I donot have the WDee board right now […]

Thanks. So it looks like both the VCM and EEPROM are tied to the sensor power sequence, but that shouldn't be the case. It should be possible to turn them on and off separately without fully powering on the others.

Also, what are the _DSM and SSDB methods for?