[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: kern/54264: audio(4) API fails to detect a capture-only device
At Thu, 06 Jun 2019 10:25:54 +0900 (JST),
Takeshi Nakayama wrote:
> The patch for uaudio.c works well on my USB speaker.
> Rev.1.7 of audio.c is workaround, so I agree with reverting it.
> But I don't think rev.1.5 is completely wrong. Variable error is
> used for result of both audio_hw_probe_fmt(..., AUMODE_PLAY) and
> audio_hw_probe_fmt(..., AUMODE_RECORD), so the result for
> AUMODE_PLAY is overwritten by its for AUMODE_RECORD.
> I think both errors should be checked and audio_hw_probe returns
> error if both are in error as of rev.1.5, or if one or the other is
> in error.
Oh, it's my wrong about rev1.5, sorry.
> Also, if such a strange hardware driver should be fixed, I think
> that you should add KASSERT or else to detect such a driver.
Yeah I will do so.
(I should have written about it in my previous mail..)
Tetsuya Isaki <isaki%pastel-flower.jp@localhost / isaki%NetBSD.org@localhost>
Main Index |
Thread Index |