May 2007 - Alsa-devel - mailman.alsa-project.org

Re: [alsa-devel] [PATCH] Disable debugging output for the ALS300 driver
by Ash Willis 29 May '07

29 May '07

> ----- Original Message ----- > From: "Takashi Iwai" <tiwai(a)suse.de> > To: "Ash Willis" <ashwillis(a)programmer.net> > Subject: Re: [alsa-devel] [PATCH] Disable debugging output for the ALS300 driver > Date: Tue, 29 May 2007 15:08:30 +0200 > > > At Sat, 26 May 2007 13:26:35 +0000, > Ash Willis wrote: > > > > Disables debugging output in the ALS300 driver. > > Also contains a whitespace cleanup and a fix for a potential bug. > > > > Signed-off-by: Ash Willis <ashwillis(a)programmer.net> > > Thanks for the patch. It seems that your MUA breaks the embedded > patch, so I had to apply it manually. Could you check whether the > latest HG is OK for you? I'm away from home right now. I'll test it as soon as I have a chance but it looks fine. > > Also I changed "if ((err = ...))" to two individual lines, which is a > newer standard kernel coding style. > Yeah, I left that for now. I'll go through the tree and tidy up remaining occurences of this style when I have some time. It seems that when people add drivers to the tree, they generally copy already existing drivers. It doesn't make much sense to be copying code that doesn't fit CodingStyle. Ash = Sports Memorabilia License Plates Signs All license plates $6 all signs $10 all frames $8 - same day shipping - fun company and selection. http://a8-asy.a8ww.net/a8-ads/adftrclick?redirectid=a44f49e42c5a283d05f0c96…

2 1

[alsa-devel] [PATCH] Disable debugging output for the ALS300 driver
by Ash Willis 29 May '07

29 May '07

Disables debugging output in the ALS300 driver. Also contains a whitespace cleanup and a fix for a potential bug. Signed-off-by: Ash Willis <ashwillis(a)programmer.net> ---------------------------------------------------- diff -r 337962e19de6 -r 8ea4fa014563 pci/als300.c --- a/pci/als300.c Fri May 25 12:18:49 2007 +0200 +++ b/pci/als300.c Sat May 26 14:04:14 2007 +0000 @@ -88,8 +88,8 @@ #define PLAYBACK_BLOCK_COUNTER 0x9A #define RECORD_BLOCK_COUNTER 0x9B -#define DEBUG_CALLS 1 -#define DEBUG_PLAY_REC 1 +#define DEBUG_CALLS 0 +#define DEBUG_PLAY_REC 0 #if DEBUG_CALLS #define snd_als300_dbgcalls(format, args...) printk(format, ##args) @@ -730,10 +730,9 @@ static int __devinit snd_als300_create(s } chip->irq = pci->irq; - snd_als300_init(chip); - if (snd_als300_ac97(chip) < 0) { + if ((err = snd_als300_ac97(chip)) < 0) { snd_printk(KERN_WARNING "Could not create ac97\n"); snd_als300_free(chip); return err;

2 1

[alsa-devel] sound on smdk2410
by Jose Henrique Spahn Torres 29 May '07

29 May '07

Hi, I am trying recent Zoltan Devai's patchs for kernel 2.6.22-rc2 on a smdk2410 board. I reached the point where the kernel recognized the uda1341 sound chip, however, when I try to play something, I get the output below. Can anyone, please, show me the path to follow to get sound working? Thanks, Henrique /home $ l drwxr-xr-x 3 root root 0 Jan 1 00:00 . drwxr-xr-x 18 root root 1024 May 18 2007 .. -r--r--r-- 1 root root 3419671 Jan 1 1970 jj.mp3 /home $ mpg123 jj.mp3 High Performance MPEG 1.0/2.0/2.5 Audio Player for Layers 1, 2 anasoc: UDA1341 <-> s3c24xx-i2s info: asoc: rate mask 0xfe asoc: min ch 2 max ch 2 asoc: min rate 8000 max rate 48000 version 0.65; written and copyright by Michael Hipp and others free software (LGPL/GPL) without any warranty but with best wishes asoc: machine hw_params failed ... (lots of asoc: messages) ... asoc: machine hw_params failed pop wq checking: Playback status: inactive waiting: yes pop wq D3 UDA1341 Playback Playing MPEG stream 1 of 1: jj.mp3 ... Title: Times Like These Artist: Jack Johnson Comment: Album: On And On Year: 2003 Genre: Acoustic MPEG 1.0 layer III, 192 kbits/s, 44100 Hz stereo [audio.c:264] error: No supported rate found! /home $

