[M] Change in coreboot[main]: soc/intel: Add check to pmc_/gpe0_different_values

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

Attention is currently required from: Eran Mitrani, Intel coreboot Reviewers, Jakub Czapiga, Jayvik Desai, Nick Vaccaro, Pranava Y N, Subrata Banik, Tarun, Werner Zeh.

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

Change subject: soc/intel: Add check to pmc_/gpe0_different_values
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/86164/comment/7a5031e7_9d3cb8a0?usp... :
PS2, Line 14: This prevents potential issues arising from a zeroed configuration
> this is a WA for an issue caused by CB:85161. […]

I am trying here to wake up my brain. If all three variables are zero how is the assert passing with the submitted code? Could you point out my mistake here ? Sorry if I am wasting your time.
```
    unsigned int dw0 = 0;	
    unsigned int dw1 = 0;	
    unsigned int dw2 = 0;
    bool all_zero = (dw0 == 0) && (dw1 == 0) && (dw2 == 0);
    bool different = (dw0 != dw1) && (dw1 != dw2) && (dw0 != dw2);
    bool result = (dw0 != dw1) && (dw1 != dw2) && (dw0 != dw2) && !all_zero;

printf("all_zero=%s\n", all_zero ? "yes" : "no");
    printf("different=%s\n", different ? "yes" : "no");
    printf("result=%s\n", result ? "yes" : "no");
```

```
all_zero=yes
different=no
result=no
```