Re: [scarthgap][RFC PATCH 3/3] tisdk-default-image: Add audio server support for am62dxx

On Fri, Jan 23, 2026 at 10:11:06AM -0600, Ryan Eatmon wrote:
> 
> 
> On 1/23/2026 12:03 AM, Paresh Bhagat wrote:
> >Add pipewire and wireplumber audio stack support for am62dxx:
> >- Include pipewire core components and tools
> >- Add ALSA and PulseAudio compatibility layers
> >- Include wireplumber session manager
> >- Add GStreamer pipewire plugin
> >- Configure pipewire with proper modules and plugins
> >
> >Signed-off-by: Paresh Bhagat <[email protected]>
> >---
> >  .../recipes-core/images/tisdk-default-image.bb  | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> >
> >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 264d1b2d..d5db6fd4 100644
> >--- a/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
> >+++ b/meta-arago-distro/recipes-core/images/tisdk-default-image.bb
> >@@ -47,3 +47,20 @@ IMAGE_INSTALL += "\
> >      ${@bb.utils.contains('TUNE_FEATURES', 'armv7a', 'valgrind', '', d)} \
> >      docker \
> >  "
> >+
> >+IMAGE_INSTALL:append:am62dxx = " \
> >+    pipewire \
> >+    pipewire-tools \
> >+    pipewire-alsa \
> >+    pipewire-pulse \
> >+    wireplumber \
> >+    speexdsp \
> >+    libpipewire \
> >+    pipewire-spa-plugins-meta \
> >+    pipewire-spa-tools \
> >+    pipewire-modules-meta \
> >+    pipewire-alsa-card-profile \
> >+    gstreamer1.0-pipewire \
> >+"
> 
> I'm sure at some point we will want to push the "more modern"
> pipewire to other boards as well.  What is the reasoning for only
> doing this on am62d?
> 
> Please create a PIPEWIRE variable that contains the above list and then:
> 
> IMAGE_INSTALL:append:am62dxx = " ${PIPEWIRE}"
> 
> Also, for ease of looking up packages, please sort the list alphabetically.
> 
> 
> >+PACKAGECONFIG:append:am62dxx:pn-pipewire = " pipewire-jack pipewire-modules pipewire-spa-plugins"

Huh? pipewire-modules and pipewire-spa-plugins are not PACKAGECONFIGs, but 
packages...


> >+PACKAGECONFIG:remove:am62dxx:pn-pipewire = " jack"

No leading space is needed here.


> Agree with Yogesh.  This is not appropriate in this file.  This
> should be in a pipewire bbappend.

Or rather generically in the distro config, as I suggested last week.

-- 
Denys