diff mbox series

[master,RFC] openmp-rtos: deprecate

Message ID 20240805043654.172479-1-denis@denix.org
State RFC
Delegated to: Ryan Eatmon
Headers show
Series [master,RFC] openmp-rtos: deprecate | expand

Commit Message

Denys Dmytriyenko Aug. 5, 2024, 4:36 a.m. UTC
From: Denys Dmytriyenko <denys@konsulko.com>

Remove openmp-rtos component as deprecated and not maintained.

This used to provide OpenMP DSP runtime components for legacy devices.

Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
---
 .../recipes-ti/openmp-rtos/openmp-rtos_git.bb | 62 -------------------
 1 file changed, 62 deletions(-)
 delete mode 100644 meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb

Comments

Andrew Davis Aug. 8, 2024, 5:17 p.m. UTC | #1
On 8/4/24 11:36 PM, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@konsulko.com>
> 
> Remove openmp-rtos component as deprecated and not maintained.
> 
> This used to provide OpenMP DSP runtime components for legacy devices.
> 

With all the OpenCL stuff gone from meta-arago, dsplib/mathlib/imglib/swtools
can also be dropped. On the side, dspdce-fw and ipumm-fw should be updated
to be shipped as binaries, same as we do with all other sysbios/ti-rtos firmware.
Then we could also drop out all the remaining sysbios/xdc bits.

Andrew

> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> ---
>   .../recipes-ti/openmp-rtos/openmp-rtos_git.bb | 62 -------------------
>   1 file changed, 62 deletions(-)
>   delete mode 100644 meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
> 
> diff --git a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb b/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
> deleted file mode 100644
> index aa4bc6cd..00000000
> --- a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
> +++ /dev/null
> @@ -1,62 +0,0 @@
> -DESCRIPTION = "TI OpenMP RTOS"
> -HOMEPAGE = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
> -LICENSE = "BSD-3-Clause"
> -
> -require recipes-ti/includes/ti-paths.inc
> -
> -inherit features_check
> -
> -REQUIRED_MACHINE_FEATURES = "dsp"
> -REQUIRED_DISTRO_FEATURES = "openmp"
> -
> -PV = "2_06_03_00"
> -PR = "r0"
> -
> -OPENMP_RTOS_GIT_URI = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
> -OPENMP_RTOS_GIT_PROTOCOL = "https"
> -OPENMP_RTOS_GIT_BRANCH = "master"
> -
> -BRANCH = "${OPENMP_RTOS_GIT_BRANCH}"
> -SRC_URI = "${OPENMP_RTOS_GIT_URI};protocol=${OPENMP_RTOS_GIT_PROTOCOL};branch=${BRANCH}"
> -
> -SRCREV = "c090eb664d9815a36ead0e14f31e102590680fb8"
> -
> -LIC_FILES_CHKSUM = "file://docs/license/omp_manifest_template.html;md5=61a6972303c0447b7c056195d7ebafee"
> -
> -DEPENDS = "common-csl-ip-rtos doxygen-native libulm ti-xdctools-native ti-ipc-rtos ti-sysbios ti-cgt6x-native zip-native"
> -
> -COMPATIBLE_MACHINE = "omap-a15"
> -PACKAGE_ARCH = "${MACHINE_ARCH}"
> -
> -S = "${WORKDIR}/git"
> -export IPC_DIR = "${IPC_INSTALL_DIR}"
> -export XDC_DIR = "${XDC_INSTALL_DIR}"
> -export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
> -export ULM_DIR ="${STAGING_DIR_TARGET}/usr/share/ti/ulm"
> -export C6636_PDK_DIR ="${PDK_INSTALL_DIR}"
> -export AM572_PDK_DIR ="${PDK_INSTALL_DIR}"
> -export XDCCGROOT = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> -
> -export BUILD_AM572 = "0"
> -
> -BUILD_AM572:omap-a15 = "1"
> -
> -RELEASE_TARGET = ""
> -RELEASE_TARGET:omap-a15 = "am57xx"
> -
> -do_compile() {
> -    make -f utils/product/Makefile .zipfile
> -}
> -
> -do_install() {
> -    install -d ${D}${OMP_INSTALL_DIR_RECIPE}
> -    cp -r ${S}/exports/openmp_dsp_${RELEASE_TARGET}_*/. -d  ${D}${OMP_INSTALL_DIR_RECIPE}
> -}
> -
> -ALLOW_EMPTY:${PN} = "1"
> -
> -FILES:${PN}-dev += " \
> -    ${OMP_INSTALL_DIR_RECIPE}/ \
> -"
> -
> -INSANE_SKIP:${PN}-dev = "arch"
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#17923): https://lists.yoctoproject.org/g/meta-ti/message/17923
> Mute This Topic: https://lists.yoctoproject.org/mt/107727040/3619733
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Denys Dmytriyenko Aug. 26, 2024, 6:46 p.m. UTC | #2
On Thu, Aug 08, 2024 at 12:17:45PM -0500, Andrew Davis wrote:
> On 8/4/24 11:36 PM, Denys Dmytriyenko wrote:
> >From: Denys Dmytriyenko <denys@konsulko.com>
> >
> >Remove openmp-rtos component as deprecated and not maintained.
> >
> >This used to provide OpenMP DSP runtime components for legacy devices.
> >
> 
> With all the OpenCL stuff gone from meta-arago, dsplib/mathlib/imglib/swtools
> can also be dropped.

