diff mbox

[U-Boot,v2,6/9] dm: core: Dont export dm_scan_fdt_node()

Message ID 20170501151852.26670-7-sjg@chromium.org
State Accepted
Commit a771a04f2eb34934a058a6c06319bd8bc5cb5fb8
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass May 1, 2017, 3:18 p.m. UTC 
This function is only used in one place. It is better to just declare it
internally since there is a simpler replacement for use outside the
driver-model core code.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

Changes in v2: None

 drivers/core/root.c | 17 +++++++++++++++--
 include/dm/root.h   | 16 ----------------
 2 files changed, 15 insertions(+), 18 deletions(-)

Comments

Tom Rini May 10, 2017, 9:43 p.m. UTC | #1 
On Mon, May 01, 2017 at 09:18:49AM -0600, Simon Glass wrote:

> This function is only used in one place. It is better to just declare it
> internally since there is a simpler replacement for use outside the
> driver-model core code.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>

Reviewed-by: Tom Rini <trini@konsulko.com>
diff mbox

Patch

diff --git a/drivers/core/root.c b/drivers/core/root.c
index 42679d047c..4bb71f3cac 100644
--- a/drivers/core/root.c
+++ b/drivers/core/root.c
@@ -207,8 +207,21 @@  int dm_scan_platdata(bool pre_reloc_only)
 }
 
 #if CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)
-int dm_scan_fdt_node(struct udevice *parent, const void *blob, int offset,
-		     bool pre_reloc_only)
+/**
+ * dm_scan_fdt_node() - Scan the device tree and bind drivers for a node
+ *
+ * This scans the subnodes of a device tree node and and creates a driver
+ * for each one.
+ *
+ * @parent: Parent device for the devices that will be created
+ * @blob: Pointer to device tree blob
+ * @offset: Offset of node to scan
+ * @pre_reloc_only: If true, bind only drivers with the DM_FLAG_PRE_RELOC
+ * flag. If false bind all drivers.
+ * @return 0 if OK, -ve on error
+ */
+static int dm_scan_fdt_node(struct udevice *parent, const void *blob,
+			    int offset, bool pre_reloc_only)
 {
 	int ret = 0, err;
 
diff --git a/include/dm/root.h b/include/dm/root.h
index 058eb98923..186cf8ba1c 100644
--- a/include/dm/root.h
+++ b/include/dm/root.h
@@ -56,22 +56,6 @@  int dm_scan_platdata(bool pre_reloc_only);
 int dm_scan_fdt(const void *blob, bool pre_reloc_only);
 
 /**
- * dm_scan_fdt_node() - Scan the device tree and bind drivers for a node
- *
- * This scans the subnodes of a device tree node and and creates a driver
- * for each one.
- *
- * @parent: Parent device for the devices that will be created
- * @blob: Pointer to device tree blob
- * @offset: Offset of node to scan
- * @pre_reloc_only: If true, bind only drivers with the DM_FLAG_PRE_RELOC
- * flag. If false bind all drivers.
- * @return 0 if OK, -ve on error
- */
-int dm_scan_fdt_node(struct udevice *parent, const void *blob, int offset,
-		     bool pre_reloc_only);
-
-/**
  * dm_scan_other() - Scan for other devices
  *
  * Some devices may not be visible to Driver Model. This weak function can