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

Hi Paresh, 

On 11:33-20260123, Paresh Bhagat via lists.yoctoproject.org 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]>
> ---

Thanks for your patch!

>  .../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 \
> +"
> +PACKAGECONFIG:append:am62dxx:pn-pipewire = " pipewire-jack pipewire-modules pipewire-spa-plugins"
> +PACKAGECONFIG:remove:am62dxx:pn-pipewire = " jack"

As per yocto docs [0], `pn-pipewire` would not work inside an image
recipe. I suggest dropping PACKAGECONFIG lines, Instead put the
PACKAGECONFIG lines in `pipewire_%.bbappend`

```
PACKAGECONFIG:append:am62dxx = " pipewire-jack pipewire-modules pipewire-spa-plugins"
PACKAGECONFIG:remove:am62dxx = " jack"
```

[0]: https://docs.yoctoproject.org/ref-manual/variables.html#term-OVERRIDES

Regards, 
- Yogesh