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

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

[Jacob Stiffler]

* This is to avoid conflict with OpenCL firmware.

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-tisdk-multimedia.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
index 81d5ee1..c51a02c 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-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

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

[Denys Dmytriyenko]

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.
> 
> 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-tisdk-multimedia.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
> index 81d5ee1..c51a02c 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-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

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

[Karthik Ramanan]

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-tisdk-multimedia.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-multimedia.bb
>> index 81d5ee1..c51a02c 100644
>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-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
>

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

[Stiffler, Jacob]

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
>

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

[R, Karthik]

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.

Regards
Karthik

-----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
>

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

[Denys Dmytriyenko]

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
> >
> 

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

[Karthik Ramanan]

On 07-Jan-15 11:42 PM, Denys Dmytriyenko wrote:
> 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,

This patch removes dspdce-fw from packagegroup-arago-tisdk-multimedia.bb

So for SDK's that are including this packagegroup, they will miss the 
dspdce-fw and so it will have to be added explicitly in some other 
packagegroup that is SDK specific.

Regards
Karthik