None of those use OpenCL and are rather self-contained (either prebuilt 
binaries, or use existing components from meta-ti, like CGT).

But they are in the same bucket of deprecated DSP-programming enablement in 
TI-SDK. Moreover, we are seeing deeper issues with old CGT versions generating 
non-reproducible code, so this class of components was pending a second round 
of clean ups...


> On the side, dspdce-fw and ipumm-fw should be updated
> to be shipped as binaries, same as we do with all other sysbios/ti-rtos firmware.
> Then we could also drop out all the remaining sysbios/xdc bits.

Yes, that was the thinking. Someone needs to add prebuilt FW binaries to 
ti-linux-firmware repo before the rest of dependencies can finally be dropped 
from meta-ti.


> Andrew
> 
> >Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> >---
> >  .../recipes-ti/openmp-rtos/openmp-rtos_git.bb | 62 -------------------
> >  1 file changed, 62 deletions(-)
> >  delete mode 100644 meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
> >
> >diff --git a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb b/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
> >deleted file mode 100644
> >index aa4bc6cd..00000000
> >--- a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
> >+++ /dev/null
> >@@ -1,62 +0,0 @@
> >-DESCRIPTION = "TI OpenMP RTOS"
> >-HOMEPAGE = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
> >-LICENSE = "BSD-3-Clause"
> >-
> >-require recipes-ti/includes/ti-paths.inc
> >-
> >-inherit features_check
> >-
> >-REQUIRED_MACHINE_FEATURES = "dsp"
> >-REQUIRED_DISTRO_FEATURES = "openmp"
> >-
> >-PV = "2_06_03_00"
> >-PR = "r0"
> >-
> >-OPENMP_RTOS_GIT_URI = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
> >-OPENMP_RTOS_GIT_PROTOCOL = "https"
> >-OPENMP_RTOS_GIT_BRANCH = "master"
> >-
> >-BRANCH = "${OPENMP_RTOS_GIT_BRANCH}"
> >-SRC_URI = "${OPENMP_RTOS_GIT_URI};protocol=${OPENMP_RTOS_GIT_PROTOCOL};branch=${BRANCH}"
> >-
> >-SRCREV = "c090eb664d9815a36ead0e14f31e102590680fb8"
> >-
> >-LIC_FILES_CHKSUM = "file://docs/license/omp_manifest_template.html;md5=61a6972303c0447b7c056195d7ebafee"
> >-
> >-DEPENDS = "common-csl-ip-rtos doxygen-native libulm ti-xdctools-native ti-ipc-rtos ti-sysbios ti-cgt6x-native zip-native"
> >-
> >-COMPATIBLE_MACHINE = "omap-a15"
> >-PACKAGE_ARCH = "${MACHINE_ARCH}"
> >-
> >-S = "${WORKDIR}/git"
> >-export IPC_DIR = "${IPC_INSTALL_DIR}"
> >-export XDC_DIR = "${XDC_INSTALL_DIR}"
> >-export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
> >-export ULM_DIR ="${STAGING_DIR_TARGET}/usr/share/ti/ulm"
> >-export C6636_PDK_DIR ="${PDK_INSTALL_DIR}"
> >-export AM572_PDK_DIR ="${PDK_INSTALL_DIR}"
> >-export XDCCGROOT = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> >-
> >-export BUILD_AM572 = "0"
> >-
> >-BUILD_AM572:omap-a15 = "1"
> >-
> >-RELEASE_TARGET = ""
> >-RELEASE_TARGET:omap-a15 = "am57xx"
> >-
> >-do_compile() {
> >-    make -f utils/product/Makefile .zipfile
> >-}
> >-
> >-do_install() {
> >-    install -d ${D}${OMP_INSTALL_DIR_RECIPE}
> >-    cp -r ${S}/exports/openmp_dsp_${RELEASE_TARGET}_*/. -d  ${D}${OMP_INSTALL_DIR_RECIPE}
> >-}
> >-
> >-ALLOW_EMPTY:${PN} = "1"
> >-
> >-FILES:${PN}-dev += " \
> >-    ${OMP_INSTALL_DIR_RECIPE}/ \
> >-"
> >-
> >-INSANE_SKIP:${PN}-dev = "arch"
Andrew Davis Aug. 26, 2024, 7:53 p.m. UTC | #3
On 8/26/24 1:46 PM, Denys Dmytriyenko wrote:
> On Thu, Aug 08, 2024 at 12:17:45PM -0500, Andrew Davis wrote:
>> On 8/4/24 11:36 PM, Denys Dmytriyenko wrote:
>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>
>>> Remove openmp-rtos component as deprecated and not maintained.
>>>
>>> This used to provide OpenMP DSP runtime components for legacy devices.
>>>
>>
>> With all the OpenCL stuff gone from meta-arago, dsplib/mathlib/imglib/swtools
>> can also be dropped.
> 
> None of those use OpenCL and are rather self-contained (either prebuilt
> binaries, or use existing components from meta-ti, like CGT).
> 

