GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/common/block/gpio_banks: add remote GPIO support 

Some AMD SoCs have a 5th GPIO bank, the remote GPIO bank, which isn't
located right after the 4th GPIO bank, but instead at a different
location inside the APCIMMIO region. A difference to the first 4 GPIO
banks is that the corresponding GPIO MUX registers aren't in a separate
bank, but at the end of the remote GPIO region. So this remote GPIO
region only supports 48 GPIOs with a 32 bit configuration register each
and has the 8 bit GPIO MUX registers beginning at offset 0xc0 in the
remote GPIO region.

For now using the remote GPIOs from verstage on PSP isn't supported. To
support this, it would need to map acpimmio_remote_gpio and update the
pointer like it already does for acpimmio_gpio0, acpimmio_iomux and a
few others.

BUG=b:194524995

Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: Ic8d7ff677a99381a5558782b80b0c4cae67602db
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56810
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
This commit is contained in:
Felix Held 2021-07-28 21:24:40 +02:00
parent 627c8443a3
commit b4fe8c5948
2 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/soc/amd/common/block/gpio_banks/gpio.c
View File

@ -24,7 +24,14 @@
/* MMIO access of new-style GPIO bank configuration registers */
static inline void *gpio_ctrl_ptr(gpio_t gpio_num)
{
return acpimmio_gpio0 + gpio_num * sizeof(uint32_t);
if (SOC_GPIO_TOTAL_PINS < AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER ||
/* Verstage on PSP would need to map acpimmio_remote_gpio */
(CONFIG(VBOOT_STARTS_BEFORE_BOOTBLOCK) && ENV_SEPARATE_VERSTAGE) ||
gpio_num < AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER)
return acpimmio_gpio0 + gpio_num * sizeof(uint32_t);
else
return acpimmio_remote_gpio +
(gpio_num - AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER) * sizeof(uint32_t);
}
static inline uint32_t gpio_read32(gpio_t gpio_num)
@ -39,7 +46,14 @@ static inline void gpio_write32(gpio_t gpio_num, uint32_t value)
static inline void *gpio_mux_ptr(gpio_t gpio_num)
{
return acpimmio_iomux + gpio_num;
if (SOC_GPIO_TOTAL_PINS < AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER ||
/* Verstage on PSP would need to map acpimmio_remote_gpio */
(CONFIG(VBOOT_STARTS_BEFORE_BOOTBLOCK) && ENV_SEPARATE_VERSTAGE) ||
gpio_num < AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER)
return acpimmio_iomux + gpio_num;
else
return acpimmio_remote_gpio + AMD_GPIO_REMOTE_GPIO_MUX_OFFSET +
(gpio_num - AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER);
}
static uint8_t get_gpio_mux(gpio_t gpio_num)

4
src/soc/amd/common/block/include/amdblocks/gpio_defs.h
View File

@ -77,6 +77,10 @@
#define AMD_GPIO_MUX_MASK 0x03
#define AMD_GPIO_FIRST_REMOTE_GPIO_NUMBER 256
/* The GPIO MUX registers for the remote GPIOs are at the end of the remote GPIO bank */
#define AMD_GPIO_REMOTE_GPIO_MUX_OFFSET 0xc0
/*
* Flags used for GPIO configuration. These provide additional information that does not go
* directly into GPIO control register. These are stored in `flags` field in soc_amd_gpio.