Skip to content

boards/youyeetoo-yy3588: name audio outputs on pipewire too#10146

Open
SuperKali wants to merge 1 commit into
armbian:mainfrom
OpenSource-YYT:yy3588-audio-naming-pipewire
Open

boards/youyeetoo-yy3588: name audio outputs on pipewire too#10146
SuperKali wants to merge 1 commit into
armbian:mainfrom
OpenSource-YYT:yy3588-audio-naming-pipewire

Conversation

@SuperKali

@SuperKali SuperKali commented Jul 8, 2026

Copy link
Copy Markdown
Member

Description

On PipeWire releases every YY3588 audio output shows up as a generic "Built-in Audio", so HDMI, DisplayPort and the onboard analog codec are impossible to tell apart in the sound settings. The board already renames them for PulseAudio through a udev rule, and this adds the matching WirePlumber drop-in so the names also show on PipeWire.

How Has This Been Tested?

Before, the three outputs were indistinguishable:

Sinks:
   Built-in Audio Stereo
   Built-in Audio Stereo
   Built-in Audio Stereo

After, each one is named:

Sinks:
   HDMI0 Audio Stereo
   DP0 Audio Stereo
   ES8388 Audio Stereo

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

Summary by CodeRabbit

  • New Features

    • Improved audio device naming on the YY3588 board for clearer device labels in system audio menus.
    • Added updated names for analog, HDMI, and DPI-IO audio cards so they are easier to identify.
  • Bug Fixes

    • Expanded audio naming support to include WirePlumber-based configuration, helping audio devices appear with more consistent descriptions across the system.

The udev SOUND_DESCRIPTION rule only renames cards on PulseAudio releases,
PipeWire (trixie, resolute and later) ignores it, so every output showed
as generic Built-in Audio. Add a WirePlumber 0.5 drop-in that sets
device.description per card, keeping the udev rule for PulseAudio. Verified
on resolute with WirePlumber 0.5.13.

Signed-off-by: SuperKali <hello@superkali.me>
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Extends the post_family_tweaks__youyeetoo_yy3588_naming_audios function in the youyeetoo-yy3588 board config to add a PulseAudio release mapping comment to existing udev rules, and to create a new WirePlumber override configuration file that updates ALSA card device.description for analog, HDMI, and DPI-IO sound cards.

Changes

Audio naming configuration

Layer / File(s) Summary
Udev naming comment update
config/boards/youyeetoo-yy3588.conf
Adds a release mapping comment above existing udev rules setting SOUND_DESCRIPTION for HDMI0, HDMI-In, DP0, and ES8388/analog cards.
WirePlumber override configuration
config/boards/youyeetoo-yy3588.conf
Creates a wireplumber.conf.d drop-in directory and file defining monitor.alsa.rules to update device.description matching the same ALSA card names.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • armbian/build#9960: Modifies the same board's ES8388/analog sound card naming that this PR's WirePlumber rules match against.

Suggested labels: BSP

Suggested reviewers: rpardini, igorpecovnik

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: adding PipeWire audio output naming for the youyeetoo-yy3588 board.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added size/small PR with less then 50 lines 08 Milestone: Third quarter release Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels Jul 8, 2026
@SuperKali SuperKali requested a review from igorpecovnik July 8, 2026 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

08 Milestone: Third quarter release Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

1 participant