mb/*/*: use ACPIMMIO common block wherever possible

TEST=boot PC Engines apu2 and launch Debian Linux

Signed-off-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Change-Id: I648167ec94367c9494c4253bec21dab20ad7b615
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37401
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>

2 files changed, 39 insertions, 19 deletions

diff --git a/src/mainboard/pcengines/apu2/gpio_ftns.c b/src/mainboard/pcengines/apu2/gpio_ftns.c
index 249ecc3494..170acca8d3 100644
--- a/src/mainboard/pcengines/apu2/gpio_ftns.c
+++ b/src/mainboard/pcengines/apu2/gpio_ftns.c
@@ -14,55 +14,75 @@
  */
 
 #include <stdint.h>
+#include <amdblocks/acpimmio.h>
+#include <console/console.h>
 #include <device/mmio.h>
 #include <FchPlatform.h>
 #include "gpio_ftns.h"
 
-void configure_gpio(u32 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting)
+static u32 gpio_read_wrapper(u32 gpio)
+{
+	if (gpio < 0x100)
+		return gpio0_read32(gpio & 0xff);
+	else if (gpio >= 0x100 && gpio < 0x200)
+		return gpio1_read32(gpio & 0xff);
+	else if (gpio >= 0x200 && gpio < 0x300)
+		return gpio2_read32(gpio & 0xff);
+
+	die("Invalid GPIO");
+}
+
+static void gpio_write_wrapper(u32 gpio, u32 setting)
+{
+	if (gpio < 0x100)
+		gpio0_write32(gpio & 0xff, setting);
+	else if (gpio >= 0x100 && gpio < 0x200)
+		gpio1_write32(gpio & 0xff, setting);
+	else if (gpio >= 0x200 && gpio < 0x300)
+		gpio2_write32(gpio & 0xff, setting);
+}
+
+void configure_gpio(u8 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting)
 {
 	u32 bdata;
 
-	bdata = read32((const volatile void *)(ACPI_MMIO_BASE + GPIO_OFFSET
-		+ gpio));
+	bdata =  gpio_read_wrapper(gpio);
 	/* out the data value to prevent glitches */
 	bdata |= (setting & GPIO_OUTPUT_ENABLE);
-	write32((volatile void *)(ACPI_MMIO_BASE + GPIO_OFFSET + gpio), bdata);
+	gpio_write_wrapper(gpio, bdata);
 
 	/* set direction and data value */
 	bdata |= (setting & (GPIO_OUTPUT_ENABLE | GPIO_OUTPUT_VALUE
 			| GPIO_PULL_UP_ENABLE | GPIO_PULL_DOWN_ENABLE));
-	write32((volatile void *)(ACPI_MMIO_BASE + GPIO_OFFSET + gpio), bdata);
+	gpio_write_wrapper(gpio, bdata);
 
-	write8((volatile void *)(ACPI_MMIO_BASE + IOMUX_OFFSET + iomux_gpio),
-			iomux_ftn & 0x3);
+	iomux_write8(iomux_gpio, iomux_ftn & 0x3);
 }
 
 u8 read_gpio(u32 gpio)
 {
-	u32 status = read32((const volatile void *)(ACPI_MMIO_BASE + GPIO_OFFSET
-			+ gpio));
-
-	return (status & GPIO_PIN_STS) ? 1 : 0;
+	return (gpio_read_wrapper(gpio) & GPIO_PIN_STS) ? 1 : 0;
 }
 
 void write_gpio(u32 gpio, u8 value)
 {
-	u32 status = read32((const volatile void *)(ACPI_MMIO_BASE + GPIO_OFFSET
-			+ gpio));
+	u32 status = gpio_read_wrapper(gpio);
 	status &= ~GPIO_OUTPUT_VALUE;
 	status |= (value > 0) ? GPIO_OUTPUT_VALUE : 0;
-	write32((volatile void *)(ACPI_MMIO_BASE + GPIO_OFFSET + gpio), status);
+	gpio_write_wrapper(gpio, status);
 }
 
 int get_spd_offset(void)
 {
 	u8 index = 0;
-	/* One SPD file contains all 4 options, determine which index to
+	/*
+	 * One SPD file contains all 4 options, determine which index to
 	 * read here, then call into the standard routines.
 	 */
-	u8 *gpio_bank0_ptr = (u8 *)(ACPI_MMIO_BASE + GPIO_BANK0_BASE);
-	if (*(gpio_bank0_ptr + (0x40 << 2) + 2) & BIT0) index |= BIT0;
-	if (*(gpio_bank0_ptr + (0x41 << 2) + 2) & BIT0) index |= BIT1;
+	if (gpio1_read8(0x02) & BIT0)
+		index |= BIT0;
+	if (gpio1_read8(0x06) & BIT0)
+		index |= BIT1;
 
 	return index;
 }
diff --git a/src/mainboard/pcengines/apu2/gpio_ftns.h b/src/mainboard/pcengines/apu2/gpio_ftns.h
index 24d6a7f772..49169be121 100644
--- a/src/mainboard/pcengines/apu2/gpio_ftns.h
+++ b/src/mainboard/pcengines/apu2/gpio_ftns.h
@@ -16,7 +16,7 @@
 #ifndef GPIO_FTNS_H
 #define GPIO_FTNS_H
 
-void configure_gpio(u32 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting);
+void configure_gpio(u8 iomux_gpio, u8 iomux_ftn, u32 gpio, u32 setting);
 u8 read_gpio(u32 gpio);
 void write_gpio(u32 gpio, u8 value);
 int get_spd_offset(void);