3 3

[alsa-devel] HDA Codec: Add support for MSI K9N Ultra
by Tobin Davis 29 May '07

29 May '07

This patch adds the MSI K9N Ultra system to the realtek patch. Signed off by Tobin Davis <tdavis(a)dsl-only.net>

2 1

[alsa-devel] ICE1724 alsa driver
by Rossco 29 May '07

29 May '07

Hi, I hope someone can help me. (tried the user list but no response) I have a hitpoint 7.1ch Envy24ht-s based card with the usual vt1616 multichannel codec and the wolfson WM8728. The card is recognised as a chaintech av710 with the only difference being that the card has spdif in as well as out. It appears to be physically similar to the maddog entertainer (would be good to know if this is also recognised as an AV710) By default 2 channel stereo in played through the "front" channel of the card through the vt1616 codec and I appear to be able to set spdif out through the mixer (but dont have another device to test at the moment). I have been able to change default output to go through the wolfson dac through an asound.state file posted on the web. I have to admit I do not quite understand what the file has done as it only appears to have removed some of the controls rather than change or add anything. Only two devices are detected for the card the vt1616 and IEC958, neither are shown as having any subdevices, therefore I am unsure how the asound.state file was able to address the wolfson chip which requires i2s input rather ac-link. This seems to indicate i2s and ac-link modes of the envy24ht-s chip are active. Most AV710 users seem happy enough to be able to route to toslink out and/or the higher quality wolfson chip, therefore there is little discussion of addressing the other channels of the card. I really want to be able to address each of the channels of the card so that I can use the convolution programme Brutefir with the card to create a multiway crossover for my loudspeakers. Is there anyway I can define each of the cards outputs so that I can select to send audio to any one of them? I am also keen to get the spdif in working and particularly the syncing to spdif in of the clock that the ice1724 chip supports. Can these features be added to the driver? Please advise of any required info, I have all the datasheets and can trace chip pins also. Thanks in advance. Ross

2 5

[alsa-devel] [PATCH] Add quirk for Supermicro PDSBA to alc883_cfg_tbl[]
by Daniel T. Chen 29 May '07

29 May '07

Attached is a diff against current alsa-kernel hg tip. Thanks, Daniel Chen

1 1

[alsa-devel] hardware not asking for more data using asyn call back
by Ashlesha Shintre 29 May '07

29 May '07

