[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/brya/var/kano: Set the ov2740 to 0 and the hi556 to 1 for SSFC

[David Wu (Code Review)]

Attention is currently required from: Tarun Tuli, Martin L Roth, Nick Vaccaro.

David Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71628 )

Change subject: mb/google/brya/var/kano: Set the ov2740 to 0 and the hi556 to 1 for SSFC
......................................................................

Patch Set 1:

(1 comment)

File src/mainboard/google/brya/variants/kano/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/71628/comment/d3744236_bad33c4f 
PS1, Line 20: 		option UFC_MIPI_OVTI2740        0
> I still don't understand why UFC_MIPI_HI556 can't just be value 3 in the UFC field, and why the SSFC […]

If set UFC_MIPI_HI556 to value 3 in the UFC field, it needs to follow SKU ID to replace mipi camera, which is inconvenient for the factory side, so the factory suggests using SSFC.
Kano is a mass production device and does not use SSFC (only one mipi camera ov2740), so we keep the original UFC filed for kano, and then add SSFC for two mipi camera for zydron.