Sound recording with OPL3SA2

Hi.

I got this message in the log when I tried to do sound recording:

"Sound: DMA (input) timed out - IRQ/DRQ config error?"

After some investigations I found a bug in the opl3sa2 driver. I have sent this patch to Scott Murray ([email protected]), but received no answer. And the bug is still in kernel 2.4.6.

So now I escalate the issue to the linux-sound mailing list....

In the attach_opl3sa2() function we look at the dma values to see if we

are using two or one channel, and configure the chip accordingly. Unfortunately the dma values are not set in the structure we look at.

Please CC any comments to me, [email protected], thanks.

Patch:

--- linux-2.4.6.org/drivers/sound/opl3sa2.c Sun Jun 24 11:31:24
2001
+++ linux-2.4.6/drivers/sound/opl3sa2.c Sun Jun 24 11:32:47 2001 @@ -703,7 +703,7 @@

       opl3sa2_write(hw_config->io_base, OPL3SA2_IRQ_CONFIG, 0x0d);

       /* Initialize DMA configuration */
-       if(hw_config->dma2 = hw_config->dma) {
+       if(cfg_mss[card].dma2 = cfg_mss[card].dma) {
               /* Want DMA configuration DMA-B: -, DMA-A: WSS-P+WSS-R
*/
               opl3sa2_write(hw_config->io_base, OPL3SA2_DMA_CONFIG,
0x03);
       }

/L

____________________________________________________________
Do You Yahoo!?
Get your free @yahoo.co.uk address at http://mail.yahoo.co.uk
or your free @yahoo.ie address at http://mail.yahoo.ie
-
To unsubscribe from this list: send the line "unsubscribe linux-sound" in the body of a message to [email protected]

I apologize for not getting back to you earlier. I've been extremely busy at the startup I work for, and on top of that, I recently lost a bunch of mail. There's a reason why linux-sound is listed in my MAINTAINERS entry, posting to it gets me off my butt. :)

Anyways, after looking at this, I prefer this IMO more correct fix:

@@ -861,8 +861,8 @@
 	/* Our own config: */
 	hw_cfg->io_base = dev->resource[4].start;
 	hw_cfg->irq     = 0;
-	hw_cfg->dma     = -1;
-	hw_cfg->dma2    = -1;
+	hw_cfg->dma     = dev->dma_resource[0].start;
+	hw_cfg->dma2    = dev->dma_resource[1].start;
 
 	/* The MSS config: */
 	mss_cfg->io_base      = dev->resource[1].start;

Lars, could you try this out and see if it works for you?

Scott

======================================Scott Murray                                        email: [email protected]
http://www.spiteful.org (coming soon)                 ICQ: 10602428
-----------------------------------------------------------------------------
     "Good, bad ... I'm the guy with the gun." - Ash, "Army of Darkness"

-
To unsubscribe from this list: send the line "unsubscribe linux-sound" in
the body of a message to [email protected]