Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41429 )
Change subject: soc/intel/jasperlake: Add ISST override support ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/41429/1/src/soc/intel/jasperlake/in... File src/soc/intel/jasperlake/include/soc/cpu.h:
https://review.coreboot.org/c/coreboot/+/41429/1/src/soc/intel/jasperlake/in... PS1, Line 37: mainboard_isst_override Do we really need this additional callback into mainboard? Can't this be handled as part of mainboard chip_init?