[U-Boot] [PATCH v3 10/16] dm: Add a 'dm' command for testing

Simon Glass sjg at chromium.org
Fri Oct 18 19:00:41 CEST 2013


Hi Marek,

On Fri, Jun 28, 2013 at 2:57 PM, Marek Vasut <marex at denx.de> wrote:

> Dear Simon Glass,
>
> > +U_BOOT_CMD(
> > +     dm,     2,      1,      do_dm,
> > +     "Driver model low level access",
> > +     "dump         Dump driver model tree\n"
> > +     "dm uclass        Dump list of instances for each uclass\n"
> > +     "dm test         Run tests"
> > +);
>
> Looking at this, remark comes to mind. We should really fix this
> inconsistency
> between the first "dump" and the rest of "dm uclass" and "dm test" etc. in
> the
> U_BOOT_CMD.
>

I think I'll change this one to 'dm tree' instead of 'dm dump' since 'dump'
is pretty vague.


> This is obviously out of scope of these patches.


Regards,
Simon


More information about the U-Boot mailing list