Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38994 )
Change subject: soc/intel/xeon_sp: Enable LPC generic IO decode range ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38994/1/src/soc/intel/xeon_sp/chip.... File src/soc/intel/xeon_sp/chip.h:
https://review.coreboot.org/c/coreboot/+/38994/1/src/soc/intel/xeon_sp/chip.... PS1, Line 89: uint32_t gen3_dec; These ranges may or may not be defined in mainboard devicetree.cb file. Need to take care of the situation where they are not defined.