[2.6.0-test6] Scratchy sound with via82xx (VT8233)

[2.6.0-test6] Scratchy sound with via82xx (VT8233)

[Kees Bakker]

Starting with 2.6.0-test6 the sound is (often) not OK. For example,
I let KDE play a sound when email arrives. Often I only hear scratchy
noise, but sometimes sound is OK.

The lcpci output for this device is:
00:11.5 Multimedia audio controller: VIA Technologies, Inc. VT8233 AC97 Audio Controller (rev 10)
        Subsystem: Micro-Star International Co., Ltd.: Unknown device 3800
        Flags: medium devsel, IRQ 10
        I/O ports at d000 [size=256]
        Capabilities: [c0] Power Management version 2

I saw the note about dxs_support, but I have the driver built-in. How do I set
dxs_support from the /proc/cmdline?

		Kees

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[James Stevenson]

Hi

i also see this in the 2.4.19 - 2.4.22 kernels
i have the following

00:11.5 Multimedia audio controller: VIA Technologies, Inc. VT8233 AC97 Audio Controller (rev 30)
	Subsystem: AOPEN Inc.: Unknown device 006a
	Control: I/O+ Mem- BusMaster- SpecCycle- MemWINV- VGASnoop- 
ParErr- Stepping- SERR- FastB2B-
	Status: Cap+ 66Mhz- UDF- FastB2B- ParErr- DEVSEL=medium >TAbort- 
	<TAbort- <MAbort- >SERR- <PERR-
	Interrupt: pin C routed to IRQ 12
	Region 0: I/O ports at e000 [size=256]
	Capabilities: <available only to root>

it plays fine then it starts scrathing then plays fine etc.. etc..

	James

On Tue, 30 Sep 2003, Kees Bakker wrote:

> Starting with 2.6.0-test6 the sound is (often) not OK. For example,
> I let KDE play a sound when email arrives. Often I only hear scratchy
> noise, but sometimes sound is OK.
> 
> The lcpci output for this device is:
> 00:11.5 Multimedia audio controller: VIA Technologies, Inc. VT8233 AC97 Audio Controller (rev 10)
>         Subsystem: Micro-Star International Co., Ltd.: Unknown device 3800
>         Flags: medium devsel, IRQ 10
>         I/O ports at d000 [size=256]
>         Capabilities: [c0] Power Management version 2
> 
> I saw the note about dxs_support, but I have the driver built-in. How do I set
> dxs_support from the /proc/cmdline?
> 
> 		Kees
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[Takashi Iwai]

At Tue, 30 Sep 2003 20:46:47 +0200,
Kees Bakker wrote:
> 
> Starting with 2.6.0-test6 the sound is (often) not OK. For example,
> I let KDE play a sound when email arrives. Often I only hear scratchy
> noise, but sometimes sound is OK.
> 
> The lcpci output for this device is:
> 00:11.5 Multimedia audio controller: VIA Technologies, Inc. VT8233 AC97 Audio Controller (rev 10)
>         Subsystem: Micro-Star International Co., Ltd.: Unknown device 3800
>         Flags: medium devsel, IRQ 10
>         I/O ports at d000 [size=256]
>         Capabilities: [c0] Power Management version 2
> 
> I saw the note about dxs_support, but I have the driver built-in. How do I set
> dxs_support from the /proc/cmdline?

pass via boot parameter:

  snd-via82xx=1,0,,-1,48000,XXX

where XXX is the value from 0 to 3 for dxs_support.
see the comment in sound/via82xx.c.


--
Takashi Iwai <tiwai dot suse.de>		ALSA Developer - www.alsa-project.org

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[James Stevenson]

> > 
> > I saw the note about dxs_support, but I have the driver built-in. How do I set
> > dxs_support from the /proc/cmdline?
> 
> pass via boot parameter:
> 
>   snd-via82xx=1,0,,-1,48000,XXX
> 
> where XXX is the value from 0 to 3 for dxs_support.
> see the comment in sound/via82xx.c.


will this work for a 2.4.x kernel as well ?

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[Takashi Iwai]

At Thu, 2 Oct 2003 21:05:59 +0100 (IST),
James Stevenson wrote:
> 
> > > 
> > > I saw the note about dxs_support, but I have the driver built-in. How do I set
> > > dxs_support from the /proc/cmdline?
> > 
> > pass via boot parameter:
> > 
> >   snd-via82xx=1,0,,-1,48000,XXX
> > 
> > where XXX is the value from 0 to 3 for dxs_support.
> > see the comment in sound/via82xx.c.
> 
> 
> will this work for a 2.4.x kernel as well ?

only if the driver is built in the kernel...


--
Takashi Iwai <tiwai dot suse.de>		ALSA Developer - www.alsa-project.org

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[James Stevenson]

On Mon, 6 Oct 2003, Takashi Iwai wrote:

> At Thu, 2 Oct 2003 21:05:59 +0100 (IST),
> James Stevenson wrote:
> > 
> > > > 
> > > > I saw the note about dxs_support, but I have the driver built-in. How do I set
> > > > dxs_support from the /proc/cmdline?
> > > 
> > > pass via boot parameter:
> > > 
> > >   snd-via82xx=1,0,,-1,48000,XXX
> > > 
> > > where XXX is the value from 0 to 3 for dxs_support.
> > > see the comment in sound/via82xx.c.
> > 
> > 
> > will this work for a 2.4.x kernel as well ?
> 
> only if the driver is built in the kernel...

