On Mon, May 24, 2021 at 04:52:01PM -0500, Nishanth Menon via lists.yoctoproject.org wrote: > On 15:26-20210524, Yogesh Siraswar wrote: > > Added KERNEL_DTB_OVERLAY_SUPPORT to individual MACHINE conf to > > compile dtbs with symbols. This fixes the issue where the dtbs > > are build without symbols causing issue with overlays. > > Might be worthwhile to add that enabling overlay support increases the > size of the dtb, hence it is done at a platform level rather than > globally to allow custom boards derived from meta-ti to not pay the > penalty of extra dtb size, yet have the flexibility to enable on need > basis. > > > > > Changes from last version: > > 1) Added human readable flag KERNEL_DTB_OVERLAY_SUPPORT to control > > DTC_FLAGS. > > 2) The KERNEL_DTB_OVERLAY_SUPPORT is also derived from build-root > > usage. > > Please don't add change log of patch version in commit message. That is > not an information that is useful in git history. > > > > > Signed-off-by: Yogesh Siraswar <yoge...@ti.com> > > --- > > conf/machine/am335x-evm.conf | 2 ++ > > conf/machine/am335x-hs-evm.conf | 2 ++ > > conf/machine/am437x-evm.conf | 2 ++ > > conf/machine/am437x-hs-evm.conf | 2 ++ > > conf/machine/am57xx-evm.conf | 2 ++ > > conf/machine/am57xx-hs-evm.conf | 3 +++ > > conf/machine/am65xx-evm.conf | 2 ++ > > conf/machine/am65xx-hs-evm.conf | 2 ++ > > conf/machine/dra7xx-evm.conf | 2 ++ > > conf/machine/dra7xx-hs-evm.conf | 3 +++ > > conf/machine/j7-evm.conf | 2 ++ > > conf/machine/j7-hs-evm.conf | 2 ++ > > conf/machine/j7200-evm.conf | 2 ++ > > conf/machine/k2g-evm.conf | 2 ++ > > conf/machine/k2g-hs-evm.conf | 2 ++ > > recipes-kernel/linux/linux-ti-mainline_git.bb | 6 ++++-- > > recipes-kernel/linux/linux-ti-staging_5.10.bb | 6 ++++-- > > recipes-kernel/linux/ti-kernel.inc | 9 +++++++++ > > 18 files changed, 49 insertions(+), 4 deletions(-) > > create mode 100644 recipes-kernel/linux/ti-kernel.inc > > > > + > > [...] > > > +def get_extra_dtc_args(d): > > + if d.getVar('KERNEL_DTB_OVERLAY_SUPPORT') == "1": > > + return "DTC_FLAGS=-@" > > + else: > > + return "" > > + > > +EXTRA_DTC_ARGS += "${@get_extra_dtc_args(d)}" > > -- > > 2.17.1 > > > > just for the record for other bb files: > linux-ti-staging-rt_5.10.bb linux-ti-staging-systest_5.10.bb > > I think this opens up other possibilities in the future to move common > operations into ti-kernel.inc and probably drop duplications
Both -rt and -systest are built on top of the main recipe, they just change defconfig and/or branch: https://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/recipes-kernel/linux/linux-ti-staging-systest_5.10.bb https://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/recipes-kernel/linux/linux-ti-staging-rt_5.10.bb -- Regards, Denys Dmytriyenko <de...@denix.org> PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964 Fingerprint: 25FC E4A5 8A72 2F69 1186 6D76 4209 0272 9A92 C964
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#13790): https://lists.yoctoproject.org/g/meta-ti/message/13790 Mute This Topic: https://lists.yoctoproject.org/mt/83061473/21656 Group Owner: meta-ti+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-