[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/skolas: Skip locking for GPP_F14 GPIO

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/72426 )

(

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: mb/google/skolas: Skip locking for GPP_F14 GPIO
......................................................................

mb/google/skolas: Skip locking for GPP_F14 GPIO

There is an existing issue for skolas boards where board wakes up
from shutdown immediately due to touchpad wake signal.

This issue was root caused to the patch which was setting GPE_EN
bits for the GPIOs before locking.
Ref: commit 38b8bf02d820 ("intelblocks: Add function to program GPE_EN before GPIO locking")

Later issue was found to be with GPP_F14 configuration for skolas
boards. While shutting down, kernel is not able to disable IRQ for
touchpad due to GPE_EN register getting locked and it is preventing
shutdown of the board.

This patch skips the locking for GPP_F14 to allow kernel to
configure it later.

BUG=b:254064671
BRANCH=None
TEST=Shutdown works on Skolas board with the patch.

Nissa Bug: 234097956

Signed-off-by: Maulik Vaghela 
Change-Id: I09cf1af1f5ab11b06073755374ee8a306984d557
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72426
Reviewed-by: Subrata Banik 
Tested-by: build bot (Jenkins) 
---
M src/mainboard/google/brya/variants/baseboard/skolas/gpio.c
1 file changed, 35 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Subrata Banik: Looks good to me, approved

diff --git a/src/mainboard/google/brya/variants/baseboard/skolas/gpio.c b/src/mainboard/google/brya/variants/baseboard/skolas/gpio.c
index fb7bc21..cd0be69 100644
--- a/src/mainboard/google/brya/variants/baseboard/skolas/gpio.c
+++ b/src/mainboard/google/brya/variants/baseboard/skolas/gpio.c
@@ -240,7 +240,7 @@
    /* F13 : GSXDOUT ==> GSPI_PCH_DI_FPMCU_DO */
    PAD_CFG_NF_LOCK(GPP_F13, NONE, NF4, LOCK_CONFIG),
    /* F14 : GSXDIN ==> TCHPAD_INT_ODL */
-	PAD_CFG_GPI_IRQ_WAKE_LOCK(GPP_F14, NONE, LEVEL, INVERT, LOCK_CONFIG),
+	PAD_CFG_GPI_IRQ_WAKE(GPP_F14, NONE, PWROK, LEVEL, INVERT),
    /* F15 : GSXSRESET# ==> FPMCU_INT_L */
    PAD_CFG_GPI_IRQ_WAKE_LOCK(GPP_F15, NONE, LEVEL, INVERT, LOCK_CONFIG),
    /* F16 : GSXCLK ==> GSPI_PCH_CS_FPMCU_R_L */