Actually i found a different solution to my problem.
This patch against 2.4.22 seems to sort out my troubles.

Was running this on saturday for around 6/7 hours didnt
skip once. From a problem that used to badly effect me
everytime large windows were redrawn in X like a X virtual
screen switch.


--- via82cxxx_audio.c	1 Sep 2003 10:50:01 -0000	1.1.1.16
+++ via82cxxx_audio.c	4 Oct 2003 10:58:44 -0000
@@ -79,14 +79,14 @@
 #define VIA_COUNTER_LIMIT	100000
 
 /* size of DMA buffers */
-#define VIA_MAX_BUFFER_DMA_PAGES	32
+#define VIA_MAX_BUFFER_DMA_PAGES	128
 
 /* buffering default values in ms */
 #define VIA_DEFAULT_FRAG_TIME		20
-#define VIA_DEFAULT_BUFFER_TIME		500
+#define VIA_DEFAULT_BUFFER_TIME		100
 
 /* the hardware has a 256 fragment limit */
-#define VIA_MIN_FRAG_NUMBER		2
+#define VIA_MIN_FRAG_NUMBER		16
 #define VIA_MAX_FRAG_NUMBER		128
 
 #define VIA_MAX_FRAG_SIZE		PAGE_SIZE

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[Kees Bakker]

[-- Attachment #1: Type: text/plain, Size: 1433 bytes --]

On Thursday 02 October 2003 12:07, you wrote:
> At Tue, 30 Sep 2003 20:46:47 +0200,
>
> Kees Bakker wrote:
> > I saw the note about dxs_support, but I have the driver built-in. How do
> > I set dxs_support from the /proc/cmdline?
>
> pass via boot parameter:
>
>   snd-via82xx=1,0,,-1,48000,XXX
>
> where XXX is the value from 0 to 3 for dxs_support.
> see the comment in sound/via82xx.c.

Ah, I see. Well, I tested my VIA in another way. I applied this patch, which
adds it to the whitelist. So far it behaves OK. Are there any particular
tests that I can run to make sure?

Also, I want draw your attention to the fact that there DOES exist
a VIA8233-Pre in the wild. Here is my lspci output on my MSI KT266
motherboard:

iris:~ # cat /proc/asound/cards
0 [V8233Pre       ]: VIA8233 - VIA 8233-Pre
                     VIA 8233-Pre at 0xd000, irq 10

iris:~ # lspci -nv -s 00:11.5
00:11.5 Class 0401: 1106:3059 (rev 10)
        Subsystem: 1462:3800
        Flags: medium devsel, IRQ 10
        I/O ports at d000 [size=256]
        Capabilities: [c0] Power Management version 2

And in case it helps, here is the output from /proc/asound/devices:
iris:~ # cat /proc/asound/devices
  1:       : sequencer
  0: [0- 0]: ctl
 17: [0- 1]: digital audio playback
 25: [0- 1]: digital audio capture
 16: [0- 0]: digital audio playback
 24: [0- 0]: digital audio capture
 33:       : timer

Let me know if you need more information.
		Kees

[-- Attachment #2: via82xx-add-kt266.patch --]
[-- Type: text/x-diff, Size: 513 bytes --]

--- linux-2.6.0/sound/pci/via82xx.c.orig	2003-10-03 18:30:25.000000000 +0200
+++ linux-2.6.0/sound/pci/via82xx.c	2003-10-03 18:31:38.615792728 +0200
@@ -1969,6 +1969,7 @@
 	static struct dxs_whitelist whitelist[] = {
 		{ .vendor = 0x1019, .device = 0x0996, .action = VIA_DXS_48K },
 		{ .vendor = 0x1297, .device = 0xc160, .action = VIA_DXS_ENABLE }, /* Shuttle SK41G */
+		{ .vendor = 0x1462, .device = 0x3800, .action = VIA_DXS_ENABLE }, /* MSI KT266 */
 		{ } /* terminator */
 	};
 	struct dxs_whitelist *w;

Re: [2.6.0-test6] Scratchy sound with via82xx (VT8233)

[Takashi Iwai]

At Fri, 3 Oct 2003 18:40:17 +0200,
Kees Bakker wrote:
> 
> [1  <text/plain; iso-8859-1 (7bit)>]
> On Thursday 02 October 2003 12:07, you wrote:
> > At Tue, 30 Sep 2003 20:46:47 +0200,
> >
> > Kees Bakker wrote:
> > > I saw the note about dxs_support, but I have the driver built-in. How do
> > > I set dxs_support from the /proc/cmdline?
> >
> > pass via boot parameter:
> >
> >   snd-via82xx=1,0,,-1,48000,XXX
> >
> > where XXX is the value from 0 to 3 for dxs_support.
> > see the comment in sound/via82xx.c.
> 
> Ah, I see. Well, I tested my VIA in another way. I applied this patch, which
> adds it to the whitelist. So far it behaves OK. Are there any particular
> tests that I can run to make sure?

if you hear the clean output of 44.1kHz playback without noises, it
must be ok. 

> Also, I want draw your attention to the fact that there DOES exist
> a VIA8233-Pre in the wild. Here is my lspci output on my MSI KT266
> motherboard:

well, i heard that from VIA-tech guys, and i believed them :)
and, it could be a bug of motherboards.  (for example, there are mobos
with the revision 0x80, which didn't exist at that time.)


anyway, please let me know if 44.1kHz playback is ok.
then i'll add your patch to cvs tree, so that it's included in the
next patchset.


thanks,

Takashi