Other way around, they were only used by OpenCL. While there could be some other
users out of tree users we don't know about, I highly doubt it at this point.

> But they are in the same bucket of deprecated DSP-programming enablement in
> TI-SDK. Moreover, we are seeing deeper issues with old CGT versions generating
> non-reproducible code, so this class of components was pending a second round
> of clean ups...
> 
> 
>> On the side, dspdce-fw and ipumm-fw should be updated
>> to be shipped as binaries, same as we do with all other sysbios/ti-rtos firmware.
>> Then we could also drop out all the remaining sysbios/xdc bits.
> 
> Yes, that was the thinking. Someone needs to add prebuilt FW binaries to
> ti-linux-firmware repo before the rest of dependencies can finally be dropped
> from meta-ti.
> 

If we want to put a timeline in place for dropping the sysbios/xdc parts,
I could add those firmware to right now. If we think this will be many
years out, then we can take our time on that. Doesn't matter much either
way to me.

Andrew

> 
>> Andrew
>>
>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
>>> ---
>>>   .../recipes-ti/openmp-rtos/openmp-rtos_git.bb | 62 -------------------
>>>   1 file changed, 62 deletions(-)
>>>   delete mode 100644 meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
>>>
>>> diff --git a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb b/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
>>> deleted file mode 100644
>>> index aa4bc6cd..00000000
>>> --- a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
>>> +++ /dev/null
>>> @@ -1,62 +0,0 @@
>>> -DESCRIPTION = "TI OpenMP RTOS"
>>> -HOMEPAGE = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
>>> -LICENSE = "BSD-3-Clause"
>>> -
>>> -require recipes-ti/includes/ti-paths.inc
>>> -
>>> -inherit features_check
>>> -
>>> -REQUIRED_MACHINE_FEATURES = "dsp"
>>> -REQUIRED_DISTRO_FEATURES = "openmp"
>>> -
>>> -PV = "2_06_03_00"
>>> -PR = "r0"
>>> -
>>> -OPENMP_RTOS_GIT_URI = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
>>> -OPENMP_RTOS_GIT_PROTOCOL = "https"
>>> -OPENMP_RTOS_GIT_BRANCH = "master"
>>> -
>>> -BRANCH = "${OPENMP_RTOS_GIT_BRANCH}"
>>> -SRC_URI = "${OPENMP_RTOS_GIT_URI};protocol=${OPENMP_RTOS_GIT_PROTOCOL};branch=${BRANCH}"
>>> -
>>> -SRCREV = "c090eb664d9815a36ead0e14f31e102590680fb8"
>>> -
>>> -LIC_FILES_CHKSUM = "file://docs/license/omp_manifest_template.html;md5=61a6972303c0447b7c056195d7ebafee"
>>> -
>>> -DEPENDS = "common-csl-ip-rtos doxygen-native libulm ti-xdctools-native ti-ipc-rtos ti-sysbios ti-cgt6x-native zip-native"
>>> -
>>> -COMPATIBLE_MACHINE = "omap-a15"
>>> -PACKAGE_ARCH = "${MACHINE_ARCH}"
>>> -
>>> -S = "${WORKDIR}/git"
>>> -export IPC_DIR = "${IPC_INSTALL_DIR}"
>>> -export XDC_DIR = "${XDC_INSTALL_DIR}"
>>> -export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
>>> -export ULM_DIR ="${STAGING_DIR_TARGET}/usr/share/ti/ulm"
>>> -export C6636_PDK_DIR ="${PDK_INSTALL_DIR}"
>>> -export AM572_PDK_DIR ="${PDK_INSTALL_DIR}"
>>> -export XDCCGROOT = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
>>> -
>>> -export BUILD_AM572 = "0"
>>> -
>>> -BUILD_AM572:omap-a15 = "1"
>>> -
>>> -RELEASE_TARGET = ""
>>> -RELEASE_TARGET:omap-a15 = "am57xx"
>>> -
>>> -do_compile() {
>>> -    make -f utils/product/Makefile .zipfile
>>> -}
>>> -
>>> -do_install() {
>>> -    install -d ${D}${OMP_INSTALL_DIR_RECIPE}
>>> -    cp -r ${S}/exports/openmp_dsp_${RELEASE_TARGET}_*/. -d  ${D}${OMP_INSTALL_DIR_RECIPE}
>>> -}
>>> -
>>> -ALLOW_EMPTY:${PN} = "1"
>>> -
>>> -FILES:${PN}-dev += " \
>>> -    ${OMP_INSTALL_DIR_RECIPE}/ \
>>> -"
>>> -
>>> -INSANE_SKIP:${PN}-dev = "arch"
diff mbox series

