Patchwork [U-Boot,v2] dts/Makefile: Build the user specified dts

login
register
mail settings
Submitter Jagannadha Sutradharudu Teki
Date Feb. 28, 2013, 8:20 p.m.
Message ID <6cb1adbd-6818-4c98-bd22-dc4ab1467b20@AM1EHSMHS015.ehs.local>
Download mbox | patch
Permalink /patch/224182/
State Accepted
Delegated to: Tom Rini
Headers show

Comments

Jagannadha Sutradharudu Teki - Feb. 28, 2013, 8:20 p.m.
This patch provides a support to build the user specified dts.

Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
---
Changes for v2:
	- Update the logic for not giving an error if DEVICE_TREE defined
	- Documented the build steps 
 doc/README.fdt-control |    8 ++++++++
 dts/Makefile           |    2 ++
 2 files changed, 10 insertions(+), 0 deletions(-)
Simon Glass - Feb. 28, 2013, 8:38 p.m.
On Thu, Feb 28, 2013 at 12:20 PM, Jagannadha Sutradharudu Teki
<jagannadha.sutradharudu-teki@xilinx.com> wrote:
> This patch provides a support to build the user specified dts.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>

Acked-by: Simon Glass <sjg@chromium.org>

Thank you.

> ---
> Changes for v2:
>         - Update the logic for not giving an error if DEVICE_TREE defined
>         - Documented the build steps
>  doc/README.fdt-control |    8 ++++++++
>  dts/Makefile           |    2 ++
>  2 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/doc/README.fdt-control b/doc/README.fdt-control
> index 85bda03..8352835 100644
> --- a/doc/README.fdt-control
> +++ b/doc/README.fdt-control
> @@ -156,6 +156,14 @@ To use this, put something like this in your board header file:
>
>  #define CONFIG_EXTRA_ENV_SETTINGS      "fdtcontroladdr=10000\0"
>
> +Build:
> +
> +After board configuration is done, fdt supported u-boot can be build in two ways:
> +1)  build the default dts which is defined from CONFIG_DEFAULT_DEVICE_TREE
> +    $ make
> +2)  build the user specified dts file
> +    $ make DEVICE_TREE=<dts-file-name>
> +
>
>  Limitations
>  -----------
> diff --git a/dts/Makefile b/dts/Makefile
> index 785104e..01364d7 100644
> --- a/dts/Makefile
> +++ b/dts/Makefile
> @@ -27,9 +27,11 @@ include $(TOPDIR)/config.mk
>
>  LIB    = $(obj)libdts.o
>
> +ifeq ($(DEVICE_TREE),)
>  $(if $(CONFIG_DEFAULT_DEVICE_TREE),,\
>  $(error Please define CONFIG_DEFAULT_DEVICE_TREE in your board header file))
>  DEVICE_TREE = $(subst ",,$(CONFIG_DEFAULT_DEVICE_TREE))
> +endif
>
>  $(if $(CONFIG_ARCH_DEVICE_TREE),,\
>  $(error Your architecture does not have device tree support enabled. \
> --
> 1.7.4
>
>
Tom Rini - April 3, 2013, 3:30 p.m.
On Thu, Feb 28, 2013 at 10:20:18AM -0000, Jagannadha Sutradharudu Teki wrote:

> This patch provides a support to build the user specified dts.
> 
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

Patch

diff --git a/doc/README.fdt-control b/doc/README.fdt-control
index 85bda03..8352835 100644
--- a/doc/README.fdt-control
+++ b/doc/README.fdt-control
@@ -156,6 +156,14 @@  To use this, put something like this in your board header file:
 
 #define CONFIG_EXTRA_ENV_SETTINGS	"fdtcontroladdr=10000\0"
 
+Build:
+
+After board configuration is done, fdt supported u-boot can be build in two ways:
+1)  build the default dts which is defined from CONFIG_DEFAULT_DEVICE_TREE
+    $ make
+2)  build the user specified dts file
+    $ make DEVICE_TREE=<dts-file-name>
+
 
 Limitations
 -----------
diff --git a/dts/Makefile b/dts/Makefile
index 785104e..01364d7 100644
--- a/dts/Makefile
+++ b/dts/Makefile
@@ -27,9 +27,11 @@  include $(TOPDIR)/config.mk
 
 LIB	= $(obj)libdts.o
 
+ifeq ($(DEVICE_TREE),)
 $(if $(CONFIG_DEFAULT_DEVICE_TREE),,\
 $(error Please define CONFIG_DEFAULT_DEVICE_TREE in your board header file))
 DEVICE_TREE = $(subst ",,$(CONFIG_DEFAULT_DEVICE_TREE))
+endif
 
 $(if $(CONFIG_ARCH_DEVICE_TREE),,\
 $(error Your architecture does not have device tree support enabled. \