[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Add VCM and NVM entry for OV8856 sensor

[Varshit B Pandya (Code Review)]

Varshit B Pandya has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41173 )

Change subject: mb/google/dedede: Add VCM and NVM entry for OV8856 sensor
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41173/6/src/mainboard/google/dedede... 
File src/mainboard/google/dedede/variants/baseboard/include/baseboard/acpi/cam1.asl:

https://review.coreboot.org/c/coreboot/+/41173/6/src/mainboard/google/dedede... 
PS6, Line 236: Device (NVM0)
> No need to expose the NVM0 device through a package in CAM1's _DSD just like how it is done for VCM?

The eeprom device will not be part of and recognized by V4L2 framework, 
the vcm attribute in sensor DSD section is to expose the VCM device to v4l2 framework, while eeprom will be registered as a standard nvmem device