Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/61424 )
Change subject: soc/intel/common/gpio: Skip GPIO PAD locking in recovery mode ......................................................................
soc/intel/common/gpio: Skip GPIO PAD locking in recovery mode
The recovery mode is meant to provide fixes for the platform deformity hence, skip locking the GPIO PAD configuration to provide the same flexibility to the platform owner while booting in recovery mode.
BUG=b:211950520 TEST=Able to build and boot the brya.
Signed-off-by: Subrata Banik subratabanik@google.com Change-Id: I0f0a3cfb2be7f2a5485679f6a3d8cb4fb407fcf4 --- M src/soc/intel/common/block/gpio/gpio.c 1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/61424/1
diff --git a/src/soc/intel/common/block/gpio/gpio.c b/src/soc/intel/common/block/gpio/gpio.c index dcec12a..3c5d3f3 100644 --- a/src/soc/intel/common/block/gpio/gpio.c +++ b/src/soc/intel/common/block/gpio/gpio.c @@ -13,6 +13,7 @@ #include <intelblocks/itss.h> #include <intelblocks/p2sb.h> #include <intelblocks/pcr.h> +#include <security/vboot/vboot_common.h> #include <soc/pci_devs.h> #include <soc/pm.h> #include <stdlib.h> @@ -616,8 +617,8 @@
int gpio_lock_pad(const gpio_t pad, enum gpio_lock_action lock_action) { - /* Skip locking GPIO PAD in early stages */ - if (ENV_ROMSTAGE_OR_BEFORE) + /* Skip locking GPIO PAD in early stages or in recovery mode */ + if (ENV_ROMSTAGE_OR_BEFORE || vboot_recovery_mode_enabled()) return -1;
const struct gpio_lock_config pads = {