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]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to