[coreboot-gerrit] Change in coreboot[master]: soc/amd/common/gpio: Clear interrupt and wake status when configuring...

[Aaron Durbin (Code Review)]

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

Change subject: soc/amd/common/gpio: Clear interrupt and wake status when configuring pads
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42877/4/src/soc/amd/common/block/gp... 
File src/soc/amd/common/block/gpio_banks/gpio.c:

https://review.coreboot.org/c/coreboot/+/42877/4/src/soc/amd/common/block/gp... 
PS4, Line 206: GPIO_WAKE_STATUS
> This will be taken care of in the event logging change to ensure that we snapshot the state before g […]

I think we'd be able to log (eventlog) things w/ b/159947207 once that work is done, but it doesn't handle the case of the kernel expecting to capture the wake. If we want to note the device that woke from kernel we shouldn't clear these.