All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm: xilinx: Fill DEVICE_TREE directly in Makefiles
@ 2020-02-13 13:53 Michal Simek
  2020-02-28 11:18 ` Michal Simek
  0 siblings, 1 reply; 2+ messages in thread
From: Michal Simek @ 2020-02-13 13:53 UTC (permalink / raw)
  To: u-boot

DEVICE_TREE variable is not exported from dts/Makefile that's why it is
necessary to setup directly before use.

Fixes: 312f2c5b14fa ("arm: xilinx: Use device tree which can be passed on cmd line")
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

It was found by more intensive testing.

Another option would be to move DEVICE_TREE setup from dts/Makefile to
Makefile but I didn't find the right place for it.

---
 board/xilinx/zynq/Makefile   | 5 +++++
 board/xilinx/zynqmp/Makefile | 5 +++++
 2 files changed, 10 insertions(+)

diff --git a/board/xilinx/zynq/Makefile b/board/xilinx/zynq/Makefile
index 6a2acee108fa..096a7aceb939 100644
--- a/board/xilinx/zynq/Makefile
+++ b/board/xilinx/zynq/Makefile
@@ -13,6 +13,11 @@ spl/board/xilinx/zynq/ps_init_gpl.o board/xilinx/zynq/ps_init_gpl.o: $(PS_INIT_F
 	$(CC) $(c_flags) -I $(srctree)/$(src) -c -o $@ $^
 endif
 
+DEVICE_TREE ?= $(CONFIG_DEFAULT_DEVICE_TREE:"%"=%)
+ifeq ($(DEVICE_TREE),)
+DEVICE_TREE := unset
+endif
+
 ifeq ($(init-objs),)
 hw-platform-y :=$(shell echo $(DEVICE_TREE))
 init-objs := $(if $(wildcard $(srctree)/$(src)/$(hw-platform-y)/ps7_init_gpl.c),\
diff --git a/board/xilinx/zynqmp/Makefile b/board/xilinx/zynqmp/Makefile
index 174f4ed24be3..398c6aaa452a 100644
--- a/board/xilinx/zynqmp/Makefile
+++ b/board/xilinx/zynqmp/Makefile
@@ -13,6 +13,11 @@ spl/board/xilinx/zynqmp/ps_init_gpl.o board/xilinx/zynqmp/ps_init_gpl.o: $(PS_IN
 	$(CC) $(c_flags) -I $(srctree)/$(src) -c -o $@ $^
 endif
 
+DEVICE_TREE ?= $(CONFIG_DEFAULT_DEVICE_TREE:"%"=%)
+ifeq ($(DEVICE_TREE),)
+DEVICE_TREE := unset
+endif
+
 ifeq ($(init-objs),)
 hw-platform-y :=$(shell echo $(DEVICE_TREE))
 init-objs := $(if $(wildcard $(srctree)/$(src)/$(hw-platform-y)/psu_init_gpl.c),\
-- 
2.25.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH] arm: xilinx: Fill DEVICE_TREE directly in Makefiles
  2020-02-13 13:53 [PATCH] arm: xilinx: Fill DEVICE_TREE directly in Makefiles Michal Simek
@ 2020-02-28 11:18 ` Michal Simek
  0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2020-02-28 11:18 UTC (permalink / raw)
  To: u-boot

?t 13. 2. 2020 v 14:53 odes?latel Michal Simek <michal.simek@xilinx.com> napsal:
>
> DEVICE_TREE variable is not exported from dts/Makefile that's why it is
> necessary to setup directly before use.
>
> Fixes: 312f2c5b14fa ("arm: xilinx: Use device tree which can be passed on cmd line")
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> It was found by more intensive testing.
>
> Another option would be to move DEVICE_TREE setup from dts/Makefile to
> Makefile but I didn't find the right place for it.
>
> ---
>  board/xilinx/zynq/Makefile   | 5 +++++
>  board/xilinx/zynqmp/Makefile | 5 +++++
>  2 files changed, 10 insertions(+)
>
> diff --git a/board/xilinx/zynq/Makefile b/board/xilinx/zynq/Makefile
> index 6a2acee108fa..096a7aceb939 100644
> --- a/board/xilinx/zynq/Makefile
> +++ b/board/xilinx/zynq/Makefile
> @@ -13,6 +13,11 @@ spl/board/xilinx/zynq/ps_init_gpl.o board/xilinx/zynq/ps_init_gpl.o: $(PS_INIT_F
>         $(CC) $(c_flags) -I $(srctree)/$(src) -c -o $@ $^
>  endif
>
> +DEVICE_TREE ?= $(CONFIG_DEFAULT_DEVICE_TREE:"%"=%)
> +ifeq ($(DEVICE_TREE),)
> +DEVICE_TREE := unset
> +endif
> +
>  ifeq ($(init-objs),)
>  hw-platform-y :=$(shell echo $(DEVICE_TREE))
>  init-objs := $(if $(wildcard $(srctree)/$(src)/$(hw-platform-y)/ps7_init_gpl.c),\
> diff --git a/board/xilinx/zynqmp/Makefile b/board/xilinx/zynqmp/Makefile
> index 174f4ed24be3..398c6aaa452a 100644
> --- a/board/xilinx/zynqmp/Makefile
> +++ b/board/xilinx/zynqmp/Makefile
> @@ -13,6 +13,11 @@ spl/board/xilinx/zynqmp/ps_init_gpl.o board/xilinx/zynqmp/ps_init_gpl.o: $(PS_IN
>         $(CC) $(c_flags) -I $(srctree)/$(src) -c -o $@ $^
>  endif
>
> +DEVICE_TREE ?= $(CONFIG_DEFAULT_DEVICE_TREE:"%"=%)
> +ifeq ($(DEVICE_TREE),)
> +DEVICE_TREE := unset
> +endif
> +
>  ifeq ($(init-objs),)
>  hw-platform-y :=$(shell echo $(DEVICE_TREE))
>  init-objs := $(if $(wildcard $(srctree)/$(src)/$(hw-platform-y)/psu_init_gpl.c),\
> --
> 2.25.0
>

Applied.
M

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-02-28 11:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-13 13:53 [PATCH] arm: xilinx: Fill DEVICE_TREE directly in Makefiles Michal Simek
2020-02-28 11:18 ` Michal Simek

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.