[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/{skylake,cannonlake,icelake}: Correct GPIO IRQ start map

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31245 )

Change subject: soc/intel/{skylake,cannonlake,icelake}: Correct GPIO IRQ start map
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/31245/1/src/soc/intel/cannonlake/include/soc... 
File src/soc/intel/cannonlake/include/soc/itss.h:

https://review.coreboot.org/#/c/31245/1/src/soc/intel/cannonlake/include/soc... 
PS1, Line 19: 24
> restoring of polarities already takes care of updating only the right bits based on start offset.

Sounds like there needs to be come comments put down explaining the implicit assumptions.