[PATCH v3] dm: core: Move "/chosen" and "/firmware" node scan

[Simon Glass]

On Tue, 18 Feb 2020 at 07:44, Patrick Delaunay <patrick.delaunay at st.com> wrote:
>
> Use the new function dm_scan_fdt_ofnode_path() to scan all the nodes
> which aren't devices themselves but may contain some:
> - "/chosen"
> - "/clocks"
> - "/firmware"
>
> The patch removes the strcmp call in recursive function dm_scan_fdt_live()
> and also corrects a conflict with the 2 applied patches in
> the commit 1712ca21924b ("dm: core: Scan /firmware node by default")
> and in the commit 747558d01457 ("dm: fdt: scan for devices under
> /firmware too"): the subnodes of "/firmware" (optee for example)
> are bound 2 times.
>
> For example the dm tree command result on STM32MP1 is:
>
> STM32MP> dm tree
>  Class     Index  Probed  Driver                Name
>  -----------------------------------------------------------
>  root          0  [ + ]   root_driver           root_driver
>  firmware      0  [   ]   psci                  |-- psci
>  sysreset      0  [   ]   psci-sysreset         |   `-- psci-sysreset
>  simple_bus    0  [ + ]   generic_simple_bus    |-- soc
> ...
>  tee           0  [ + ]   optee                 |-- optee
> ...
>  tee           1  [   ]   optee                 `-- optee
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> Tested-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
> Also reported here:
> https://lists.denx.de/pipermail/u-boot/2020-January/395427.html
>
>
> Changes in v3:
> - update test-fdt: use dm_extended_scan_fdt instead of dm_scan_fdt
>
> Changes in v2:
> - update commit message (Serie-cc => Series-cc)
>
>  drivers/core/root.c | 52 +++++++++++++++------------------------------
>  test/dm/test-fdt.c  |  2 +-
>  2 files changed, 18 insertions(+), 36 deletions(-)

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