Skip to contentSkip to conversation

ARM: dts: Makefile: Use board to decide to compile dts

PATCH Superseded Wenyou Yang 4 participants 4 messages Sep 18, 2016 – Sep 23, 2016 Arm (32-bit)

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: dts: Makefile: Use board to decide to compile dts

To avoid to compile all AT91 SoCs' board dts files, use the board to decide to build its corresponding dts, instead of CONFIG_AT91FAMILY.

---

 arch/arm/dts/Makefile | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
arch/arm/dts/Makefile +4 -4
index 223124e..2ac1f20 100644
@@ -2,10 +2,10 @@
 # SPDX-License-Identifier:	GPL-2.0+
 #
 
-dtb-$(CONFIG_AT91FAMILY) += at91sam9260-smartweb.dtb \
-	at91sam9g20-taurus.dtb \
-	at91sam9g45-corvus.dtb \
-	at91sam9g45-gurnard.dtb
+dtb-$(CONFIG_TARGET_SMARTWEB) += at91sam9260-smartweb.dtb
+dtb-$(CONFIG_TARGET_TAURUS) += at91sam9g20-taurus.dtb
+dtb-$(CONFIG_TARGET_CORVUS) += at91sam9g45-corvus.dtb
+dtb-$(CONFIG_TARGET_GURNARD) += at91sam9g45-gurnard.dtb
 
 dtb-$(CONFIG_S5PC100) += s5pc1xx-smdkc100.dtb
 dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
Signature · 1 line
2.7.4
#1 lore raw

PATCH 1/1 ARM: dts: Makefile: Use board to decide to compile dts

Hello Wenyou Yang,

Am 18.09.2016 um 09:40 schrieb Wenyou Yang:

Quoted 9 lines

To avoid to compile all AT91 SoCs' board dts files, use the board to decide to build its corresponding dts, instead of CONFIG_AT91FAMILY.

---

  arch/arm/dts/Makefile | 8 ++++----
  1 file changed, 4 insertions(+), 4 deletions(-)

bye,
Heiko

Quoted 21 lines
arch/arm/dts/Makefile +4 -4
index 223124e..2ac1f20 100644
@@ -2,10 +2,10 @@
  # SPDX-License-Identifier:	GPL-2.0+
  #
 
-dtb-$(CONFIG_AT91FAMILY) += at91sam9260-smartweb.dtb \
-	at91sam9g20-taurus.dtb \
-	at91sam9g45-corvus.dtb \
-	at91sam9g45-gurnard.dtb
+dtb-$(CONFIG_TARGET_SMARTWEB) += at91sam9260-smartweb.dtb
+dtb-$(CONFIG_TARGET_TAURUS) += at91sam9g20-taurus.dtb
+dtb-$(CONFIG_TARGET_CORVUS) += at91sam9g45-corvus.dtb
+dtb-$(CONFIG_TARGET_GURNARD) += at91sam9g45-gurnard.dtb
 
  dtb-$(CONFIG_S5PC100) += s5pc1xx-smdkc100.dtb
  dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
Signature · 2 lines
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
#2 lore raw

PATCH 1/1 ARM: dts: Makefile: Use board to decide to compile dts

In reply to Wenyou Yang

Hi,

Quoted 9 lines · Wenyou Yang

To avoid to compile all AT91 SoCs' board dts files, use the board to decide to build its corresponding dts, instead of CONFIG_AT91FAMILY.

---

 arch/arm/dts/Makefile | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

What is the purpose of this? It seems unnecessary to me. Compiling a few more .dts files will be quick.

Quoted 21 lines
arch/arm/dts/Makefile +4 -5
index 223124e..2ac1f20 100644
@@ -2,10 +2,10 @@
 # SPDX-License-Identifier:     GPL-2.0+
 #
 
-dtb-$(CONFIG_AT91FAMILY) += at91sam9260-smartweb.dtb \
-       at91sam9g20-taurus.dtb \
-       at91sam9g45-corvus.dtb \
-       at91sam9g45-gurnard.dtb
+dtb-$(CONFIG_TARGET_SMARTWEB) += at91sam9260-smartweb.dtb
+dtb-$(CONFIG_TARGET_TAURUS) += at91sam9g20-taurus.dtb
+dtb-$(CONFIG_TARGET_CORVUS) += at91sam9g45-corvus.dtb
+dtb-$(CONFIG_TARGET_GURNARD) += at91sam9g45-gurnard.dtb
 
 dtb-$(CONFIG_S5PC100) += s5pc1xx-smdkc100.dtb
 dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
--
Quoted 2 lines

2.7.4

Regards,
Simon

#3 lore raw

PATCH 1/1 ARM: dts: Makefile: Use board to decide to compile dts

Quoted 17 lines

-----Original Message-----
From: sjg at google.com [mailto:sjg at google.com] On Behalf Of Simon Glass
Sent: 2016?9?23? 12:17
To: Wenyou Yang - A41535 <[email protected]>
Cc: U-Boot Mailing List <[email protected]>; Stephen Warren <[email protected]>; Andreas Bie?mann <[email protected]>; Wenyou Yang - A41535 <[email protected]>
Subject: Re: [PATCH v1] ARM: dts: Makefile: Use board to decide to compile dts

Hi,

Quoted 3 lines · Wenyou Yang

To avoid to compile all AT91 SoCs' board dts files, use the board to decide to build its corresponding dts, instead of CONFIG_AT91FAMILY.

Quoted 8 lines
Quoted 5 lines

---

 arch/arm/dts/Makefile | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

What is the purpose of this? It seems unnecessary to me. Compiling a few more .dts files will be quick.

Why we need to compile all AT91FAMILY dts files, not be used for a specific AT91 board?

It is enough to compile the dts file used for the target board. Others is unnecessary.

Quoted 19 lines
Quoted 19 lines
arch/arm/dts/Makefile index renamed from arch/arm/dts/Makefile +0 -0

223124e..2ac1f20 100644

arch/arm/dts/Makefile +4 -4
@@ -2,10 +2,10 @@
 # SPDX-License-Identifier:     GPL-2.0+
 #
 
-dtb-$(CONFIG_AT91FAMILY) += at91sam9260-smartweb.dtb \
-       at91sam9g20-taurus.dtb \
-       at91sam9g45-corvus.dtb \
-       at91sam9g45-gurnard.dtb
+dtb-$(CONFIG_TARGET_SMARTWEB) += at91sam9260-smartweb.dtb
+dtb-$(CONFIG_TARGET_TAURUS) += at91sam9g20-taurus.dtb
+dtb-$(CONFIG_TARGET_CORVUS) += at91sam9g45-corvus.dtb
+dtb-$(CONFIG_TARGET_GURNARD) += at91sam9g45-gurnard.dtb
 
Quoted 8 lines
Quoted 5 lines
arch/arm/dts/Makefile +0 -1
 dtb-$(CONFIG_S5PC100) += s5pc1xx-smdkc100.dtb
 dtb-$(CONFIG_S5PC110) += s5pc1xx-goni.dtb
--

2.7.4

Regards,
Simon

Best Regards,
Wenyou Yang