* [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver
@ 2017-11-03 7:16 Kever Yang
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
` (4 more replies)
0 siblings, 5 replies; 10+ messages in thread
From: Kever Yang @ 2017-11-03 7:16 UTC (permalink / raw)
To: u-boot
Use a common driver for all Rockchip SOC instead of one for each SoC.
Use driver_data for reg offset.
Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
---
Changes in v3: None
Changes in v2:
- use udev priv member instead of driver_data
arch/arm/include/asm/arch-rockchip/clock.h | 5 +++
drivers/sysreset/Makefile | 12 +-----
drivers/sysreset/sysreset_rk3036.c | 45 ----------------------
drivers/sysreset/sysreset_rk3128.c | 45 ----------------------
drivers/sysreset/sysreset_rk3188.c | 62 ------------------------------
drivers/sysreset/sysreset_rk322x.c | 45 ----------------------
drivers/sysreset/sysreset_rk3288.c | 47 ----------------------
drivers/sysreset/sysreset_rk3328.c | 45 ----------------------
drivers/sysreset/sysreset_rk3368.c | 62 ------------------------------
drivers/sysreset/sysreset_rk3399.c | 45 ----------------------
drivers/sysreset/sysreset_rockchip.c | 47 ++++++++++++++++++++++
drivers/sysreset/sysreset_rv1108.c | 46 ----------------------
12 files changed, 53 insertions(+), 453 deletions(-)
delete mode 100644 drivers/sysreset/sysreset_rk3036.c
delete mode 100644 drivers/sysreset/sysreset_rk3128.c
delete mode 100644 drivers/sysreset/sysreset_rk3188.c
delete mode 100644 drivers/sysreset/sysreset_rk322x.c
delete mode 100644 drivers/sysreset/sysreset_rk3288.c
delete mode 100644 drivers/sysreset/sysreset_rk3328.c
delete mode 100644 drivers/sysreset/sysreset_rk3368.c
delete mode 100644 drivers/sysreset/sysreset_rk3399.c
create mode 100644 drivers/sysreset/sysreset_rockchip.c
delete mode 100644 drivers/sysreset/sysreset_rv1108.c
diff --git a/arch/arm/include/asm/arch-rockchip/clock.h b/arch/arm/include/asm/arch-rockchip/clock.h
index 641df58..736b260 100644
--- a/arch/arm/include/asm/arch-rockchip/clock.h
+++ b/arch/arm/include/asm/arch-rockchip/clock.h
@@ -39,6 +39,11 @@ static inline int rk_pll_id(enum rk_clk_id clk_id)
return clk_id - 1;
}
+struct sysreset_reg {
+ unsigned int glb_srst_fst_value;
+ unsigned int glb_srst_snd_value;
+};
+
/**
* clk_get_divisor() - Calculate the required clock divisior
*
diff --git a/drivers/sysreset/Makefile b/drivers/sysreset/Makefile
index 6f0dfda..20f371b 100644
--- a/drivers/sysreset/Makefile
+++ b/drivers/sysreset/Makefile
@@ -9,19 +9,9 @@ obj-$(CONFIG_SYSRESET_PSCI) += sysreset_psci.o
obj-$(CONFIG_SYSRESET_SYSCON) += sysreset_syscon.o
obj-$(CONFIG_SYSRESET_WATCHDOG) += sysreset_watchdog.o
-ifndef CONFIG_SPL_BUILD
-obj-$(CONFIG_ROCKCHIP_RK3036) += sysreset_rk3036.o
-endif
-obj-$(CONFIG_ROCKCHIP_RK3128) += sysreset_rk3128.o
-obj-$(CONFIG_ROCKCHIP_RK3188) += sysreset_rk3188.o
-obj-$(CONFIG_ROCKCHIP_RK322X) += sysreset_rk322x.o
-obj-$(CONFIG_ROCKCHIP_RK3288) += sysreset_rk3288.o
-obj-$(CONFIG_ROCKCHIP_RK3328) += sysreset_rk3328.o
-obj-$(CONFIG_ROCKCHIP_RK3368) += sysreset_rk3368.o
-obj-$(CONFIG_ROCKCHIP_RK3399) += sysreset_rk3399.o
-obj-$(CONFIG_ROCKCHIP_RV1108) += sysreset_rv1108.o
obj-$(CONFIG_SANDBOX) += sysreset_sandbox.o
obj-$(CONFIG_ARCH_SNAPDRAGON) += sysreset_snapdragon.o
obj-$(CONFIG_ARCH_STI) += sysreset_sti.o
obj-$(CONFIG_TARGET_XTFPGA) += sysreset_xtfpga.o
obj-$(CONFIG_ARCH_ASPEED) += sysreset_ast.o
+obj-$(CONFIG_ARCH_ROCKCHIP) += sysreset_rockchip.o
diff --git a/drivers/sysreset/sysreset_rk3036.c b/drivers/sysreset/sysreset_rk3036.c
deleted file mode 100644
index b3d2113..0000000
--- a/drivers/sysreset/sysreset_rk3036.c
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * (C) Copyright 2015 Rockchip Electronics Co., Ltd
- *
- * SPDX-License-Identifier: GPL-2.0+
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3036.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rk3036_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3036_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- writel(0xeca8, &cru->cru_glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- writel(0xfdb9, &cru->cru_glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3036_sysreset = {
- .request = rk3036_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3036) = {
- .name = "rk3036_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3036_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk3128.c b/drivers/sysreset/sysreset_rk3128.c
deleted file mode 100644
index 5aab8ec..0000000
--- a/drivers/sysreset/sysreset_rk3128.c
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * (C) Copyright 2017 Rockchip Electronics Co., Ltd
- *
- * SPDX-License-Identifier: GPL-2.0+
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3128.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rk3128_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3128_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- writel(0xeca8, &cru->cru_glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- writel(0xfdb9, &cru->cru_glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3128_sysreset = {
- .request = rk3128_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3128) = {
- .name = "rk3128_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3128_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk3188.c b/drivers/sysreset/sysreset_rk3188.c
deleted file mode 100644
index 053a634..0000000
--- a/drivers/sysreset/sysreset_rk3188.c
+++ /dev/null
@@ -1,62 +0,0 @@
-/*
- * (C) Copyright 2015 Google, Inc
- *
- * SPDX-License-Identifier: GPL-2.0
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <syscon.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3188.h>
-#include <asm/arch/grf_rk3188.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rk3188_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3188_cru *cru = rockchip_get_cru();
- struct rk3188_grf *grf;
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
- if (IS_ERR(grf))
- return -EPROTONOSUPPORT;
-
- /*
- * warm-reset keeps the remap value,
- * so make sure it's disabled.
- */
- rk_clrsetreg(&grf->soc_con0,
- NOC_REMAP_MASK << NOC_REMAP_SHIFT,
- 0 << NOC_REMAP_SHIFT);
-
- rk_clrreg(&cru->cru_mode_con, 0xffff);
- writel(0xeca8, &cru->cru_glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- rk_clrreg(&cru->cru_mode_con, 0xffff);
- writel(0xfdb9, &cru->cru_glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3188_sysreset = {
- .request = rk3188_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3188) = {
- .name = "rk3188_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3188_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk322x.c b/drivers/sysreset/sysreset_rk322x.c
deleted file mode 100644
index 5fce79b..0000000
--- a/drivers/sysreset/sysreset_rk322x.c
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * (C) Copyright 2017 Rockchip Electronics Co., Ltd
- *
- * SPDX-License-Identifier: GPL-2.0+
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk322x.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rk322x_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk322x_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- writel(0xeca8, &cru->cru_glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- writel(0xfdb9, &cru->cru_glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk322x_sysreset = {
- .request = rk322x_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk322x) = {
- .name = "rk322x_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk322x_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk3288.c b/drivers/sysreset/sysreset_rk3288.c
deleted file mode 100644
index 0aad1c2..0000000
--- a/drivers/sysreset/sysreset_rk3288.c
+++ /dev/null
@@ -1,47 +0,0 @@
-/*
- * (C) Copyright 2015 Google, Inc
- *
- * SPDX-License-Identifier: GPL-2.0
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3288.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rk3288_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3288_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- rk_clrreg(&cru->cru_mode_con, 0xffff);
- writel(0xeca8, &cru->cru_glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- rk_clrreg(&cru->cru_mode_con, 0xffff);
- writel(0xfdb9, &cru->cru_glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3288_sysreset = {
- .request = rk3288_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3288) = {
- .name = "rk3288_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3288_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk3328.c b/drivers/sysreset/sysreset_rk3328.c
deleted file mode 100644
index 7b9af09..0000000
--- a/drivers/sysreset/sysreset_rk3328.c
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * (C) Copyright 2016 Rockchip Electronics Co., Ltd
- *
- * SPDX-License-Identifier: GPL-2.0
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3328.h>
-#include <asm/arch/hardware.h>
-#include <asm/io.h>
-#include <linux/err.h>
-
-int rk3328_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3328_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- writel(0xeca8, &cru->glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- writel(0xfdb9, &cru->glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3328_sysreset = {
- .request = rk3328_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3328) = {
- .name = "rk3328_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3328_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk3368.c b/drivers/sysreset/sysreset_rk3368.c
deleted file mode 100644
index de62921..0000000
--- a/drivers/sysreset/sysreset_rk3368.c
+++ /dev/null
@@ -1,62 +0,0 @@
-/*
- * (C) Copyright Rockchip Electronics Co., Ltd
- *
- * SPDX-License-Identifier: GPL-2.0
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3368.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-static void rk3368_pll_enter_slow_mode(struct rk3368_cru *cru)
-{
- struct rk3368_pll *pll;
- int i;
-
- for (i = 0; i < 6; i++) {
- pll = &cru->pll[i];
- rk_clrreg(&pll->con3, PLL_MODE_MASK);
- }
-}
-
-static int rk3368_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3368_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- rk3368_pll_enter_slow_mode(cru);
- rk_clrsetreg(&cru->glb_rst_con, PMU_GLB_SRST_CTRL_MASK,
- PMU_RST_BY_SND_GLB_SRST << PMU_GLB_SRST_CTRL_SHIFT);
- writel(0xeca8, &cru->glb_srst_snd_val);
- break;
- case SYSRESET_COLD:
- rk3368_pll_enter_slow_mode(cru);
- rk_clrsetreg(&cru->glb_rst_con, PMU_GLB_SRST_CTRL_MASK,
- PMU_RST_BY_FST_GLB_SRST << PMU_GLB_SRST_CTRL_SHIFT);
- writel(0xfdb9, &cru->glb_srst_fst_val);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3368_sysreset = {
- .request = rk3368_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3368) = {
- .name = "rk3368_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3368_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rk3399.c b/drivers/sysreset/sysreset_rk3399.c
deleted file mode 100644
index 9a55546..0000000
--- a/drivers/sysreset/sysreset_rk3399.c
+++ /dev/null
@@ -1,45 +0,0 @@
-/*
- * (C) Copyright 2016 Rockchip Electronics Co., Ltd
- *
- * SPDX-License-Identifier: GPL-2.0
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rk3399.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rk3399_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rk3399_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
- switch (type) {
- case SYSRESET_WARM:
- writel(0xeca8, &cru->glb_srst_snd_value);
- break;
- case SYSRESET_COLD:
- writel(0xfdb9, &cru->glb_srst_fst_value);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rk3399_sysreset = {
- .request = rk3399_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rk3399) = {
- .name = "rk3399_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rk3399_sysreset,
-};
diff --git a/drivers/sysreset/sysreset_rockchip.c b/drivers/sysreset/sysreset_rockchip.c
new file mode 100644
index 0000000..894975f
--- /dev/null
+++ b/drivers/sysreset/sysreset_rockchip.c
@@ -0,0 +1,47 @@
+/*
+ * (C) Copyright 2017 Rockchip Electronics Co., Ltd
+ *
+ * SPDX-License-Identifier: GPL-2.0
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <errno.h>
+#include <sysreset.h>
+#include <asm/io.h>
+#include <asm/arch/clock.h>
+#include <asm/arch/cru_rk3328.h>
+#include <asm/arch/hardware.h>
+#include <linux/err.h>
+
+int rockchip_sysreset_request(struct udevice *dev, enum sysreset_t type)
+{
+ struct sysreset_reg *offset = dev_get_priv(dev);
+ unsigned long cru_base = (unsigned long)rockchip_get_cru();
+
+ if (IS_ERR_VALUE(cru_base))
+ return (int)cru_base;
+
+ switch (type) {
+ case SYSRESET_WARM:
+ writel(0xeca8, cru_base + offset->glb_srst_snd_value);
+ break;
+ case SYSRESET_COLD:
+ writel(0xfdb9, cru_base + offset->glb_srst_fst_value);
+ break;
+ default:
+ return -EPROTONOSUPPORT;
+ }
+
+ return -EINPROGRESS;
+}
+
+static struct sysreset_ops rockchip_sysreset = {
+ .request = rockchip_sysreset_request,
+};
+
+U_BOOT_DRIVER(sysreset_rockchip) = {
+ .name = "rockchip_sysreset",
+ .id = UCLASS_SYSRESET,
+ .ops = &rockchip_sysreset,
+};
diff --git a/drivers/sysreset/sysreset_rv1108.c b/drivers/sysreset/sysreset_rv1108.c
deleted file mode 100644
index 9d8e9f7..0000000
--- a/drivers/sysreset/sysreset_rv1108.c
+++ /dev/null
@@ -1,46 +0,0 @@
-/*
- * (C) Copyright 2015 Rockchip Electronics Co., Ltd
- * Author: Andy Yan <andy.yan@rock-chips.com>
- * SPDX-License-Identifier: GPL-2.0+
- */
-
-#include <common.h>
-#include <dm.h>
-#include <errno.h>
-#include <sysreset.h>
-#include <asm/io.h>
-#include <asm/arch/clock.h>
-#include <asm/arch/cru_rv1108.h>
-#include <asm/arch/hardware.h>
-#include <linux/err.h>
-
-int rv1108_sysreset_request(struct udevice *dev, enum sysreset_t type)
-{
- struct rv1108_cru *cru = rockchip_get_cru();
-
- if (IS_ERR(cru))
- return PTR_ERR(cru);
-
- switch (type) {
- case SYSRESET_WARM:
- writel(0xeca8, &cru->glb_srst_snd_val);
- break;
- case SYSRESET_COLD:
- writel(0xfdb9, &cru->glb_srst_fst_val);
- break;
- default:
- return -EPROTONOSUPPORT;
- }
-
- return -EINPROGRESS;
-}
-
-static struct sysreset_ops rv1108_sysreset = {
- .request = rv1108_sysreset_request,
-};
-
-U_BOOT_DRIVER(sysreset_rv1108) = {
- .name = "rv1108_sysreset",
- .id = UCLASS_SYSRESET,
- .ops = &rv1108_sysreset,
-};
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding
2017-11-03 7:16 [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver Kever Yang
@ 2017-11-03 7:16 ` Kever Yang
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, " Philipp Tomsich
` (3 more replies)
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver Philipp Tomsich
` (3 subsequent siblings)
4 siblings, 4 replies; 10+ messages in thread
From: Kever Yang @ 2017-11-03 7:16 UTC (permalink / raw)
To: u-boot
Using priv for new sysreset driver binding.
Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
---
Changes in v3:
- do not init priv if driver bind fail
Changes in v2:
- use udev priv member instead of driver_data
drivers/clk/rockchip/clk_rk3036.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk3128.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk3188.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk322x.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk3288.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk3328.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk3368.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rk3399.c | 17 ++++++++++++++---
drivers/clk/rockchip/clk_rv1108.c | 17 ++++++++++++++---
9 files changed, 126 insertions(+), 27 deletions(-)
diff --git a/drivers/clk/rockchip/clk_rk3036.c b/drivers/clk/rockchip/clk_rk3036.c
index 83f4ae6..280ebb9 100644
--- a/drivers/clk/rockchip/clk_rk3036.c
+++ b/drivers/clk/rockchip/clk_rk3036.c
@@ -330,11 +330,22 @@ static int rk3036_clk_probe(struct udevice *dev)
static int rk3036_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3036_sysreset", "reset", &dev);
- if (ret)
- debug("Warning: No RK3036 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3036_cru,
+ cru_glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk3036_cru,
+ cru_glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return 0;
}
diff --git a/drivers/clk/rockchip/clk_rk3128.c b/drivers/clk/rockchip/clk_rk3128.c
index ace154f..c75cbf8 100644
--- a/drivers/clk/rockchip/clk_rk3128.c
+++ b/drivers/clk/rockchip/clk_rk3128.c
@@ -325,11 +325,22 @@ static int rk3128_clk_probe(struct udevice *dev)
static int rk3128_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3128_sysreset", "reset", &dev);
- if (ret)
- debug("Warning: No RK3128 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3128_cru,
+ cru_glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk3128_cru,
+ cru_glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return 0;
}
diff --git a/drivers/clk/rockchip/clk_rk3188.c b/drivers/clk/rockchip/clk_rk3188.c
index 8c2c9bc..fca6899 100644
--- a/drivers/clk/rockchip/clk_rk3188.c
+++ b/drivers/clk/rockchip/clk_rk3188.c
@@ -573,11 +573,22 @@ static int rk3188_clk_probe(struct udevice *dev)
static int rk3188_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3188_sysreset", "reset", &dev);
- if (ret)
- debug("Warning: No rk3188 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3188_cru,
+ cru_glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk3188_cru,
+ cru_glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return 0;
}
diff --git a/drivers/clk/rockchip/clk_rk322x.c b/drivers/clk/rockchip/clk_rk322x.c
index e87267d..ff52b55 100644
--- a/drivers/clk/rockchip/clk_rk322x.c
+++ b/drivers/clk/rockchip/clk_rk322x.c
@@ -385,11 +385,22 @@ static int rk322x_clk_probe(struct udevice *dev)
static int rk322x_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk322x_sysreset", "reset", &dev);
- if (ret)
- debug("Warning: No RK322x reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk322x_cru,
+ cru_glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk322x_cru,
+ cru_glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return 0;
}
diff --git a/drivers/clk/rockchip/clk_rk3288.c b/drivers/clk/rockchip/clk_rk3288.c
index a133810..ac53239 100644
--- a/drivers/clk/rockchip/clk_rk3288.c
+++ b/drivers/clk/rockchip/clk_rk3288.c
@@ -859,11 +859,22 @@ static int rk3288_clk_probe(struct udevice *dev)
static int rk3288_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3288_sysreset", "reset", &dev);
- if (ret)
- debug("Warning: No RK3288 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3288_cru,
+ cru_glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk3288_cru,
+ cru_glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return 0;
}
diff --git a/drivers/clk/rockchip/clk_rk3328.c b/drivers/clk/rockchip/clk_rk3328.c
index 540d910..4d522a7 100644
--- a/drivers/clk/rockchip/clk_rk3328.c
+++ b/drivers/clk/rockchip/clk_rk3328.c
@@ -597,11 +597,22 @@ static int rk3328_clk_ofdata_to_platdata(struct udevice *dev)
static int rk3328_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3328_sysreset", "reset", &dev);
- if (ret)
- printf("Warning: No RK3328 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3328_cru,
+ glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk3328_cru,
+ glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return ret;
}
diff --git a/drivers/clk/rockchip/clk_rk3368.c b/drivers/clk/rockchip/clk_rk3368.c
index 3661769..bfeef39 100644
--- a/drivers/clk/rockchip/clk_rk3368.c
+++ b/drivers/clk/rockchip/clk_rk3368.c
@@ -526,11 +526,22 @@ static int rk3368_clk_ofdata_to_platdata(struct udevice *dev)
static int rk3368_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3368_sysreset", "reset", &dev);
- if (ret)
- pr_err("bind RK3368 reset driver failed: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3368_cru,
+ glb_srst_fst_val);
+ priv->glb_srst_snd_value = offsetof(struct rk3368_cru,
+ glb_srst_snd_val);
+ sys_child->priv = priv;
+ }
return ret;
}
diff --git a/drivers/clk/rockchip/clk_rk3399.c b/drivers/clk/rockchip/clk_rk3399.c
index 6f85a38..e21d056 100644
--- a/drivers/clk/rockchip/clk_rk3399.c
+++ b/drivers/clk/rockchip/clk_rk3399.c
@@ -1033,11 +1033,22 @@ static int rk3399_clk_ofdata_to_platdata(struct udevice *dev)
static int rk3399_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rk3399_sysreset", "reset", &dev);
- if (ret)
- printf("Warning: No RK3399 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rk3399_cru,
+ glb_srst_fst_value);
+ priv->glb_srst_snd_value = offsetof(struct rk3399_cru,
+ glb_srst_snd_value);
+ sys_child->priv = priv;
+ }
return 0;
}
diff --git a/drivers/clk/rockchip/clk_rv1108.c b/drivers/clk/rockchip/clk_rv1108.c
index 55741c3..a119548 100644
--- a/drivers/clk/rockchip/clk_rv1108.c
+++ b/drivers/clk/rockchip/clk_rv1108.c
@@ -223,11 +223,22 @@ static int rv1108_clk_probe(struct udevice *dev)
static int rv1108_clk_bind(struct udevice *dev)
{
int ret;
+ struct udevice *sys_child;
+ struct sysreset_reg *priv;
/* The reset driver does not have a device node, so bind it here */
- ret = device_bind_driver(gd->dm_root, "rv1108_sysreset", "reset", &dev);
- if (ret)
- pr_err("No Rv1108 reset driver: ret=%d\n", ret);
+ ret = device_bind_driver(dev, "rockchip_sysreset", "sysreset",
+ &sys_child);
+ if (ret) {
+ debug("Warning: No sysreset driver: ret=%d\n", ret);
+ } else {
+ priv = malloc(sizeof(struct sysreset_reg));
+ priv->glb_srst_fst_value = offsetof(struct rv1108_cru,
+ glb_srst_fst_val);
+ priv->glb_srst_snd_value = offsetof(struct rv1108_cru,
+ glb_srst_snd_val);
+ sys_child->priv = priv;
+ }
return 0;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 2/2] rockchip: clock: update sysreset driver bingding
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
@ 2017-11-06 17:28 ` Philipp Tomsich
2017-11-07 13:54 ` Philipp Tomsich
` (2 subsequent siblings)
3 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-06 17:28 UTC (permalink / raw)
To: u-boot
> Using priv for new sysreset driver binding.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> ---
>
> Changes in v3:
> - do not init priv if driver bind fail
>
> Changes in v2:
> - use udev priv member instead of driver_data
>
> drivers/clk/rockchip/clk_rk3036.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3128.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3188.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk322x.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3288.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3328.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3368.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3399.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rv1108.c | 17 ++++++++++++++---
> 9 files changed, 126 insertions(+), 27 deletions(-)
>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 2/2] rockchip: clock: update sysreset driver bingding
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, " Philipp Tomsich
@ 2017-11-07 13:54 ` Philipp Tomsich
2017-11-07 14:22 ` Philipp Tomsich
2017-11-07 14:40 ` Philipp Tomsich
3 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-07 13:54 UTC (permalink / raw)
To: u-boot
> Using priv for new sysreset driver binding.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v3:
> - do not init priv if driver bind fail
>
> Changes in v2:
> - use udev priv member instead of driver_data
>
> drivers/clk/rockchip/clk_rk3036.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3128.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3188.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk322x.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3288.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3328.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3368.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3399.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rv1108.c | 17 ++++++++++++++---
> 9 files changed, 126 insertions(+), 27 deletions(-)
>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 2/2] rockchip: clock: update sysreset driver bingding
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, " Philipp Tomsich
2017-11-07 13:54 ` Philipp Tomsich
@ 2017-11-07 14:22 ` Philipp Tomsich
2017-11-07 14:40 ` Philipp Tomsich
3 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-07 14:22 UTC (permalink / raw)
To: u-boot
> Using priv for new sysreset driver binding.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v3:
> - do not init priv if driver bind fail
>
> Changes in v2:
> - use udev priv member instead of driver_data
>
> drivers/clk/rockchip/clk_rk3036.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3128.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3188.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk322x.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3288.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3328.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3368.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3399.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rv1108.c | 17 ++++++++++++++---
> 9 files changed, 126 insertions(+), 27 deletions(-)
>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 2/2] rockchip: clock: update sysreset driver bingding
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
` (2 preceding siblings ...)
2017-11-07 14:22 ` Philipp Tomsich
@ 2017-11-07 14:40 ` Philipp Tomsich
3 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-07 14:40 UTC (permalink / raw)
To: u-boot
> Using priv for new sysreset driver binding.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v3:
> - do not init priv if driver bind fail
>
> Changes in v2:
> - use udev priv member instead of driver_data
>
> drivers/clk/rockchip/clk_rk3036.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3128.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3188.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk322x.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3288.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3328.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3368.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rk3399.c | 17 ++++++++++++++---
> drivers/clk/rockchip/clk_rv1108.c | 17 ++++++++++++++---
> 9 files changed, 126 insertions(+), 27 deletions(-)
>
Applied to u-boot-rockchip/next, thanks!
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver
2017-11-03 7:16 [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver Kever Yang
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
@ 2017-11-06 17:28 ` Philipp Tomsich
2017-11-07 13:54 ` Philipp Tomsich
` (2 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-06 17:28 UTC (permalink / raw)
To: u-boot
> Use a common driver for all Rockchip SOC instead of one for each SoC.
> Use driver_data for reg offset.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> ---
>
> Changes in v3: None
> Changes in v2:
> - use udev priv member instead of driver_data
>
> arch/arm/include/asm/arch-rockchip/clock.h | 5 +++
> drivers/sysreset/Makefile | 12 +-----
> drivers/sysreset/sysreset_rk3036.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3128.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3188.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk322x.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3288.c | 47 ----------------------
> drivers/sysreset/sysreset_rk3328.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3368.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk3399.c | 45 ----------------------
> drivers/sysreset/sysreset_rockchip.c | 47 ++++++++++++++++++++++
> drivers/sysreset/sysreset_rv1108.c | 46 ----------------------
> 12 files changed, 53 insertions(+), 453 deletions(-)
> delete mode 100644 drivers/sysreset/sysreset_rk3036.c
> delete mode 100644 drivers/sysreset/sysreset_rk3128.c
> delete mode 100644 drivers/sysreset/sysreset_rk3188.c
> delete mode 100644 drivers/sysreset/sysreset_rk322x.c
> delete mode 100644 drivers/sysreset/sysreset_rk3288.c
> delete mode 100644 drivers/sysreset/sysreset_rk3328.c
> delete mode 100644 drivers/sysreset/sysreset_rk3368.c
> delete mode 100644 drivers/sysreset/sysreset_rk3399.c
> create mode 100644 drivers/sysreset/sysreset_rockchip.c
> delete mode 100644 drivers/sysreset/sysreset_rv1108.c
>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver
2017-11-03 7:16 [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver Kever Yang
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver Philipp Tomsich
@ 2017-11-07 13:54 ` Philipp Tomsich
2017-11-07 14:22 ` Philipp Tomsich
2017-11-07 14:40 ` Philipp Tomsich
4 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-07 13:54 UTC (permalink / raw)
To: u-boot
> Use a common driver for all Rockchip SOC instead of one for each SoC.
> Use driver_data for reg offset.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v3: None
> Changes in v2:
> - use udev priv member instead of driver_data
>
> arch/arm/include/asm/arch-rockchip/clock.h | 5 +++
> drivers/sysreset/Makefile | 12 +-----
> drivers/sysreset/sysreset_rk3036.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3128.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3188.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk322x.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3288.c | 47 ----------------------
> drivers/sysreset/sysreset_rk3328.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3368.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk3399.c | 45 ----------------------
> drivers/sysreset/sysreset_rockchip.c | 47 ++++++++++++++++++++++
> drivers/sysreset/sysreset_rv1108.c | 46 ----------------------
> 12 files changed, 53 insertions(+), 453 deletions(-)
> delete mode 100644 drivers/sysreset/sysreset_rk3036.c
> delete mode 100644 drivers/sysreset/sysreset_rk3128.c
> delete mode 100644 drivers/sysreset/sysreset_rk3188.c
> delete mode 100644 drivers/sysreset/sysreset_rk322x.c
> delete mode 100644 drivers/sysreset/sysreset_rk3288.c
> delete mode 100644 drivers/sysreset/sysreset_rk3328.c
> delete mode 100644 drivers/sysreset/sysreset_rk3368.c
> delete mode 100644 drivers/sysreset/sysreset_rk3399.c
> create mode 100644 drivers/sysreset/sysreset_rockchip.c
> delete mode 100644 drivers/sysreset/sysreset_rv1108.c
>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver
2017-11-03 7:16 [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver Kever Yang
` (2 preceding siblings ...)
2017-11-07 13:54 ` Philipp Tomsich
@ 2017-11-07 14:22 ` Philipp Tomsich
2017-11-07 14:40 ` Philipp Tomsich
4 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-07 14:22 UTC (permalink / raw)
To: u-boot
> Use a common driver for all Rockchip SOC instead of one for each SoC.
> Use driver_data for reg offset.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v3: None
> Changes in v2:
> - use udev priv member instead of driver_data
>
> arch/arm/include/asm/arch-rockchip/clock.h | 5 +++
> drivers/sysreset/Makefile | 12 +-----
> drivers/sysreset/sysreset_rk3036.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3128.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3188.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk322x.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3288.c | 47 ----------------------
> drivers/sysreset/sysreset_rk3328.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3368.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk3399.c | 45 ----------------------
> drivers/sysreset/sysreset_rockchip.c | 47 ++++++++++++++++++++++
> drivers/sysreset/sysreset_rv1108.c | 46 ----------------------
> 12 files changed, 53 insertions(+), 453 deletions(-)
> delete mode 100644 drivers/sysreset/sysreset_rk3036.c
> delete mode 100644 drivers/sysreset/sysreset_rk3128.c
> delete mode 100644 drivers/sysreset/sysreset_rk3188.c
> delete mode 100644 drivers/sysreset/sysreset_rk322x.c
> delete mode 100644 drivers/sysreset/sysreset_rk3288.c
> delete mode 100644 drivers/sysreset/sysreset_rk3328.c
> delete mode 100644 drivers/sysreset/sysreset_rk3368.c
> delete mode 100644 drivers/sysreset/sysreset_rk3399.c
> create mode 100644 drivers/sysreset/sysreset_rockchip.c
> delete mode 100644 drivers/sysreset/sysreset_rv1108.c
>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver
2017-11-03 7:16 [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver Kever Yang
` (3 preceding siblings ...)
2017-11-07 14:22 ` Philipp Tomsich
@ 2017-11-07 14:40 ` Philipp Tomsich
4 siblings, 0 replies; 10+ messages in thread
From: Philipp Tomsich @ 2017-11-07 14:40 UTC (permalink / raw)
To: u-boot
> Use a common driver for all Rockchip SOC instead of one for each SoC.
> Use driver_data for reg offset.
>
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v3: None
> Changes in v2:
> - use udev priv member instead of driver_data
>
> arch/arm/include/asm/arch-rockchip/clock.h | 5 +++
> drivers/sysreset/Makefile | 12 +-----
> drivers/sysreset/sysreset_rk3036.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3128.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3188.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk322x.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3288.c | 47 ----------------------
> drivers/sysreset/sysreset_rk3328.c | 45 ----------------------
> drivers/sysreset/sysreset_rk3368.c | 62 ------------------------------
> drivers/sysreset/sysreset_rk3399.c | 45 ----------------------
> drivers/sysreset/sysreset_rockchip.c | 47 ++++++++++++++++++++++
> drivers/sysreset/sysreset_rv1108.c | 46 ----------------------
> 12 files changed, 53 insertions(+), 453 deletions(-)
> delete mode 100644 drivers/sysreset/sysreset_rk3036.c
> delete mode 100644 drivers/sysreset/sysreset_rk3128.c
> delete mode 100644 drivers/sysreset/sysreset_rk3188.c
> delete mode 100644 drivers/sysreset/sysreset_rk322x.c
> delete mode 100644 drivers/sysreset/sysreset_rk3288.c
> delete mode 100644 drivers/sysreset/sysreset_rk3328.c
> delete mode 100644 drivers/sysreset/sysreset_rk3368.c
> delete mode 100644 drivers/sysreset/sysreset_rk3399.c
> create mode 100644 drivers/sysreset/sysreset_rockchip.c
> delete mode 100644 drivers/sysreset/sysreset_rv1108.c
>
Applied to u-boot-rockchip/next, thanks!
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-11-07 14:40 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-03 7:16 [U-Boot] [PATCH v3 1/2] rockchip: sysreset: merge into one common driver Kever Yang
2017-11-03 7:16 ` [U-Boot] [PATCH v3 2/2] rockchip: clock: update sysreset driver bingding Kever Yang
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, " Philipp Tomsich
2017-11-07 13:54 ` Philipp Tomsich
2017-11-07 14:22 ` Philipp Tomsich
2017-11-07 14:40 ` Philipp Tomsich
2017-11-06 17:28 ` [U-Boot] [U-Boot, v3, 1/2] rockchip: sysreset: merge into one common driver Philipp Tomsich
2017-11-07 13:54 ` Philipp Tomsich
2017-11-07 14:22 ` Philipp Tomsich
2017-11-07 14:40 ` Philipp Tomsich
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.