Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
On Thu, Sep 13, 2018 at 05:20:04PM -0400, Xu, Jianzhong wrote:
>Denys,
>
>Thanks for the cleaned up patches. I applied those patches and removed
>COMPATIBLE_MACHINE and resubmitted.
>
>Regards,
>
>Jianzhong
>
> --
>
>From: meta-ti-boun...@yoctoproject.org
>on behalf of Dmytriyenko, Denys
>Sent: Thursday, September 13, 2018 2:35 PM
>To: Stiffler, Jacob
>Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
>build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
>DSPLIB.
>
>Jake, Jianzhong,
>
>I have re-posted the cleaned up patches I had in rocko-next. Please use
>those
>for fixing the COMPATIBLE_MACHINE issue. For now I'll drop them, as I need
>rocko-next freed up.
>
>Denys
>
>On Thu, Sep 13, 2018 at 12:34:40PM -0400, Denys Dmytriyenko wrote:
>> On Thu, Sep 13, 2018 at 12:01:32PM -0400, Stiffler, Jacob wrote:
>> >That will work as c66x is a subset of the keystone SOC family.
>> >
>> >
>> >
>> >I just don’t think that specifying COMPATIBLE_MACHINE is necessary
>as
>> >these packages are independent of the architecture that is running
>Linux.
>>
>> Agree.
>>
>>
>> >And due to this, we should also remove the ‘PACKAGE_ARCH =
>> >${MACHINE_ARCH}”’ as there is no difference between the resulting
>packages
>> >for any machine.
>>
>> Also agree.
>>
>>
>> >IMHO, COMPATIBLE_MACHINE has been overused and it creates more
>problems
>> >that it solves.
>>
>> I've been saying this for years! Still, the response from developers was
>that
>> they only tested it on one platform, hence it's only compatible with
>this one
>> platform...
>>
>>
>> Jake,
>>
>> Please work with your team to explain this and stop abusing
>COMPATIBLE_MACHINE
>> variable. We'll also need to work with RTOS team for the same...
>>
> > --
> > Denys
> >
> >
>> >
>> >
>> >
> > > - Jake
>> >
>> >
>> >
>> >From: Xu, Jianzhong
>> >Sent: Thursday, September 13, 2018 11:45 AM
>> >To: Dmytriyenko, Denys; Stiffler, Jacob
>> >Cc: meta-ti@yoctoproject.org
>> >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB
>recipe to
>> >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB
>and
>> >DSPLIB.
>> >
>> >
>> >
>> >Denys,
>> >
>> >
>> >
>> >Would the following fix the problems you've seen:
> > >
> > >
> > >
> > >COMPATIBLE_MACHINE = "omap-a15|keystone"
> > >
>> >
>> >
>> >This was in DSPLIB recipes before.
>> >
>> >
>> >
>> >Thanks,
>> >
>> >Jianzhong
>> >
>> >
>
>> >
>> >From: Dmytriyenko, Denys
>> >Sent: Thursday, September 13, 2018 11:26:45 AM
>> >To: Stiffler, Jacob
>> >Cc: Xu, Jianzhong; meta-ti@yoctoproject.org
>> >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB
>recipe to
>> >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB
>and
>> >DSPLIB.
>> >
>> >
>> >
>> >On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote:
>> >>
>> >>
>> >> On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote:
>> >> >Same for dra7xx-evm...
>> >> >
>> >> >
>> >> >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys
>wrote:
>> >> >>And it crashed and burned right away:
>> >> >>
>> >
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Denys,
Thanks for the cleaned up patches. I applied those patches and removed
COMPATIBLE_MACHINE and resubmitted.
Regards,
Jianzhong
From: meta-ti-boun...@yoctoproject.org on
behalf of Dmytriyenko, Denys
Sent: Thursday, September 13, 2018 2:35 PM
To: Stiffler, Jacob
Cc: meta-ti@yoctoproject.org
Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build
DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Jake, Jianzhong,
I have re-posted the cleaned up patches I had in rocko-next. Please use those
for fixing the COMPATIBLE_MACHINE issue. For now I'll drop them, as I need
rocko-next freed up.
Denys
On Thu, Sep 13, 2018 at 12:34:40PM -0400, Denys Dmytriyenko wrote:
> On Thu, Sep 13, 2018 at 12:01:32PM -0400, Stiffler, Jacob wrote:
> >That will work as c66x is a subset of the keystone SOC family.
> >
> >
> >
> >I just don’t think that specifying COMPATIBLE_MACHINE is necessary as
> >these packages are independent of the architecture that is running Linux.
>
> Agree.
>
>
> >And due to this, we should also remove the ‘PACKAGE_ARCH =
> >${MACHINE_ARCH}”’ as there is no difference between the resulting
> > packages
> >for any machine.
>
> Also agree.
>
>
> >IMHO, COMPATIBLE_MACHINE has been overused and it creates more problems
> >that it solves.
>
> I've been saying this for years! Still, the response from developers was that
> they only tested it on one platform, hence it's only compatible with this one
> platform...
>
>
> Jake,
>
> Please work with your team to explain this and stop abusing COMPATIBLE_MACHINE
> variable. We'll also need to work with RTOS team for the same...
>
> --
> Denys
>
>
> >
> >
> >
> >- Jake
> >
> >
> >
> >From: Xu, Jianzhong
> > Sent: Thursday, September 13, 2018 11:45 AM
> >To: Dmytriyenko, Denys; Stiffler, Jacob
> >Cc: meta-ti@yoctoproject.org
> >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
> >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
> >DSPLIB.
> >
> >
> >
> >Denys,
> >
> >
> >
> >Would the following fix the problems you've seen:
> >
> >
> >
> >COMPATIBLE_MACHINE = "omap-a15|keystone"
> >
> >
> >
> >This was in DSPLIB recipes before.
> >
> >
> >
> > Thanks,
> >
> > Jianzhong
> >
> > ----
> >
> >From: Dmytriyenko, Denys
> >Sent: Thursday, September 13, 2018 11:26:45 AM
> >To: Stiffler, Jacob
> >Cc: Xu, Jianzhong; meta-ti@yoctoproject.org
> >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
> >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
> >DSPLIB.
> >
> >
> >
> >On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote:
> >>
> >>
> >> On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote:
> >> >Same for dra7xx-evm...
> >> >
> >> >
> >> >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote:
> >> >>And it crashed and burned right away:
> >> >>
> >> >>ERROR: Nothing PROVIDES 'dsplib-c66x' (but
> >
> > /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb
> >DEPENDS on or otherwise requires it)
> >> >>dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not
> >in COMPATIBLE_MACHINE)
> >> >>NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is
> >unbuildable, removing...
> >> >>
> >> >>Jake,
> >> >>
> >> >>Did you tested it on am57xx-hs-evm?
> >>
> >> I had only built for am57xx-evm as there are no machine overrides in
> >these
> >> recipes because they are DSP architecture-specific.
> > >
> > > Looks like we should change "COMPATIBLE_MACHINE" to, at the very least,
> >> include what was there before (omap-a15|keystone). But as these are
> >specific
> >> to the DSP architecture, should COMPATIBLE_MACHINE even be necessary?
> >
> >Either way should be fine. Currently those patches break too
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Jake, Jianzhong,
I have re-posted the cleaned up patches I had in rocko-next. Please use those
for fixing the COMPATIBLE_MACHINE issue. For now I'll drop them, as I need
rocko-next freed up.
Denys
On Thu, Sep 13, 2018 at 12:34:40PM -0400, Denys Dmytriyenko wrote:
> On Thu, Sep 13, 2018 at 12:01:32PM -0400, Stiffler, Jacob wrote:
> >That will work as c66x is a subset of the keystone SOC family.
> >
> >
> >
> >I just don’t think that specifying COMPATIBLE_MACHINE is necessary as
> >these packages are independent of the architecture that is running Linux.
>
> Agree.
>
>
> >And due to this, we should also remove the ‘PACKAGE_ARCH =
> >${MACHINE_ARCH}”’ as there is no difference between the resulting
> > packages
> >for any machine.
>
> Also agree.
>
>
> >IMHO, COMPATIBLE_MACHINE has been overused and it creates more problems
> >that it solves.
>
> I've been saying this for years! Still, the response from developers was that
> they only tested it on one platform, hence it's only compatible with this one
> platform...
>
>
> Jake,
>
> Please work with your team to explain this and stop abusing
> COMPATIBLE_MACHINE
> variable. We'll also need to work with RTOS team for the same...
>
> --
> Denys
>
>
> >
> >
> >
> >- Jake
> >
> >
> >
> > From: Xu, Jianzhong
> >Sent: Thursday, September 13, 2018 11:45 AM
> > To: Dmytriyenko, Denys; Stiffler, Jacob
> >Cc: meta-ti@yoctoproject.org
> >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
> >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
> >DSPLIB.
> >
> >
> >
> >Denys,
> >
> >
> >
> >Would the following fix the problems you've seen:
> >
> >
> >
> >COMPATIBLE_MACHINE = "omap-a15|keystone"
> >
> >
> >
> >This was in DSPLIB recipes before.
> >
> >
> >
> > Thanks,
> >
> >Jianzhong
> >
> > --------
> >
> >From: Dmytriyenko, Denys
> >Sent: Thursday, September 13, 2018 11:26:45 AM
> >To: Stiffler, Jacob
> >Cc: Xu, Jianzhong; meta-ti@yoctoproject.org
> >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
> >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
> >DSPLIB.
> >
> >
> >
> >On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote:
> >>
> >>
> >> On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote:
> >> >Same for dra7xx-evm...
> >> >
> >> >
> >> >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote:
> >> >>And it crashed and burned right away:
> >> >>
> >> >>ERROR: Nothing PROVIDES 'dsplib-c66x' (but
> >
> > /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb
> >DEPENDS on or otherwise requires it)
> >> >>dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not
> >in COMPATIBLE_MACHINE)
> >> >>NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is
> >unbuildable, removing...
> >> >>
> >> >>Jake,
> >> >>
> >> >>Did you tested it on am57xx-hs-evm?
> >>
> >> I had only built for am57xx-evm as there are no machine overrides in
> >these
> >> recipes because they are DSP architecture-specific.
> > >
> >> Looks like we should change "COMPATIBLE_MACHINE" to, at the very least,
> >> include what was there before (omap-a15|keystone). But as these are
> >specific
> >> to the DSP architecture, should COMPATIBLE_MACHINE even be necessary?
> >
> >Either way should be fine. Currently those patches break too many
> >platforms -
> >dra7 GP and both am5/dra7 HS.
> >
> >> >>Denys
> >> >>
> >> >>
> >> >>>-Original Message-
> >> >>>From: Denys Dmytriyenko [mailto:de...@ti.com]
> >> >>>Sent: Wednesday, September 12, 2018 4:21 PM
> >
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
On Thu, Sep 13, 2018 at 12:01:32PM -0400, Stiffler, Jacob wrote:
>That will work as c66x is a subset of the keystone SOC family.
>
>
>
>I just don’t think that specifying COMPATIBLE_MACHINE is necessary as
>these packages are independent of the architecture that is running Linux.
Agree.
>And due to this, we should also remove the ‘PACKAGE_ARCH =
>${MACHINE_ARCH}”’ as there is no difference between the resulting packages
>for any machine.
Also agree.
>IMHO, COMPATIBLE_MACHINE has been overused and it creates more problems
>that it solves.
I've been saying this for years! Still, the response from developers was that
they only tested it on one platform, hence it's only compatible with this one
platform...
Jake,
Please work with your team to explain this and stop abusing COMPATIBLE_MACHINE
variable. We'll also need to work with RTOS team for the same...
--
Denys
>
>
>
>- Jake
>
>
>
>From: Xu, Jianzhong
>Sent: Thursday, September 13, 2018 11:45 AM
>To: Dmytriyenko, Denys; Stiffler, Jacob
>Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
>build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
>DSPLIB.
>
>
>
>Denys,
>
>
>
>Would the following fix the problems you've seen:
>
>
>
>COMPATIBLE_MACHINE = "omap-a15|keystone"
>
>
>
>This was in DSPLIB recipes before.
>
>
>
>Thanks,
>
>Jianzhong
>
>
>
>From: Dmytriyenko, Denys
>Sent: Thursday, September 13, 2018 11:26:45 AM
> To: Stiffler, Jacob
> Cc: Xu, Jianzhong; meta-ti@yoctoproject.org
>Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
>build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
>DSPLIB.
>
>
>
>On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote:
>>
>>
>> On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote:
>> >Same for dra7xx-evm...
>> >
>> >
>> >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote:
>> >>And it crashed and burned right away:
>> >>
>> >>ERROR: Nothing PROVIDES 'dsplib-c66x' (but
>
> /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb
>DEPENDS on or otherwise requires it)
>> >>dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not
>in COMPATIBLE_MACHINE)
>> >>NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is
>unbuildable, removing...
>> >>
>> >>Jake,
>> >>
>> >>Did you tested it on am57xx-hs-evm?
>>
>> I had only built for am57xx-evm as there are no machine overrides in
>these
>> recipes because they are DSP architecture-specific.
>>
>> Looks like we should change "COMPATIBLE_MACHINE" to, at the very least,
>> include what was there before (omap-a15|keystone). But as these are
>specific
>> to the DSP architecture, should COMPATIBLE_MACHINE even be necessary?
>
> Either way should be fine. Currently those patches break too many
>platforms -
> dra7 GP and both am5/dra7 HS.
>
>> >>Denys
>> >>
>> >>
>> >>>-Original Message-
>> >>>From: Denys Dmytriyenko [mailto:de...@ti.com]
>> >>>Sent: Wednesday, September 12, 2018 4:21 PM
>> >>>To: Xu, Jianzhong
>> >>>Cc: Stiffler, Jacob; meta-ti@yoctoproject.org
>> >>>Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe
>to build
>> >>>DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and
>DSPLIB.
>> >>>
>> >>>Thanks, merged into rocko-next for testing.
>> >>>
>> >>>There were lots of small non-critical things I had to fix, like
>whitespaces,
>> >>>indentation, etc. But the biggest one is that you have to follow
>OpenEmbedded
>> >>>Commit Message Guidelines:
>> >>>
>> >>>https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
>> >>>
>> >>>
>&g
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
That will work as c66x is a subset of the keystone SOC family.
I just don't think that specifying COMPATIBLE_MACHINE is necessary as these
packages are independent of the architecture that is running Linux.
And due to this, we should also remove the 'PACKAGE_ARCH = ${MACHINE_ARCH}"' as
there is no difference between the resulting packages for any machine.
IMHO, COMPATIBLE_MACHINE has been overused and it creates more problems that it
solves.
- Jake
From: Xu, Jianzhong
Sent: Thursday, September 13, 2018 11:45 AM
To: Dmytriyenko, Denys; Stiffler, Jacob
Cc: meta-ti@yoctoproject.org
Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build
DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Denys,
Would the following fix the problems you've seen:
COMPATIBLE_MACHINE = "omap-a15|keystone"
This was in DSPLIB recipes before.
Thanks,
Jianzhong
From: Dmytriyenko, Denys
Sent: Thursday, September 13, 2018 11:26:45 AM
To: Stiffler, Jacob
Cc: Xu, Jianzhong; meta-ti@yoctoproject.org
Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build
DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote:
>
>
> On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote:
> >Same for dra7xx-evm...
> >
> >
> >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote:
> >>And it crashed and burned right away:
> >>
> >>ERROR: Nothing PROVIDES 'dsplib-c66x' (but
> >>/OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb
> >> DEPENDS on or otherwise requires it)
> >>dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in
> >>COMPATIBLE_MACHINE)
> >>NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is
> >>unbuildable, removing...
> >>
> >>Jake,
> >>
> >>Did you tested it on am57xx-hs-evm?
>
> I had only built for am57xx-evm as there are no machine overrides in these
> recipes because they are DSP architecture-specific.
>
> Looks like we should change "COMPATIBLE_MACHINE" to, at the very least,
> include what was there before (omap-a15|keystone). But as these are specific
> to the DSP architecture, should COMPATIBLE_MACHINE even be necessary?
Either way should be fine. Currently those patches break too many platforms -
dra7 GP and both am5/dra7 HS.
> >>Denys
> >>
> >>
> >>>-----Original Message-
> >>>From: Denys Dmytriyenko [mailto:de...@ti.com]
> >>>Sent: Wednesday, September 12, 2018 4:21 PM
> >>>To: Xu, Jianzhong
> >>>Cc: Stiffler, Jacob; meta-ti@yoctoproject.org
> >>>Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to
> >>>build
> >>>DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
> >>>
> >>>Thanks, merged into rocko-next for testing.
> >>>
> >>>There were lots of small non-critical things I had to fix, like
> >>>whitespaces,
> >>>indentation, etc. But the biggest one is that you have to follow
> >>>OpenEmbedded
> >>>Commit Message Guidelines:
> >>>
> >>>https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
> >>>
> >>>
> >>>On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote:
> >>>>Denys,
> >>>>
> >>>>
> >>>>
> >>>>You're right. The versioning was inconsistent. Please use
> >>>>[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3],
> >>>> [EXTERNAL,1/3].
> >>>>The latter two should have had v2 in it. Please see highlights below.
> >>>>
> >>>>
> >>>>
> >>>>
> >>>> ++
> >>>>|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | |
> >>>> || |
> >>>>|build DSPLIB from source. Updated |2018-09-06|Jianzhong
> >>>> Xu||New|
> >>>>|includes/ti-paths.inc for MATHLIB and | |
> >>>> || |
> >>>>|DSPLIB.| |
> >>>> || |
> >>>>
> >>>> |---+--+++---|
> >>>>|[
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Denys, Would the following fix the problems you've seen: COMPATIBLE_MACHINE = "omap-a15|keystone" This was in DSPLIB recipes before. Thanks, Jianzhong From: Dmytriyenko, Denys Sent: Thursday, September 13, 2018 11:26:45 AM To: Stiffler, Jacob Cc: Xu, Jianzhong; meta-ti@yoctoproject.org Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote: > > > On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote: > >Same for dra7xx-evm... > > > > > >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote: > >>And it crashed and burned right away: > >> > >>ERROR: Nothing PROVIDES 'dsplib-c66x' (but > >>/OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb > >> DEPENDS on or otherwise requires it) > >>dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in > >>COMPATIBLE_MACHINE) > >>NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is > >>unbuildable, removing... > >> > >>Jake, > >> > >>Did you tested it on am57xx-hs-evm? > > I had only built for am57xx-evm as there are no machine overrides in these > recipes because they are DSP architecture-specific. > > Looks like we should change "COMPATIBLE_MACHINE" to, at the very least, > include what was there before (omap-a15|keystone). But as these are specific > to the DSP architecture, should COMPATIBLE_MACHINE even be necessary? Either way should be fine. Currently those patches break too many platforms - dra7 GP and both am5/dra7 HS. > >>Denys > >> > >> > >>>-Original Message----- > >>>From: Denys Dmytriyenko [mailto:de...@ti.com] > >>>Sent: Wednesday, September 12, 2018 4:21 PM > >>>To: Xu, Jianzhong > >>>Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > >>>Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > >>>build > >>>DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > >>> > >>>Thanks, merged into rocko-next for testing. > >>> > >>>There were lots of small non-critical things I had to fix, like > >>>whitespaces, > >>>indentation, etc. But the biggest one is that you have to follow > >>>OpenEmbedded > >>>Commit Message Guidelines: > >>> > >>>https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines > >>> > >>> > >>>On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: > >>>>Denys, > >>>> > >>>> > >>>> > >>>>You're right. The versioning was inconsistent. Please use > >>>>[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], > >>>> [EXTERNAL,1/3]. > >>>>The latter two should have had v2 in it. Please see highlights below. > >>>> > >>>> > >>>> > >>>> > >>>> ++ > >>>>|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | | > >>>> || | > >>>>|build DSPLIB from source. Updated |2018-09-06|Jianzhong > >>>> Xu||New| > >>>>|includes/ti-paths.inc for MATHLIB and | | > >>>> || | > >>>>|DSPLIB.| | > >>>> || | > >>>> > >>>> |---+--+++---| > >>>>|[EXTERNAL] Added DSPLIB recipe to build| | > >>>> || | > >>>>|DSPLIB from source. Updated|2018-09-06|Jianzhong > >>>> Xu||New| > >>>>|includes/ti-paths.inc for MATHLIB and | | > >>>> || | > >>>>|DSPLIB.| | > >>>> || | > >>>> > >>>> |---+--+++---| > >>>>|[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong > >>>Xu||New| > >>>>|DSPLIB from source.
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
On Thu, Sep 13, 2018 at 07:26:12AM -0400, Jacob Stiffler wrote: > > > On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote: > >Same for dra7xx-evm... > > > > > >On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote: > >>And it crashed and burned right away: > >> > >>ERROR: Nothing PROVIDES 'dsplib-c66x' (but > >>/OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb > >> DEPENDS on or otherwise requires it) > >>dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in > >>COMPATIBLE_MACHINE) > >>NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is > >>unbuildable, removing... > >> > >>Jake, > >> > >>Did you tested it on am57xx-hs-evm? > > I had only built for am57xx-evm as there are no machine overrides in these > recipes because they are DSP architecture-specific. > > Looks like we should change "COMPATIBLE_MACHINE" to, at the very least, > include what was there before (omap-a15|keystone). But as these are specific > to the DSP architecture, should COMPATIBLE_MACHINE even be necessary? Either way should be fine. Currently those patches break too many platforms - dra7 GP and both am5/dra7 HS. > >>Denys > >> > >> > >>>-Original Message- > >>>From: Denys Dmytriyenko [mailto:de...@ti.com] > >>>Sent: Wednesday, September 12, 2018 4:21 PM > >>>To: Xu, Jianzhong > >>>Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > >>>Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > >>>build > >>>DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > >>> > >>>Thanks, merged into rocko-next for testing. > >>> > >>>There were lots of small non-critical things I had to fix, like > >>>whitespaces, > >>>indentation, etc. But the biggest one is that you have to follow > >>>OpenEmbedded > >>>Commit Message Guidelines: > >>> > >>>https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines > >>> > >>> > >>>On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: > >>>>Denys, > >>>> > >>>> > >>>> > >>>>You're right. The versioning was inconsistent. Please use > >>>>[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], > >>>> [EXTERNAL,1/3]. > >>>>The latter two should have had v2 in it. Please see highlights below. > >>>> > >>>> > >>>> > >>>> > >>>> ++ > >>>>|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | | > >>>> || | > >>>>|build DSPLIB from source. Updated |2018-09-06|Jianzhong > >>>> Xu||New| > >>>>|includes/ti-paths.inc for MATHLIB and | | > >>>> || | > >>>>|DSPLIB.| | > >>>> || | > >>>> > >>>> |---+--+++---| > >>>>|[EXTERNAL] Added DSPLIB recipe to build| | > >>>> || | > >>>>|DSPLIB from source. Updated|2018-09-06|Jianzhong > >>>> Xu||New| > >>>>|includes/ti-paths.inc for MATHLIB and | | > >>>> || | > >>>>|DSPLIB.| | > >>>> || | > >>>> > >>>> |---+--+++---| > >>>>|[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong > >>>Xu||New| > >>>>|DSPLIB from source.| | > >>>> || | > >>>> > >>>> |-----------------------+--+++---| > >>>>|[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong > >>>Xu||New| > >>>>|build MATHLIB from source. | | > >>>> || | > >>>> > >>>&g
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
I don't know what you mean by "tried". Yes, am57xx-evm built fine. I don't build c6x platforms. On Thu, Sep 13, 2018 at 11:24:55AM -0400, Xu, Jianzhong wrote: > Hi Denys, > > Have you tried am57xx-evm or c667x-evm? > > Thanks, > Jianzhong > > -Original Message- > From: Stiffler, Jacob > Sent: Thursday, September 13, 2018 7:26 AM > To: Dmytriyenko, Denys; Xu, Jianzhong > Cc: meta-ti@yoctoproject.org > Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build > DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > > > > On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote: > > Same for dra7xx-evm... > > > > > > On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote: > >> And it crashed and burned right away: > >> > >> ERROR: Nothing PROVIDES 'dsplib-c66x' (but > >> /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb > >> DEPENDS on or otherwise requires it) > >> dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in > >> COMPATIBLE_MACHINE) > >> NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is > >> unbuildable, removing... > >> > >> Jake, > >> > >> Did you tested it on am57xx-hs-evm? > > I had only built for am57xx-evm as there are no machine overrides in > these recipes because they are DSP architecture-specific. > > Looks like we should change "COMPATIBLE_MACHINE" to, at the very least, > include what was there before (omap-a15|keystone). But as these are > specific to the DSP architecture, should COMPATIBLE_MACHINE even be > necessary? > > >> Denys > >> > >> > >>> -Original Message- > >>> From: Denys Dmytriyenko [mailto:de...@ti.com] > >>> Sent: Wednesday, September 12, 2018 4:21 PM > >>> To: Xu, Jianzhong > >>> Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > >>> Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > >>> build > >>> DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > >>> > >>> Thanks, merged into rocko-next for testing. > >>> > >>> There were lots of small non-critical things I had to fix, like > >>> whitespaces, > >>> indentation, etc. But the biggest one is that you have to follow > >>> OpenEmbedded > >>> Commit Message Guidelines: > >>> > >>> https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines > >>> > >>> > >>> On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: > >>>> Denys, > >>>> > >>>> > >>>> > >>>> You're right. The versioning was inconsistent. Please use > >>>> [EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], > >>>> [EXTERNAL,1/3]. > >>>> The latter two should have had v2 in it. Please see highlights below. > >>>> > >>>> > >>>> > >>>> > >>>> ++ > >>>> |[EXTERNAL,3/3,v2] Added DSPLIB recipe to | | > >>>> || | > >>>> |build DSPLIB from source. Updated |2018-09-06|Jianzhong > >>>> Xu||New| > >>>> |includes/ti-paths.inc for MATHLIB and | | > >>>> || | > >>>> |DSPLIB.| | > >>>> || | > >>>> > >>>> |---+--+++---| > >>>> |[EXTERNAL] Added DSPLIB recipe to build| | > >>>> || | > >>>> |DSPLIB from source. Updated|2018-09-06|Jianzhong > >>>> Xu||New| > >>>> |includes/ti-paths.inc for MATHLIB and | | > >>>> || | > >>>> |DSPLIB.| | > >>>> || | > >>>> > >>>> |---+--+++---| > >>>> |[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong > >>> Xu||New| > >>>&
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Hi Denys, Have you tried am57xx-evm or c667x-evm? Thanks, Jianzhong -Original Message- From: Stiffler, Jacob Sent: Thursday, September 13, 2018 7:26 AM To: Dmytriyenko, Denys; Xu, Jianzhong Cc: meta-ti@yoctoproject.org Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote: > Same for dra7xx-evm... > > > On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote: >> And it crashed and burned right away: >> >> ERROR: Nothing PROVIDES 'dsplib-c66x' (but >> /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb >> DEPENDS on or otherwise requires it) >> dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in >> COMPATIBLE_MACHINE) >> NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is >> unbuildable, removing... >> >> Jake, >> >> Did you tested it on am57xx-hs-evm? I had only built for am57xx-evm as there are no machine overrides in these recipes because they are DSP architecture-specific. Looks like we should change "COMPATIBLE_MACHINE" to, at the very least, include what was there before (omap-a15|keystone). But as these are specific to the DSP architecture, should COMPATIBLE_MACHINE even be necessary? >> Denys >> >> >>> -Original Message- >>> From: Denys Dmytriyenko [mailto:de...@ti.com] >>> Sent: Wednesday, September 12, 2018 4:21 PM >>> To: Xu, Jianzhong >>> Cc: Stiffler, Jacob; meta-ti@yoctoproject.org >>> Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to >>> build >>> DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. >>> >>> Thanks, merged into rocko-next for testing. >>> >>> There were lots of small non-critical things I had to fix, like whitespaces, >>> indentation, etc. But the biggest one is that you have to follow >>> OpenEmbedded >>> Commit Message Guidelines: >>> >>> https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines >>> >>> >>> On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: >>>> Denys, >>>> >>>> >>>> >>>> You're right. The versioning was inconsistent. Please use >>>> [EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. >>>> The latter two should have had v2 in it. Please see highlights below. >>>> >>>> >>>> >>>> >>>> ++ >>>> |[EXTERNAL,3/3,v2] Added DSPLIB recipe to | ||| >>>> | >>>> |build DSPLIB from source. Updated |2018-09-06|Jianzhong >>>> Xu||New| >>>> |includes/ti-paths.inc for MATHLIB and | ||| >>>> | >>>> |DSPLIB.| ||| >>>> | >>>> >>>> |---+--+++---| >>>> |[EXTERNAL] Added DSPLIB recipe to build| ||| >>>> | >>>> |DSPLIB from source. Updated|2018-09-06|Jianzhong >>>> Xu||New| >>>> |includes/ti-paths.inc for MATHLIB and | ||| >>>> | >>>> |DSPLIB.| ||| >>>> | >>>> >>>> |---+--+++---| >>>> |[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong >>> Xu||New| >>>> |DSPLIB from source.| ||| >>>> | >>>> >>>> |---+--+++---| >>>> |[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong >>> Xu||New| >>>> |build MATHLIB from source. | | || >>>> | >>>> >>>> |---------------+------+++---| >>>> |[EXTERNAL,1/3] Added SWTOOLS which is | ||| >>>> | >>>> |needed to build processor libs (DSPLIB,
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
On 9/12/2018 6:52 PM, Denys Dmytriyenko wrote: Same for dra7xx-evm... On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote: And it crashed and burned right away: ERROR: Nothing PROVIDES 'dsplib-c66x' (but /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb DEPENDS on or otherwise requires it) dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in COMPATIBLE_MACHINE) NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is unbuildable, removing... Jake, Did you tested it on am57xx-hs-evm? I had only built for am57xx-evm as there are no machine overrides in these recipes because they are DSP architecture-specific. Looks like we should change "COMPATIBLE_MACHINE" to, at the very least, include what was there before (omap-a15|keystone). But as these are specific to the DSP architecture, should COMPATIBLE_MACHINE even be necessary? Denys -Original Message- From: Denys Dmytriyenko [mailto:de...@ti.com] Sent: Wednesday, September 12, 2018 4:21 PM To: Xu, Jianzhong Cc: Stiffler, Jacob; meta-ti@yoctoproject.org Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. Thanks, merged into rocko-next for testing. There were lots of small non-critical things I had to fix, like whitespaces, indentation, etc. But the biggest one is that you have to follow OpenEmbedded Commit Message Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: Denys, You're right. The versioning was inconsistent. Please use [EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. The latter two should have had v2 in it. Please see highlights below. ++ |[EXTERNAL,3/3,v2] Added DSPLIB recipe to | ||| | |build DSPLIB from source. Updated |2018-09-06|Jianzhong Xu||New| |includes/ti-paths.inc for MATHLIB and | ||| | |DSPLIB.| ||| | |---+--+++---| |[EXTERNAL] Added DSPLIB recipe to build| ||| | |DSPLIB from source. Updated|2018-09-06|Jianzhong Xu||New| |includes/ti-paths.inc for MATHLIB and | ||| | |DSPLIB.| ||| | |---+--+++---| |[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong Xu||New| |DSPLIB from source.| ||| | |---+--+++---| |[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong Xu||New| |build MATHLIB from source. | ||| | |---+--+++---| |[EXTERNAL,1/3] Added SWTOOLS which is | ||| | |needed to build processor libs (DSPLIB,|2018-09-04|Jianzhong Xu||New| |MATHLIB, etc) from source. | ||| | ++ Thanks, Jianzhong -Original Message- From: Dmytriyenko, Denys Sent: Wednesday, September 12, 2018 3:13 PM To: Stiffler, Jacob Cc: Xu, Jianzhong; meta-ti@yoctoproject.org Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. Jake, I'll give it a look - I assume you want the 3 most recent patches, I don't think versioning was consistent... Denys On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler wrote: > ACK > > > Denys, > > Can you apply these patches to the ti2018.03 branch? > > > Thank you, > > Jake > > > On 9/6/2018 1:46 PM, Jianzhong Xu wrote: > >Signed-off-by: Jianzhong Xu > >--- > >Change from previous version of patch > >Corrected typo: MATHLIB_C674INSTALL_XXX should have been MATHLIB_C674_INSTALL_XXX > >--- > >--- > > recipes-ti/dsplib/dsplib-c64xp_git.bb| 9 > > recipes-ti/dsplib/dsplib-c66x.inc| 30 > > recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb | 9 > > recipes-ti/dsplib/dsplib-c66x_git.bb | 10
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Same for dra7xx-evm... On Wed, Sep 12, 2018 at 04:37:48PM -0400, Dmytriyenko, Denys wrote: > And it crashed and burned right away: > > ERROR: Nothing PROVIDES 'dsplib-c66x' (but > /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb > DEPENDS on or otherwise requires it) > dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in > COMPATIBLE_MACHINE) > NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is > unbuildable, removing... > > Jake, > > Did you tested it on am57xx-hs-evm? > > Denys > > > > -Original Message- > > From: Denys Dmytriyenko [mailto:de...@ti.com] > > Sent: Wednesday, September 12, 2018 4:21 PM > > To: Xu, Jianzhong > > Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > > Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > > build > > DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > > > > Thanks, merged into rocko-next for testing. > > > > There were lots of small non-critical things I had to fix, like whitespaces, > > indentation, etc. But the biggest one is that you have to follow > > OpenEmbedded > > Commit Message Guidelines: > > > > https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines > > > > > > On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: > > >Denys, > > > > > > > > > > > >You're right. The versioning was inconsistent. Please use > > >[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. > > >The latter two should have had v2 in it. Please see highlights below. > > > > > > > > > > > > > > > ++ > > >|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | ||| > > > | > > >|build DSPLIB from source. Updated |2018-09-06|Jianzhong > > > Xu||New| > > >|includes/ti-paths.inc for MATHLIB and | ||| > > > | > > >|DSPLIB.| ||| > > > | > > > > > > |---+--+++---| > > >|[EXTERNAL] Added DSPLIB recipe to build| ||| > > > | > > >|DSPLIB from source. Updated|2018-09-06|Jianzhong > > > Xu||New| > > >|includes/ti-paths.inc for MATHLIB and | ||| > > > | > > >|DSPLIB.| ||| > > > | > > > > > > |---+--+++---| > > >|[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong > > Xu||New| > > >|DSPLIB from source.| ||| > > > | > > > > > > |---+--+++---| > > >|[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong > > Xu||New| > > >|build MATHLIB from source. | ||| > > > | > > > > > > |---+--+++---| > > >|[EXTERNAL,1/3] Added SWTOOLS which is | ||| > > > | > > >|needed to build processor libs (DSPLIB,|2018-09-04|Jianzhong > > > Xu||New| > > >|MATHLIB, etc) from source. | ||| > > > | > > > > > > ++ > > > > > > > > > > > >Thanks, > > > > > >Jianzhong > > > > > > > > > > > >-Original Message- > > >From: Dmytriyenko, Denys > > >Sent: Wednesday, September 12, 2018 3:13 PM > > >To: Stiffler, Jacob > > >Cc: Xu, Jianzhong; meta-ti@yoctoproject.org > > >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > > >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and > > >DSPLIB. > > > > > > > > > > > >Jake, > > > > > > > > > > > >I'
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Denys, Please try c667x-evm. Thanks, Jianzhong -Original Message- From: Dmytriyenko, Denys Sent: Wednesday, September 12, 2018 4:38 PM To: Dmytriyenko, Denys; Xu, Jianzhong Cc: Stiffler, Jacob; meta-ti@yoctoproject.org Subject: RE: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. And it crashed and burned right away: ERROR: Nothing PROVIDES 'dsplib-c66x' (but /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb DEPENDS on or otherwise requires it) dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in COMPATIBLE_MACHINE) NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is unbuildable, removing... Jake, Did you tested it on am57xx-hs-evm? Denys > -Original Message- > From: Denys Dmytriyenko [mailto:de...@ti.com] > Sent: Wednesday, September 12, 2018 4:21 PM > To: Xu, Jianzhong > Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build > DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > > Thanks, merged into rocko-next for testing. > > There were lots of small non-critical things I had to fix, like whitespaces, > indentation, etc. But the biggest one is that you have to follow OpenEmbedded > Commit Message Guidelines: > > https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines > > > On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: > >Denys, > > > > > > > >You're right. The versioning was inconsistent. Please use > >[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. > >The latter two should have had v2 in it. Please see highlights below. > > > > > > > > > > ++ > >|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | ||| > > | > >|build DSPLIB from source. Updated |2018-09-06|Jianzhong > > Xu||New| > >|includes/ti-paths.inc for MATHLIB and | ||| > > | > >|DSPLIB.| ||| > > | > > > > |---+--+++---| > >|[EXTERNAL] Added DSPLIB recipe to build| ||| > > | > >|DSPLIB from source. Updated|2018-09-06|Jianzhong > > Xu||New| > >|includes/ti-paths.inc for MATHLIB and | ||| > > | > >|DSPLIB.| ||| > > | > > > > |---+--+++---| > >|[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong > Xu||New| > >|DSPLIB from source.| ||| > > | > > > > |---+--+++---| > >|[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong > Xu||New| > >|build MATHLIB from source. | ||| > > | > > > > |---+--+++---| > >|[EXTERNAL,1/3] Added SWTOOLS which is | ||| > > | > >|needed to build processor libs (DSPLIB,|2018-09-04|Jianzhong > > Xu||New| > >|MATHLIB, etc) from source. | ||| > > | > > > > +----------------------------+ > > > > > > > > Thanks, > > > >Jianzhong > > > > > > > >-Original Message- > >From: Dmytriyenko, Denys > >Sent: Wednesday, September 12, 2018 3:13 PM > >To: Stiffler, Jacob > >Cc: Xu, Jianzhong; meta-ti@yoctoproject.org > >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and > >DSPLIB. > > > > > > > >Jake, > > > > > > > >I'll give it a look - I assume you want the 3 most recent patches, I > > don't > > > >think versioning was consistent... > > > > > > > >Denys > > > > > > > > > > > >On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
And it crashed and burned right away: ERROR: Nothing PROVIDES 'dsplib-c66x' (but /OE/arago-rocko-next-build/sources/meta-arago/meta-arago-extras/recipes-apps/video-analytics-demo/qt-opencv-opencl-opengl-multithreaded_git.bb DEPENDS on or otherwise requires it) dsplib-c66x was skipped: incompatible with machine am57xx-hs-evm (not in COMPATIBLE_MACHINE) NOTE: Runtime target 'qt-opencv-opencl-opengl-multithreaded-src' is unbuildable, removing... Jake, Did you tested it on am57xx-hs-evm? Denys > -Original Message- > From: Denys Dmytriyenko [mailto:de...@ti.com] > Sent: Wednesday, September 12, 2018 4:21 PM > To: Xu, Jianzhong > Cc: Stiffler, Jacob; meta-ti@yoctoproject.org > Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build > DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB. > > Thanks, merged into rocko-next for testing. > > There were lots of small non-critical things I had to fix, like whitespaces, > indentation, etc. But the biggest one is that you have to follow OpenEmbedded > Commit Message Guidelines: > > https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines > > > On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: > >Denys, > > > > > > > >You're right. The versioning was inconsistent. Please use > >[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. > >The latter two should have had v2 in it. Please see highlights below. > > > > > > > > > > ++ > >|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | ||| > > | > >|build DSPLIB from source. Updated |2018-09-06|Jianzhong > > Xu||New| > >|includes/ti-paths.inc for MATHLIB and | ||| > > | > >|DSPLIB.| ||| > > | > > > > |---+--+++---| > >|[EXTERNAL] Added DSPLIB recipe to build| ||| > > | > >|DSPLIB from source. Updated|2018-09-06|Jianzhong > > Xu||New| > >|includes/ti-paths.inc for MATHLIB and | ||| > > | > >|DSPLIB.| ||| > > | > > > > |---+--+++---| > >|[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong > Xu||New| > >|DSPLIB from source.| ||| > > | > > > > |---+--+++---| > >|[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong > Xu||New| > >|build MATHLIB from source. | ||| > > | > > > > |---+--+++---| > >|[EXTERNAL,1/3] Added SWTOOLS which is | ||| > > | > >|needed to build processor libs (DSPLIB,|2018-09-04|Jianzhong > > Xu||New| > >|MATHLIB, etc) from source. | ||| > > | > > > > +----------------------------+ > > > > > > > > Thanks, > > > >Jianzhong > > > > > > > >-Original Message- > >From: Dmytriyenko, Denys > >Sent: Wednesday, September 12, 2018 3:13 PM > >To: Stiffler, Jacob > >Cc: Xu, Jianzhong; meta-ti@yoctoproject.org > >Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to > >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and > >DSPLIB. > > > > > > > >Jake, > > > > > > > >I'll give it a look - I assume you want the 3 most recent patches, I > > don't > > > >think versioning was consistent... > > > > > > > >Denys > > > > > > > > > > > >On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler wrote: > > > >> ACK > > > >> > > > >> > > > >> Denys, > > > >> > > > >> Can you apply these patches to the ti2018.03 branch? > > > >> > > > >> > > > >> Thank you, > > > >> > > > &
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Thanks, merged into rocko-next for testing. There were lots of small non-critical things I had to fix, like whitespaces, indentation, etc. But the biggest one is that you have to follow OpenEmbedded Commit Message Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines On Wed, Sep 12, 2018 at 03:40:42PM -0400, Xu, Jianzhong wrote: >Denys, > > > >You're right. The versioning was inconsistent. Please use >[EXTERNAL,3/3,v2] and newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. >The latter two should have had v2 in it. Please see highlights below. > > > >++ >|[EXTERNAL,3/3,v2] Added DSPLIB recipe to | ||| | >|build DSPLIB from source. Updated |2018-09-06|Jianzhong Xu||New| >|includes/ti-paths.inc for MATHLIB and | ||| | >|DSPLIB.| ||| | >|---+--+++---| >|[EXTERNAL] Added DSPLIB recipe to build| ||| | >|DSPLIB from source. Updated|2018-09-06|Jianzhong Xu||New| >|includes/ti-paths.inc for MATHLIB and | ||| | >|DSPLIB.| ||| | >|---+--+++---| >|[EXTERNAL,3/3] Added DSPLIB recipe to build|2018-09-04|Jianzhong Xu||New| >|DSPLIB from source.| ||| | >|---+--+++---| >|[EXTERNAL,2/3] Added MATHLIB recipe to |2018-09-04|Jianzhong Xu||New| >|build MATHLIB from source. | ||| | >|---+--+++---| >|[EXTERNAL,1/3] Added SWTOOLS which is | ||| | >|needed to build processor libs (DSPLIB,|2018-09-04|Jianzhong Xu||New| >|MATHLIB, etc) from source. | ||| | >++ > > > >Thanks, > >Jianzhong > > > >-Original Message- >From: Dmytriyenko, Denys >Sent: Wednesday, September 12, 2018 3:13 PM > To: Stiffler, Jacob > Cc: Xu, Jianzhong; meta-ti@yoctoproject.org > Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to >build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and >DSPLIB. > > > >Jake, > > > >I'll give it a look - I assume you want the 3 most recent patches, I don't > >think versioning was consistent... > > > >Denys > > > > > >On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler wrote: > >> ACK > >> > >> > >> Denys, > >> > >> Can you apply these patches to the ti2018.03 branch? > >> > >> > >> Thank you, > >> > >> Jake > >> > >> > >> On 9/6/2018 1:46 PM, Jianzhong Xu wrote: > >> >Signed-off-by: Jianzhong Xu > >> >--- > >> >Change from previous version of patch > >> >Corrected typo: MATHLIB_C674INSTALL_XXX should have been >MATHLIB_C674_INSTALL_XXX > >> >--- > >> >--- > >> > recipes-ti/dsplib/dsplib-c64xp_git.bb| 9 > >> > recipes-ti/dsplib/dsplib-c66x.inc| 30 > >> > recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb | 9 > >> > recipes-ti/dsplib/dsplib-c66x_git.bb | 10 > >> > recipes-ti/dsplib/dsplib-c674x_git.bb| 9 > >> > recipes-ti/dsplib/dsplib.inc | 79 > > >> > recipes-ti/includes/ti-paths.inc | 8 +++- > >> > 7 files changed, 114 insertions(+), 40 deletions(-) > >> > create mode 100644 recipes-ti/dsplib/dsplib-c64xp_git.bb > >> > delete mode 100644 recipes-ti/dsplib/dsplib-c66x.inc > >> > delete mode 100644 recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb > >> > create mode 100644 recipes-ti/dsplib/dsplib-c66x_git.bb > >> > create mode 100644 recipes-ti/dsplib/dsplib-c674x_git.bb > >> > create mode 100
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Denys,
You're right. The versioning was inconsistent. Please use [EXTERNAL,3/3,v2] and
newer versions of [EXTERNAL,2/3], [EXTERNAL,1/3]. The latter two should have
had v2 in it. Please see highlights below.
[EXTERNAL,3/3,v2] Added DSPLIB recipe to build DSPLIB from source. Updated
includes/ti-paths.inc for MATHLIB and
DSPLIB.<http://patchwork.dal.design.ti.com/patch/40555/>
2018-09-06
Jianzhong Xu<mailto:x...@ti.com>
New
[EXTERNAL] Added DSPLIB recipe to build DSPLIB from source. Updated
includes/ti-paths.inc for MATHLIB and
DSPLIB.<http://patchwork.dal.design.ti.com/patch/40553/>
2018-09-06
Jianzhong Xu<mailto:x...@ti.com>
New
[EXTERNAL,3/3] Added DSPLIB recipe to build DSPLIB from
source.<http://patchwork.dal.design.ti.com/patch/40451/>
2018-09-04
Jianzhong Xu<mailto:x...@ti.com>
New
[EXTERNAL,2/3] Added MATHLIB recipe to build MATHLIB from
source.<http://patchwork.dal.design.ti.com/patch/40450/>
2018-09-04
Jianzhong Xu<mailto:x...@ti.com>
New
[EXTERNAL,1/3] Added SWTOOLS which is needed to build processor libs (DSPLIB,
MATHLIB, etc) from source.<http://patchwork.dal.design.ti.com/patch/40449/>
2018-09-04
Jianzhong Xu<mailto:x...@ti.com>
New
Thanks,
Jianzhong
-Original Message-
From: Dmytriyenko, Denys
Sent: Wednesday, September 12, 2018 3:13 PM
To: Stiffler, Jacob
Cc: Xu, Jianzhong; meta-ti@yoctoproject.org
Subject: Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build
DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Jake,
I'll give it a look - I assume you want the 3 most recent patches, I don't
think versioning was consistent...
Denys
On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler wrote:
> ACK
>
>
> Denys,
>
> Can you apply these patches to the ti2018.03 branch?
>
>
> Thank you,
>
> Jake
>
>
> On 9/6/2018 1:46 PM, Jianzhong Xu wrote:
> >Signed-off-by: Jianzhong Xu
> >---
> >Change from previous version of patch
> >Corrected typo: MATHLIB_C674INSTALL_XXX should have been
> >MATHLIB_C674_INSTALL_XXX
> >---
> >---
> > recipes-ti/dsplib/dsplib-c64xp_git.bb| 9
> > recipes-ti/dsplib/dsplib-c66x.inc| 30
> > recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb | 9
> > recipes-ti/dsplib/dsplib-c66x_git.bb | 10
> > recipes-ti/dsplib/dsplib-c674x_git.bb| 9
> > recipes-ti/dsplib/dsplib.inc | 79
> >
> > recipes-ti/includes/ti-paths.inc | 8 +++-
> > 7 files changed, 114 insertions(+), 40 deletions(-)
> > create mode 100644 recipes-ti/dsplib/dsplib-c64xp_git.bb
> > delete mode 100644 recipes-ti/dsplib/dsplib-c66x.inc
> > delete mode 100644 recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
> > create mode 100644 recipes-ti/dsplib/dsplib-c66x_git.bb
> > create mode 100644 recipes-ti/dsplib/dsplib-c674x_git.bb
> > create mode 100644 recipes-ti/dsplib/dsplib.inc
> >
> >diff --git a/recipes-ti/dsplib/dsplib-c64xp_git.bb
> >b/recipes-ti/dsplib/dsplib-c64xp_git.bb
> >new file mode 100644
> >index 000..1038d44
> >--- /dev/null
> >+++ b/recipes-ti/dsplib/dsplib-c64xp_git.bb
> >@@ -0,0 +1,9 @@
> >+MATHLIB_INSTALL_DIR = "${MATHLIB_C64P_INSTALL_DIR}/packages"
> >+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C64P_INSTALL_DIR_RECIPE}"
> >+DSPLIB_INSTALL_DIR = "${DSPLIB_C64P_INSTALL_DIR}"
> >+DSPLIB_BUILD_TARGET = "c64Px"
> >+
> >+DEPENDS = ""
> >+
> >+include dsplib.inc
> >+
> >diff --git a/recipes-ti/dsplib/dsplib-c66x.inc
> >b/recipes-ti/dsplib/dsplib-c66x.inc
> >deleted file mode 100644
> >index 7ff99c2..000
> >--- a/recipes-ti/dsplib/dsplib-c66x.inc
> >+++ /dev/null
> >@@ -1,30 +0,0 @@
> >-DESCRIPTION = "DSPLIB contains optimized general-purpose signal-processing
> >routines for TI DSPs"
> >-HOMEPAGE = "http://www.ti.com/tool/sprc265;
> >-SECTION = "devel"
> >-LICENSE = "BSD-3-Clause"
> >-
> >-require ../includes/ti-paths.inc
> >-require ../includes/ti-staging.inc
> >-require ../includes/ti-unpack.inc
> >-
> >-COMPATIBLE_MACHINE = "omap-a15|keystone"
> >-
> >-SRC_URI =
> >"http://software-dl.ti.com/sdoemb/sdoemb_public_sw/dsplib/${PV}/exports/${BINFILE};name=dsplib;
> >-
> >-BINFILE = "dsplib_c66x_${PV}_Linux.bin"
> >-TI_BIN_UNPK_ARGS = "--mode silent --prefix ${S}"
> >-TI_BIN_UNPK_CMDS = ""
> >-
> >-do_install()
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
On 9/12/2018 3:13 PM, Denys Dmytriyenko wrote:
Jake,
I'll give it a look - I assume you want the 3 most recent patches, I don't
think versioning was consistent...
Yes. I build tested these patches:
http://patchwork.dal.design.ti.com/patch/40449/
http://patchwork.dal.design.ti.com/patch/40450/
http://patchwork.dal.design.ti.com/patch/40555/
Denys
On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler wrote:
ACK
Denys,
Can you apply these patches to the ti2018.03 branch?
Thank you,
Jake
On 9/6/2018 1:46 PM, Jianzhong Xu wrote:
Signed-off-by: Jianzhong Xu
---
Change from previous version of patch
Corrected typo: MATHLIB_C674INSTALL_XXX should have been
MATHLIB_C674_INSTALL_XXX
---
---
recipes-ti/dsplib/dsplib-c64xp_git.bb| 9
recipes-ti/dsplib/dsplib-c66x.inc| 30
recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb | 9
recipes-ti/dsplib/dsplib-c66x_git.bb | 10
recipes-ti/dsplib/dsplib-c674x_git.bb| 9
recipes-ti/dsplib/dsplib.inc | 79
recipes-ti/includes/ti-paths.inc | 8 +++-
7 files changed, 114 insertions(+), 40 deletions(-)
create mode 100644 recipes-ti/dsplib/dsplib-c64xp_git.bb
delete mode 100644 recipes-ti/dsplib/dsplib-c66x.inc
delete mode 100644 recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
create mode 100644 recipes-ti/dsplib/dsplib-c66x_git.bb
create mode 100644 recipes-ti/dsplib/dsplib-c674x_git.bb
create mode 100644 recipes-ti/dsplib/dsplib.inc
diff --git a/recipes-ti/dsplib/dsplib-c64xp_git.bb
b/recipes-ti/dsplib/dsplib-c64xp_git.bb
new file mode 100644
index 000..1038d44
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib-c64xp_git.bb
@@ -0,0 +1,9 @@
+MATHLIB_INSTALL_DIR = "${MATHLIB_C64P_INSTALL_DIR}/packages"
+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C64P_INSTALL_DIR_RECIPE}"
+DSPLIB_INSTALL_DIR = "${DSPLIB_C64P_INSTALL_DIR}"
+DSPLIB_BUILD_TARGET = "c64Px"
+
+DEPENDS = ""
+
+include dsplib.inc
+
diff --git a/recipes-ti/dsplib/dsplib-c66x.inc
b/recipes-ti/dsplib/dsplib-c66x.inc
deleted file mode 100644
index 7ff99c2..000
--- a/recipes-ti/dsplib/dsplib-c66x.inc
+++ /dev/null
@@ -1,30 +0,0 @@
-DESCRIPTION = "DSPLIB contains optimized general-purpose signal-processing routines
for TI DSPs"
-HOMEPAGE = "http://www.ti.com/tool/sprc265;
-SECTION = "devel"
-LICENSE = "BSD-3-Clause"
-
-require ../includes/ti-paths.inc
-require ../includes/ti-staging.inc
-require ../includes/ti-unpack.inc
-
-COMPATIBLE_MACHINE = "omap-a15|keystone"
-
-SRC_URI =
"http://software-dl.ti.com/sdoemb/sdoemb_public_sw/dsplib/${PV}/exports/${BINFILE};name=dsplib;
-
-BINFILE = "dsplib_c66x_${PV}_Linux.bin"
-TI_BIN_UNPK_ARGS = "--mode silent --prefix ${S}"
-TI_BIN_UNPK_CMDS = ""
-
-do_install() {
-CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
-install -d ${D}${DSPLIB_C66_INSTALL_DIR_RECIPE}
-cp ${CP_ARGS} ${S}/* ${D}${DSPLIB_C66_INSTALL_DIR_RECIPE}
-}
-
-ALLOW_EMPTY_${PN} = "1"
-FILES_${PN}-dev += "${DSPLIB_C66_INSTALL_DIR_RECIPE}"
-
-INHIBIT_PACKAGE_STRIP = "1"
-INHIBIT_SYSROOT_STRIP = "1"
-INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
-INSANE_SKIP_${PN}-dev = "arch"
diff --git a/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
b/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
deleted file mode 100644
index f1273f3..000
--- a/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
+++ /dev/null
@@ -1,9 +0,0 @@
-require dsplib-c66x.inc
-
-PV = "3_4_0_0"
-PR = "r1"
-
-LIC_FILES_CHKSUM =
"file://packages/ti/dsplib/dsplib.h;beginline=9;endline=35;md5=17733f9e1100f8a628512f47c4bf676a"
-
-SRC_URI[dsplib.md5sum] = "66785604729c153dcad5282097dee8bf"
-SRC_URI[dsplib.sha256sum] =
"e88909b6271ce6c4c62e2ab90181b7adb36c7f28f5c52836668eb35888785ce8"
diff --git a/recipes-ti/dsplib/dsplib-c66x_git.bb
b/recipes-ti/dsplib/dsplib-c66x_git.bb
new file mode 100644
index 000..cbc25d0
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib-c66x_git.bb
@@ -0,0 +1,10 @@
+MATHLIB_INSTALL_DIR = "${MATHLIB_C66_INSTALL_DIR}/packages"
+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C66_INSTALL_DIR_RECIPE}"
+DSPLIB_INSTALL_DIR = "${DSPLIB_C66_INSTALL_DIR}"
+DSPLIB_BUILD_TARGET = "c66x"
+
+DEPENDS = "mathlib-c66x"
+
+include dsplib.inc
+
+
diff --git a/recipes-ti/dsplib/dsplib-c674x_git.bb
b/recipes-ti/dsplib/dsplib-c674x_git.bb
new file mode 100644
index 000..07c6892
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib-c674x_git.bb
@@ -0,0 +1,9 @@
+MATHLIB_INSTALL_DIR = "${MATHLIB_C674_INSTALL_DIR}/packages"
+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C674_INSTALL_DIR_RECIPE}"
+DSPLIB_INSTALL_DIR = "${DSPLIB_C674_INSTALL_DIR}"
+DSPLIB_BUILD_TARGET = "c674x"
+
+DEPENDS = "mathlib-c674x"
+
+include dsplib.inc
+
diff --git a/recipes-ti/dsplib/dsplib.inc b/recipes-ti/dsplib/dsplib.inc
new file mode 100644
index 000..66fe0fb
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib.inc
@@ -0,0 +1,79 @@
+SUMMARY = "DSPLIB contains optimized signal processing functions for TI DSPs"
+HOMEPAGE = "http://git.ti.com/ep-processor-libraries/dsplib;
+LICENSE
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
Jake,
I'll give it a look - I assume you want the 3 most recent patches, I don't
think versioning was consistent...
Denys
On Tue, Sep 11, 2018 at 09:17:49AM -0400, Jacob Stiffler wrote:
> ACK
>
>
> Denys,
>
> Can you apply these patches to the ti2018.03 branch?
>
>
> Thank you,
>
> Jake
>
>
> On 9/6/2018 1:46 PM, Jianzhong Xu wrote:
> >Signed-off-by: Jianzhong Xu
> >---
> >Change from previous version of patch
> >Corrected typo: MATHLIB_C674INSTALL_XXX should have been
> >MATHLIB_C674_INSTALL_XXX
> >---
> >---
> > recipes-ti/dsplib/dsplib-c64xp_git.bb| 9
> > recipes-ti/dsplib/dsplib-c66x.inc| 30
> > recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb | 9
> > recipes-ti/dsplib/dsplib-c66x_git.bb | 10
> > recipes-ti/dsplib/dsplib-c674x_git.bb| 9
> > recipes-ti/dsplib/dsplib.inc | 79
> >
> > recipes-ti/includes/ti-paths.inc | 8 +++-
> > 7 files changed, 114 insertions(+), 40 deletions(-)
> > create mode 100644 recipes-ti/dsplib/dsplib-c64xp_git.bb
> > delete mode 100644 recipes-ti/dsplib/dsplib-c66x.inc
> > delete mode 100644 recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
> > create mode 100644 recipes-ti/dsplib/dsplib-c66x_git.bb
> > create mode 100644 recipes-ti/dsplib/dsplib-c674x_git.bb
> > create mode 100644 recipes-ti/dsplib/dsplib.inc
> >
> >diff --git a/recipes-ti/dsplib/dsplib-c64xp_git.bb
> >b/recipes-ti/dsplib/dsplib-c64xp_git.bb
> >new file mode 100644
> >index 000..1038d44
> >--- /dev/null
> >+++ b/recipes-ti/dsplib/dsplib-c64xp_git.bb
> >@@ -0,0 +1,9 @@
> >+MATHLIB_INSTALL_DIR = "${MATHLIB_C64P_INSTALL_DIR}/packages"
> >+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C64P_INSTALL_DIR_RECIPE}"
> >+DSPLIB_INSTALL_DIR = "${DSPLIB_C64P_INSTALL_DIR}"
> >+DSPLIB_BUILD_TARGET = "c64Px"
> >+
> >+DEPENDS = ""
> >+
> >+include dsplib.inc
> >+
> >diff --git a/recipes-ti/dsplib/dsplib-c66x.inc
> >b/recipes-ti/dsplib/dsplib-c66x.inc
> >deleted file mode 100644
> >index 7ff99c2..000
> >--- a/recipes-ti/dsplib/dsplib-c66x.inc
> >+++ /dev/null
> >@@ -1,30 +0,0 @@
> >-DESCRIPTION = "DSPLIB contains optimized general-purpose signal-processing
> >routines for TI DSPs"
> >-HOMEPAGE = "http://www.ti.com/tool/sprc265;
> >-SECTION = "devel"
> >-LICENSE = "BSD-3-Clause"
> >-
> >-require ../includes/ti-paths.inc
> >-require ../includes/ti-staging.inc
> >-require ../includes/ti-unpack.inc
> >-
> >-COMPATIBLE_MACHINE = "omap-a15|keystone"
> >-
> >-SRC_URI =
> >"http://software-dl.ti.com/sdoemb/sdoemb_public_sw/dsplib/${PV}/exports/${BINFILE};name=dsplib;
> >-
> >-BINFILE = "dsplib_c66x_${PV}_Linux.bin"
> >-TI_BIN_UNPK_ARGS = "--mode silent --prefix ${S}"
> >-TI_BIN_UNPK_CMDS = ""
> >-
> >-do_install() {
> >-CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
> >-install -d ${D}${DSPLIB_C66_INSTALL_DIR_RECIPE}
> >-cp ${CP_ARGS} ${S}/* ${D}${DSPLIB_C66_INSTALL_DIR_RECIPE}
> >-}
> >-
> >-ALLOW_EMPTY_${PN} = "1"
> >-FILES_${PN}-dev += "${DSPLIB_C66_INSTALL_DIR_RECIPE}"
> >-
> >-INHIBIT_PACKAGE_STRIP = "1"
> >-INHIBIT_SYSROOT_STRIP = "1"
> >-INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> >-INSANE_SKIP_${PN}-dev = "arch"
> >diff --git a/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
> >b/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
> >deleted file mode 100644
> >index f1273f3..000
> >--- a/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
> >+++ /dev/null
> >@@ -1,9 +0,0 @@
> >-require dsplib-c66x.inc
> >-
> >-PV = "3_4_0_0"
> >-PR = "r1"
> >-
> >-LIC_FILES_CHKSUM =
> >"file://packages/ti/dsplib/dsplib.h;beginline=9;endline=35;md5=17733f9e1100f8a628512f47c4bf676a"
> >-
> >-SRC_URI[dsplib.md5sum] = "66785604729c153dcad5282097dee8bf"
> >-SRC_URI[dsplib.sha256sum] =
> >"e88909b6271ce6c4c62e2ab90181b7adb36c7f28f5c52836668eb35888785ce8"
> >diff --git a/recipes-ti/dsplib/dsplib-c66x_git.bb
> >b/recipes-ti/dsplib/dsplib-c66x_git.bb
> >new file mode 100644
> >index 000..cbc25d0
> >--- /dev/null
> >+++ b/recipes-ti/dsplib/dsplib-c66x_git.bb
> >@@ -0,0 +1,10 @@
> >+MATHLIB_INSTALL_DIR = "${MATHLIB_C66_INSTALL_DIR}/packages"
> >+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C66_INSTALL_DIR_RECIPE}"
> >+DSPLIB_INSTALL_DIR = "${DSPLIB_C66_INSTALL_DIR}"
> >+DSPLIB_BUILD_TARGET = "c66x"
> >+
> >+DEPENDS = "mathlib-c66x"
> >+
> >+include dsplib.inc
> >+
> >+
> >diff --git a/recipes-ti/dsplib/dsplib-c674x_git.bb
> >b/recipes-ti/dsplib/dsplib-c674x_git.bb
> >new file mode 100644
> >index 000..07c6892
> >--- /dev/null
> >+++ b/recipes-ti/dsplib/dsplib-c674x_git.bb
> >@@ -0,0 +1,9 @@
> >+MATHLIB_INSTALL_DIR = "${MATHLIB_C674_INSTALL_DIR}/packages"
> >+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C674_INSTALL_DIR_RECIPE}"
> >+DSPLIB_INSTALL_DIR = "${DSPLIB_C674_INSTALL_DIR}"
> >+DSPLIB_BUILD_TARGET = "c674x"
> >+
> >+DEPENDS = "mathlib-c674x"
> >+
> >+include dsplib.inc
> >+
> >diff --git a/recipes-ti/dsplib/dsplib.inc b/recipes-ti/dsplib/dsplib.inc
> >new file mode 100644
> >index 000..66fe0fb
> >--- /dev/null
> >+++
Re: [meta-ti] [EXTERNAL] [PATCH 3/3 v2] Added DSPLIB recipe to build DSPLIB from source. Updated includes/ti-paths.inc for MATHLIB and DSPLIB.
ACK
Denys,
Can you apply these patches to the ti2018.03 branch?
Thank you,
Jake
On 9/6/2018 1:46 PM, Jianzhong Xu wrote:
Signed-off-by: Jianzhong Xu
---
Change from previous version of patch
Corrected typo: MATHLIB_C674INSTALL_XXX should have been
MATHLIB_C674_INSTALL_XXX
---
---
recipes-ti/dsplib/dsplib-c64xp_git.bb| 9
recipes-ti/dsplib/dsplib-c66x.inc| 30
recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb | 9
recipes-ti/dsplib/dsplib-c66x_git.bb | 10
recipes-ti/dsplib/dsplib-c674x_git.bb| 9
recipes-ti/dsplib/dsplib.inc | 79
recipes-ti/includes/ti-paths.inc | 8 +++-
7 files changed, 114 insertions(+), 40 deletions(-)
create mode 100644 recipes-ti/dsplib/dsplib-c64xp_git.bb
delete mode 100644 recipes-ti/dsplib/dsplib-c66x.inc
delete mode 100644 recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
create mode 100644 recipes-ti/dsplib/dsplib-c66x_git.bb
create mode 100644 recipes-ti/dsplib/dsplib-c674x_git.bb
create mode 100644 recipes-ti/dsplib/dsplib.inc
diff --git a/recipes-ti/dsplib/dsplib-c64xp_git.bb
b/recipes-ti/dsplib/dsplib-c64xp_git.bb
new file mode 100644
index 000..1038d44
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib-c64xp_git.bb
@@ -0,0 +1,9 @@
+MATHLIB_INSTALL_DIR = "${MATHLIB_C64P_INSTALL_DIR}/packages"
+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C64P_INSTALL_DIR_RECIPE}"
+DSPLIB_INSTALL_DIR = "${DSPLIB_C64P_INSTALL_DIR}"
+DSPLIB_BUILD_TARGET = "c64Px"
+
+DEPENDS = ""
+
+include dsplib.inc
+
diff --git a/recipes-ti/dsplib/dsplib-c66x.inc
b/recipes-ti/dsplib/dsplib-c66x.inc
deleted file mode 100644
index 7ff99c2..000
--- a/recipes-ti/dsplib/dsplib-c66x.inc
+++ /dev/null
@@ -1,30 +0,0 @@
-DESCRIPTION = "DSPLIB contains optimized general-purpose signal-processing routines
for TI DSPs"
-HOMEPAGE = "http://www.ti.com/tool/sprc265;
-SECTION = "devel"
-LICENSE = "BSD-3-Clause"
-
-require ../includes/ti-paths.inc
-require ../includes/ti-staging.inc
-require ../includes/ti-unpack.inc
-
-COMPATIBLE_MACHINE = "omap-a15|keystone"
-
-SRC_URI =
"http://software-dl.ti.com/sdoemb/sdoemb_public_sw/dsplib/${PV}/exports/${BINFILE};name=dsplib;
-
-BINFILE = "dsplib_c66x_${PV}_Linux.bin"
-TI_BIN_UNPK_ARGS = "--mode silent --prefix ${S}"
-TI_BIN_UNPK_CMDS = ""
-
-do_install() {
-CP_ARGS="-Prf --preserve=mode,timestamps --no-preserve=ownership"
-install -d ${D}${DSPLIB_C66_INSTALL_DIR_RECIPE}
-cp ${CP_ARGS} ${S}/* ${D}${DSPLIB_C66_INSTALL_DIR_RECIPE}
-}
-
-ALLOW_EMPTY_${PN} = "1"
-FILES_${PN}-dev += "${DSPLIB_C66_INSTALL_DIR_RECIPE}"
-
-INHIBIT_PACKAGE_STRIP = "1"
-INHIBIT_SYSROOT_STRIP = "1"
-INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
-INSANE_SKIP_${PN}-dev = "arch"
diff --git a/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
b/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
deleted file mode 100644
index f1273f3..000
--- a/recipes-ti/dsplib/dsplib-c66x_3.4.0.0.bb
+++ /dev/null
@@ -1,9 +0,0 @@
-require dsplib-c66x.inc
-
-PV = "3_4_0_0"
-PR = "r1"
-
-LIC_FILES_CHKSUM =
"file://packages/ti/dsplib/dsplib.h;beginline=9;endline=35;md5=17733f9e1100f8a628512f47c4bf676a"
-
-SRC_URI[dsplib.md5sum] = "66785604729c153dcad5282097dee8bf"
-SRC_URI[dsplib.sha256sum] =
"e88909b6271ce6c4c62e2ab90181b7adb36c7f28f5c52836668eb35888785ce8"
diff --git a/recipes-ti/dsplib/dsplib-c66x_git.bb
b/recipes-ti/dsplib/dsplib-c66x_git.bb
new file mode 100644
index 000..cbc25d0
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib-c66x_git.bb
@@ -0,0 +1,10 @@
+MATHLIB_INSTALL_DIR = "${MATHLIB_C66_INSTALL_DIR}/packages"
+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C66_INSTALL_DIR_RECIPE}"
+DSPLIB_INSTALL_DIR = "${DSPLIB_C66_INSTALL_DIR}"
+DSPLIB_BUILD_TARGET = "c66x"
+
+DEPENDS = "mathlib-c66x"
+
+include dsplib.inc
+
+
diff --git a/recipes-ti/dsplib/dsplib-c674x_git.bb
b/recipes-ti/dsplib/dsplib-c674x_git.bb
new file mode 100644
index 000..07c6892
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib-c674x_git.bb
@@ -0,0 +1,9 @@
+MATHLIB_INSTALL_DIR = "${MATHLIB_C674_INSTALL_DIR}/packages"
+DSPLIB_INSTALL_DIR_RECIPE = "${DSPLIB_C674_INSTALL_DIR_RECIPE}"
+DSPLIB_INSTALL_DIR = "${DSPLIB_C674_INSTALL_DIR}"
+DSPLIB_BUILD_TARGET = "c674x"
+
+DEPENDS = "mathlib-c674x"
+
+include dsplib.inc
+
diff --git a/recipes-ti/dsplib/dsplib.inc b/recipes-ti/dsplib/dsplib.inc
new file mode 100644
index 000..66fe0fb
--- /dev/null
+++ b/recipes-ti/dsplib/dsplib.inc
@@ -0,0 +1,79 @@
+SUMMARY = "DSPLIB contains optimized signal processing functions for TI DSPs"
+HOMEPAGE = "http://git.ti.com/ep-processor-libraries/dsplib;
+LICENSE = "BSD-3-Clause"
+
+require ../includes/ti-paths.inc
+
+PV = "3_4_0_1"
+PR = "r0"
+
+COMPATIBLE_MACHINE = "am57xx-evm|keystone|c66x|omapl1"
+
+DEPENDS += "ti-cgt6x-native \
+ ti-xdctools-native \
+ ti-sysbios \
+ doxygen-native \
+ zip-native \
+ swtools "
+
+S = "${WORKDIR}/git"
+
+SRC_URI =
"git://git.ti.com/ep-processor-libraries/dsplib.git;protocol=git;branch=master"
