multimedia/audio: improve minimal-build handling for playback and record#334
Open
smuppand wants to merge 3 commits intoqualcomm-linux:mainfrom
Open
multimedia/audio: improve minimal-build handling for playback and record#334smuppand wants to merge 3 commits intoqualcomm-linux:mainfrom
smuppand wants to merge 3 commits intoqualcomm-linux:mainfrom
Conversation
Introduce shared helpers for backend management detection and ALSA playback/capture probing. Improve audio fallback behavior for images without pipewire/pulseaudio services. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
vnarapar
reviewed
Mar 13, 2026
| if [ -z "${__INIT_ENV_LOADED:-}" ]; then | ||
| # shellcheck disable=SC1090 | ||
| . "$INIT_ENV" | ||
| __INIT_ENV_LOADED=1 |
Contributor
There was a problem hiding this comment.
why is the spacing removed?
Contributor
Author
There was a problem hiding this comment.
why is the spacing removed?
There is no functional reason for the spacing change — that was unintended during the edit.
vnarapar
reviewed
Mar 13, 2026
| SEARCH="$SCRIPT_DIR" | ||
|
|
||
| while [ "$SEARCH" != "/" ]; do | ||
| if [ -f "$SEARCH/init_env" ]; then |
Contributor
There was a problem hiding this comment.
Any reason why spacing is removed? If so these should be in different commit
Contributor
Author
There was a problem hiding this comment.
Any reason why spacing is removed? If so these should be in different commit
There is no functional reason for the spacing change — that was unintended during the edit.
smuppand
force-pushed
the
audio
branch
6 times, most recently
from
c188be1 to
739ed38
Compare
Handle minimal images without managed audio services by falling back to direct ALSA playback. Keep full-build behavior unchanged while improving minimal-build execution and logging. Add a dedicated AudioPlayback LAVA YAML for minimal images using direct ALSA playback and custom clip path support. Fixed shellcheck errors. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
Try direct ALSA capture on minimal images and report a clear SKIP when no usable capture profile is available. Preserve existing full-build backend flow and logging. Add a dedicated AudioRecord LAVA YAML for minimal images using ALSA capture probing and clear skip behavior. Fixed shellcheck reported lint issues. Signed-off-by: Srikanth Muppandam <smuppand@qti.qualcomm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR improves audio test behavior on minimal builds where systemctl may exist but managed audio services such as PipeWire/PulseAudio are not available.
Changes
Expected behavior