Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44270 )
Change subject: soc/intel/jasperlake: Configure IPU based on devicetree ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44270/6/src/soc/intel/jasperlake/ro... File src/soc/intel/jasperlake/romstage/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/44270/6/src/soc/intel/jasperlake/ro... PS6, Line 70: /* By default IPU is enabled, need to disable in FSP if pci device is disabled */
nit: not needed?
Done