[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede/var/storo: Add camera support

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Martin Roth, Henry Sun, zanxi chen, Paul Menzel, Andy Yeh, Weimin Wu, ShawnX Tu.
Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50995 )

Change subject: mb/google/dedede/var/storo: Add camera support
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2:
> henry is correct. no need check whether EEPROM is provisioned in coreboot.

If a second source is introduced at a later point in time, we need to differentiate identify unprovisioned scenario.

Because in the factory on the first boot, SSFC will be unprovisioned and read as 0. That will cause us to apply the power sequencing for the first camera because the default SSFC value is 0.

Andy,

I am not sure why camera EEPROM does not need to be provisioned. If we don't do it now, then it will be hard to identify the camera source at the factory and provision the SSFC fields in CBI EEPROM.