rockchip: refactor gpio driver

The gpio of rockchip SoCs(rk3288 & rk3399) are the same IP,
moving the gpio code of rk3288 to common then can be reused on rk3399.

BRANCH=none
BUG=chrome-os-partner:51537
TEST=build and boot into chromeos on veyron_jerry

Change-Id: I10a4b9d32afe60fd52512f2ad0007e9d2785033b
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: 1c0c4b4b999790b0be7b0eeb70d2a7a86158f779
Original-Change-Id: If13b7760108831d81e8e8c950cdf61724d497b17
Original-Signed-off-by: Shunqian Zheng <zhengsq@rock-chips.com>
Original-Reviewed-on: https://chromium-review.googlesource.com/339846
Original-Commit-Ready: Vadim Bendebury <vbendeb@chromium.org>
Original-Tested-by: Vadim Bendebury <vbendeb@chromium.org>
Original-Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://review.coreboot.org/14712
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>

3 files changed, 23 insertions, 114 deletions

diff --git a/src/soc/rockchip/rk3288/Makefile.inc b/src/soc/rockchip/rk3288/Makefile.inc
index 553bd00ba5..1eab8f2c52 100644
--- a/src/soc/rockchip/rk3288/Makefile.inc
+++ b/src/soc/rockchip/rk3288/Makefile.inc
@@ -25,6 +25,7 @@ endif
 bootblock-y += timer.c
 bootblock-y += clock.c
 bootblock-y += ../common/spi.c
+bootblock-y += ../common/gpio.c
 bootblock-y += gpio.c
 bootblock-y += ../common/i2c.c
 bootblock-$(CONFIG_SOFTWARE_I2C) += software_i2c.c
@@ -33,6 +34,7 @@ bootblock-y += ../common/rk808.c
 verstage-y += ../common/spi.c
 verstage-y += timer.c
 verstage-$(CONFIG_DRIVERS_UART) += ../common/uart.c
+verstage-y += ../common/gpio.c
 verstage-y += gpio.c
 verstage-y += clock.c
 libverstage-y += crypto.c
@@ -45,6 +47,7 @@ romstage-$(CONFIG_DRIVERS_UART) += ../common/uart.c
 romstage-y += ../common/i2c.c
 romstage-$(CONFIG_SOFTWARE_I2C) += software_i2c.c
 romstage-y += clock.c
+romstage-y += ../common/gpio.c
 romstage-y += gpio.c
 romstage-y += ../common/spi.c
 romstage-y += sdram.c
@@ -60,6 +63,7 @@ ramstage-$(CONFIG_SOFTWARE_I2C) += software_i2c.c
 ramstage-y += clock.c
 ramstage-y += ../common/spi.c
 ramstage-y += sdram.c
+ramstage-y += ../common/gpio.c
 ramstage-y += gpio.c
 ramstage-y += ../common/rk808.c
 ramstage-y += ../common/pwm.c
diff --git a/src/soc/rockchip/rk3288/gpio.c b/src/soc/rockchip/rk3288/gpio.c
index 35b4b7ee88..8a15b85908 100644
--- a/src/soc/rockchip/rk3288/gpio.c
+++ b/src/soc/rockchip/rk3288/gpio.c
@@ -16,66 +16,37 @@
 #include <arch/io.h>
 #include <console/console.h>
 #include <gpio.h>
+#include <soc/gpio.h>
 #include <soc/grf.h>
 #include <soc/pmu.h>
 #include <soc/soc.h>
 #include <stdlib.h>
 