Patch

diff --git a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb b/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
deleted file mode 100644
index aa4bc6cd..00000000
--- a/meta-ti-extras/recipes-ti/openmp-rtos/openmp-rtos_git.bb
+++ /dev/null
@@ -1,62 +0,0 @@ 
-DESCRIPTION = "TI OpenMP RTOS"
-HOMEPAGE = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
-LICENSE = "BSD-3-Clause"
-
-require recipes-ti/includes/ti-paths.inc
-
-inherit features_check
-
-REQUIRED_MACHINE_FEATURES = "dsp"
-REQUIRED_DISTRO_FEATURES = "openmp"
-
-PV = "2_06_03_00"
-PR = "r0"
-
-OPENMP_RTOS_GIT_URI = "git://git.ti.com/git/openmp/ti-openmp-dsp-runtime.git"
-OPENMP_RTOS_GIT_PROTOCOL = "https"
-OPENMP_RTOS_GIT_BRANCH = "master"
-
-BRANCH = "${OPENMP_RTOS_GIT_BRANCH}"
-SRC_URI = "${OPENMP_RTOS_GIT_URI};protocol=${OPENMP_RTOS_GIT_PROTOCOL};branch=${BRANCH}"
-
-SRCREV = "c090eb664d9815a36ead0e14f31e102590680fb8"
-
-LIC_FILES_CHKSUM = "file://docs/license/omp_manifest_template.html;md5=61a6972303c0447b7c056195d7ebafee"
-
-DEPENDS = "common-csl-ip-rtos doxygen-native libulm ti-xdctools-native ti-ipc-rtos ti-sysbios ti-cgt6x-native zip-native"
-
-COMPATIBLE_MACHINE = "omap-a15"
-PACKAGE_ARCH = "${MACHINE_ARCH}"
-
-S = "${WORKDIR}/git"
-export IPC_DIR = "${IPC_INSTALL_DIR}"
-export XDC_DIR = "${XDC_INSTALL_DIR}"
-export BIOS_DIR = "${SYSBIOS_INSTALL_DIR}"
-export ULM_DIR ="${STAGING_DIR_TARGET}/usr/share/ti/ulm"
-export C6636_PDK_DIR ="${PDK_INSTALL_DIR}"
-export AM572_PDK_DIR ="${PDK_INSTALL_DIR}"
-export XDCCGROOT = "${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
-
-export BUILD_AM572 = "0"
-
-BUILD_AM572:omap-a15 = "1"
-
-RELEASE_TARGET = ""
-RELEASE_TARGET:omap-a15 = "am57xx"
-
-do_compile() {
-    make -f utils/product/Makefile .zipfile
-}
-
-do_install() {
-    install -d ${D}${OMP_INSTALL_DIR_RECIPE}
-    cp -r ${S}/exports/openmp_dsp_${RELEASE_TARGET}_*/. -d  ${D}${OMP_INSTALL_DIR_RECIPE}
-}
-
-ALLOW_EMPTY:${PN} = "1"
-
-FILES:${PN}-dev += " \
-    ${OMP_INSTALL_DIR_RECIPE}/ \
-"
-
-INSANE_SKIP:${PN}-dev = "arch"