diff mbox series

[v2] dm: core: remove the duplicated function dm_ofnode_pre_reloc

Message ID 20200403113844.v2.1.If3d7baadb24504a63742fe82f963f481b030fafa@changeid
State Accepted
Commit 5c9c9bc9571f23d4c8bab18fc50fc1238da0c6c1
Headers show
Series [v2] dm: core: remove the duplicated function dm_ofnode_pre_reloc | expand

Commit Message

Patrick Delaunay April 3, 2020, 9:39 a.m. UTC
The content dm_ofnode_pre_reloc() is identical with ofnode_pre_reloc()
defined in drivers/core/ofnode.c and used only three times:
- drivers/core/lists.c:lists_bind_fdt()
- drivers/clk/at91/pmc.c::at91_clk_sub_device_bind
- drivers/clk/altera/clk-arria10.c::socfpga_a10_clk_bind

So this function dm_ofnode_pre_reloc can be removed and replaced
by these function calls by ofnode_pre_reloc().

Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
---
Hi Simon,

It is a rebased patch for http://patchwork.ozlabs.org/patch/1035799/
marked as superseded but never resent (and I forget it).

Compilation is OK on travis:
https://travis-ci.org/github/patrickdelaunay/u-boot/builds/670029232

Patrick


Changes in v2:
- rebase on master
- udpate dm_ofnode_pre_reloc call in at91_clk_sub_device_bind and
  socfpga_a10_clk_bind

 drivers/clk/altera/clk-arria10.c |  2 +-
 drivers/clk/at91/pmc.c           |  2 +-
 drivers/core/lists.c             |  2 +-
 drivers/core/util.c              | 28 ----------------------------
 include/dm/util.h                | 27 ---------------------------
 5 files changed, 3 insertions(+), 58 deletions(-)

Comments

Simon Glass April 8, 2020, 2:57 a.m. UTC | #1
On Fri, 3 Apr 2020 at 03:39, Patrick Delaunay <patrick.delaunay at st.com> wrote:
>
> The content dm_ofnode_pre_reloc() is identical with ofnode_pre_reloc()
> defined in drivers/core/ofnode.c and used only three times:
> - drivers/core/lists.c:lists_bind_fdt()
> - drivers/clk/at91/pmc.c::at91_clk_sub_device_bind
> - drivers/clk/altera/clk-arria10.c::socfpga_a10_clk_bind
>
> So this function dm_ofnode_pre_reloc can be removed and replaced
> by these function calls by ofnode_pre_reloc().
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> ---
> Hi Simon,
>
> It is a rebased patch for http://patchwork.ozlabs.org/patch/1035799/
> marked as superseded but never resent (and I forget it).
>
> Compilation is OK on travis:
> https://travis-ci.org/github/patrickdelaunay/u-boot/builds/670029232
>
> Patrick
>
>
> Changes in v2:
> - rebase on master
> - udpate dm_ofnode_pre_reloc call in at91_clk_sub_device_bind and
>   socfpga_a10_clk_bind
>
>  drivers/clk/altera/clk-arria10.c |  2 +-
>  drivers/clk/at91/pmc.c           |  2 +-
>  drivers/core/lists.c             |  2 +-
>  drivers/core/util.c              | 28 ----------------------------
>  include/dm/util.h                | 27 ---------------------------
>  5 files changed, 3 insertions(+), 58 deletions(-)
>

Acked-by: Simon Glass <sjg at chromium.org>
Simon Glass April 10, 2020, 7:31 p.m. UTC | #2
On Fri, 3 Apr 2020 at 03:39, Patrick Delaunay <patrick.delaunay at st.com> wrote:
>
> The content dm_ofnode_pre_reloc() is identical with ofnode_pre_reloc()
> defined in drivers/core/ofnode.c and used only three times:
> - drivers/core/lists.c:lists_bind_fdt()
> - drivers/clk/at91/pmc.c::at91_clk_sub_device_bind
> - drivers/clk/altera/clk-arria10.c::socfpga_a10_clk_bind
>
> So this function dm_ofnode_pre_reloc can be removed and replaced
> by these function calls by ofnode_pre_reloc().
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> ---
> Hi Simon,
>
> It is a rebased patch for http://patchwork.ozlabs.org/patch/1035799/
> marked as superseded but never resent (and I forget it).
>
> Compilation is OK on travis:
> https://travis-ci.org/github/patrickdelaunay/u-boot/builds/670029232
>
> Patrick
>
>
> Changes in v2:
> - rebase on master
> - udpate dm_ofnode_pre_reloc call in at91_clk_sub_device_bind and
>   socfpga_a10_clk_bind
>
>  drivers/clk/altera/clk-arria10.c |  2 +-
>  drivers/clk/at91/pmc.c           |  2 +-
>  drivers/core/lists.c             |  2 +-
>  drivers/core/util.c              | 28 ----------------------------
>  include/dm/util.h                | 27 ---------------------------
>  5 files changed, 3 insertions(+), 58 deletions(-)
>

Acked-by: Simon Glass <sjg at chromium.org>

