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: Wed, 26 Feb 2020 20:29:18 +0200

 Hi,
 
 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
 
 Regards,
 Andrius V
 
 On Wed, Feb 26, 2020 at 4:25 PM Tetsuya Isaki <isaki%pastel-flower.jp@localhost> wrote:
 >
 > At Tue, 25 Feb 2020 23:40:00 +0000 (UTC),
 > vezhlys%gmail.com@localhost wrote:
 > > auvia0 at pci0 dev 17 function 5: VIA Technologies VT8237 AC'97 Audio (rev 0x60)
 > > 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: blksize not configured 23040 -> 512
 > > audio0: autoconfiguration error: configuring playback mode failed
 >
 > Would you try this patch?
 >
 > diff --git a/sys/dev/pci/auvia.c b/sys/dev/pci/auvia.c
 > index a80314177..954c4c8fa 100644
 > --- a/sys/dev/pci/auvia.c
 > +++ b/sys/dev/pci/auvia.c
 > @@ -749,8 +749,7 @@ auvia_round_blocksize(void *addr, int blk,
 >         if (sc->sc_flags & AUVIA_FLAGS_VT8233 && blk < 288)
 >                 blk = 288;
 >
 > -       /* Avoid too many dma_ops. */
 > -       return uimin((blk & -32), AUVIA_MINBLKSZ);
 > +       return (blk & -32);
 >  }
 >
 >  static int
 >
 > ---
 > Tetsuya Isaki <isaki%pastel-flower.jp@localhost / isaki%NetBSD.org@localhost>