summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2020-06-01 09:37:05 +0300
committerPatrick Georgi <pgeorgi@google.com>2020-06-07 21:49:12 +0000
commit7051a40b0be77c26dfeb929b059a24ff8a36bc78 (patch)
tree34d05538d442c1b10368126130a414271eb75604
parentda6fcb13f1d8169ae6697d592c34cf4c19a2a0e2 (diff)
downloadcoreboot-7051a40b0be77c26dfeb929b059a24ff8a36bc78.tar.xz
mb/pcengines/apu2: Change GPIO configuration functions
The definitions of GPIO_xx equal IOMUX_GPIO_xx shifted by two. Change-Id: I0ee821c71c88bf535122a9526862a9d1e68bd755 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/42038 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Michał Żygowski <michal.zygowski@3mdeb.com>
-rw-r--r--src/mainboard/pcengines/apu2/gpio_ftns.c12
-rw-r--r--src/mainboard/pcengines/apu2/gpio_ftns.h48
-rw-r--r--src/mainboard/pcengines/apu2/romstage.c30
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 <FchPlatform.h>
#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);
}