[coreboot-gerrit] Change in coreboot[master]: drivers/intel/mipi_camera: Add support to handle CIO2 device

[Tim Wawrzynczak (Code Review)]

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

Change subject: drivers/intel/mipi_camera: Add support to handle CIO2 device
......................................................................

Patch Set 14:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/42470/12/src/drivers/intel/mipi_cam... 
PS12, Line 87: 	for (i = 0; i < config->cio2_num_ports; ++i) {
             : 		free(ep_table_name[i]);
             : 		free(port_table_name[i]);
             : 		free(port_name[i]);
             : 	}
> Hrm, given the style's suggestion to use malloc() as a last resort another option is to use somethin […]

And that alternative uses an extra 384 bytes of stack space... the memory has to come from somewhere either way. I don't think either option is great, it's just an artifact of the current acpi_dp_* API, which already is a heavy malloc() user.