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

[Paul Menzel (Code Review)]

Paul Menzel 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 7: Code-Review+1

(2 comments)

Is somebody working on unifying the camera ASL code, so it’s automatically generated for example? For example, besides some values, the structure is very similar to.

src/mainboard/google/volteer/variants/baseboard/include/baseboard/acpi/mipi_camera.asl

https://review.coreboot.org/c/coreboot/+/41173/7//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/41173/7//COMMIT_MSG@11 
PS7, Line 11: BUG=None
No bug?

https://review.coreboot.org/c/coreboot/+/41173/7//COMMIT_MSG@14 
PS7, Line 14: and able to capture image using world facing camera.
Did the camera work before?