On Thu, Dec 15, 2022 at 09:38:00AM -0600, Ryan Eatmon via lists.yoctoproject.org wrote: > Add support for a new variable DEVICETREE_PATTERN that we > can set in each machine config that will let us choose > which dtb/dtbo files to include rather than picking > them all. > > Signed-off-by: Ryan Eatmon <reat...@ti.com> > --- > meta-ti-bsp/conf/machine/am62xx-evm.conf | 2 ++ > meta-ti-bsp/conf/machine/am62xx-lp-evm.conf | 2 ++ > meta-ti-bsp/conf/machine/beagle-x15.conf | 2 ++ > meta-ti-bsp/conf/machine/beaglebone.conf | 2 ++ > meta-ti-bsp/conf/machine/include/am64xx.inc | 2 ++ > meta-ti-bsp/conf/machine/include/am65xx.inc | 2 ++ > meta-ti-bsp/conf/machine/include/ti33x.inc | 2 ++ > meta-ti-bsp/conf/machine/j7200-evm.conf | 2 ++ > meta-ti-bsp/conf/machine/j721e-evm.conf | 2 ++ > meta-ti-bsp/conf/machine/j721s2-evm.conf | 2 ++ > meta-ti-bsp/conf/machine/j784s4-evm.conf | 2 ++ > .../recipes-kernel/linux/linux-ti-next_git.bb | 17 +++++++++++------ > 12 files changed, 33 insertions(+), 6 deletions(-) > > diff --git a/meta-ti-bsp/conf/machine/am62xx-evm.conf > b/meta-ti-bsp/conf/machine/am62xx-evm.conf > index 73859d5f..97927b48 100644 > --- a/meta-ti-bsp/conf/machine/am62xx-evm.conf > +++ b/meta-ti-bsp/conf/machine/am62xx-evm.conf > @@ -11,4 +11,6 @@ KERNEL_DEVICETREE = " \ > ti/k3-am625-sk-csi2-ov5640.dtbo \ > " > > +DEVICETREE_PATTERN = "ti/k3-am625-*" > + > UBOOT_MACHINE = "am62x_evm_a53_defconfig" > diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf > b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf > index ef8e8692..a29e1407 100644 > --- a/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf > +++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm.conf > @@ -13,4 +13,6 @@ KERNEL_DEVICETREE = " \ > ti/k3-am625-sk-csi2-ov5640.dtbo \ > " > > +DEVICETREE_PATTERN = "ti/k3-am62*" > + > UBOOT_MACHINE = "am62x_lpsk_a53_defconfig" > diff --git a/meta-ti-bsp/conf/machine/beagle-x15.conf > b/meta-ti-bsp/conf/machine/beagle-x15.conf > index c7503266..14467f24 100644 > --- a/meta-ti-bsp/conf/machine/beagle-x15.conf > +++ b/meta-ti-bsp/conf/machine/beagle-x15.conf > @@ -6,6 +6,8 @@ require conf/machine/include/dra7xx.inc > > KERNEL_DEVICETREE = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revb1.dtb > am57xx-beagle-x15-revc.dtb" > > +DEVICETREE_PATTERN = "am57xx-beagle-*" > + > MACHINE_GUI_CLASS = "bigscreen" > > SERIAL_CONSOLES = "115200;ttyS2" > diff --git a/meta-ti-bsp/conf/machine/beaglebone.conf > b/meta-ti-bsp/conf/machine/beaglebone.conf > index 413a960d..76035a4b 100644 > --- a/meta-ti-bsp/conf/machine/beaglebone.conf > +++ b/meta-ti-bsp/conf/machine/beaglebone.conf > @@ -8,6 +8,8 @@ KERNEL_DEVICETREE = "am335x-bone.dtb am335x-boneblue.dtb \ > am335x-boneblack.dtb am335x-boneblack-wireless.dtb \ > am335x-bonegreen.dtb am335x-bonegreen-wireless.dtb" > > +DEVICETREE_PATTERN = "am335x-*" > + > MACHINE_GUI_CLASS = "bigscreen" > MACHINE_FEATURES += "screen" > > diff --git a/meta-ti-bsp/conf/machine/include/am64xx.inc > b/meta-ti-bsp/conf/machine/include/am64xx.inc > index 2fca1ac0..62c01347 100644 > --- a/meta-ti-bsp/conf/machine/include/am64xx.inc > +++ b/meta-ti-bsp/conf/machine/include/am64xx.inc > @@ -14,6 +14,8 @@ KERNEL_DEVICETREE = " \ > ti/k3-am642-evm-nand.dtbo \ > " > > +DEVICETREE_PATTERN = "ti/k3-am642-*" > + > # Default tiboot3.bin on AM64x is for SR2.0 HS-FS > BBMULTICONFIG = "k3r5-sr2-hs-fs" > do_image[mcdepends] = "mc::k3r5-sr2-hs-fs:ti-sci-fw:do_deploy" > diff --git a/meta-ti-bsp/conf/machine/include/am65xx.inc > b/meta-ti-bsp/conf/machine/include/am65xx.inc > index ef31ea52..5bb84de5 100644 > --- a/meta-ti-bsp/conf/machine/include/am65xx.inc > +++ b/meta-ti-bsp/conf/machine/include/am65xx.inc > @@ -26,6 +26,8 @@ KERNEL_DEVICETREE = " \ > ti/k3-am654-evm-ov5640.dtbo \ > " > > +DEVICETREE_PATTERN = "ti/k3-am654-*" > + > BBMULTICONFIG += "k3r5-sr2" > > # Since J721e does not support multi-cert tiboot3.bin add standalone > sysfw.itb > diff --git a/meta-ti-bsp/conf/machine/include/ti33x.inc > b/meta-ti-bsp/conf/machine/include/ti33x.inc > index d9381555..a327d29e 100644 > --- a/meta-ti-bsp/conf/machine/include/ti33x.inc > +++ b/meta-ti-bsp/conf/machine/include/ti33x.inc > @@ -30,6 +30,8 @@ KERNEL_DEVICETREE = " \ > > KERNEL_DEVICETREE += "${@oe.utils.conditional('ENABLE_TI_UIO_DEVICES', '1', > 'am335x-icev2-pru-excl-uio.dtb', '', d)}" > > +DEVICETREE_PATTERN = "am335x-*" > + > UBOOT_ARCH = "arm" > UBOOT_MACHINE = "am335x_evm_config" > > diff --git a/meta-ti-bsp/conf/machine/j7200-evm.conf > b/meta-ti-bsp/conf/machine/j7200-evm.conf > index 077d2f41..fdffd7a8 100644 > --- a/meta-ti-bsp/conf/machine/j7200-evm.conf > +++ b/meta-ti-bsp/conf/machine/j7200-evm.conf > @@ -11,6 +11,8 @@ KERNEL_DEVICETREE = " \ > ti/k3-j7200-common-proc-board.dtb \ > " > > +DEVICETREE_PATTERN = "ti/k3-j7200-*" > + > UBOOT_MACHINE = "j7200_evm_a72_config" > > do_image[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy" > diff --git a/meta-ti-bsp/conf/machine/j721e-evm.conf > b/meta-ti-bsp/conf/machine/j721e-evm.conf > index d28d16d1..9a22cba9 100644 > --- a/meta-ti-bsp/conf/machine/j721e-evm.conf > +++ b/meta-ti-bsp/conf/machine/j721e-evm.conf > @@ -16,4 +16,6 @@ KERNEL_DEVICETREE = " \ > ti/k3-j721e-sk.dtb \ > " > > +DEVICETREE_PATTERN = "ti/k3-j721e-*" > + > UBOOT_MACHINE = "j721e_evm_a72_config" > diff --git a/meta-ti-bsp/conf/machine/j721s2-evm.conf > b/meta-ti-bsp/conf/machine/j721s2-evm.conf > index d02c8258..bd99728c 100644 > --- a/meta-ti-bsp/conf/machine/j721s2-evm.conf > +++ b/meta-ti-bsp/conf/machine/j721s2-evm.conf > @@ -15,6 +15,8 @@ KERNEL_DEVICETREE = " \ > ti/k3-j721s2-gesi-exp-board.dtbo \ > " > > +DEVICETREE_PATTERN = "ti/k3-j721s2-*" > + > UBOOT_MACHINE = "j721s2_evm_a72_defconfig" > > do_image[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy" > diff --git a/meta-ti-bsp/conf/machine/j784s4-evm.conf > b/meta-ti-bsp/conf/machine/j784s4-evm.conf > index f88006c3..a27c82ec 100644 > --- a/meta-ti-bsp/conf/machine/j784s4-evm.conf > +++ b/meta-ti-bsp/conf/machine/j784s4-evm.conf > @@ -16,6 +16,8 @@ KERNEL_DEVICETREE = " \ > ti/k3-j784s4-evm.dtb \ > " > > +DEVICETREE_PATTERN = "ti/k3-j784s4-*" > + > UBOOT_MACHINE = "j784s4_evm_a72_defconfig" > > do_image[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy" > diff --git a/meta-ti-bsp/recipes-kernel/linux/linux-ti-next_git.bb > b/meta-ti-bsp/recipes-kernel/linux/linux-ti-next_git.bb > index e2908951..6eb75673 100644 > --- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-next_git.bb > +++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-next_git.bb > @@ -19,21 +19,26 @@ PV = "6.1.0-rc1+git${SRCPV}" > > SRC_URI = > "git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master" > > -def list_dtbs(dts_dir, dts_subdir): > +def list_dtbs(dts_dir, dts_subdir, dts_pattern): > import os > import fnmatch > - matches = [] > + all_matches = [] > + final_matches = [] > > for root,dirnames,filenames in os.walk(os.path.join(dts_dir, > dts_subdir)): > for filename in fnmatch.filter(filenames, '*.dtb'): > - matches.append(os.path.join(dts_subdir, filename)) > + all_matches.append(os.path.join(dts_subdir, filename)) > for filename in fnmatch.filter(filenames, '*.dtbo'): > - matches.append(os.path.join(dts_subdir, filename)) > - return ' '.join(matches) > + all_matches.append(os.path.join(dts_subdir, filename)) > + for filename in fnmatch.filter(all_matches, dts_pattern): > + final_matches.append(filename) > + return ' '.join(final_matches)
I feel like with this change DTS_SUBDIR is not being used optimally. You could either: 1. simplify DEVICETREE_PATTERN to not have "ti/" subdir prefix and automatically add DTS_SUBDIR here in the code, but then it would be a bit inconsistent with the KERNEL_DEVICETREE in the machine configs 2. or completely eliminate DTS_SUBDIR and reverse the code above to first match against DEVICETREE_PATTERN and then further filter for *.dtb/*.dtbo > + > +DEVICETREE_PATTERN ?= "*" > > DTS_SUBDIR = "" > DTS_SUBDIR:k3 = "ti" > -KERNEL_DEVICETREE = "${@list_dtbs('${B}/arch/${ARCH}/boot/dts', > '${DTS_SUBDIR}')}" > +KERNEL_DEVICETREE = "${@list_dtbs('${B}/arch/${ARCH}/boot/dts', > '${DTS_SUBDIR}', '${DEVICETREE_PATTERN}')}" > DEFCONFIG_NAME = "multi_v7_defconfig" > DEFCONFIG_NAME:k3 = "defconfig" > KERNEL_CONFIG_COMMAND = "oe_runmake -C ${S} O=${B} ${DEFCONFIG_NAME}" > -- > 2.17.1 >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#15488): https://lists.yoctoproject.org/g/meta-ti/message/15488 Mute This Topic: https://lists.yoctoproject.org/mt/95690764/21656 Group Owner: meta-ti+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-