From 7051a40b0be77c26dfeb929b059a24ff8a36bc78 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ky=C3=B6sti=20M=C3=A4lkki?= Date: Mon, 1 Jun 2020 09:37:05 +0300 Subject: mb/pcengines/apu2: Change GPIO configuration functions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The definitions of GPIO_xx equal IOMUX_GPIO_xx shifted by two. Change-Id: I0ee821c71c88bf535122a9526862a9d1e68bd755 Signed-off-by: Kyösti Mälkki Reviewed-on: https://review.coreboot.org/c/coreboot/+/42038 Tested-by: build bot (Jenkins) Reviewed-by: Michał Żygowski --- src/mainboard/pcengines/apu2/gpio_ftns.c | 12 +++++--- src/mainboard/pcengines/apu2/gpio_ftns.h | 48 ++++++++++---------------------- src/mainboard/pcengines/apu2/romstage.c | 30 ++++++++++---------- 3 files changed, 38 insertions(+), 52 deletions(-) diff --git a/src/mainboard/pcengines/apu2/gpio_ftns.c b/src/mainboard/pcengines/apu2/gpio_ftns.c index 1969a885b8..c249c2da5c 100644 --- a/src/mainboard/pcengines/apu2/gpio_ftns.c +++ b/src/mainboard/pcengines/apu2/gpio_ftns.c @@ -7,8 +7,10 @@ #include #include "gpio_ftns.h" -static u32 gpio_read_wrapper(u32 gpio) +static u32 gpio_read_wrapper(u32 iomux_gpio) { + u32 gpio = iomux_gpio << 2; + if (gpio < 0x100) return gpio0_read32(gpio & 0xff); else if (gpio >= 0x100 && gpio < 0x200) @@ -19,8 +21,10 @@ static u32 gpio_read_wrapper(u32 gpio) die("Invalid GPIO"); } -static void gpio_write_wrapper(u32 gpio, u32 setting) +static void gpio_write_wrapper(u32 iomux_gpio, u32 setting) { + u32 gpio = iomux_gpio << 2; + if (gpio < 0x100) gpio0_write32(gpio & 0xff, setting); else if (gpio >= 0x100 && gpio < 0x200) @@ -29,7 +33,7 @@ static void gpio_write_wrapper(u32 gpio, u32 setting) gpio2_write32(gpio & 0xff, setting); } -void configure_gpio(u8 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting) +void configure_gpio(u32 gpio, u8 iomux_ftn, u32 setting) { u32 bdata; @@ -43,7 +47,7 @@ void configure_gpio(u8 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting) | GPIO_PULL_UP_ENABLE | GPIO_PULL_DOWN_ENABLE)); gpio_write_wrapper(gpio, bdata); - iomux_write8(iomux_gpio, iomux_ftn & 0x3); + iomux_write8(gpio, iomux_ftn & 0x3); } u8 read_gpio(u32 gpio) diff --git a/src/mainboard/pcengines/apu2/gpio_ftns.h b/src/mainboard/pcengines/apu2/gpio_ftns.h index 7cc03e832b..d1e76de029 100644 --- a/src/mainboard/pcengines/apu2/gpio_ftns.h +++ b/src/mainboard/pcengines/apu2/gpio_ftns.h @@ -3,48 +3,30 @@ #ifndef GPIO_FTNS_H #define GPIO_FTNS_H -void configure_gpio(u8 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting); +void configure_gpio(u32 gpio, u8 iomux_ftn, u32 setting); u8 read_gpio(u32 gpio); void write_gpio(u32 gpio, u8 value); int get_spd_offset(void); -#define IOMUX_OFFSET 0xD00 -#define GPIO_OFFSET 0x1500 - // // Based on PC Engines APU2C and APU3A schematics // http://www.pcengines.ch/schema/apu2c.pdf // http://www.pcengines.ch/schema/apu3a.pdf // -#define IOMUX_GPIO_22 0x09 // MODESW (APU5) -#define IOMUX_GPIO_32 0x59 // MODESW (SIMSWAP2 on APU5) -#define IOMUX_GPIO_33 0x5A // SIMSWAP (SIMSWAP3 on APU5) -#define IOMUX_GPIO_49 0x40 // STRAP0 -#define IOMUX_GPIO_50 0x41 // STRAP1 -#define IOMUX_GPIO_51 0x42 // PE3 Reset (SIM1 Reset on APU5) -#define IOMUX_GPIO_55 0x43 // PE4 Reset (SIM2 Reset on APU5) -#define IOMUX_GPIO_57 0x44 // LED1# -#define IOMUX_GPIO_58 0x45 // LED2# -#define IOMUX_GPIO_59 0x46 // LED3# -#define IOMUX_GPIO_64 0x47 // PE3_WDIS (SIM3 Reset on APU5) -#define IOMUX_GPIO_66 0x5B // SPKR -#define IOMUX_GPIO_68 0x48 // PE4_WDIS (SIMSWAP1 on APU5) -#define IOMUX_GPIO_71 0x4D // PROCHOT - -#define GPIO_22 0x24 // MODESW (APU5) -#define GPIO_32 0x164 // MODESW (SIMSWAP2 on APU5) -#define GPIO_33 0x168 // SIMSWAP (SIMSWAP3 on APU5) -#define GPIO_49 0x100 // STRAP0 -#define GPIO_50 0x104 // STRAP1 -#define GPIO_51 0x108 // PE3 Reset (SIM1 Reset on APU5) -#define GPIO_55 0x10C // PE4 Reset (SIM2 Reset on APU5) -#define GPIO_57 0x110 // LED1# -#define GPIO_58 0x114 // LED2# -#define GPIO_59 0x118 // LED3# -#define GPIO_64 0x11C // PE3_WDIS (SIM3 Reset on APU5) -#define GPIO_66 0x16C // SPKR -#define GPIO_68 0x120 // PE4_WDIS (SIMSWAP1 on APU5) -#define GPIO_71 0x134 // PROCHOT +#define GPIO_22 0x09 // MODESW (APU5) +#define GPIO_32 0x59 // MODESW (SIMSWAP2 on APU5) +#define GPIO_33 0x5A // SIMSWAP (SIMSWAP3 on APU5) +#define GPIO_49 0x40 // STRAP0 +#define GPIO_50 0x41 // STRAP1 +#define GPIO_51 0x42 // PE3 Reset (SIM1 Reset on APU5) +#define GPIO_55 0x43 // PE4 Reset (SIM2 Reset on APU5) +#define GPIO_57 0x44 // LED1# +#define GPIO_58 0x45 // LED2# +#define GPIO_59 0x46 // LED3# +#define GPIO_64 0x47 // PE3_WDIS (SIM3 Reset on APU5) +#define GPIO_66 0x5B // SPKR +#define GPIO_68 0x48 // PE4_WDIS (SIMSWAP1 on APU5) +#define GPIO_71 0x4D // PROCHOT #define GPIO_OUTPUT_ENABLE BIT23 #define GPIO_OUTPUT_VALUE BIT22 diff --git a/src/mainboard/pcengines/apu2/romstage.c b/src/mainboard/pcengines/apu2/romstage.c index d098bee700..0fc472ff4b 100644 --- a/src/mainboard/pcengines/apu2/romstage.c +++ b/src/mainboard/pcengines/apu2/romstage.c @@ -36,18 +36,18 @@ static void early_lpc_init(void) // Configure output disabled, value low, pull up/down disabled // if (CONFIG(BOARD_PCENGINES_APU5)) { - configure_gpio(IOMUX_GPIO_22, Function0, GPIO_22, setting); + configure_gpio(GPIO_22, Function0, setting); } if (CONFIG(BOARD_PCENGINES_APU2) || CONFIG(BOARD_PCENGINES_APU3) || CONFIG(BOARD_PCENGINES_APU4)) { - configure_gpio(IOMUX_GPIO_32, Function0, GPIO_32, setting); + configure_gpio(GPIO_32, Function0, setting); } - configure_gpio(IOMUX_GPIO_49, Function2, GPIO_49, setting); - configure_gpio(IOMUX_GPIO_50, Function2, GPIO_50, setting); - configure_gpio(IOMUX_GPIO_71, Function0, GPIO_71, setting); + configure_gpio(GPIO_49, Function2, setting); + configure_gpio(GPIO_50, Function2, setting); + configure_gpio(GPIO_71, Function0, setting); // // Configure output enabled, value low, pull up/down disabled @@ -55,12 +55,12 @@ static void early_lpc_init(void) setting = GPIO_OUTPUT_ENABLE; if (CONFIG(BOARD_PCENGINES_APU3) || CONFIG(BOARD_PCENGINES_APU4)) { - configure_gpio(IOMUX_GPIO_33, Function0, GPIO_33, setting); + configure_gpio(GPIO_33, Function0, setting); } - configure_gpio(IOMUX_GPIO_57, Function1, GPIO_57, setting); - configure_gpio(IOMUX_GPIO_58, Function1, GPIO_58, setting); - configure_gpio(IOMUX_GPIO_59, Function3, GPIO_59, setting); + configure_gpio(GPIO_57, Function1, setting); + configure_gpio(GPIO_58, Function1, setting); + configure_gpio(GPIO_59, Function3, setting); // // Configure output enabled, value high, pull up/down disabled @@ -68,12 +68,12 @@ static void early_lpc_init(void) setting = GPIO_OUTPUT_ENABLE | GPIO_OUTPUT_VALUE; if (CONFIG(BOARD_PCENGINES_APU5)) { - configure_gpio(IOMUX_GPIO_32, Function0, GPIO_32, setting); - configure_gpio(IOMUX_GPIO_33, Function0, GPIO_33, setting); + configure_gpio(GPIO_32, Function0, setting); + configure_gpio(GPIO_33, Function0, setting); } - configure_gpio(IOMUX_GPIO_51, Function2, GPIO_51, setting); - configure_gpio(IOMUX_GPIO_55, Function3, GPIO_55, setting); - configure_gpio(IOMUX_GPIO_64, Function2, GPIO_64, setting); - configure_gpio(IOMUX_GPIO_68, Function0, GPIO_68, setting); + configure_gpio(GPIO_51, Function2, setting); + configure_gpio(GPIO_55, Function3, setting); + configure_gpio(GPIO_64, Function2, setting); + configure_gpio(GPIO_68, Function0, setting); } -- cgit v1.2.3