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

On 1/23/2026 11:01 AM, Denys Dmytriyenko wrote:
> 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.

Good point.


-- 
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS