Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 25 Feb 2025 11:48:27 GMT
From:      Christos Margiolis <christos@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: f121de8fbcbc - main - mixer(3): Do not skip devices with no volume control
Message-ID:  <202502251148.51PBmR84051272@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by christos:

URL: https://cgit.FreeBSD.org/src/commit/?id=f121de8fbcbc8aa82f9fb59039c502406b7c88b6

commit f121de8fbcbc8aa82f9fb59039c502406b7c88b6
Author:     Christos Margiolis <christos@FreeBSD.org>
AuthorDate: 2025-02-25 11:44:49 +0000
Commit:     Christos Margiolis <christos@FreeBSD.org>
CommitDate: 2025-02-25 11:44:49 +0000

    mixer(3): Do not skip devices with no volume control
    
    Some devices might not have a software volume control, but still be
    able to be set as recording sources.
    
    PR:             279787
    Sponsored by:   The FreeBSD Foundation
    MFC after:      1 week
    Reviewed by:    emaste
    Differential Revision:  https://reviews.freebsd.org/D48730
---
 lib/libmixer/mixer.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/libmixer/mixer.c b/lib/libmixer/mixer.c
index 4abbed9b27d3..2be3a48eb755 100644
--- a/lib/libmixer/mixer.c
+++ b/lib/libmixer/mixer.c
@@ -113,14 +113,14 @@ dunit:
 
 	TAILQ_INIT(&m->devs);
 	for (i = 0; i < SOUND_MIXER_NRDEVICES; i++) {
-		if (!MIX_ISDEV(m, i))
+		if (!MIX_ISDEV(m, i) && !MIX_ISREC(m, i))
 			continue;
 		if ((dp = calloc(1, sizeof(struct mix_dev))) == NULL)
 			goto fail;
 		dp->parent_mixer = m;
 		dp->devno = i;
 		dp->nctl = 0;
-		if (_mixer_readvol(dp) < 0)
+		if (MIX_ISDEV(m, i) && _mixer_readvol(dp) < 0)
 			goto fail;
 		(void)strlcpy(dp->name, names[i], sizeof(dp->name));
 		TAILQ_INIT(&dp->ctls);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202502251148.51PBmR84051272>