* [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. @ 2015-01-06 14:53 Jacob Stiffler 2015-01-06 18:18 ` Denys Dmytriyenko 0 siblings, 1 reply; 7+ messages in thread From: Jacob Stiffler @ 2015-01-06 14:53 UTC (permalink / raw) To: meta-arago * 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 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. 2015-01-06 14:53 [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm Jacob Stiffler @ 2015-01-06 18:18 ` Denys Dmytriyenko 2015-01-07 3:29 ` Karthik Ramanan 0 siblings, 1 reply; 7+ messages in thread From: Denys Dmytriyenko @ 2015-01-06 18:18 UTC (permalink / raw) To: Jacob Stiffler; +Cc: meta-arago 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. 2015-01-06 18:18 ` Denys Dmytriyenko @ 2015-01-07 3:29 ` Karthik Ramanan 2015-01-07 12:15 ` Stiffler, Jacob 0 siblings, 1 reply; 7+ messages in thread From: Karthik Ramanan @ 2015-01-07 3:29 UTC (permalink / raw) To: Jacob Stiffler; +Cc: meta-arago 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 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. 2015-01-07 3:29 ` Karthik Ramanan @ 2015-01-07 12:15 ` Stiffler, Jacob 2015-01-07 12:31 ` R, Karthik 0 siblings, 1 reply; 7+ messages in thread From: Stiffler, Jacob @ 2015-01-07 12:15 UTC (permalink / raw) To: R, Karthik; +Cc: meta-arago@arago-project.org 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 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. 2015-01-07 12:15 ` Stiffler, Jacob @ 2015-01-07 12:31 ` R, Karthik 2015-01-07 18:12 ` Denys Dmytriyenko 0 siblings, 1 reply; 7+ messages in thread From: R, Karthik @ 2015-01-07 12:31 UTC (permalink / raw) To: Stiffler, Jacob; +Cc: meta-arago@arago-project.org 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 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. 2015-01-07 12:31 ` R, Karthik @ 2015-01-07 18:12 ` Denys Dmytriyenko 2015-01-08 6:34 ` Karthik Ramanan 0 siblings, 1 reply; 7+ messages in thread From: Denys Dmytriyenko @ 2015-01-07 18:12 UTC (permalink / raw) To: R, Karthik; +Cc: meta-arago@arago-project.org 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 > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm. 2015-01-07 18:12 ` Denys Dmytriyenko @ 2015-01-08 6:34 ` Karthik Ramanan 0 siblings, 0 replies; 7+ messages in thread From: Karthik Ramanan @ 2015-01-08 6:34 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: meta-arago@arago-project.org, R, Karthik 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-01-08 6:34 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-01-06 14:53 [PATCH] packagegroup-arago-tisdk-multimedia: Remove dspdce-fw from dra7xx-evm Jacob Stiffler 2015-01-06 18:18 ` Denys Dmytriyenko 2015-01-07 3:29 ` Karthik Ramanan 2015-01-07 12:15 ` Stiffler, Jacob 2015-01-07 12:31 ` R, Karthik 2015-01-07 18:12 ` Denys Dmytriyenko 2015-01-08 6:34 ` Karthik Ramanan
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.