Attention is currently required from: Selma Bensaid, Paul Menzel, Bernardo Perez Priego, Andy Yeh, kiran2.kumar@intel.com. ShawnX Tu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58374 )
Change subject: mb/google/brya: Update camera NVM parameters ......................................................................
Patch Set 5:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58374/comment/454996d6_bf8810cf PS5, Line 9: Change HID name from INT3499 to PRP0001 along with size and : address width.
Why?
Refer to eeprom discussion thread (https://partnerissuetracker.corp.google.com/issues/170924309), it defined the NVM nodes naming & udev rules with Google.
https://review.coreboot.org/c/coreboot/+/58374/comment/49807d68_0968a500 PS5, Line 13: Test=Boot board, camera nvm parameters should be as expected.
Checked how?
# cat /sys/bus/i2c/devices/i2c-PRP0001:01/eeprom > ./ov8856_eeprom_test.bin # hexdump -C ov8856_eeprom_test.bin > ov8856_eeprom_dump.log
You should see the result be same as module nvm file by vendor provided or meet the Intel nvm calibration format. (e.g. first 4 bytes be 0x01, 0x03, 0x01, 0x00)