[OE-core] [PATCH 1/1] linux-yocto: Add SCSI support when ptest is in DISTRO_FEATURES

[Bruce Ashfield]

On Wed, Jun 19, 2019 at 6:51 AM Richard Purdie
<richard.purdie at linuxfoundation.org> 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 <just.another.mariano at gmail.com>
> > ---
> >  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.

Bruce

>
> Cheers,
>
> Richard
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II