[PATCH] tisdk-server-extra-rootfs-image: only add multimedia when alsa is enabled

[PATCH] tisdk-server-extra-rootfs-image: only add multimedia when alsa is enabled

[Denys Dmytriyenko]

From: Denys Dmytriyenko <denys@ti.com>

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 .../recipes-core/images/tisdk-server-extra-rootfs-image.bb              | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-image.bb b/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-image.bb
index 432f421..51237cb 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-image.bb
@@ -3,7 +3,7 @@ require recipes-core/images/tisdk-server-rootfs-image.bb
 IMAGE_INSTALL += "\
     packagegroup-arago-test-addons \
     packagegroup-arago-tisdk-addons-extra \
-    packagegroup-arago-tisdk-multimedia \
+    ${@base_contains('MACHINE_FEATURES', 'alsa', 'packagegroup-arago-tisdk-multimedia', '',d)} \
 "
 
 export IMAGE_BASENAME = "tisdk-server-extra-rootfs-image"
-- 
2.2.0

Re: [PATCH] tisdk-server-extra-rootfs-image: only add multimedia when alsa is enabled

[Gou, Hongmei]

Hi Denys,

Multimedia/gstreamer packagegroup is needed for all K2 platforms due to the reasons below:

1. OpenCV is intended for all K2 platforms, which has dependency on gstreamer
2. The dsp66 GST plugin (a new feature in the coming Processor SDK 2.0.2 release) is intended for all K2 platforms, which needs gstreamer also

Thanks,
Hongmei

> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> Sent: Monday, March 07, 2016 4:19 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] tisdk-server-extra-rootfs-image: only add
> multimedia when alsa is enabled
> 
> From: Denys Dmytriyenko <denys@ti.com>
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  .../recipes-core/images/tisdk-server-extra-rootfs-image.bb              | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> image.bb b/meta-arago-distro/recipes-core/images/tisdk-server-extra-
> rootfs-image.bb
> index 432f421..51237cb 100644
> --- a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> image.bb
> +++ b/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> image.bb
> @@ -3,7 +3,7 @@ require recipes-core/images/tisdk-server-rootfs-image.bb
>  IMAGE_INSTALL += "\
>      packagegroup-arago-test-addons \
>      packagegroup-arago-tisdk-addons-extra \
> -    packagegroup-arago-tisdk-multimedia \
> +    ${@base_contains('MACHINE_FEATURES', 'alsa', 'packagegroup-arago-
> tisdk-multimedia', '',d)} \
>  "
> 
>  export IMAGE_BASENAME = "tisdk-server-extra-rootfs-image"
> --
> 2.2.0
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH] tisdk-server-extra-rootfs-image: only add multimedia when alsa is enabled

[Denys Dmytriyenko]

Hongmei,

So, why not include packagegroup-*-gst instead if you only need gstreamer? 
What kind of multimedia there is for Keystone?

-- 
Denys


On Mon, Mar 07, 2016 at 10:40:43PM +0000, Gou, Hongmei wrote:
> Hi Denys,
> 
> Multimedia/gstreamer packagegroup is needed for all K2 platforms due to the reasons below:
> 
> 1. OpenCV is intended for all K2 platforms, which has dependency on gstreamer
> 2. The dsp66 GST plugin (a new feature in the coming Processor SDK 2.0.2 release) is intended for all K2 platforms, which needs gstreamer also
> 
> Thanks,
> Hongmei
> 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > Sent: Monday, March 07, 2016 4:19 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH] tisdk-server-extra-rootfs-image: only add
> > multimedia when alsa is enabled
> > 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  .../recipes-core/images/tisdk-server-extra-rootfs-image.bb              | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > image.bb b/meta-arago-distro/recipes-core/images/tisdk-server-extra-
> > rootfs-image.bb
> > index 432f421..51237cb 100644
> > --- a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > image.bb
> > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > image.bb
> > @@ -3,7 +3,7 @@ require recipes-core/images/tisdk-server-rootfs-image.bb
> >  IMAGE_INSTALL += "\
> >      packagegroup-arago-test-addons \
> >      packagegroup-arago-tisdk-addons-extra \
> > -    packagegroup-arago-tisdk-multimedia \
> > +    ${@base_contains('MACHINE_FEATURES', 'alsa', 'packagegroup-arago-
> > tisdk-multimedia', '',d)} \
> >  "
> > 
> >  export IMAGE_BASENAME = "tisdk-server-extra-rootfs-image"
> > --
> > 2.2.0
> > 
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH] tisdk-server-extra-rootfs-image: only add multimedia when alsa is enabled

[Gou, Hongmei]

Hi Denys,

In the current recipes, packagegroup-arago-tisdk-multimedia is equivalent to packagegroup-*-gst for keystone. We used packagegroup-arago-tisdk-multimedia to allow additions in the future for keystone, e.g., testing clips. For dsp66 gst plugins, we are planning some demos from file to file, or using Ethernet as the input/output source.

