[U-Boot] [PATCH] dts/Makefile: Build the user specified dts

Simon Glass sjg at chromium.org
Thu Feb 28 15:38:17 CET 2013


Hi Jagannadha,

On Thu, Feb 28, 2013 at 6:33 AM, Jagannadha Sutradharudu Teki
<jagannadha.sutradharudu-teki at xilinx.com> wrote:
> This patch provides a support to build the user specified dts.
> If the user is not specifying any dts, then it should build the
> dts from CONFIG_DEFAULT_DEVICE_TREE.
>
> $ make -> choose the default dts, defined as CONFIG_DEFAULT_DEVICE_TREE
> $ make DEVICE_TREE=<dts-file-name> -> choose the specified dts from the user
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jaganna at xilinx.com>

This is a useful patch thank you.

> ---
>  dts/Makefile |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/dts/Makefile b/dts/Makefile
> index 922c78c..dd6c1fa 100644
> --- a/dts/Makefile
> +++ b/dts/Makefile
> @@ -29,7 +29,9 @@ LIB   = $(obj)libdts.o
>
>  $(if $(CONFIG_DEFAULT_DEVICE_TREE),,\
>  $(error Please define CONFIG_DEFAULT_DEVICE_TREE in your board header file))
> +ifneq ($(DEVICE_TREE),y)
>  DEVICE_TREE = $(subst ",,$(CONFIG_DEFAULT_DEVICE_TREE))
> +endif

It would be better if we could not give an error if DEVICE_TREE is
defined, even if there is no CONFIG_DEFAULT_DEVICE_TREE.

>
>  $(if $(CONFIG_ARCH_DEVICE_TREE),,\
>  $(error Your architecture does not have device tree support enabled. \
> --
> 1.7.4
>
>

Regards,
Simon


More information about the U-Boot mailing list