dm: README: recommend u-boot.dtb to try driver-model on sandbox

PATCH New Masahiro Yamada 3 participants 8 messages Dec 19, 2014 – Dec 29, 2014

Share & tools

Masahiro Yamada

author

2014-12-19 05:16 UTC #0 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

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 <[email protected]>

---

 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

doc/driver-model/README.txt +2 -2

index 3e2f622..eafa825 100644
@@ -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
 
    (type 'reset' to exit U-Boot)

Signature · 1 line

1.9.1

Jagan Teki

2014-12-19 13:13 UTC #1 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

Quoted 14 lines · Masahiro Yamada

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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644

doc/driver-model/README.txt How to try it · line 41

Quoted 8 lines

doc/driver-model/README.txt +2 -2

@@ -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?

Quoted 6 lines

doc/driver-model/README.txt +0 -1
    (type 'reset' to exit U-Boot)
 
--
1.9.1

thanks!

Signature · 1 line

Jagan.

Masahiro Yamada

author

2014-12-19 13:39 UTC #2 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

Jagan,

Quoted 18 lines · On Fri, 19 Dec 2014 18:43:37 +0530 Jagan Teki <[email protected]> wrote:

Quoted 17 lines · Masahiro Yamada
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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644
@@ -36,9 +36,9 @@ How to try it
 
 Build U-Boot sandbox and run it:
 

Quoted 7 lines

doc/driver-model/README.txt How to try it · line 41
Quoted 5 lines
doc/driver-model/README.txt +2 -2
-   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.

Best Regards
Masahiro Yamada

Simon Glass

2014-12-19 20:48 UTC #3 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

Quoted 32 lines · Masahiro Yamada

Jagan,
Quoted 18 lines · On Fri, 19 Dec 2014 18:43:37 +0530 Jagan Teki <[email protected]> wrote:
Quoted 17 lines · Masahiro Yamada
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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644
@@ -36,9 +36,9 @@ How to try it
 
 Build U-Boot sandbox and run it:
 
Quoted 7 lines
doc/driver-model/README.txt How to try it · line 41
Quoted 5 lines
doc/driver-model/README.txt +2 -2
-   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?

Quoted 3 lines

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 <[email protected]>

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.

Regards,
Simon

Jagan Teki

2014-12-20 07:06 UTC #4 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

Quoted 37 lines · Simon Glass

Quoted 32 lines · Masahiro Yamada
Jagan,
Quoted 18 lines · On Fri, 19 Dec 2014 18:43:37 +0530 Jagan Teki <[email protected]> wrote:
Quoted 17 lines · Masahiro Yamada
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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644
@@ -36,9 +36,9 @@ How to try it
 
 Build U-Boot sandbox and run it:
 
Quoted 7 lines
doc/driver-model/README.txt How to try it · line 41
Quoted 5 lines
doc/driver-model/README.txt +2 -2
-   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?
Quoted 3 lines
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.

Quoted 5 lines

Acked-by Simon Glass <[email protected]>
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.

In that case may be better list the options, on file itself.

Acked-by Jagannadha Sutradharudu Teki <[email protected]>

thanks!

Signature · 1 line

Jagan.

Masahiro Yamada

author

2014-12-22 02:49 UTC #5 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

In reply to Simon Glass

Hi Simon,

Quoted 37 lines · On Fri, 19 Dec 2014 13:48:16 -0700 Simon Glass <[email protected]> wrote:

Quoted 32 lines · Masahiro Yamada
Jagan,
Quoted 18 lines · On Fri, 19 Dec 2014 18:43:37 +0530 Jagan Teki <[email protected]> wrote:
Quoted 17 lines · Masahiro Yamada
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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644
@@ -36,9 +36,9 @@ How to try it
 
 Build U-Boot sandbox and run it:
 
Quoted 7 lines
doc/driver-model/README.txt How to try it · line 41
Quoted 5 lines
doc/driver-model/README.txt +2 -2
-   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?
Quoted 3 lines
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.

Quoted 5 lines

Acked-by Simon Glass <[email protected]>
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

Simon Glass

2014-12-22 19:57 UTC #6 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

Hi Masahiro,

Quoted 46 lines · Masahiro Yamada

Hi Simon,
Quoted 39 lines · On Fri, 19 Dec 2014 13:48:16 -0700 Simon Glass <[email protected]> wrote:
Quoted 32 lines · Masahiro Yamada
Jagan,
Quoted 18 lines · On Fri, 19 Dec 2014 18:43:37 +0530 Jagan Teki <[email protected]> wrote:
Quoted 17 lines · Masahiro Yamada
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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644
@@ -36,9 +36,9 @@ How to try it
 
 Build U-Boot sandbox and run it:
 
Quoted 7 lines
doc/driver-model/README.txt How to try it · line 41
Quoted 5 lines
doc/driver-model/README.txt +2 -2
-   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?
Quoted 3 lines
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 <[email protected]>

Quoted 8 lines

Quoted 3 lines
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

Simon Glass

2014-12-29 22:04 UTC #7 lore raw

PATCH 1/1 dm: README: recommend u-boot.dtb to try driver-model on sandbox

Quoted 53 lines · Simon Glass

Hi Masahiro,
Quoted 50 lines · Masahiro Yamada
Hi Simon,
Quoted 42 lines · On Fri, 19 Dec 2014 13:48:16 -0700 Simon Glass <[email protected]> wrote:
Quoted 32 lines · Masahiro Yamada
Jagan,
Quoted 18 lines · On Fri, 19 Dec 2014 18:43:37 +0530 Jagan Teki <[email protected]> wrote:
Quoted 17 lines · Masahiro Yamada
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 <[email protected]>
---
 doc/driver-model/README.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
doc/driver-model/README.txt +0 -0
index 3e2f622..eafa825 100644
@@ -36,9 +36,9 @@ How to try it
 
 Build U-Boot sandbox and run it:
 
Quoted 7 lines
doc/driver-model/README.txt How to try it · line 41
Quoted 5 lines
doc/driver-model/README.txt +2 -2
-   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?
Quoted 3 lines
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 <[email protected]>
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.

Quoted 8 lines

Quoted 4 lines
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.