Thanks,
Hongmei

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, March 07, 2016 5:44 PM
> To: Gou, Hongmei
> Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] tisdk-server-extra-rootfs-image: only add
> multimedia when alsa is enabled
> 
> Hongmei,
> 
> So, why not include packagegroup-*-gst instead if you only need gstreamer?
> What kind of multimedia there is for Keystone?
> 
> --
> Denys
> 
> 
> On Mon, Mar 07, 2016 at 10:40:43PM +0000, Gou, Hongmei wrote:
> > Hi Denys,
> >
> > Multimedia/gstreamer packagegroup is needed for all K2 platforms due to
> the reasons below:
> >
> > 1. OpenCV is intended for all K2 platforms, which has dependency on
> gstreamer
> > 2. The dsp66 GST plugin (a new feature in the coming Processor SDK 2.0.2
> release) is intended for all K2 platforms, which needs gstreamer also
> >
> > Thanks,
> > Hongmei
> >
> > > -----Original Message-----
> > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > Sent: Monday, March 07, 2016 4:19 PM
> > > To: meta-arago@arago-project.org
> > > Subject: [meta-arago] [PATCH] tisdk-server-extra-rootfs-image: only add
> > > multimedia when alsa is enabled
> > >
> > > From: Denys Dmytriyenko <denys@ti.com>
> > >
> > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > ---
> > >  .../recipes-core/images/tisdk-server-extra-rootfs-image.bb              | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-extra-
> rootfs-
> > > image.bb b/meta-arago-distro/recipes-core/images/tisdk-server-extra-
> > > rootfs-image.bb
> > > index 432f421..51237cb 100644
> > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > > image.bb
> > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > > image.bb
> > > @@ -3,7 +3,7 @@ require recipes-core/images/tisdk-server-rootfs-
> image.bb
> > >  IMAGE_INSTALL += "\
> > >      packagegroup-arago-test-addons \
> > >      packagegroup-arago-tisdk-addons-extra \
> > > -    packagegroup-arago-tisdk-multimedia \
> > > +    ${@base_contains('MACHINE_FEATURES', 'alsa', 'packagegroup-
> arago-
> > > tisdk-multimedia', '',d)} \
> > >  "
> > >
> > >  export IMAGE_BASENAME = "tisdk-server-extra-rootfs-image"
> > > --
> > > 2.2.0
> > >
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH] tisdk-server-extra-rootfs-image: only add multimedia when alsa is enabled

[Denys Dmytriyenko]

So, dsp66 gst plugins would go into gst packagegroup anyway. As of demos and 
clips, sure multimedia packagegroup does make sense. But usually you update 
packagegroup at the end, not in the beginning. That's why right now, when I 
looked at it, it didn't make sense to include multimedia packagegroup for all 
keystones. Ok, I will not change it, but only under the assumption that you 
will be submitting mentioned updates in the near future. Thanks.

-- 
Denys


On Mon, Mar 07, 2016 at 06:41:07PM -0500, Gou, Hongmei wrote:
> Hi Denys,
> 
> In the current recipes, packagegroup-arago-tisdk-multimedia is equivalent to 
> packagegroup-*-gst for keystone. We used packagegroup-arago-tisdk-multimedia 
> to allow additions in the future for keystone, e.g., testing clips. For 
> dsp66 gst plugins, we are planning some demos from file to file, or using 
> Ethernet as the input/output source.
> 
> Thanks,
> Hongmei
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Monday, March 07, 2016 5:44 PM
> > To: Gou, Hongmei
> > Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [PATCH] tisdk-server-extra-rootfs-image: only add
> > multimedia when alsa is enabled
> > 
> > Hongmei,
> > 
> > So, why not include packagegroup-*-gst instead if you only need gstreamer?
> > What kind of multimedia there is for Keystone?
> > 
> > --
> > Denys
> > 
> > 
> > On Mon, Mar 07, 2016 at 10:40:43PM +0000, Gou, Hongmei wrote:
> > > Hi Denys,
> > >
> > > Multimedia/gstreamer packagegroup is needed for all K2 platforms due to
> > the reasons below:
> > >
> > > 1. OpenCV is intended for all K2 platforms, which has dependency on
> > gstreamer
> > > 2. The dsp66 GST plugin (a new feature in the coming Processor SDK 2.0.2
> > release) is intended for all K2 platforms, which needs gstreamer also
> > >
> > > Thanks,
> > > Hongmei
> > >
> > > > -----Original Message-----
> > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > > > Sent: Monday, March 07, 2016 4:19 PM
> > > > To: meta-arago@arago-project.org
> > > > Subject: [meta-arago] [PATCH] tisdk-server-extra-rootfs-image: only add
> > > > multimedia when alsa is enabled
> > > >
> > > > From: Denys Dmytriyenko <denys@ti.com>
> > > >
> > > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > > ---
> > > >  .../recipes-core/images/tisdk-server-extra-rootfs-image.bb              | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-server-extra-
> > rootfs-
> > > > image.bb b/meta-arago-distro/recipes-core/images/tisdk-server-extra-
> > > > rootfs-image.bb
> > > > index 432f421..51237cb 100644
> > > > --- a/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > > > image.bb
> > > > +++ b/meta-arago-distro/recipes-core/images/tisdk-server-extra-rootfs-
> > > > image.bb
> > > > @@ -3,7 +3,7 @@ require recipes-core/images/tisdk-server-rootfs-
> > image.bb
> > > >  IMAGE_INSTALL += "\
> > > >      packagegroup-arago-test-addons \
> > > >      packagegroup-arago-tisdk-addons-extra \
> > > > -    packagegroup-arago-tisdk-multimedia \
> > > > +    ${@base_contains('MACHINE_FEATURES', 'alsa', 'packagegroup-
> > arago-
> > > > tisdk-multimedia', '',d)} \
> > > >  "
> > > >
> > > >  export IMAGE_BASENAME = "tisdk-server-extra-rootfs-image"
> > > > --
> > > > 2.2.0
> > > >
> > > > _______________________________________________
> > > > meta-arago mailing list
> > > > meta-arago@arago-project.org
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago