On Wed, 2019-06-19 at 08:24 -0400, Bruce Ashfield wrote: > On Wed, Jun 19, 2019 at 6:51 AM Richard Purdie > <[email protected]> wrote: > > On Tue, 2019-06-18 at 15:10 -0500, Mariano López wrote: > > > The scsi_debug module can't be built in some architectures as > > > they > > > have > > > SCSI support disabled by default. This patch will enable SCSI > > > support > > > as > > > well as the scsi_debug module when ptest is in DISTRO_FEATURES. > > > > > > Signed-off-by: Mariano López <[email protected]> > > > --- > > > meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb | 5 ++++- > > > meta/recipes-kernel/linux/linux-yocto-rt_5.0.bb | 5 ++++- > > > meta/recipes-kernel/linux/linux-yocto_4.19.bb | 5 ++++- > > > meta/recipes-kernel/linux/linux-yocto_5.0.bb | 5 ++++- > > > 4 files changed, 16 insertions(+), 4 deletions(-) > > > > > > diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > index 0836dc7ea0..e6c8ee320c 100644 > > > --- a/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > +++ b/meta/recipes-kernel/linux/linux-yocto-rt_4.19.bb > > > @@ -41,4 +41,7 @@ KERNEL_FEATURES_append = " > > > ${KERNEL_EXTRA_FEATURES}" > > > KERNEL_FEATURES_append_qemuall=" cfg/virtio.scc" > > > KERNEL_FEATURES_append_qemux86=" cfg/sound.scc > > > cfg/paravirt_kvm.scc" > > > KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc > > > cfg/paravirt_kvm.scc" > > > -KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES" > > > , > > > "ptest", " features/scsi/scsi-debug.scc", "" ,d)}" > > > +KERNEL_FEATURES_append = "${@bb.utils.contains("DISTRO_FEATURES" > > > , > > > "ptest", "\ > > > + features/scsi/scsi.scc \ > > > + features/scsi/scsi-debug.scc \ > > > + ","" ,d)}" > > > > Question to Bruce - can we mark scsi-debug.scc as having a > > dependency > > on scsi.scc? > > With the goal that we can only list one of the fragments in the > KERNEL_FEATURES append ? > We can do something like that, I just want to make sure I'm grok'ing > the end goal.
Yes, it just seems odd that you'd have to remember to list both. If listing both makes sense I'm ok with it, I just thought it worth asking that question as it does seem like a missing dependency. Cheers, Richard -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
