[M] Change in coreboot[main]: drivers/intel/fsp2_0: Simplify FSP global reset definition

[Subrata Banik (Code Review)]

Attention is currently required from: Andrey Petrov, Dinesh Gehlot, Eran Mitrani, Jakub Czapiga, Jayvik Desai, Jérémy Compostella, Kapil Porwal, Lean Sheng Tan, Nick Vaccaro, Paul Menzel, Pranava Y N, Rishika Raj, Ronak Kanabar, Sean Rhodes, Tarun, Werner Zeh.

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

Change subject: drivers/intel/fsp2_0: Simplify FSP global reset definition
......................................................................

Patch Set 11:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/84356/comment/9d3d1c8a_1e276b30?usp... :
PS11, Line 35: CONFIG_FSP_STATUS_GLOBAL_RESET
can we implement a wrapper here instead which can convert 32-bit FSP status into 64-bit looking at if the FSP/platform is x86_64 compatible rather we start passing index?

```
#define CONVERT_TO_64BIT(value) (CONFIG_PLATFORM_USES_FSP2_X86_32 \
                 ? value : \
                 ((uint64_t)(value & 0xf0000000) << 32) | (value & 0xff))

#define FSP_STATUS_WRAPPER(x) CONVERT_TO_64BIT(x)
```

```
if (status == FSP_STATUS_WRAPPER(CONFIG_FSP_STATUS_GLOBAL_RESET))
```

in this case, you don't need to change the code significantly and no need to introduce new terminology like index, reset type. what we need is a seamless translation between 32-bit and 64-bit FSP reset status. Which has been done by FSP spec as 32-bit value 0x40000003 to 64-bit value 0x4000000000000003