Change in coreboot[master]: drivers/intel/mipi_camera: Support adding camera power resource
Sugnan Prabhu S has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41624 )
Change subject: drivers/intel/mipi_camera: Support adding camera power resource ......................................................................
Patch Set 16:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41624/12/src/drivers/intel/mipi_cam... File src/drivers/intel/mipi_camera/camera.c:
https://review.coreboot.org/c/coreboot/+/41624/12/src/drivers/intel/mipi_cam... PS12, Line 537: acpigen_write_store_op_to_namestr(0, "STA");
This implementation is based on some of the older platforms PowerResource. […]
OS can guarantee for a specific device, _ON and _OFF are called appropriately. But if multiple devices share the same power resource, OS does not know this. such cases do exists. Like camera sensor, eeprom and the VCM(device that control lens position) share the same power rails. Also for some platforms, worldfacing and userfacing cameras share the same power rails.
-
Sugnan Prabhu S (Code Review)