Re: Re: snd-es1968 (+ snd-cs4236) trouble

[Takashi Iwai <tiwai@suse.de>]

At Tue, 13 Dec 2005 08:09:05 +0100,
Rene Herman wrote:
> 
> Rene Herman wrote:
> 
> > I'll try to see if I can find anything in the 1969 or 1978 docs.
> > Please beat me to it though... :-)
> 
> The minimal patch I need to have things functional, is not enabling TDMA 
> in sound/pci/es1968.c:
> 
> @@ -2197,10 +2197,7 @@ static void snd_es1968_chip_init(es1968_
>          /* Config Reg A */
>          pci_read_config_word(pci, ESM_CONFIG_A, &w);
> 
> -       /*      Use TDMA for now. TDMA works on all boards, so while its
> -        *      not the most efficient its the simplest. */
>          w &= ~DMA_CLEAR;        /* Clear DMA bits */
> -       w |= DMA_TDMA;          /* TDMA on */
> 
> With that applied, things seem to be working fine again. With the bits 
> cleared, it's set to "Distributed DMA" which is disabled via the DDMA 
> control register (0x60), so this should be okay? The comment's a bit 
> puzzling but doesn't this stuff apply only to the legacy DMA, which 
> isn't used in this driver, anyway?

Good catch.  I thought TDMA is the timer DMA for es1968, and never
thought it conflicts with ISA DMAs.

Just to be sure, doesn't it break es1968 itself, right?


> While looking there, I also noticed two bits that are reversed between 
> documentation and source. One of them in fact the "Enable Legacy Audio" 
> bit, so I expected that to be the trouble at first, although it wasn't. 
> In the second attached patch, this bit is set in accordance with the 
> documentation. Works for me... with the bit set either way, I don't seem 
> to be experiencing conflicts with the CS4236 (which also has an enabled 
> SBPro emulation) so I just trusted the documentation.
> 
> The other one is the "Emulate ISA timing on PCI bit". The code said and 
> commented that it enabled the emulation, but according to the datasheet 
> it was disabling it instead. I've left it the same, but changed the name 
> of the define and the comment.
> 
> This card has Canyon-3D (ES1970MS-3D) written on the main chip, but it 
> advertises itself as a 125d:1978, rev 10, which should be a Maestro-2E, 
> so I've been looking at that datasheet...
> 
> As said, on this TerraTec SoundSystem DMX, version 1.2, I need the first 
> patch at minimum. The second one also does the bit reversals, and works 
> for me. Please apply?

I'll apply it to CVS after releasing 1.0.11-rc1.  (Don't worry, rc2
will come soon later :) 


Thanks,

Takashi


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click