Hi, In my asynchronous playback program, the hardware stops asking for any data and the program gets stuck after one call to the callback function. I have pasted the program below -- please let me know if something is wrong - Thanks, Ashlesha. /* Reading wav file into a buffer and then trying to play it */ > > #include <stdio.h> > #include </usr/include/alsa/asoundlib.h> > #include </usr/include/alsa/pcm.h> > #include <sys/types.h> > #include <unistd.h> > #include <fcntl.h> > > #define SIZE 128 > > short int *buffer; > int ind=10, fd1; > static snd_pcm_t *handle; > static char *device = "default"; /* playback device > */ > snd_output_t *output = NULL; > static short int *wto, *rfrom, buflen; > > void MyCallback(snd_async_handler_t *pcm_callback); > > int main() > { > int i,j,n,length,count,a,err,k=0; > char c; > snd_pcm_sframes_t frames; > snd_async_handler_t *pcm_callback; > > > > if((fd1=open("SA2.WAV",O_RDONLY,0))==-1) > printf("error opening wav file\n"); > > count=0; > while(count++<40) > a=read(fd1,&c,sizeof(char)); > > a=read(fd1,&length,sizeof(int)); > > n=length/SIZE; > > printf("length = %d\n",length); > > buflen = 4*SIZE; > buffer = (short int *) malloc (buflen*sizeof(short int)); > > wto = NULL; > rfrom = &buffer[0]; > > count=0; > a=1; > while(count<buflen && a>0) > a=read(fd1,&buffer[count++],sizeof(short int)); > if(a<0) > printf("error in reading from wav file\n"); > > if((err = snd_pcm_open(&handle, device, SND_PCM_STREAM_PLAYBACK, 0)) < > 0) { > printf("error opening device: %s\n",snd_strerror(err)); > return -1; > } > > if((err = snd_pcm_set_params(handle, > SND_PCM_FORMAT_S16_LE, > SND_PCM_ACCESS_RW_INTERLEAVED, > 1, > 16000, > 1, > 1000)) < 0) { /* 2 sec */ > printf("Playback open error: %s\n", snd_strerror(err)); > return -2; > } > printf("n=%d\n",n); > for (i=0; i<50; i++) > { > printf("i=%d\n",i); > frames = snd_pcm_writei(handle, rfrom, SIZE); > if(frames < 0) // underrun > { > printf("underrun recovery\n"); > frames = snd_pcm_recover(handle, frames,0); > if(frames < 0){ > printf("error in recovery\n"); > return -3; > } > } > if(frames >0 && frames < SIZE) > printf("expected to write %d, wrote %d\n",SIZE,frames); > printf("distance between ptrs before reinit is %d\n",rfrom - wto); > if(rfrom < &buffer[buflen-1]) > { > wto = rfrom; > rfrom = rfrom + SIZE; > } > else > { > printf("rfm at the end, rfrom -buffer[buflen-1] > =%d\n",(*rfrom - buffer[buflen-1])); > wto = rfrom; > rfrom = &buffer[0]; > } > for(k=0; k<SIZE; k++) > { > a=read(fd1,(wto+k),sizeof(short int)); > if(a<0) > printf("error in reading from wav file k = %d\n"); > } > printf("buffer[i*SIZE] - rfrom = %d\n",(buffer[(i+1)*SIZE] - > *rfrom)); > > } > > /* Async Handler */ > > // err = > snd_async_add_pcm_handler(&pcm_callback,handle,MyCallback,buffer); > err = snd_async_add_pcm_handler(&pcm_callback,handle,MyCallback,NULL); > if(err<0) > printf("add pcm handler error = %d\n%s\n",err,snd_strerror(err)); > > err = snd_pcm_start(handle); > if(err<0) > printf("error in starting snd pcm start err > :%s\n",snd_strerror(err)); > > while (1) { > if(wto == NULL) > break; > sleep(1); > } > > err = snd_pcm_close(handle); > if(err<0) > printf("error in closing pcm device: %s\n",snd_strerror(err)); > close(fd1); > return 0; > } > > void MyCallback(snd_async_handler_t *pcm_callback) > { > > snd_pcm_t *pcm_handle = snd_async_handler_get_pcm(pcm_callback); > snd_pcm_sframes_t avail; > int count,a; > snd_pcm_uframes_t period_size = 64; > snd_pcm_sframes_t frames; > //short int *bufferin = > snd_async_handler_get_callback_private(pcm_callback); > > while((avail=snd_pcm_avail_update(pcm_handle)) >= period_size) > { > printf("available frames = %d\n",avail); > frames = snd_pcm_writei(pcm_handle, rfrom, SIZE); > if(frames < 0) // underrun > { > printf("underrun recovery\n"); > frames = snd_pcm_prepare(pcm_handle); > printf("error from snd_pcm_prepare is: %d\n",frames); > if(frames < 0){ > printf("error in recovery\n"); > // break; > } > } > > if(frames >0 && frames < SIZE) > printf("expected to write %d, wrote %d\n",SIZE,frames); > count = 0; > a=1; > while(count<SIZE && a>0){ > a=read(fd1,(wto+count),sizeof(short int)); //write to > the buffer from file > ++count; > } > printf("count = %d\n",count); > if(a <0){ //EOF > wto = NULL; > printf("end of file reached\n"); > break; > } > else{ > if(rfrom < &buffer[buflen-1]) > { > wto = wto+count; > rfrom = rfrom + SIZE; > } > else > { > wto = wto + count; > rfrom = &buffer[0]; > } //location of where to start > writing from next > } > ++ind; > } > > printf("going out of callback, ind = %d\n",ind); > } > > >

2 1

[alsa-devel] async playback works despite error in snd_pcm_start.
by Ashlesha Shintre 28 May '07

28 May '07

Hi, I managed to write code to playback sound asynchronously -- however, this code works (i.e data is read from .wav file and output to the playback device) despite the snd_pcm_start giving a "file descriptor in bad state" error.. can someone explain this please? Thanks, Ashlesha.

