[PATCH] arm64: dts: mediatek: Makefile: Generate symbols for DTBO support

AngeloGioacchino Del Regno posted 1 patch 3 months ago
arch/arm64/boot/dts/mediatek/Makefile | 4 ++++
1 file changed, 4 insertions(+)
[PATCH] arm64: dts: mediatek: Makefile: Generate symbols for DTBO support
Posted by AngeloGioacchino Del Regno 3 months ago
Add DTC_FLAGS '-@' for mt7986a-bananapi-bpi-r3 and -mini to
instruct the devicetree compiler to enable generation of symbols.

This allows proper support for Device Tree Overlay(s) for those
boards; future boards that need DTBO support are expected to add
their own DTC_FLAGS_{dtb-name}.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
 arch/arm64/boot/dts/mediatek/Makefile | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile
index 56d551da917e..5dbb695d053f 100644
--- a/arch/arm64/boot/dts/mediatek/Makefile
+++ b/arch/arm64/boot/dts/mediatek/Makefile
@@ -84,3 +84,7 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt8390-genio-700-evk.dtb
 dtb-$(CONFIG_ARCH_MEDIATEK) += mt8395-kontron-3-5-sbc-i1200.dtb
 dtb-$(CONFIG_ARCH_MEDIATEK) += mt8395-radxa-nio-12l.dtb
 dtb-$(CONFIG_ARCH_MEDIATEK) += mt8516-pumpkin.dtb
+
+# Device tree overlays support
+DTC_FLAGS_mt7986a-bananapi-bpi-r3 := -@
+DTC_FLAGS_mt7986a-bananapi-bpi-r3-mini := -@
-- 
2.45.2
Re: [PATCH] arm64: dts: mediatek: Makefile: Generate symbols for DTBO support
Posted by Conor Dooley 3 months ago
On Thu, Jun 20, 2024 at 12:18:30PM +0200, AngeloGioacchino Del Regno wrote:
> Add DTC_FLAGS '-@' for mt7986a-bananapi-bpi-r3 and -mini to
> instruct the devicetree compiler to enable generation of symbols.
> 
> This allows proper support for Device Tree Overlay(s) for those
> boards; future boards that need DTBO support are expected to add
> their own DTC_FLAGS_{dtb-name}.
> 
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

Adding these symbols has on a few occasions lead to people asking if the
dt maintainers were okay with it, so:
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>