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 13:
(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]); : } one last thing, so you know it's completely optional to free() memory in coreboot, see the section about memory allocation: https://review.coreboot.org/plugins/gitiles/coreboot/+/refs/heads/master/Doc...