[meta-arago][scarthgap/master][PATCH 1/2] packagegroup-arago-gst: Disable vpe and ducati gstreamer plugins

[meta-arago][scarthgap/master][PATCH 1/2] packagegroup-arago-gst: Disable vpe and ducati gstreamer plugins

[Andrew Davis]

These are not buildable due to missing dependencies, not due to the BSP
version or missing hardware. So do not disable the MACHINE_FEATURE for
the hardware, instead simply remove them from the builds until they
can be fixed.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 meta-arago-distro/conf/distro/include/branding-core.inc       | 3 ---
 meta-arago-distro/conf/distro/include/branding-ltsprep.inc    | 3 ---
 .../recipes-core/packagegroups/packagegroup-arago-gst.bb      | 4 ++--
 3 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/meta-arago-distro/conf/distro/include/branding-core.inc b/meta-arago-distro/conf/distro/include/branding-core.inc
index 1d52ed60..48fe36e1 100644
--- a/meta-arago-distro/conf/distro/include/branding-core.inc
+++ b/meta-arago-distro/conf/distro/include/branding-core.inc
@@ -8,9 +8,6 @@
 # Raise priority for sample IPC FW images
 ALTERNATIVE_PRIORITY:pn-ti-ipc-rtos = "15"
 
-# Remove mmip - DRM allocator, KMS sink, Wayland sink need porting to GST 1.18
-MACHINE_FEATURES:remove = "mmip"
-
 # GC320 support requires out-of-tree drivers not available in until LTS prep is
 # complete. Disable for now.
 # Remove gc320 - the driver fails to build with gcc12
diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
index 72bfe0dc..415eee8d 100644
--- a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
+++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
@@ -1,8 +1,5 @@
 TI_PREFERRED_BSP = "ti-6_12"
 
-# Remove mmip - DRM allocator, KMS sink, Wayland sink need porting to GST 1.18
-MACHINE_FEATURES:remove = "mmip"
-
 # GC320 support requires out-of-tree drivers not available in until LTS prep is
 # complete. Disable for now.
 MACHINE_FEATURES:remove = "gc320"
diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
index 3bc3bfc1..0aee4773 100644
--- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-gst.bb
@@ -22,8 +22,8 @@ GSTREAMER_DEPS:append:dra7xx = " \
 "
 
 GSTREAMER_ACCEL_MM = ""
-GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
-GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
+#GSTREAMER_ACCEL_MM:omap-a15 = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', "gstreamer1.0-plugins-ducati", '', d)}"
+#GSTREAMER_ACCEL_MM:append:dra7xx = "${@bb.utils.contains('MACHINE_FEATURES', 'mmip', " gstreamer1.0-plugins-vpe", '', d)}"
 
 RDEPENDS:${PN} = "\
     ${GSTREAMER_DEPS} \
-- 
2.39.2

[meta-arago][scarthgap/master][PATCH 2/2] branding: Disable missing hardware support in the BSP layer

[Andrew Davis]

Currently MACHINE_FEATURES are disabled based on branding, but the
branding is not what determines if these are functional, the BSP version
does. While we do select the BSP version here, what is supported or not
by a given version is a property of the BSP layer. Disabling missing
MACHINE_FEATURES for each version should be handled there.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 meta-arago-distro/conf/distro/include/branding-core.inc    | 5 -----
 meta-arago-distro/conf/distro/include/branding-ltsprep.inc | 4 ----
 2 files changed, 9 deletions(-)

diff --git a/meta-arago-distro/conf/distro/include/branding-core.inc b/meta-arago-distro/conf/distro/include/branding-core.inc
index 48fe36e1..3eed685a 100644
--- a/meta-arago-distro/conf/distro/include/branding-core.inc
+++ b/meta-arago-distro/conf/distro/include/branding-core.inc
@@ -7,8 +7,3 @@
 
 # Raise priority for sample IPC FW images
 ALTERNATIVE_PRIORITY:pn-ti-ipc-rtos = "15"
-
-# GC320 support requires out-of-tree drivers not available in until LTS prep is
-# complete. Disable for now.
-# Remove gc320 - the driver fails to build with gcc12
-MACHINE_FEATURES:remove = "gc320"
diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
index 415eee8d..21e33662 100644
--- a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
+++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
@@ -1,5 +1 @@
 TI_PREFERRED_BSP = "ti-6_12"
-
-# GC320 support requires out-of-tree drivers not available in until LTS prep is
-# complete. Disable for now.
-MACHINE_FEATURES:remove = "gc320"
-- 
2.39.2

Re: [meta-arago][scarthgap/master][PATCH 2/2] branding: Disable missing hardware support in the BSP layer

[Andrew Davis]

On 12/17/24 3:01 PM, Andrew Davis wrote:
> Currently MACHINE_FEATURES are disabled based on branding, but the
> branding is not what determines if these are functional, the BSP version
> does. While we do select the BSP version here, what is supported or not
> by a given version is a property of the BSP layer. Disabling missing
> MACHINE_FEATURES for each version should be handled there.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>   meta-arago-distro/conf/distro/include/branding-core.inc    | 5 -----
>   meta-arago-distro/conf/distro/include/branding-ltsprep.inc | 4 ----
>   2 files changed, 9 deletions(-)
> 
> diff --git a/meta-arago-distro/conf/distro/include/branding-core.inc b/meta-arago-distro/conf/distro/include/branding-core.inc
> index 48fe36e1..3eed685a 100644
> --- a/meta-arago-distro/conf/distro/include/branding-core.inc
> +++ b/meta-arago-distro/conf/distro/include/branding-core.inc
> @@ -7,8 +7,3 @@
>   
>   # Raise priority for sample IPC FW images
>   ALTERNATIVE_PRIORITY:pn-ti-ipc-rtos = "15"
> -
> -# GC320 support requires out-of-tree drivers not available in until LTS prep is
> -# complete. Disable for now.
> -# Remove gc320 - the driver fails to build with gcc12
> -MACHINE_FEATURES:remove = "gc320"
> diff --git a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> index 415eee8d..21e33662 100644
> --- a/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> +++ b/meta-arago-distro/conf/distro/include/branding-ltsprep.inc
> @@ -1,5 +1 @@
>   TI_PREFERRED_BSP = "ti-6_12"
> -
> -# GC320 support requires out-of-tree drivers not available in until LTS prep is
> -# complete. Disable for now.
> -MACHINE_FEATURES:remove = "gc320"

This has a equal opposite patch[0] over in meta-ti disabling these in the
BSP layer. That will need to go in first to prevent GC320 support from
being enabled in the meantime (gc320 doesn't build that would cause failures).

[0] https://lists.yoctoproject.org/g/meta-ti/message/18126

Andrew