1 0

Re: [alsa-devel] another ALSA question
by Jaroslav Kysela 28 May '07

28 May '07

I'm CCing alsa-devel because this discussion might be interesting for other developers, too. On Mon, 28 May 2007, Maarten Lankhorst wrote: > Jaroslav Kysela schreef: > > On Mon, 28 May 2007, Maarten Lankhorst wrote: > > > >> After playing around a little more with alsa I tried to do manipulation > >> of pointer using the instructions, I wrote some simple code either does > >> rewind or forward to get to the right point, taking in respect the > >> amount of delay and where to go. > >> > >> However it doesn't seem to work: Looking at the source code it seems > >> rewinding isn't implemented, using the #if 0'd code there gives problems > >> when doing snd_pcm_forward later: It then ends up at the pointer where > >> the sound is currently being played at. > >> > > > > Not all alsa-lib plugins are capable or have implemented this fine > > position control (check error value). > > > For a plugin like dmix, I would expect it to be implemented, especially > since most cards sold are integrated cards that don't have multiple > sound buffers. Go ahead and send us patches. We're doing open source :-) Anyway any rewind operation mens "undo" - it will eat extra CPU ticks for nothing. > >> As workaround I used the following in my own code: > >> typeof(snd_pcm_forward) snd_pcm_mmap_appl_forward; > >> #define snd_pcm_forward snd_pcm_mmap_appl_forward > >> > >> typeof(snd_pcm_rewind) snd_pcm_mmap_appl_backward; > >> #define snd_pcm_rewind snd_pcm_mmap_appl_backward > >> > >> This works, but I wonder if there is a cleaner way to do it. > >> > > > > It won't work with all plugins and mentioned functions are not part of > > public API. > > > As I work in wine on it's DirectSound implementation, I can tell it > relies heavily on the fact that it can lock a buffer at an arbitrary > position, and that it works with absolute pointers, rather then > relative. So if snd_pcm_rewind/snd_pcm_forward don't work as expected, > an alsa driver for it would become nearly useless, I don't expect it to > always work, but I would expect it to work for a common plugin like dmix > at least. > >> Also, I noticed some audio glitches when using this in combination with > >> alsa-lib's resampling, is that because of the way it is done or is > >> something else wrong? > >> > > > > You should not use mentioned functions. > > > > You can try to use the direct hardware access (hw:CARD_NAME) devices > > (you'll lose all stream conversions from the ALSA side) or use an > > intermediate buffer for fine position control and configure ALSA to use > > smallest ring buffer as possible (to minimalize latency). > > > > Jarosla > I wish that would work, directsound expects to have access to a fixed > size buffer, and if I go for direct hardware access, I lose the ability > to work properly with most cheap cards on the market, since people want > to be able to listen to music on background while gaming. For now it > seems I have to use the internal calls in case of dmix, or modify > already committed data, which is what winealsa's current driver does. I > don't know what is the least of those 2 evils, or if there is a third > option. I'm open for advice. I'm afraid, that you're trying to exactly copy DirectSound API. Playing with the buffer position in this way is always dangerous, because it's really hard to predict the final result in the multitasking environment. Think about these proposals: 1) what sample (ring buffer) rewritting is supposed to do - maybe you can change only not yet commited samples with satisfactory results 2) use a small ALSA ring buffer and use own "big" ring buffer for changes before you commit samples to ALSA's ring buffer - you'll increase a bit sample flow latency, but you can do anything you want (you just add extra buffering to avoid rewind / forward for ALSA API) 3) fix (or better implement) dmix plugin rewind function Jaroslav ----- Jaroslav Kysela <perex(a)suse.cz> Linux Kernel Sound Maintainer ALSA Project, SUSE Labs

1 0

[alsa-devel] [ALSA] ALSA Power Management, Drivers behaving unexpectedly after suspend/resume cycle
by Nobin Mathew 28 May '07

28 May '07

I am implementing power management in my ALSA sound drivers. ALSA drivers are not working properly after resume. All registers values are proper and I am able to read back the codec register contents. If I do a playback after resume then I am getting lots of "underrun". After a reboot playback is working fine. Is there any known issues with ALSA power management? Why Ubuntu and Redhat does ALSA modules removal before suspend and insertion after resume?

3 10