[meta-arago][scarthgap/master][PATCH 1/3] ti-demos: Add pdm-anomaly-detection for all platforms

[meta-arago][scarthgap/master][PATCH 1/3] ti-demos: Add pdm-anomaly-detection for all platforms

[Andrew Davis]

There is nothing in this demo that makes it specific to any board.
Enable this for all devices.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 .../recipes-core/packagegroups/ti-demos.bb             | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
index 7bccb4de..91e469e7 100644
--- a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
@@ -1,17 +1,9 @@
 SUMMARY = "TI World packagegroup"
 LICENSE = "MIT"
 
-PACKAGE_ARCH = "${MACHINE_ARCH}"
-
 inherit packagegroup
 
-PDM_ANOMALY_PKG = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
-
-PDM_ANOMALY = ""
-PDM_ANOMALY:ti33x = "${PDM_ANOMALY_PKG}"
-PDM_ANOMALY:ti43x = "${PDM_ANOMALY_PKG}"
-PDM_ANOMALY:omap-a15 = "${PDM_ANOMALY_PKG}"
-PDM_ANOMALY:am65xx = "${PDM_ANOMALY_PKG}"
+PDM_ANOMALY = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
 
 RDEPENDS:${PN} = "\
     ${PDM_ANOMALY} \
-- 
2.39.2

[meta-arago][scarthgap/master][PATCH 2/3] ti-demos: Remove this packagegroup

[Andrew Davis]

Having a packagegroup for a single package isn't really needed.
If we end up with more demos and want to collect them we can do
that when needed.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 .../recipes-core/images/tisdk-default-image.bb         |  2 +-
 .../recipes-core/packagegroups/ti-demos.bb             | 10 ----------
 .../recipes-core/packagegroups/ti-world.bb             |  2 +-
 3 files changed, 2 insertions(+), 12 deletions(-)
 delete mode 100644 meta-arago-distro/recipes-core/packagegroups/ti-demos.bb

diff --git a/meta-arago-distro/recipes-core/images/tisdk-default-image.bb b/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
index faae53d3..ce3c3879 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
@@ -24,7 +24,7 @@ IMAGE_INSTALL += "\
     packagegroup-arago-tisdk-addons \
     packagegroup-arago-tisdk-addons-extra \
     ${@bb.utils.contains('DISTRO_FEATURES','opengl','packagegroup-arago-tisdk-hmi','packagegroup-arago-base-tisdk-server-extra',d)} \
-    ti-demos \
+    ${@bb.utils.contains('DISTRO_FEATURES','opengl','pdm-anomaly-detection','', d)} \
     ${ARAGO_DEFAULT_IMAGE_EXTRA_INSTALL} \
     packagegroup-arago-tisdk-sysrepo \
 "
diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
deleted file mode 100644
index 91e469e7..00000000
--- a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
+++ /dev/null
@@ -1,10 +0,0 @@
-SUMMARY = "TI World packagegroup"
-LICENSE = "MIT"
-
-inherit packagegroup
-
-PDM_ANOMALY = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
-
-RDEPENDS:${PN} = "\
-    ${PDM_ANOMALY} \
-"
diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-world.bb b/meta-arago-distro/recipes-core/packagegroups/ti-world.bb
index 438dd8d0..96bfcac8 100644
--- a/meta-arago-distro/recipes-core/packagegroups/ti-world.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/ti-world.bb
@@ -50,6 +50,6 @@ RDEPENDS:${PN} = "\
     ${DEVTOOLS} \
     ${@bb.utils.contains('TUNE_FEATURES', 'armv7a', 'valgrind', '', d)} \
     packagegroup-arago-misc \
-    ti-demos \
+    ${@bb.utils.contains('DISTRO_FEATURES','opengl','pdm-anomaly-detection','', d)} \
     ${PREFERRED_PROVIDER_virtual/docker} \
 "
-- 
2.39.2

[meta-arago][scarthgap/master][PATCH 3/3] packagegroup-arago-tisdk-hmi: Enable HMI demos for all platforms

[Andrew Davis]

There is nothing in this demo that makes it specific to any board.
Enable these for all devices.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 .../packagegroup-arago-tisdk-hmi.bb           | 27 +------------------
 1 file changed, 1 insertion(+), 26 deletions(-)

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
index a8261bf2..ca528d3d 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-hmi.bb
@@ -2,35 +2,10 @@ SUMMARY = "Task to add HMI related packages"
 LICENSE = "MIT"
 PR = "r0"
 
