On Mon, May 24, 2021 at 05:29:15PM -0500, Yogesh Siraswar via lists.yoctoproject.org wrote: > > On 5/24/21 5:02 PM, Denys Dmytriyenko wrote: > >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. > Will add in new version > >> > >>>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. > Will remove it
Should instead go below --- line. > >>>Signed-off-by: Yogesh Siraswar <[email protected]> > >>>--- > >>> 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 > I guess no action here. > > -- Regards, Denys Dmytriyenko <[email protected]> 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 (#13793): https://lists.yoctoproject.org/g/meta-ti/message/13793 Mute This Topic: https://lists.yoctoproject.org/mt/83061473/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
