[M] Change in coreboot[main]: [RFC] soc/intel/fsp_reset: Drop global-reset sanity check

[Jérémy Compostella (Code Review)]

Attention is currently required from: Andrey Petrov, Dinesh Gehlot, Eran Mitrani, Jakub Czapiga, Jayvik Desai, Kapil Porwal, Lean Sheng Tan, Nick Vaccaro, Nico Huber, Pranava Y N, Ronak Kanabar, Sean Rhodes, Subrata Banik, Tarun, Werner Zeh.

Jérémy Compostella has posted comments on this change by Nico Huber. ( https://review.coreboot.org/c/coreboot/+/84558?usp=email )

Change subject: [RFC] soc/intel/fsp_reset: Drop global-reset sanity check
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/common/fsp_reset.c:

https://review.coreboot.org/c/coreboot/+/84558/comment/e73dd5c4_8d645b22?usp... :
PS1, Line 78: 		/* Use RESET_REQUIRED_3 status for global resets. Needs
            : 		   to be kept in sync with chipset_handle_reset() above. */
            : 		return FSP_STATUS_RESET_REQUIRED_3;
> I like the idea of getting rid of the check as a simplification of the code as this is the only type […]

but then, of course, this long list of changes raises the question: are we really simplifying anything ?