[U-Boot] [PATCH] dm: README: recommend u-boot.dtb to try driver-model on sandbox

[Masahiro Yamada]

Hi Simon,



On Fri, 19 Dec 2014 13:48:16 -0700
Simon Glass <sjg at chromium.org> wrote:

> On 19 December 2014 at 06:39, Masahiro Yamada <yamada.m at jp.panasonic.com> wrote:
> > Jagan,
> >
> > On Fri, 19 Dec 2014 18:43:37 +0530
> > Jagan Teki <jagannadh.teki at gmail.com> wrote:
> >
> >> On 19 December 2014 at 10:46, Masahiro Yamada <yamada.m at jp.panasonic.com> wrote:
> >> > To enjoy driver-model on sandbox, using device tree is recommended.
> >> > While we are here, change sandbox_config to sandbox_defconfig too.
> >> >
> >> > Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
> >> > ---
> >> >
> >> >  doc/driver-model/README.txt | 4 ++--
> >> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >> >
> >> > diff --git a/doc/driver-model/README.txt b/doc/driver-model/README.txt
> >> > index 3e2f622..eafa825 100644
> >> > --- a/doc/driver-model/README.txt
> >> > +++ b/doc/driver-model/README.txt
> >> > @@ -36,9 +36,9 @@ How to try it
> >> >
> >> >  Build U-Boot sandbox and run it:
> >> >
> >> > -   make sandbox_config
> >> > +   make sandbox_defconfig
> >> >     make
> >> > -   ./u-boot
> >> > +   ./u-boot -d u-boot.dtb
> >>
> >> this -d is an optional I suppose without dtb we can able to access dm isn't it?
> >
> >
> > Without -d, DM can still have access to platform data, but
> > arch/sandbox/sandbox.dts provies us more features.
> > With "-d u-boot.dtb", you will notice a difference at glance: "cyan" terminal.
> 
> Acked-by: Simon Glass <sjg at chromium.org>
> 
> Masahiro, do you think U-Boot should automatically pick up u-boot.dtb
> in the same directory as the executable, unless -d gives another file?
> I suppose we could add a '--nodtb' option too.

I generally do not like programs to do what I do not ask to do behind the scene,
so I personally think it should be as it is.

(If you want to save "-d u-boot.dtb", you can use CONFIG_OF_EMBED)



Best Regards
Masahiro Yamada