[coreboot-gerrit] Change in coreboot[master]: drivers/intel/mipi_camera: SSDT changes to add PLD

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42469 )

Change subject: drivers/intel/mipi_camera: SSDT changes to add PLD
......................................................................

Patch Set 12:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42469/12/src/drivers/intel/mipi_cam... 
File src/drivers/intel/mipi_camera/camera.c:

https://review.coreboot.org/c/coreboot/+/42469/12/src/drivers/intel/mipi_cam... 
PS12, Line 19: 	if (!config->pld.ignore_color)
             : 		config->pld.ignore_color = 1;
             : 
             : 	if (!config->pld.visible)
             : 		config->pld.visible = 1;
Won't this effectively just always set ignore_color and visible to 1? Either the devicetree sets it to 1 or it gets set to 1 here, so why bother with a config option?