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

[Subrata Banik (Code Review)]

Attention is currently required from: Dinesh Gehlot, Eran Mitrani, Intel coreboot Reviewers, Jakub Czapiga, Jayvik Desai, Kapil Porwal, Matt DeVillier, Nick Vaccaro, Pranava Y N, Sean Rhodes, Subrata Banik, Tarun, Werner Zeh.

Hello Dinesh Gehlot, Eran Mitrani, Intel coreboot Reviewers, Jakub Czapiga, Jayvik Desai, Kapil Porwal, Matt DeVillier, Nick Vaccaro, Pranava Y N, Sean Rhodes, Tarun, Werner Zeh, build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/86164?usp=email

to look at the new patch set (#2).

The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)

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

soc/intel: Add check to pmc_/gpe0_different_values

The pmc_/gpe0_different_values function previously only checked if the
dw0, dw1, and dw2 values were different from each other.  This commit
adds a check to ensure that the values are not all zero, as this can
represent an invalid configuration.

This prevents potential issues arising from a zeroed configuration
by ensuring the assertion only passes if the values are different
and at least one of them is non-zero.

TEST=Built and booted normally. No assertion failure observed.

Change-Id: Ie66d6dbcf49d5400b3fc3e4da113a569fe52dd51
Signed-off-by: Subrata Banik 
---
M src/soc/intel/alderlake/pmutil.c
M src/soc/intel/apollolake/pmutil.c
M src/soc/intel/cannonlake/pmutil.c
M src/soc/intel/elkhartlake/pmutil.c
M src/soc/intel/jasperlake/pmutil.c
M src/soc/intel/meteorlake/pmutil.c
M src/soc/intel/pantherlake/pmutil.c
M src/soc/intel/skylake/pmutil.c
M src/soc/intel/tigerlake/pmutil.c
9 files changed, 63 insertions(+), 9 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/86164/2