Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm.

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Jan 07, 2015 at 07:31:41AM -0500, R, Karthik wrote:
> Jake,
> 
> ACK for this patch.
> 
> Ok. I understand. 
> 
> But then we would need the specific SDKs to add either add the "opencl" or 
> "dspdce-fw" into the respective packagegroups as per their need.

> I just confirmed that the arago-glsdk-multimedia-image includes the 
> dspdce-fw explicity.

What do you mean includes it explicitly?

So, this comes back to the original suggestion to include either of the 2 FW 
images conditionally - do you mean to handle this by SDK branding?

-- 
Denys


> -----Original Message-----
> From: Stiffler, Jacob 
> Sent: Wednesday, January 07, 2015 5:46 PM
> To: R, Karthik
> Cc: Dmytriyenko, Denys; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm.
> 
> Karthik,
> 
> Both "opencl" and "dspdce-fw" supply DSP firmware (i.e. "/lib/firmware/dra7-dsp1-fw.xe66").
> 
> Does MM use the DSP on dra7xx-evm? We have not seen any issues in our testing.
> 
> Thanks,
> Jake
> 
> -----Original Message-----
> From: R, Karthik
> Sent: Tuesday, January 06, 2015 10:29 PM
> To: Stiffler, Jacob
> Cc: Dmytriyenko, Denys; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm.
> 
> Jake,
> 
> 
> On 06-Jan-15 11:48 PM, Denys Dmytriyenko wrote:
> > So, this will break MM for dra7xx-evm, but we still don't have OpenCL 
> > enabled by default. Should this be more conditional, if both of those can't co-exist?
> 
> >
> >
> > On Tue, Jan 06, 2015 at 09:53:22AM -0500, Jacob Stiffler wrote:
> >> * This is to avoid conflict with OpenCL firmware.
> >>
> 
> Could you please give more details about the conflict with OpenCL firmware?
> 
> Regards
> Karthik
> 
> >> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >> ---
> >>   .../packagegroup-arago-tisdk-multimedia.bb         |    3 +--
> >>   1 file changed, 1 insertion(+), 2 deletions(-)
> >>
> >> diff --git
> >> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tis
> >> dk-multimedia.bb
> >> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tis
> >> dk-multimedia.bb
> >> index 81d5ee1..c51a02c 100644
> >> ---
> >> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tis
> >> dk-multimedia.bb
> >> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago
> >> +++ -tisdk-multimedia.bb
> >> @@ -1,6 +1,6 @@
> >>   DESCRIPTION = "Task to add multimedia related packages"
> >>   LICENSE = "MIT"
> >> -PR = "r10"
> >> +PR = "r11"
> >>
> >>   inherit packagegroup
> >>
> >> @@ -49,7 +49,6 @@ ACCEL_FW_append_omap5-evm = " \
> >>
> >>   ACCEL_FW_append_dra7xx-evm = " \
> >>       ${@base_contains('MACHINE_FEATURES', 'mmip', 'ipumm-fw', '', d)} \
> >> -    dspdce-fw \
> >>       vis \
> >>       "
> >>
> >> --
> >> 1.7.9.5
> >>
> >> _______________________________________________
> >> 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
> >
>