Re: kern/55017: audio0 autoconfiguration error on auvia

[Andrius V <vezhlys%gmail.com@localhost>]

The following reply was made to PR kern/55017; it has been noted by GNATS.

From: Andrius V <vezhlys%gmail.com@localhost>
To: Tetsuya Isaki <isaki%pastel-flower.jp@localhost>
Cc: gnats-bugs%netbsd.org@localhost, kern-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
	netbsd-bugs%netbsd.org@localhost
Subject: Re: kern/55017: audio0 autoconfiguration error on auvia
Date: Sat, 29 Feb 2020 14:41:11 +0200

 --00000000000058341e059fb64561
 Content-Type: text/plain; charset="UTF-8"
 
 Thank you for the quick fix and detailed explanation.
 
 Regards,
 Andrius V
 
 On Fri, Feb 28, 2020, 15:39 Tetsuya Isaki <isaki%pastel-flower.jp@localhost> wrote:
 
 > Hello,
 >
 > At Wed, 26 Feb 2020 20:29:18 +0200,
 > Andrius V wrote:
 > > Patch works:
 > >
 > > auvia0 at pci0 dev 17 function 5: VIA Technologies VT8237 AC'97 Audio
 > (rev 0x60)
 > > allocated pic ioapic0 type level pin 22 level 7 to cpu0 slot 6 idt entry
 > 112
 > > auvia0: interrupting at ioapic0 pin 22
 > > auvia0: ac97: VIA Technologies VT1616i codec; headphone, 18 bit DAC,
 > > 18 bit ADC, KS Waves 3D
 > > auvia0: ac97: ext id 0x1c1<LDAC,SDAC,CDAC,VRA>
 > > audio0 at auvia0: playback, capture, full duplex, independent
 > > audio0: slinear_le:16 6ch 48000Hz, blk 40ms for playback
 > > audio0: slinear_le:16 2ch 48000Hz, blk 40ms for recording
 >
 > I've commited it (and will pullup -9 later).
 >
 > In 8.99.39 audio(4) has been changed to attach with the number of maximum
 > channels that hardware supports.  But auvia(4) had following mismatch:
 > - auvia_round_blocksize returns 512 bytes (in most case).
 > - audio(4) layer requires that round_blocksize returns a multiple of
 >   framesize, (precision/8 * channels).
 > This is not a problem for 2ch/4ch, but is a problem for 6ch.
 >
 > Thank you for reporting.
 > ---
 > Tetsuya Isaki <isaki%pastel-flower.jp@localhost / isaki%NetBSD.org@localhost>
 >
 
 --00000000000058341e059fb64561
 Content-Type: text/html; charset="UTF-8"
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"auto"><div>Thank you for the quick fix and detailed explanation=
 .</div><div dir=3D"auto"><br></div><div dir=3D"auto">Regards,</div><div dir=
 =3D"auto">Andrius V<br><br><div class=3D"gmail_quote" dir=3D"auto"><div dir=
 =3D"ltr" class=3D"gmail_attr">On Fri, Feb 28, 2020, 15:39 Tetsuya Isaki &lt=
 ;<a href=3D"mailto:isaki%pastel-flower.jp@localhost";>isaki%pastel-flower.jp@localhost</a>&gt; w=
 rote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex=
 ;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
 <br>
 At Wed, 26 Feb 2020 20:29:18 +0200,<br>
 Andrius V wrote:<br>
 &gt; Patch works:<br>
 &gt; <br>
 &gt; auvia0 at pci0 dev 17 function 5: VIA Technologies VT8237 AC&#39;97 Au=
 dio (rev 0x60)<br>
 &gt; allocated pic ioapic0 type level pin 22 level 7 to cpu0 slot 6 idt ent=
 ry 112<br>
 &gt; auvia0: interrupting at ioapic0 pin 22<br>
 &gt; auvia0: ac97: VIA Technologies VT1616i codec; headphone, 18 bit DAC,<b=
 r>
 &gt; 18 bit ADC, KS Waves 3D<br>
 &gt; auvia0: ac97: ext id 0x1c1&lt;LDAC,SDAC,CDAC,VRA&gt;<br>
 &gt; audio0 at auvia0: playback, capture, full duplex, independent<br>
 &gt; audio0: slinear_le:16 6ch 48000Hz, blk 40ms for playback<br>
 &gt; audio0: slinear_le:16 2ch 48000Hz, blk 40ms for recording<br>
 <br>
 I&#39;ve commited it (and will pullup -9 later).<br>
 <br>
 In 8.99.39 audio(4) has been changed to attach with the number of maximum<b=
 r>
 channels that hardware supports.=C2=A0 But auvia(4) had following mismatch:=
 <br>
 - auvia_round_blocksize returns 512 bytes (in most case).<br>
 - audio(4) layer requires that round_blocksize returns a multiple of<br>
 =C2=A0 framesize, (precision/8 * channels).<br>
 This is not a problem for 2ch/4ch, but is a problem for 6ch.<br>
 <br>
 Thank you for reporting.<br>
 ---<br>
 Tetsuya Isaki &lt;<a href=3D"mailto:isaki%pastel-flower.jp@localhost"; target=3D"_blan=
 k" rel=3D"noreferrer">isaki%pastel-flower.jp@localhost</a> / isaki%NetBSD.org@localhost&gt;<br>
 </blockquote></div></div></div>
 
 --00000000000058341e059fb64561--