Skip to content

qcs6490: Fix sound card cannot be detected#9615

Open
kasimling wants to merge 1 commit intoarmbian:mainfrom
kasimling:q6a_fix_audio_on_boot
Open

qcs6490: Fix sound card cannot be detected#9615
kasimling wants to merge 1 commit intoarmbian:mainfrom
kasimling:q6a_fix_audio_on_boot

Conversation

@kasimling
Copy link
Copy Markdown
Contributor

@kasimling kasimling commented Mar 29, 2026

It seems wcd938x driver module is not loaded automatically by default. Also use GITHUB_SOURCE to respect proxy settings for building.

You can refer to https://forum.armbian.com/topic/57121-latest-armbian-build-hdmi-audio-support-fix/ for more issue details.

Summary by CodeRabbit

  • Chores
    • Enhanced audio support initialization for Radxa Dragon Q6a devices
    • Improved build configuration management with more maintainable dependency resolution

It seems wcd938x driver module is not loaded automatically by default.
Also use GITHUB_SOURCE to respect proxy settings for building.
@kasimling kasimling requested a review from HeyMeco as a code owner March 29, 2026 14:22
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d972ddf0-92d1-45fd-b503-c41777c287b1

📥 Commits

Reviewing files that changed from the base of the PR and between 2d17191 and cfa7144.

📒 Files selected for processing (1)
  • config/boards/radxa-dragon-q6a.conf

📝 Walkthrough

Walkthrough

Configuration updates to the Radxa Dragon Q6A board setup: enhanced audio module initialization in the initramfs by creating modules-load.d directory and configuring the WCD938X sound codec module, and refactored the ALSA UCM package URL to use a variable reference instead of a hardcoded URL.

Changes

Cohort / File(s) Summary
Audio Module Configuration
config/boards/radxa-dragon-q6a.conf
Extended BSP firmware initialization to create etc/modules-load.d/ directory and add wcd938x.conf for loading the snd-soc-wcd938x module. Updated ALSA UCM installation to use ${GITHUB_SOURCE} variable instead of hardcoded GitHub URL base.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A rabbit's tune for audio so sweet,
Where modules load and codecs meet,
From hardcoded paths to variables true,
The Dragon's song rings fresh and new! 🎵

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'qcs6490: Fix sound card cannot be detected' directly corresponds to the main objective of the PR, which addresses an issue where the wcd938x driver module is not loaded automatically, preventing sound card detection.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 and usage tips.

@github-actions github-actions bot added 05 Milestone: Second quarter release size/small PR with less then 50 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels Mar 29, 2026
@HeyMeco
Copy link
Copy Markdown
Collaborator

HeyMeco commented Mar 29, 2026

Thanks for sending this in. Will take a look at it today with some other Q6A improvements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

05 Milestone: Second 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.

2 participants