Applied to u-boot-dm/next, thanks!
diff mbox series

Patch

diff --git a/drivers/clk/altera/clk-arria10.c b/drivers/clk/altera/clk-arria10.c
index affeb31fc2..0a9bf13ac4 100644
--- a/drivers/clk/altera/clk-arria10.c
+++ b/drivers/clk/altera/clk-arria10.c
@@ -258,7 +258,7 @@  static int socfpga_a10_clk_bind(struct udevice *dev)
 			continue;
 
 		if (pre_reloc_only &&
-		    !dm_ofnode_pre_reloc(offset_to_ofnode(offset)))
+		    !ofnode_pre_reloc(offset_to_ofnode(offset)))
 			continue;
 
 		ret = device_bind_driver_to_node(dev, "clk-a10", name,
diff --git a/drivers/clk/at91/pmc.c b/drivers/clk/at91/pmc.c
index 6b55ec59d6..f5808449a6 100644
--- a/drivers/clk/at91/pmc.c
+++ b/drivers/clk/at91/pmc.c
@@ -61,7 +61,7 @@  int at91_clk_sub_device_bind(struct udevice *dev, const char *drv_name)
 	     offset > 0;
 	     offset = fdt_next_subnode(fdt, offset)) {
 		if (pre_reloc_only &&
-		    !dm_ofnode_pre_reloc(offset_to_ofnode(offset)))
+		    !ofnode_pre_reloc(offset_to_ofnode(offset)))
 			continue;
 		/*
 		 * If this node has "compatible" property, this is not
diff --git a/drivers/core/lists.c b/drivers/core/lists.c
index 68204c303f..c7db14ed56 100644
--- a/drivers/core/lists.c
+++ b/drivers/core/lists.c
@@ -175,7 +175,7 @@  int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp,
 			continue;
 
 		if (pre_reloc_only) {
-			if (!dm_ofnode_pre_reloc(node) &&
+			if (!ofnode_pre_reloc(node) &&
 			    !(entry->flags & DM_FLAG_PRE_RELOC)) {
 				log_debug("Skipping device pre-relocation\n");
 				return 0;
diff --git a/drivers/core/util.c b/drivers/core/util.c
index 69f83755f0..25b0d76f43 100644
--- a/drivers/core/util.c
+++ b/drivers/core/util.c
@@ -33,34 +33,6 @@  int list_count_items(struct list_head *head)
 	return count;
 }
 
-#if CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)
-bool dm_ofnode_pre_reloc(ofnode node)
-{
-#if defined(CONFIG_SPL_BUILD) || defined(CONFIG_TPL_BUILD)
-	/* for SPL and TPL the remaining nodes after the fdtgrep 1st pass
-	 * had property dm-pre-reloc or u-boot,dm-spl/tpl.
-	 * They are removed in final dtb (fdtgrep 2nd pass)
-	 */
-	return true;
-#else
-	if (ofnode_read_bool(node, "u-boot,dm-pre-reloc"))
-		return true;
-	if (ofnode_read_bool(node, "u-boot,dm-pre-proper"))
-		return true;
-
-	/*
-	 * In regular builds individual spl and tpl handling both
-	 * count as handled pre-relocation for later second init.
-	 */
-	if (ofnode_read_bool(node, "u-boot,dm-spl") ||
-	    ofnode_read_bool(node, "u-boot,dm-tpl"))
-		return true;
-
-	return false;
-#endif
-}
-#endif
-
 #if !CONFIG_IS_ENABLED(OF_PLATDATA)
 int pci_get_devfn(struct udevice *dev)
 {
diff --git a/include/dm/util.h b/include/dm/util.h
index 0ccb3fbadf..23f8deb14e 100644
--- a/include/dm/util.h
+++ b/include/dm/util.h
@@ -42,31 +42,4 @@  static inline void dm_dump_devres(void)
 /* Dump out a list of drivers */
 void dm_dump_drivers(void);
 
-/**
- * Check if an of node should be or was bound before relocation.
- *
- * Devicetree nodes can be marked as needed to be bound
- * in the loader stages via special devicetree properties.
- *
- * Before relocation this function can be used to check if nodes
- * are required in either SPL or TPL stages.
- *
- * After relocation and jumping into the real U-Boot binary
- * it is possible to determine if a node was bound in one of
- * SPL/TPL stages.
- *
- * There are 4 settings currently in use
- * - u-boot,dm-pre-proper: U-Boot proper pre-relocation only
- * - u-boot,dm-pre-reloc: legacy and indicates any of TPL or SPL
- *   Existing platforms only use it to indicate nodes needed in
- *   SPL. Should probably be replaced by u-boot,dm-spl for
- *   existing platforms.
- * - u-boot,dm-spl: SPL and U-Boot pre-relocation
- * - u-boot,dm-tpl: TPL and U-Boot pre-relocation
- * @node: of node
- *
- * Returns true if node is needed in SPL/TL, false otherwise.
- */
-bool dm_ofnode_pre_reloc(ofnode node);
-
 #endif