[PATCH v1 00/12] Bind Syscon driver with OF_REAL

[PATCH v1 00/12] Bind Syscon driver with OF_REAL

[Anand Moon]

Bind syscon driver with dm_scan_fdt_dev
As OF_PLATDATA is no longer enabled, the related
code is removed.

Compiled and tested on RK3399 and RV1126 board.

Thanks
-Anand

Anand Moon (12):
  arm: rockchip: rk3399: Remove unused syscon drivers code
  arch: rockchip: rv1126: Update syscon driver binding method
  arm: rockchip: rv1126: Remove unused syscon drivers code
  arm: rockchip: px30: Remove unused syscon drivers code
  arch: rockchip: rk3066: Update syscon driver binding method
  arm: rockchip: rk3066: Remove unused syscon drivers code
  arch: rockchip: rk3368: Update syscon driver binding method
  arm: rockchip: rk3368: Remove unused syscon drivers code
  arch: rockchip: rk3188: Update syscon driver binding method
  arm: rockchip: rk3188: Remove unused syscon drivers code
  arch: rockchip: rk3288: Update syscon driver binding method
  arm: rockchip: rk3288: Remove unused syscon drivers code

 arch/arm/mach-rockchip/px30/syscon_px30.c     |  31 ----
 arch/arm/mach-rockchip/rk3066/syscon_rk3066.c |  34 +---
 arch/arm/mach-rockchip/rk3188/syscon_rk3188.c |  34 +---
 arch/arm/mach-rockchip/rk3288/syscon_rk3288.c | 157 +-----------------
 arch/arm/mach-rockchip/rk3368/syscon_rk3368.c |  41 +----
 arch/arm/mach-rockchip/rk3399/syscon_rk3399.c |  45 -----
 arch/arm/mach-rockchip/rv1126/syscon_rv1126.c |  27 +--
 7 files changed, 14 insertions(+), 355 deletions(-)


base-commit: 6becf9ba1ab82af6f4fcf9f4d0da38f9c75212d2
-- 
2.46.0

[PATCH v1 01/12] arm: rockchip: rk3399: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code, As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3399/syscon_rk3399.c | 45 -------------------
 1 file changed, 45 deletions(-)

