Hi, > From: Simon Glass <s...@chromium.org> > Sent: lundi 17 février 2020 04:56 > > 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)?
I forget to execute it, sorry. Work in progress.... > Regards, > Simon Patrick