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

[Simon Glass]

Hi Masahiro,

On 21 December 2014 at 19:49, Masahiro Yamada <yamada.m at jp.panasonic.com> wrote:
> 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)

OK, let's leave it then. I suppose another option would be -D to cause
it to search for u-boot.dtb. I do find it something of a pain to type
each time.

Regards,
Simon