diff --git a/arch/arm/mach-rockchip/rk3399/syscon_rk3399.c b/arch/arm/mach-rockchip/rk3399/syscon_rk3399.c
index b92ad54ede5..5895fa373ec 100644
--- a/arch/arm/mach-rockchip/rk3399/syscon_rk3399.c
+++ b/arch/arm/mach-rockchip/rk3399/syscon_rk3399.c
@@ -25,48 +25,3 @@ U_BOOT_DRIVER(syscon_rk3399) = {
 #endif
 	.of_match = rk3399_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-static int rk3399_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-
-U_BOOT_DRIVER(rockchip_rk3399_grf) = {
-	.name = "rockchip_rk3399_grf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3399_syscon_ids,
-	.bind = rk3399_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3399_pmugrf) = {
-	.name = "rockchip_rk3399_pmugrf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3399_syscon_ids + 1,
-	.bind = rk3399_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3399_pmusgrf) = {
-	.name = "rockchip_rk3399_pmusgrf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3399_syscon_ids + 2,
-	.bind = rk3399_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3399_cic) = {
-	.name = "rockchip_rk3399_cic",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3399_syscon_ids + 3,
-	.bind = rk3399_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3399_pmu) = {
-	.name = "rockchip_rk3399_pmu",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3399_syscon_ids + 4,
-	.bind = rk3399_syscon_bind_of_plat,
-};
-#endif
-- 
2.46.0

[PATCH v1 02/12] arch: rockchip: rv1126: Update syscon driver binding method

[Anand Moon]

Updated the binding method to use `dm_scan_fdt_dev` when
`OF_REAL` is enabled.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rv1126/syscon_rv1126.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c b/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c
index 67d2f18a8d0..d66d84c836d 100644
--- a/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c
+++ b/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c
@@ -18,6 +18,9 @@ static const struct udevice_id rv1126_syscon_ids[] = {
 U_BOOT_DRIVER(syscon_rv1126) = {
 	.name = "rv1126_syscon",
 	.id = UCLASS_SYSCON,
+#if CONFIG_IS_ENABLED(OF_REAL)
+	.bind = dm_scan_fdt_dev,
+#endif
 	.of_match = rv1126_syscon_ids,
 };
 
-- 
2.46.0

[PATCH v1 03/12] arm: rockchip: rv1126: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code, As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rv1126/syscon_rv1126.c | 24 -------------------
 1 file changed, 24 deletions(-)

diff --git a/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c b/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c
index d66d84c836d..4c0690f1ba2 100644
--- a/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c
+++ b/arch/arm/mach-rockchip/rv1126/syscon_rv1126.c
@@ -23,27 +23,3 @@ U_BOOT_DRIVER(syscon_rv1126) = {
 #endif
 	.of_match = rv1126_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-static int rv1126_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-
-U_BOOT_DRIVER(rockchip_rv1126_pmu) = {
-	.name = "rockchip_rv1126_pmu",
-	.id = UCLASS_SYSCON,
-	.of_match = rv1126_syscon_ids,
-	.bind = rv1126_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rv1126_pmugrf) = {
-	.name = "rockchip_rv1126_pmugrf",
-	.id = UCLASS_SYSCON,
-	.of_match = rv1126_syscon_ids + 1,
-	.bind = rv1126_syscon_bind_of_plat,
-};
-#endif
-- 
2.46.0

[PATCH v1 04/12] arm: rockchip: px30: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code. As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/px30/syscon_px30.c | 31 -----------------------
 1 file changed, 31 deletions(-)

diff --git a/arch/arm/mach-rockchip/px30/syscon_px30.c b/arch/arm/mach-rockchip/px30/syscon_px30.c
index 893a5234baa..cc5adc71569 100644
--- a/arch/arm/mach-rockchip/px30/syscon_px30.c
+++ b/arch/arm/mach-rockchip/px30/syscon_px30.c
@@ -23,34 +23,3 @@ U_BOOT_DRIVER(syscon_px30) = {
 #endif
 	.of_match = px30_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-static int px30_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-
-U_BOOT_DRIVER(rockchip_px30_pmu) = {
-	.name = "rockchip_px30_pmu",
-	.id = UCLASS_SYSCON,
-	.of_match = px30_syscon_ids,
-	.bind = px30_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_px30_pmugrf) = {
-	.name = "rockchip_px30_pmugrf",
-	.id = UCLASS_SYSCON,
-	.of_match = px30_syscon_ids + 1,
-	.bind = px30_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_px30_grf) = {
-	.name = "rockchip_px30_grf",
-	.id = UCLASS_SYSCON,
-	.of_match = px30_syscon_ids + 2,
-	.bind = px30_syscon_bind_of_plat,
-};
-#endif
-- 
2.46.0

[PATCH v1 05/12] arch: rockchip: rk3066: Update syscon driver binding method

[Anand Moon]

Updated the binding method to use `dm_scan_fdt_dev` when
OF_REAL` is enabled.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3066/syscon_rk3066.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c b/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c
index ff269b53b54..4e13aeca03e 100644
--- a/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c
+++ b/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c
@@ -19,6 +19,9 @@ static const struct udevice_id rk3066_syscon_ids[] = {
 U_BOOT_DRIVER(syscon_rk3066) = {
 	.name = "rk3066_syscon",
 	.id = UCLASS_SYSCON,
+#if CONFIG_IS_ENABLED(OF_REAL)
+	.bind = dm_scan_fdt_dev,
+#endif
 	.of_match = rk3066_syscon_ids,
 };
 
-- 
2.46.0

[PATCH v1 06/12] arm: rockchip: rk3066: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code. As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3066/syscon_rk3066.c | 31 -------------------
 1 file changed, 31 deletions(-)

diff --git a/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c b/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c
index 4e13aeca03e..18f006610d0 100644
--- a/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c
+++ b/arch/arm/mach-rockchip/rk3066/syscon_rk3066.c
@@ -24,34 +24,3 @@ U_BOOT_DRIVER(syscon_rk3066) = {
 #endif
 	.of_match = rk3066_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-static int rk3066_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-
-U_BOOT_DRIVER(rockchip_rk3066_noc) = {
-	.name = "rockchip_rk3066_noc",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3066_syscon_ids,
-	.bind = rk3066_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3066_grf) = {
-	.name = "rockchip_rk3066_grf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3066_syscon_ids + 1,
-	.bind = rk3066_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3066_pmu) = {
-	.name = "rockchip_rk3066_pmu",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3066_syscon_ids + 2,
-	.bind = rk3066_syscon_bind_of_plat,
-};
-#endif
-- 
2.46.0

[PATCH v1 07/12] arch: rockchip: rk3368: Update syscon driver binding method

[Anand Moon]

Updated the binding method to use `dm_scan_fdt_dev` when
OF_REAL` is enabled.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3368/syscon_rk3368.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c b/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c
index 7389c028364..89caa6c310c 100644
--- a/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c
+++ b/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c
@@ -25,6 +25,9 @@ static const struct udevice_id rk3368_syscon_ids[] = {
 U_BOOT_DRIVER(syscon_rk3368) = {
 	.name = "rk3368_syscon",
 	.id = UCLASS_SYSCON,
+#if CONFIG_IS_ENABLED(OF_REAL)
+	.bind = dm_scan_fdt_dev,
+#endif
 	.of_match = rk3368_syscon_ids,
 };
 
-- 
2.46.0

[PATCH v1 08/12] arm: rockchip: rk3368: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code. As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3368/syscon_rk3368.c | 38 -------------------
 1 file changed, 38 deletions(-)

diff --git a/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c b/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c
index 89caa6c310c..0381a22453f 100644
--- a/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c
+++ b/arch/arm/mach-rockchip/rk3368/syscon_rk3368.c
@@ -30,41 +30,3 @@ U_BOOT_DRIVER(syscon_rk3368) = {
 #endif
 	.of_match = rk3368_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-static int rk3368_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-
-U_BOOT_DRIVER(rockchip_rk3368_grf) = {
-	.name = "rockchip_rk3368_grf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3368_syscon_ids,
-	.bind = rk3368_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3368_pmugrf) = {
-	.name = "rockchip_rk3368_pmugrf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3368_syscon_ids + 1,
-	.bind = rk3368_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3368_msch) = {
-	.name = "rockchip_rk3368_msch",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3368_syscon_ids + 2,
-	.bind = rk3368_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3368_sgrf) = {
-	.name = "rockchip_rk3368_sgrf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3368_syscon_ids + 3,
-	.bind = rk3368_syscon_bind_of_plat,
-};
-#endif
-- 
2.46.0

[PATCH v1 09/12] arch: rockchip: rk3188: Update syscon driver binding method

[Anand Moon]

Updated the binding method to use `dm_scan_fdt_dev` when
OF_REAL` is enabled.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3188/syscon_rk3188.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c b/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c
index 6df054e5b27..bcd7870c503 100644
--- a/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c
+++ b/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c
@@ -19,6 +19,9 @@ static const struct udevice_id rk3188_syscon_ids[] = {
 U_BOOT_DRIVER(syscon_rk3188) = {
 	.name = "rk3188_syscon",
 	.id = UCLASS_SYSCON,
+#if CONFIG_IS_ENABLED(OF_REAL)
+	.bind = dm_scan_fdt_dev,
+#endif
 	.of_match = rk3188_syscon_ids,
 };
 
-- 
2.46.0

[PATCH v1 10/12] arm: rockchip: rk3188: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code. As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3188/syscon_rk3188.c | 31 -------------------
 1 file changed, 31 deletions(-)

diff --git a/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c b/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c
index bcd7870c503..94dbd6f8644 100644
--- a/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c
+++ b/arch/arm/mach-rockchip/rk3188/syscon_rk3188.c
@@ -24,34 +24,3 @@ U_BOOT_DRIVER(syscon_rk3188) = {
 #endif
 	.of_match = rk3188_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-static int rk3188_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-
-U_BOOT_DRIVER(rockchip_rk3188_noc) = {
-	.name = "rockchip_rk3188_noc",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3188_syscon_ids,
-	.bind = rk3188_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3188_grf) = {
-	.name = "rockchip_rk3188_grf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3188_syscon_ids + 1,
-	.bind = rk3188_syscon_bind_of_plat,
-};
-
-U_BOOT_DRIVER(rockchip_rk3188_pmu) = {
-	.name = "rockchip_rk3188_pmu",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3188_syscon_ids + 2,
-	.bind = rk3188_syscon_bind_of_plat,
-};
-#endif
-- 
2.46.0

[PATCH v1 11/12] arch: rockchip: rk3288: Update syscon driver binding method

[Anand Moon]

Updated the binding method to use `dm_scan_fdt_dev` when
`OF_REAL` is enabled.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3288/syscon_rk3288.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c b/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c
index 6413d0a88a1..4030e919134 100644
--- a/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c
+++ b/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c
@@ -23,6 +23,9 @@ static const struct udevice_id rk3288_syscon_ids[] = {
 U_BOOT_DRIVER(syscon_rk3288) = {
 	.name = "rk3288_syscon",
 	.id = UCLASS_SYSCON,
+#if CONFIG_IS_ENABLED(OF_REAL)
+	.bind = dm_scan_fdt_dev,
+#endif
 	.of_match = rk3288_syscon_ids,
 };
 
-- 
2.46.0

[PATCH v1 12/12] arm: rockchip: rk3288: Remove unused syscon drivers code

[Anand Moon]

Since commit 95397385091d ("treewide: Use OF_REAL instead of !OF_PLATDATA"),
the OF_PLATDATA code. As OF_PLATDATA is no longer enabled, the related
code is removed.

Signed-off-by: Anand Moon <linux.amoon@gmail.com>
---
 arch/arm/mach-rockchip/rk3288/syscon_rk3288.c | 156 ------------------
 1 file changed, 156 deletions(-)

diff --git a/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c b/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c
index 4030e919134..e2e1eb624ec 100644
--- a/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c
+++ b/arch/arm/mach-rockchip/rk3288/syscon_rk3288.c
@@ -28,159 +28,3 @@ U_BOOT_DRIVER(syscon_rk3288) = {
 #endif
 	.of_match = rk3288_syscon_ids,
 };
-
-#if CONFIG_IS_ENABLED(OF_PLATDATA)
-#if IS_ENABLED(CONFIG_FDT_64BIT)
-struct rockchip_rk3288_noc_plat {
-	struct dtd_rockchip_rk3288_noc dtplat;
-};
-
-struct rockchip_rk3288_grf_plat {
-	struct dtd_rockchip_rk3288_grf dtplat;
-};
-
-struct rockchip_rk3288_sgrf_plat {
-	struct dtd_rockchip_rk3288_sgrf dtplat;
-};
-
-struct rockchip_rk3288_pmu_plat {
-	struct dtd_rockchip_rk3288_pmu dtplat;
-};
-
-static int rk3288_noc_bind_of_plat(struct udevice *dev)
-{
-	struct rockchip_rk3288_noc_plat *plat = dev_get_plat(dev);
-	struct syscon_uc_info *priv = dev_get_uclass_priv(dev);
-	int size = dev->uclass->uc_drv->per_device_auto;
-
-	if (size && !priv) {
-		priv = calloc(1, size);
-		if (!priv)
-			return -ENOMEM;
-		dev_set_uclass_priv(dev, priv);
-	}
-
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return regmap_init_mem_plat(dev, plat->dtplat.reg, sizeof(plat->dtplat.reg[0]),
-				    ARRAY_SIZE(plat->dtplat.reg) / 2, &priv->regmap);
-}
-
-static int rk3288_grf_bind_of_plat(struct udevice *dev)
-{
-	struct rockchip_rk3288_grf_plat *plat = dev_get_plat(dev);
-	struct syscon_uc_info *priv = dev_get_uclass_priv(dev);
-	int size = dev->uclass->uc_drv->per_device_auto;
-
-	if (size && !priv) {
-		priv = calloc(1, size);
-		if (!priv)
-			return -ENOMEM;
-		dev_set_uclass_priv(dev, priv);
-	}
-
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return regmap_init_mem_plat(dev, plat->dtplat.reg, sizeof(plat->dtplat.reg[0]),
-				    ARRAY_SIZE(plat->dtplat.reg) / 2, &priv->regmap);
-}
-
-static int rk3288_sgrf_bind_of_plat(struct udevice *dev)
-{
-	struct rockchip_rk3288_sgrf_plat *plat = dev_get_plat(dev);
-	struct syscon_uc_info *priv = dev_get_uclass_priv(dev);
-	int size = dev->uclass->uc_drv->per_device_auto;
-
-	if (size && !priv) {
-		priv = calloc(1, size);
-		if (!priv)
-			return -ENOMEM;
-		dev_set_uclass_priv(dev, priv);
-	}
-
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return regmap_init_mem_plat(dev, plat->dtplat.reg, sizeof(plat->dtplat.reg[0]),
-				    ARRAY_SIZE(plat->dtplat.reg) / 2, &priv->regmap);
-}
-
-static int rk3288_pmu_bind_of_plat(struct udevice *dev)
-{
-	struct rockchip_rk3288_pmu_plat *plat = dev_get_plat(dev);
-	struct syscon_uc_info *priv = dev_get_uclass_priv(dev);
-	int size = dev->uclass->uc_drv->per_device_auto;
-
-	if (size && !priv) {
-		priv = calloc(1, size);
-		if (!priv)
-			return -ENOMEM;
-		dev_set_uclass_priv(dev, priv);
-	}
-
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return regmap_init_mem_plat(dev, plat->dtplat.reg, sizeof(plat->dtplat.reg[0]),
-				    ARRAY_SIZE(plat->dtplat.reg) / 2, &priv->regmap);
-}
-#else
-static int rk3288_syscon_bind_of_plat(struct udevice *dev)
-{
-	dev->driver_data = dev->driver->of_match->data;
-	debug("syscon: %s %d\n", dev->name, (uint)dev->driver_data);
-
-	return 0;
-}
-#endif
-
-U_BOOT_DRIVER(rockchip_rk3288_noc) = {
-	.name = "rockchip_rk3288_noc",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3288_syscon_ids,
-#if IS_ENABLED(CONFIG_FDT_64BIT)
-	.bind = rk3288_noc_bind_of_plat,
-	.plat_auto = sizeof(struct rockchip_rk3288_noc_plat),
-#else
-	.bind = rk3288_syscon_bind_of_plat,
-#endif
-};
-
-U_BOOT_DRIVER(rockchip_rk3288_grf) = {
-	.name = "rockchip_rk3288_grf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3288_syscon_ids + 1,
-#if IS_ENABLED(CONFIG_FDT_64BIT)
-	.bind = rk3288_grf_bind_of_plat,
-	.plat_auto = sizeof(struct rockchip_rk3288_grf_plat),
-#else
-	.bind = rk3288_syscon_bind_of_plat,
-#endif
-};
-
-U_BOOT_DRIVER(rockchip_rk3288_sgrf) = {
-	.name = "rockchip_rk3288_sgrf",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3288_syscon_ids + 2,
-#if IS_ENABLED(CONFIG_FDT_64BIT)
-	.bind = rk3288_sgrf_bind_of_plat,
-	.plat_auto = sizeof(struct rockchip_rk3288_sgrf_plat),
-#else
-	.bind = rk3288_syscon_bind_of_plat,
-#endif
-};
-
-U_BOOT_DRIVER(rockchip_rk3288_pmu) = {
-	.name = "rockchip_rk3288_pmu",
-	.id = UCLASS_SYSCON,
-	.of_match = rk3288_syscon_ids + 3,
-#if IS_ENABLED(CONFIG_FDT_64BIT)
-	.bind = rk3288_pmu_bind_of_plat,
-	.plat_auto = sizeof(struct rockchip_rk3288_pmu_plat),
-#else
-	.bind = rk3288_syscon_bind_of_plat,
-#endif
-};
-#endif
-- 
2.46.0

Re: [PATCH v1 00/12] Bind Syscon driver with OF_REAL

[Kever Yang]

Hi Anand,

On 2024/8/6 12:38, Anand Moon wrote:
> Bind syscon driver with dm_scan_fdt_dev
> As OF_PLATDATA is no longer enabled, the related
> code is removed.

The OF_PLATDATA is no removed, basically it's use in TPL/SPL for code size,

and the SPL_OF_PLATDATA and TPL_PLATDAT still available,

The OF_REAL is only to replace "OF_CONTROL && !OF_PLATDATA."

Thanks,
- Kever
>
> Compiled and tested on RK3399 and RV1126 board.
>
> Thanks
> -Anand
>
> Anand Moon (12):
>    arm: rockchip: rk3399: Remove unused syscon drivers code
>    arch: rockchip: rv1126: Update syscon driver binding method
>    arm: rockchip: rv1126: Remove unused syscon drivers code
>    arm: rockchip: px30: Remove unused syscon drivers code
>    arch: rockchip: rk3066: Update syscon driver binding method
>    arm: rockchip: rk3066: Remove unused syscon drivers code
>    arch: rockchip: rk3368: Update syscon driver binding method
>    arm: rockchip: rk3368: Remove unused syscon drivers code
>    arch: rockchip: rk3188: Update syscon driver binding method
>    arm: rockchip: rk3188: Remove unused syscon drivers code
>    arch: rockchip: rk3288: Update syscon driver binding method
>    arm: rockchip: rk3288: Remove unused syscon drivers code
>
>   arch/arm/mach-rockchip/px30/syscon_px30.c     |  31 ----
>   arch/arm/mach-rockchip/rk3066/syscon_rk3066.c |  34 +---
>   arch/arm/mach-rockchip/rk3188/syscon_rk3188.c |  34 +---
>   arch/arm/mach-rockchip/rk3288/syscon_rk3288.c | 157 +-----------------
>   arch/arm/mach-rockchip/rk3368/syscon_rk3368.c |  41 +----
>   arch/arm/mach-rockchip/rk3399/syscon_rk3399.c |  45 -----
>   arch/arm/mach-rockchip/rv1126/syscon_rv1126.c |  27 +--
>   7 files changed, 14 insertions(+), 355 deletions(-)
>
>
> base-commit: 6becf9ba1ab82af6f4fcf9f4d0da38f9c75212d2

Re: [PATCH v1 00/12] Bind Syscon driver with OF_REAL

[Anand Moon]

Hi Kever,

On Sat, 14 Sept 2024 at 15:30, Kever Yang <kever.yang@rock-chips.com> wrote:
>
> Hi Anand,
>
> On 2024/8/6 12:38, Anand Moon wrote:
>
> Bind syscon driver with dm_scan_fdt_dev
> As OF_PLATDATA is no longer enabled, the related
> code is removed.
>
> The OF_PLATDATA is no removed, basically it's use in TPL/SPL for code size,
>
> and the SPL_OF_PLATDATA and TPL_PLATDAT still available,
>
> The OF_REAL is only to replace "OF_CONTROL && !OF_PLATDATA."
>

Thanks, for this update.
.
> Thanks,
> - Kever
>

Thanks

-Anand

> Compiled and tested on RK3399 and RV1126 board.
>
> Thanks
> -Anand
>
> Anand Moon (12):
>   arm: rockchip: rk3399: Remove unused syscon drivers code
>   arch: rockchip: rv1126: Update syscon driver binding method
>   arm: rockchip: rv1126: Remove unused syscon drivers code
>   arm: rockchip: px30: Remove unused syscon drivers code
>   arch: rockchip: rk3066: Update syscon driver binding method
>   arm: rockchip: rk3066: Remove unused syscon drivers code
>   arch: rockchip: rk3368: Update syscon driver binding method
>   arm: rockchip: rk3368: Remove unused syscon drivers code
>   arch: rockchip: rk3188: Update syscon driver binding method
>   arm: rockchip: rk3188: Remove unused syscon drivers code
>   arch: rockchip: rk3288: Update syscon driver binding method
>   arm: rockchip: rk3288: Remove unused syscon drivers code
>
>  arch/arm/mach-rockchip/px30/syscon_px30.c     |  31 ----
>  arch/arm/mach-rockchip/rk3066/syscon_rk3066.c |  34 +---
>  arch/arm/mach-rockchip/rk3188/syscon_rk3188.c |  34 +---
>  arch/arm/mach-rockchip/rk3288/syscon_rk3288.c | 157 +-----------------
>  arch/arm/mach-rockchip/rk3368/syscon_rk3368.c |  41 +----
>  arch/arm/mach-rockchip/rk3399/syscon_rk3399.c |  45 -----
>  arch/arm/mach-rockchip/rv1126/syscon_rv1126.c |  27 +--
>  7 files changed, 14 insertions(+), 355 deletions(-)
>
>
> base-commit: 6becf9ba1ab82af6f4fcf9f4d0da38f9c75212d2