-PACKAGE_ARCH = "${MACHINE_ARCH}"
-
 inherit packagegroup
 
-HMI = " \
-"
-
-
-HMI:append:ti33x = " \
+RDEPENDS:${PN} = "\
     evse-hmi \
     protection-relays-hmi \
     mmwavegesture-hmi \
 "
-
-HMI:append:ti43x = " \
-    evse-hmi \
-    mmwavegesture-hmi \
-"
-
-HMI:append:omap-a15 = " \
-    evse-hmi \
-    mmwavegesture-hmi \
-"
-
-HMI:append:am65xx = " \
-    evse-hmi \
-    mmwavegesture-hmi \
-"
-
-RDEPENDS:${PN} = "\
-	${HMI} \
-"
-- 
2.39.2

Re: [meta-arago][scarthgap/master][PATCH 1/3] ti-demos: Add pdm-anomaly-detection for all platforms

[Denys Dmytriyenko]

On Tue, Dec 17, 2024 at 03:02:04PM -0600, Andrew Davis via lists.yoctoproject.org wrote:
> There is nothing in this demo that makes it specific to any board.
> Enable this for all devices.

Why fix up ti-demos.bb here and completely remove it in the next patch of the 
series?


> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  .../recipes-core/packagegroups/ti-demos.bb             | 10 +---------
>  1 file changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
> index 7bccb4de..91e469e7 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
> @@ -1,17 +1,9 @@
>  SUMMARY = "TI World packagegroup"
>  LICENSE = "MIT"
>  
> -PACKAGE_ARCH = "${MACHINE_ARCH}"
> -
>  inherit packagegroup
>  
> -PDM_ANOMALY_PKG = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
> -
> -PDM_ANOMALY = ""
> -PDM_ANOMALY:ti33x = "${PDM_ANOMALY_PKG}"
> -PDM_ANOMALY:ti43x = "${PDM_ANOMALY_PKG}"
> -PDM_ANOMALY:omap-a15 = "${PDM_ANOMALY_PKG}"
> -PDM_ANOMALY:am65xx = "${PDM_ANOMALY_PKG}"
> +PDM_ANOMALY = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
>  
>  RDEPENDS:${PN} = "\
>      ${PDM_ANOMALY} \
> -- 
> 2.39.2

Re: [meta-arago][scarthgap/master][PATCH 1/3] ti-demos: Add pdm-anomaly-detection for all platforms

[Andrew Davis]

On 12/18/24 3:42 PM, Denys Dmytriyenko wrote:
> On Tue, Dec 17, 2024 at 03:02:04PM -0600, Andrew Davis via lists.yoctoproject.org wrote:
>> There is nothing in this demo that makes it specific to any board.
>> Enable this for all devices.
> 
> Why fix up ti-demos.bb here and completely remove it in the next patch of the
> series?
> 

Felt better to do this in two steps, especially when each step could break
something different :)

> 
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   .../recipes-core/packagegroups/ti-demos.bb             | 10 +---------
>>   1 file changed, 1 insertion(+), 9 deletions(-)
>>
>> diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
>> index 7bccb4de..91e469e7 100644
>> --- a/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
>> +++ b/meta-arago-distro/recipes-core/packagegroups/ti-demos.bb
>> @@ -1,17 +1,9 @@
>>   SUMMARY = "TI World packagegroup"
>>   LICENSE = "MIT"
>>   
>> -PACKAGE_ARCH = "${MACHINE_ARCH}"
>> -
>>   inherit packagegroup
>>   
>> -PDM_ANOMALY_PKG = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
>> -
>> -PDM_ANOMALY = ""
>> -PDM_ANOMALY:ti33x = "${PDM_ANOMALY_PKG}"
>> -PDM_ANOMALY:ti43x = "${PDM_ANOMALY_PKG}"
>> -PDM_ANOMALY:omap-a15 = "${PDM_ANOMALY_PKG}"
>> -PDM_ANOMALY:am65xx = "${PDM_ANOMALY_PKG}"
>> +PDM_ANOMALY = "${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'pdm-anomaly-detection', '', d)}"
>>   
>>   RDEPENDS:${PN} = "\
>>       ${PDM_ANOMALY} \
>> -- 
>> 2.39.2