Change in coreboot[master]: soc/intel/denverton_ns: Enable MC Exception

[Julien Viard de Galbert (Code Review)]

Julien Viard de Galbert has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25434 )

Change subject: soc/intel/denverton_ns: Enable MC Exception
......................................................................

Patch Set 19:

(2 comments)

This one should be okay now.

https://review.coreboot.org/c/coreboot/+/25434/3/src/soc/intel/denverton_ns/... 
File src/soc/intel/denverton_ns/cpu.c:

https://review.coreboot.org/c/coreboot/+/25434/3/src/soc/intel/denverton_ns/... 
PS3, Line 63: 	/* TODO install a fallback MC handler for each core in case OS does
            : 	   not provide one. Is it really needed ? */
> What would the fallback MC handler do if an MC occurred?

No idea, to be honest I remember reading this in the BiosWriterGuide I no longer have access to...

https://review.coreboot.org/c/coreboot/+/25434/12/src/soc/intel/denverton_ns... 
File src/soc/intel/denverton_ns/cpu.c:

https://review.coreboot.org/c/coreboot/+/25434/12/src/soc/intel/denverton_ns... 
PS12, Line 65: needed ?
> Ping

Removed the space.