[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Move interrupt and wake status clear

[Josie Nordrum (Code Review)]

Josie Nordrum has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44640 )

Change subject: soc/amd/common: Move interrupt and wake status clear
......................................................................

soc/amd/common: Move interrupt and wake status clear

Move interrupt status and wake status clearing to after GPIO config.

BUG=b:164892883, b:165342107
TEST=None
BRANCH=None

Signed-off-by: Josie Nordrum 
Change-Id: If4a5db4bfa6a2ee9827f38e9595f487a4dcfac2c
---
M src/soc/amd/common/block/gpio_banks/gpio.c
1 file changed, 2 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/44640/1

diff --git a/src/soc/amd/common/block/gpio_banks/gpio.c b/src/soc/amd/common/block/gpio_banks/gpio.c
index 7fb6622..74ea696 100644
--- a/src/soc/amd/common/block/gpio_banks/gpio.c
+++ b/src/soc/amd/common/block/gpio_banks/gpio.c
@@ -214,10 +214,9 @@

soc_gpio_hook(gpio, mux);

-		/* Clear interrupt and wake status (write 1-to-clear bits) */
-		control |= GPIO_INT_STATUS | GPIO_WAKE_STATUS;
    	__gpio_setbits32(gpio, PAD_CFG_MASK, control);
-
+		/* Clear interrupt and wake status (write 1-to-clear bits) */
+		__gpio_or32(gpio, GPIO_INT_STATUS | GPIO_WAKE_STATUS);
    	if (control_flags == 0)
    		continue;