[src/trunk]: src/sys/dev Be even more carefull with devices that support only...

[nat <nat%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/b3e29466fce1
branches:  trunk
changeset: 824198:b3e29466fce1
user:      nat <nat%NetBSD.org@localhost>
date:      Sat May 27 13:55:58 2017 +0000

description:
Be even more carefull with devices that support only play back OR capture.

Addresses PR kern/52256.

diffstat:

 sys/dev/audio.c |  18 +++++++++++-------
 1 files changed, 11 insertions(+), 7 deletions(-)

diffs (61 lines):

diff -r ef549f9c4a39 -r b3e29466fce1 sys/dev/audio.c
--- a/sys/dev/audio.c   Sat May 27 13:11:50 2017 +0000
+++ b/sys/dev/audio.c   Sat May 27 13:55:58 2017 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: audio.c,v 1.352 2017/05/23 07:57:26 nat Exp $  */
+/*     $NetBSD: audio.c,v 1.353 2017/05/27 13:55:58 nat Exp $  */
 
 /*-
  * Copyright (c) 2016 Nathanial Sloss <nathanialsloss%yahoo.com.au@localhost>
@@ -148,7 +148,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: audio.c,v 1.352 2017/05/23 07:57:26 nat Exp $");
+__KERNEL_RCSID(0, "$NetBSD: audio.c,v 1.353 2017/05/27 13:55:58 nat Exp $");
 
 #include "audio.h"
 #if NAUDIO > 0
@@ -2011,8 +2011,8 @@
        struct audio_chan *chan;
        int error;
 
+       chan = SIMPLEQ_FIRST(&sc->sc_audiochan);
        if (vc == NULL) {
-               chan = SIMPLEQ_FIRST(&sc->sc_audiochan);
                vc = chan->vc;
                sc->sc_pr.blksize = vc->sc_mrr.blksize;
                sc->sc_rr.blksize = vc->sc_mrr.blksize;
@@ -2020,7 +2020,8 @@
 
        DPRINTF(("audio_initbufs: mode=0x%x\n", vc->sc_mode));
        hw = sc->hw_if;
-       if (audio_can_capture(sc) || (vc->sc_open & AUOPEN_READ)) {
+       if (audio_can_capture(sc) &&
+               ((vc->sc_open & AUOPEN_READ) || vc == chan->vc)) {
                audio_init_ringbuffer(sc, &vc->sc_mrr,
                    AUMODE_RECORD);
                if (sc->sc_opens == 0 && hw->init_input &&
@@ -2034,7 +2035,8 @@
        if (vc == SIMPLEQ_FIRST(&sc->sc_audiochan)->vc)
                sc->sc_rr.blksize = vc->sc_mrr.blksize;
 
-       if (audio_can_playback(sc) || (vc->sc_open & AUOPEN_WRITE)) {
+       if (audio_can_playback(sc) &&
+               ((vc->sc_open & AUOPEN_WRITE) || vc == chan->vc)) {
                audio_init_ringbuffer(sc, &vc->sc_mpr,
                    AUMODE_PLAY);
                vc->sc_sil_count = 0;
@@ -2191,8 +2193,10 @@
                        }
                }
                audio_initbufs(sc, NULL);
-               audio_init_ringbuffer(sc, &sc->sc_pr, AUMODE_PLAY);
-               audio_init_ringbuffer(sc, &sc->sc_rr, AUMODE_RECORD);
+               if (audio_can_playback(sc))
+                       audio_init_ringbuffer(sc, &sc->sc_pr, AUMODE_PLAY);
+               if (audio_can_capture(sc))
+                       audio_init_ringbuffer(sc, &sc->sc_rr, AUMODE_RECORD);
                sc->schedule_wih = false;
                sc->schedule_rih = false;
                sc->sc_last_drops = 0;