On Wed, May 8, 2024 at 11:26 PM Khem Raj <[email protected]> wrote: > > On Wed, May 8, 2024 at 8:16 PM Bruce Ashfield <[email protected]> > wrote: > > > > On Wed, May 8, 2024 at 9:22 PM Khem Raj <[email protected]> wrote: > > > > > > Signed-off-by: Khem Raj <[email protected]> > > > Cc: Bruce Ashfield <[email protected]> > > > --- > > > meta/recipes-kernel/linux/linux-yocto_6.6.bb | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > > b/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > > index 71bfbf9e032..4503ec94ae9 100644 > > > --- a/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > > +++ b/meta/recipes-kernel/linux/linux-yocto_6.6.bb > > > @@ -66,6 +66,7 @@ KERNEL_FEATURES:append:qemux86-64=" cfg/sound.scc > > > cfg/paravirt_kvm.scc" > > > KERNEL_FEATURES:append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", > > > " cfg/x32.scc", "", d)}" > > > KERNEL_FEATURES:append = " ${@bb.utils.contains("DISTRO_FEATURES", > > > "ptest", " features/scsi/scsi-debug.scc features/nf_tables/nft_test.scc", > > > "", d)}" > > > KERNEL_FEATURES:append = " ${@bb.utils.contains("DISTRO_FEATURES", > > > "ptest", " features/gpio/mockup.scc features/gpio/sim.scc", "", d)}" > > > > We are getting quite a few of these now and that isn't a problem. But > > I think we should > > probably start putting comments above the enable lines so casual users > > will know why > > the various fragments are being enabled. > > > > For example, I don't really know what the team driver is, and why > > ptest needs it! > > when we expand ptests for more than core layer, then certain packages > e.g. libteam in meta-oe [1] this config option is needed to be > enabled. >
Cool. I don't have any concerns with the enablement itself, I'd just prefer that we put a line above it in the file to indicate what depends on that configuration so there's no confusion later. Bruce > [1] > https://git.openembedded.org/meta-openembedded/commit/?id=df35bef3ae6411900efdfc3ea5ade23820e9af2f > > > > > Bruce > > > > > +KERNEL_FEATURES:append = " ${@bb.utils.contains("DISTRO_FEATURES", > > > "ptest", " features/net/team/team.scc", "", d)}" > > > KERNEL_FEATURES:append:powerpc =" arch/powerpc/powerpc-debug.scc" > > > KERNEL_FEATURES:append:powerpc64 =" arch/powerpc/powerpc-debug.scc" > > > KERNEL_FEATURES:append:powerpc64le =" arch/powerpc/powerpc-debug.scc" > > > -- > > > 2.45.0 > > > > > > > > > -- > > - Thou shalt not follow the NULL pointer, for chaos and madness await > > thee at its end > > - "Use the force Harry" - Gandalf, Star Trek II -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#199175): https://lists.openembedded.org/g/openembedded-core/message/199175 Mute This Topic: https://lists.openembedded.org/mt/105993917/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
