From 222852a26476767d38337a0d3d629babace616fe Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Sun, 24 Jul 2022 23:47:56 +0530 Subject: soc/intel/gpio: Update GPIO Lock configuration recommendation This patch updates the GPIO lock configuration recommendation kconfig string to ensure the SoC user can select the correct config as applicable for the SoC. Note: From MTL onwards GPIO lock config can be performed using PCR write (MMIO write) and the GPIO team has confirmed this. BUG=b:213596994 TEST=Able to fix below GPIO lock config error msg on MTL with `SOC_INTEL_COMMON_BLOCK_GPIO_LOCK_USING_PCR` kconfig enabled. [INFO ] Locking pad configuration using SBI [INFO ] gpio_pad_config_lock_using_sbi: Locking pad 73 configuration [ERROR] SBI Failure: Transaction Status = 1 [ERROR] Failed to lock GPIO PAD, response = 1 Signed-off-by: Subrata Banik Change-Id: Icab1e4849b8e08ee1c695c924599f1513774178f Reviewed-on: https://review.coreboot.org/c/coreboot/+/66113 Tested-by: build bot (Jenkins) Reviewed-by: Tarun Tuli Reviewed-by: Eric Lai --- src/soc/intel/common/block/gpio/Kconfig | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src/soc/intel/common') diff --git a/src/soc/intel/common/block/gpio/Kconfig b/src/soc/intel/common/block/gpio/Kconfig index 03a3b092c0..a1e98dbe7d 100644 --- a/src/soc/intel/common/block/gpio/Kconfig +++ b/src/soc/intel/common/block/gpio/Kconfig @@ -40,7 +40,7 @@ config SOC_INTEL_COMMON_BLOCK_GPIO_LOCK_USING_SBI bool default n help - From TGL PCH onwards,`Pad Configuration Lock` can only be set or cleared + Till ADL PCH,`Pad Configuration Lock` can only be set or cleared using non-posted sideband write. config SOC_INTEL_COMMON_BLOCK_GPIO_LOCK_USING_PCR @@ -50,5 +50,7 @@ config SOC_INTEL_COMMON_BLOCK_GPIO_LOCK_USING_PCR help SoC user to select this config if `Pad Configuration Lock` can only be set or cleared using private configuration register (PCR) write. + Starting with MTL SoC, the recommendation is to use PCR for locking down the + GPIO configuration. endif -- cgit v1.2.3