From patchwork Wed Jan 8 03:24:02 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Thirupathaiah Annapureddy X-Patchwork-Id: 239248 List-Id: U-Boot discussion From: thiruan at linux.microsoft.com (Thirupathaiah Annapureddy) Date: Tue, 7 Jan 2020 19:24:02 -0800 Subject: [PATCH] Revert "dm: core: Scan "/firmware" node by default" Message-ID: <20200108032402.26721-1-thiruan@linux.microsoft.com> Subnodes under "/firmware" node are scanned twice in dm_scan_fdt_node and dm_extended_scan_fdt. This patch removes the double scanning. This reverts commit 1712ca21924bff678f19fd3141f435408d23bdbf. Signed-off-by: Thirupathaiah Annapureddy --- drivers/core/root.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/core/root.c b/drivers/core/root.c index e85643819e..daf7c61190 100644 --- a/drivers/core/root.c +++ b/drivers/core/root.c @@ -342,14 +342,8 @@ int dm_extended_scan_fdt(const void *blob, bool pre_reloc_only) } ret = dm_scan_fdt_ofnode_path("/clocks", pre_reloc_only); - if (ret) { - debug("scan for /clocks failed: %d\n", ret); - return ret; - } - - ret = dm_scan_fdt_ofnode_path("/firmware", pre_reloc_only); if (ret) - debug("scan for /firmware failed: %d\n", ret); + debug("scan for /clocks failed: %d\n", ret); return ret; }