[M] Change in coreboot[main]: soc/intel: Assert if `pmc_/gpe0_dwX` values are not unique

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/85161?usp=email )

Change subject: soc/intel: Assert if `pmc_/gpe0_dwX` values are not unique
......................................................................

Patch Set 5:

(1 comment)

Patchset:

PS5:
> FYI, this breaks a ton of devices which use the default GPE routing, which previously worked fine but now assert since no routes defined in devicetree

HI Matt,

Can you help me understand how this affects you? If, as you say, lots of devices are failing due to static assertions, why did the build-bots mark this CL as V+1? Shouldn't a static assertion cause the build to fail?

Suggestion, if you are thinking avoiding assertion check for undeclared GPE (with default value 0) is good enough ? But note, as per GPE PMC description, zero is also a valid GPE group assignment.