merged. Bruce
In message: [linux-yocto][linux-yocto v5.10/standard/nxp-sdk-5.10/nxp-soc & v5.10/standard/preempt-rt/nxp-sdk-5.10/nxp-soc][PATCH 1/1] Revert "scripts: Makefile: Enable creation of _symbols_ DT node for overlays" on 25/02/2022 Xiaolei Wang wrote: > From: Jason Liu <jason.hui....@nxp.com> > > commit 3c02b480dc20f5f65afc7d51625b5fda40525672 from > https://source.codeaurora.org/external/imx/linux-imx > > 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....@nxp.com> > Signed-off-by: Vladimir Oltean <vladimir.olt...@nxp.com> > Signed-off-by: Xiaolei Wang <xiaolei.w...@windriver.com> > --- > scripts/Makefile.lib | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > index b904c6b07860..94133708889d 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= \ > -- > 2.25.1 >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#10991): https://lists.yoctoproject.org/g/linux-yocto/message/10991 Mute This Topic: https://lists.yoctoproject.org/mt/89387286/21656 Group Owner: linux-yocto+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
