[coreboot-gerrit] Change in coreboot[master]: [UNTESTED] soc/intel/{xeon-sp,icl,tgl,jsl,ehl,adl}: add NMI_{EN,STS} ...

[Lance Zhao (Code Review)]

Lance Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48102 )

Change subject: [UNTESTED] soc/intel/{xeon-sp,icl,tgl,jsl,ehl,adl}: add NMI_{EN,STS} registers
......................................................................

Patch Set 18:

(1 comment)

https://review.coreboot.org/c/coreboot/+/48102/18/src/soc/intel/xeon_sp/incl... 
File src/soc/intel/xeon_sp/include/soc/lewisburg_pch_gpio_defs.h:

https://review.coreboot.org/c/coreboot/+/48102/18/src/soc/intel/xeon_sp/incl... 
PS18, Line 666: #define GPI_NMI_STS_0		0x160
              : #define GPI_NMI_EN_0		0x170
That's not match with EDS 547817.that' 164/174 for community 0, 160/164/168/170/174/178 for community 1 ...