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

[Simon Glass]

On 22 December 2014 at 12:57, Simon Glass <sjg at chromium.org> wrote:
> 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.

This is a sandbox commit.

Applied to u-boot-x86 branch misc.