build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35225 )
Change subject: soc/intel/common/block/cse: Move me_read_config32() to common code ......................................................................
Patch Set 3:
(8 comments)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cse/cse.c:
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 564: if (offset == PCI_ME_HFSTS1) trailing whitespace
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... File src/soc/intel/common/block/include/intelblocks/cse.h:
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 62: <<<<<<< HEAD spaces required around that '<' (ctx:OxW)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 65: ======= spaces required around that '==' (ctx:ExO)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 65: ======= spaces required around that '==' (ctx:OxO)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 65: ======= spaces required around that '==' (ctx:OxO)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 65: ======= spaces required around that '=' (ctx:OxE)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 79: >>>>>>> b2d10c8321... src/soc/intel/common/block/cse: Add helper functions to CSE lib spaces required around that '>' (ctx:OxW)
https://review.coreboot.org/c/coreboot/+/35225/3/src/soc/intel/common/block/... PS3, Line 79: >>>>>>> b2d10c8321... src/soc/intel/common/block/cse: Add helper functions to CSE lib spaces required around that ':' (ctx:VxW)