-struct rk3288_gpio_regs *gpio_port[] = {
-	(struct rk3288_gpio_regs *)0xff750000,
-	(struct rk3288_gpio_regs *)0xff780000,
-	(struct rk3288_gpio_regs *)0xff790000,
-	(struct rk3288_gpio_regs *)0xff7a0000,
-	(struct rk3288_gpio_regs *)0xff7b0000,
-	(struct rk3288_gpio_regs *)0xff7c0000,
-	(struct rk3288_gpio_regs *)0xff7d0000,
-	(struct rk3288_gpio_regs *)0xff7e0000,
-	(struct rk3288_gpio_regs *)0xff7f0000
-};
-
-enum {
-	PULLNONE = 0,
-	PULLUP,
-	PULLDOWN
+struct rockchip_gpio_regs *gpio_port[] = {
+	(struct rockchip_gpio_regs *)0xff750000,
+	(struct rockchip_gpio_regs *)0xff780000,
+	(struct rockchip_gpio_regs *)0xff790000,
+	(struct rockchip_gpio_regs *)0xff7a0000,
+	(struct rockchip_gpio_regs *)0xff7b0000,
+	(struct rockchip_gpio_regs *)0xff7c0000,
+	(struct rockchip_gpio_regs *)0xff7d0000,
+	(struct rockchip_gpio_regs *)0xff7e0000,
+	(struct rockchip_gpio_regs *)0xff7f0000
 };
 
 #define PMU_GPIO_PORT 0
 
-static void __gpio_input(gpio_t gpio, u32 pull)
+int is_pmu_gpio(gpio_t gpio)
 {
-	clrbits_le32(&gpio_port[gpio.port]->swporta_ddr, 1 << gpio.num);
 	if (gpio.port == PMU_GPIO_PORT)
-		clrsetbits_le32(&rk3288_pmu->gpio0pull[gpio.bank],
-				3 << (gpio.idx * 2),  pull << (gpio.idx * 2));
-	else
-		write32(&rk3288_grf->gpio1_p[(gpio.port - 1)][gpio.bank],
-			RK_CLRSETBITS(3 << (gpio.idx * 2),
-				   pull << (gpio.idx * 2)));
-}
-
-void gpio_input(gpio_t gpio)
-{
-	__gpio_input(gpio, PULLNONE);
-}
-
-void gpio_input_pulldown(gpio_t gpio)
-{
-	__gpio_input(gpio, PULLDOWN);
-}
-
-void gpio_input_pullup(gpio_t gpio)
-{
-	__gpio_input(gpio, PULLUP);
-}
-
-int gpio_get(gpio_t gpio)
-{
-	return (read32(&gpio_port[gpio.port]->ext_porta) >> gpio.num) & 0x1;
+		return 1;
+	return 0;
 }
 
-void gpio_output(gpio_t gpio, int value)
+void *gpio_grf_reg(gpio_t gpio)
 {
-	setbits_le32(&gpio_port[gpio.port]->swporta_ddr, 1 << gpio.num);
-	clrsetbits_le32(&gpio_port[gpio.port]->swporta_dr, 1 << gpio.num,
-							   !!value << gpio.num);
+	if (is_pmu_gpio(gpio))
+		return &rk3288_pmu->gpio0pull[gpio.bank];
+	/* There is one pmu gpio, gpio0 , so " - 1" */
+	return &rk3288_grf->gpio1_p[(gpio.port - 1)][gpio.bank];
 }
diff --git a/src/soc/rockchip/rk3288/include/soc/gpio.h b/src/soc/rockchip/rk3288/include/soc/gpio.h
deleted file mode 100644
index 05b30b9627..0000000000
--- a/src/soc/rockchip/rk3288/include/soc/gpio.h
+++ /dev/null
@@ -1,66 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright 2014 Rockchip Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#ifndef __SOC_ROCKCHIP_RK3288_GPIO_H__
-#define __SOC_ROCKCHIP_RK3288_GPIO_H__
-
-#include <types.h>
-
-#define GPIO(p, b, i) ((gpio_t){.port = p, .bank = GPIO_##b, .idx = i})
-
-struct rk3288_gpio_regs {
-	u32 swporta_dr;
-	u32 swporta_ddr;
-	u32 reserved0[(0x30 - 0x08) / 4];
-	u32 inten;
-	u32 intmask;
-	u32 inttype_level;
-	u32 int_polarity;
-	u32 int_status;
-	u32 int_rawstatus;
-	u32 debounce;
-	u32 porta_eoi;
-	u32 ext_porta;
-	u32 reserved1[(0x60 - 0x54) / 4];
-	u32 ls_sync;
-};
-check_member(rk3288_gpio_regs, ls_sync, 0x60);
-
-typedef union {
-	u32 raw;
-	struct {
-		u16 port;
-		union {
-			struct {
-				u16 num:5;
-				u16 :11;
-			};
-			struct {
-				u16 idx:3;
-				u16 bank:2;
-				u16 :11;
-			};
-		};
-	};
-} gpio_t;
-
-enum {
-	GPIO_A = 0,
-	GPIO_B,
-	GPIO_C,
-	GPIO_D,
-};
-
-#endif	/* __SOC_ROCKCHIP_RK3288_GPIO_H__ */