Hi Patrick, On Thu, 13 Feb 2020 at 11:48, Patrick Delaunay <patrick.delau...@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.delau...@st.com> > --- > > Changes in v2: > - update commit message (Serie-cc => Series-cc) > > drivers/core/root.c | 52 +++++++++++++++------------------------------ > 1 file changed, 17 insertions(+), 35 deletions(-)
This looks good to me, but please can you address the test failure (make qcheck)? Regards, Simon