From 3c02b480dc20f5f65afc7d51625b5fda40525672 Mon Sep 17 00:00:00 2001
From: Jason Liu <jason.hui.liu@nxp.com>
Date: Mon, 5 Jul 2021 17:15:23 +0300
Subject: [PATCH] Revert "scripts: Makefile: Enable creation of _symbols_ DT
 node for overlays"

This reverts commit 11db97cab885454945c4e36d86a27864e994bad3.

The blamed commit changes the common Makefile and force creation of
__symbols__ node on the generated dtb files for all platforms.

This is not good and not acceptible under some cases due to this change
will increase the final DTB size a lot and bring big impact for others
who does not need creation of _symbols_ DT node for overlays.For
example, on i.MX OP-TEE, the maxsize of DT is 1MB, this patch will break
some of the i.MX6/i.MX7 boards to boot with OP-TEE enabled.

A similar patch was submitted and rejected by the community. The
following post discusses this:

https://lore.kernel.org/patchwork/patch/821645/

LS1028A-QDS, which needs the "-@" build flag for device tree overlays,
now sets the "DTC_FLAGS_fsl-ls1028a-qds" GNU Make variable in
arch/arm64/boot/dts/freescale/Makefile, and the support for overlays is
already built for it, so the reverted patch was not even necessary.

Signed-off-by: Jason Liu <jason.hui.liu@nxp.com>
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
 scripts/Makefile.lib | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib
index b904c6b078608b..94133708889d7f 100644
--- a/scripts/Makefile.lib
+++ b/scripts/Makefile.lib
@@ -297,9 +297,6 @@ endif
 
 DTC_FLAGS += $(DTC_FLAGS_$(basetarget))
 
-# Enable creation of _symbols_ node for DT overlays
-DTC_FLAGS += -@
-
 # Generate an assembly file to wrap the output of the device tree compiler
 quiet_cmd_dt_S_dtb= DTB     $@
 cmd_dt_S_dtb=						\
-- 
GitLab