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

[alsa-devel] [PATCH 0/2]ad1938 dai and board driver clean-up according to feedback
by Barry Song 20 Jul '09

20 Jul '09

Hi Mark, Update bf5xx-tdm dai(patch 1) and bf5xx-ad1938 machine(patch 2) drivers according to your feedback. Thanks Barry

3 4

[alsa-devel] [PATCH] ALSA: hda - Add quirk for Gateway T6834c laptop
by 宋浩 20 Jul '09

20 Jul '09

From: Hao Song <baritono.tux(a)gmail.com> Gateway T6834c laptops need EAPD always on while the default behavior for the STAC9205 reference board is to turn it off upon every HP plug. By using the special "eapd" model, which is first introduced for Gateway T1616 laptops for this same reason, this peculiarity can be properly handled. Signed-off-by: Hao Song <baritono.tux(a)gmail.com> --- sound/pci/hda/patch_sigmatel.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/sound/pci/hda/patch_sigmatel.c b/sound/pci/hda/patch_sigmatel.c index 41b5b3a..d9efe5c 100644 --- a/sound/pci/hda/patch_sigmatel.c +++ b/sound/pci/hda/patch_sigmatel.c @@ -2379,6 +2379,7 @@ static struct snd_pci_quirk stac9205_cfg_tbl[] = { "Dell Vostro 1500", STAC_9205_DELL_M42), /* Gateway */ SND_PCI_QUIRK(0x107b, 0x0565, "Gateway T1616", STAC_9205_EAPD), + SND_PCI_QUIRK(0x107b, 0x0560, "Gateway T6834c", STAC_9205_EAPD), {} /* terminator */ }; -- 1.6.0.4 -- 宋浩 Song, Hao Institute of Theoretical Computer Science and Communications Room 117, Ho Sin Hang Engineering Building Chinese University of Hong Kong, Hong Kong, P.R.China E-mail: baritono.tux(a)gmail.com Office Tel: (+8600852)-31634255

2 1

Re: [alsa-devel] New Audio Driver Design
by Daniel Mack 20 Jul '09

20 Jul '09

On Thu, Jul 16, 2009 at 09:13:28AM +0900, jassi brar wrote: > On Thu, Jul 16, 2009 at 5:22 AM, Daniel Mack <daniel(a)caiaq.de> wrote: > > Are these decoder instruction something you need to do at runtime? Or > > would it suffice to provide them once when the driver is probed? > > The decoder insn, params atleast, depend upon the mp3 file i want the dsp to > decode and play. > So, let us assume, i have to load decoder insns and params into the dsp for > every mp3 file playback. Then the alsa control API seems appropriate. Unless there is any other interface for that specific purpose I'm not aware of. > Also, by what interface name(like I2S/PCM/AC97) shud my driver get encoded > mp3 data from my mp3-player and then decode that mp3 data by dsp and play > the output pcm stream? > I am afraid i must bypass/hack ALSA stack to directly get mp3 data from > user-space. Am i right? Have a look at include/sound/pcm.h - there are more formats defined than just linear PCM, and I think SNDRV_PCM_FMTBIT_MPEG could be something you can use. But I'm not sure and never used something like that before, so maybe anyone else can elaborate more? Daniel

1 0

[alsa-devel] CS42L52 codec
by Anna White 20 Jul '09

20 Jul '09

Hello, I am working with CS42L52 codec under Linux. I have CS42L52 driver code, but I need ALSA configuration files. Does somebody have ALSA configuration files ? Or recomendation how to write them ? Thanks for responces. P.S. It may be related to thread: http://mailman.alsa-project.org/pipermail/alsa-devel/2008-February/006130.h…

2 1

Re: [alsa-devel] [PATCH] hda: add bounds checking for the codec command fields v2
by Wu Fengguang 20 Jul '09

20 Jul '09

On Fri, Jul 17, 2009 at 06:46:01PM +0800, Takashi Iwai wrote: > At Fri, 17 Jul 2009 18:41:05 +0800, > Guo, Chaohong wrote: > > > > Although it does address this issue, I am not comfortable with this fixing. > > It seems more like a workaround than fix. > > No, it's rather for catching a bug. This is definitely neither > "workaround" nor "fix". Yes, I wrote this mainly for catching unknown bugs. > > Moreover, if long format node > > ID is used in the future, the code will cause little trouble for maintaining. > > Well, the current code doesn't support the long id (as the restriction > of HD-audio controller side), so we'd need major changes in anyway. > Thus this check is no big issue for maintenance, at least to me :) Does the HDA spec define cmd format that accept long form NID? If so, can you point me to the specific location please? Thanks. > > what I want is to fix it during parsing connection list, and verify the node > > id is valid there . > > Heh, this was already fixed :) AFAIK, snd_hda_get_connections() won't return NIDs bigger than 0x7f(short) or 0x7ffff(long). Thanks, Fengguang

2 1

[alsa-devel] [RFC] tty (or char) bus?
by Janusz Krzysztofik 19 Jul '09

19 Jul '09

Hi, In my attempt to add support for contols to a voice modem codec sound device driver, I found that in order to talk to the modem, it would be convenient if I can get access to a tty device from inside the kernel in a way similiar to that available form userspace. AFAICS, even if tty lowlevel write() could be used unmodified, a convenient way of reading characters from a tty is missing and should be implemented in a line discipline. Please correct me if I am wrong. OTOH, I found that some kind of abstraction layer for acccessing devices over a tty could be convenient. Instead of allocating a new line discipline for each specific device, sometimes found on a specific board only, why not just create a new bus type? Implemented as a line discipline, activated (hot-plugged) from userspace with ldattach, that new bus adapter would give kernel level access to an arbitrary device hanging off a tty. As the bus can be assumed point-to-point, a single generic device could be registered automatically in order to trigger that bus registerd drivers' probes (vendor/model id queries, for example). If not enough, an ioctl and a ldattach replacement could be provided for setting up a device id that would match a driver id (something like inputattach does for N_MOUSE). Once detected by a voice modem codec driver tty bus probe(), a codec subdevice could then be registered with a sound card. Please let me know if you like the idea. If not, please push me in a better direction. Many of the new code could be probably based on currently existing drivers and line discplines. As I'm not familiar enough with the linux kernel code, unlike most of you, so please give me some hints on what specific drivers should I look at to get examples best matching my idea. Thanks, Janusz

3 3

[alsa-devel] [PATCH] alsa-info.sh: include 1 line of dmesg context
by Wu Fengguang 19 Jul '09

19 Jul '09

So as to include possible ALSA messages without the common keywords, as well as be more confident on the completeness/coverage of the report. It also shows nice '--' separators between the dmesg ranges. Signed-off-by: Wu Fengguang <fengguang.wu(a)intel.com> --- utils/alsa-info.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- alsa-driver.orig/utils/alsa-info.sh +++ alsa-driver/utils/alsa-info.sh @@ -247,7 +247,7 @@ withdmesg() { echo "!!ALSA/HDA dmesg" >> $FILE echo "!!------------------" >> $FILE echo "" >> $FILE - dmesg | grep -E 'ALSA|HDA|HDMI|sound|hda.codec|hda.intel' >> $FILE + dmesg | grep -C1 -E 'ALSA|HDA|HDMI|sound|hda.codec|hda.intel' >> $FILE echo "" >> $FILE echo "" >> $FILE }

2 1

[alsa-devel] [PATCH 6/6] sound/isa: convert nested spin_lock_irqsave to spin_lock
by Julia Lawall 19 Jul '09

19 Jul '09

From: Julia Lawall <julia(a)diku.dk> If spin_lock_irqsave is called twice in a row with the same second argument, the interrupt state at the point of the second call overwrites the value saved by the first call. Indeed, the second call does not need to save the interrupt state, so it is changed to a simple spin_lock. The semantic match that finds this problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // <smpl> @@ expression lock1,lock2; expression flags; @@ *spin_lock_irqsave(lock1,flags) ... when != flags *spin_lock_irqsave(lock2,flags) // </smpl> Signed-off-by: Julia Lawall <julia(a)diku.dk> --- sound/isa/gus/gus_pcm.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/isa/gus/gus_pcm.c b/sound/isa/gus/gus_pcm.c index edb11ee..2dcf45b 100644 --- a/sound/isa/gus/gus_pcm.c +++ b/sound/isa/gus/gus_pcm.c @@ -795,13 +795,13 @@ static int snd_gf1_pcm_volume_put(struct snd_kcontrol *kcontrol, struct snd_ctl_ if (!(pcmp->flags & SNDRV_GF1_PCM_PFLG_ACTIVE)) continue; /* load real volume - better precision */ - spin_lock_irqsave(&gus->reg_lock, flags); + spin_lock(&gus->reg_lock); snd_gf1_select_voice(gus, pvoice->number); snd_gf1_ctrl_stop(gus, SNDRV_GF1_VB_VOLUME_CONTROL); vol = pvoice == pcmp->pvoices[0] ? gus->gf1.pcm_volume_level_left : gus->gf1.pcm_volume_level_right; snd_gf1_write16(gus, SNDRV_GF1_VW_VOLUME, vol); pcmp->final_volume = 1; - spin_unlock_irqrestore(&gus->reg_lock, flags); + spin_unlock(&gus->reg_lock); } spin_unlock_irqrestore(&gus->voice_alloc, flags); return change;

2 1

[alsa-devel] [PATCH 0/3] new support for ad1938 codec
by Barry Song 18 Jul '09

18 Jul '09

Hi Liam&Mark, The following series add support for ad1938 codec *PATCH 1: ad1938 codec driver *PATCH 2: blackfin I2S(in TDM mode) DAI driver Even though TDM mode can be as part of I2S DAI, but there are so much difference in configuration and data flow, it's very ugly to integrate I2S and TDM into a module. So we use the DAI type, but split normal I2S and TDM into individual files. *PATCH 3: board driver to connect bf5xx with ad1938 -- Thanks, Barry

2 7

[alsa-devel] snd_pcm_avail_update returns bogus values with the OSS PCM plugin
by Sindre Aamås 17 Jul '09

17 Jul '09

Hi. For the attached test case snd_pcm_avail_update returns values higher than the buffer size when run with the OSS PCM plugin. Output of alsa-info.sh is at http://www.alsa-project.org/db/?f=77a873e55d5d6ca50d67a035a9d6b7e10cca179d . --- alsa_oss_pcm_avail_bug.c --- /* * Public domain test case to illustrate a bug in the ALSA OSS PCM plugin * where snd_pcm_avail_update returns bogus values. */ #include <alsa/asoundlib.h> #include <stdio.h> #include <stdlib.h> #include <string.h> static short buf[803 * 2]; static snd_pcm_t *pcm_handle = NULL; void close_pcm(void) { if (pcm_handle) snd_pcm_close(pcm_handle); pcm_handle = NULL; } static int set_pcm_parameters(void) { snd_pcm_hw_params_t *hwparams; snd_pcm_hw_params_alloca(&hwparams); if (snd_pcm_hw_params_any(pcm_handle, hwparams) < 0) { fprintf(stderr, "snd_pcm_hw_params_any failed.\n"); return -1; } if (snd_pcm_hw_params_set_access(pcm_handle, hwparams, SND_PCM_ACCESS_RW_INTERLEAVED) < 0) { fprintf(stderr, "snd_pcm_hw_params_set_access failed.\n"); return -1; } if (snd_pcm_hw_params_set_format(pcm_handle, hwparams, SND_PCM_FORMAT_S16) < 0) { fprintf(stderr, "snd_pcm_hw_params_set_format failed.\n"); return -1; } { unsigned rate = 48000; if (snd_pcm_hw_params_set_rate_near(pcm_handle, hwparams, &rate, 0) < 0) { fprintf(stderr, "snd_pcm_hw_params_set_rate_near failed.\n"); return -1; } } if (snd_pcm_hw_params_set_channels(pcm_handle, hwparams, 2) < 0) { fprintf(stderr, "snd_pcm_hw_params_set_channels failed.\n"); return -1; } { unsigned ulatency = 100000; if (snd_pcm_hw_params_set_buffer_time_near(pcm_handle, hwparams, &ulatency, 0) < 0) { fprintf(stderr, "snd_pcm_hw_params_set_buffer_time_near failed.\n"); return -1; } } if (snd_pcm_hw_params(pcm_handle, hwparams) < 0) { fprintf(stderr, "snd_pcm_hw_params failed.\n"); return -1; } { snd_pcm_uframes_t bSize = 0; if (snd_pcm_hw_params_get_buffer_size(hwparams, &bSize) < 0) { fprintf(stderr, "snd_pcm_hw_params_get_buffer_size failed\n"); return -1; } printf("buffer_size: %u\n", (unsigned int) bSize); } return 0; } static void test_loop(int n) { while (n--) { snd_pcm_sframes_t sframes = snd_pcm_avail_update(pcm_handle); printf("snd_pcm_avail: %d\n", (int) sframes); printf("writing %d frames\n", (int) sizeof(buf) / 4); sframes = snd_pcm_writei(pcm_handle, buf, sizeof(buf) / 4); if (sframes != sizeof(buf) / 4) { if (sframes < 0) snd_pcm_prepare(pcm_handle); printf("snd_pcm_writei returned %d\n", (int) sframes); } } } int main(int argc, char **argv) { atexit(close_pcm); if (argc < 2) { printf("usage: %s <ALSA pcm device>\n", argv[0]); return 0; } if (snd_pcm_open(&pcm_handle, argv[1], SND_PCM_STREAM_PLAYBACK, 0) < 0) { fprintf(stderr, "Error opening PCM device %s\n", argv[1]); pcm_handle = NULL; return -1; } if (set_pcm_parameters() < 0) return -1; memset(buf, 0, sizeof(buf)); test_loop(500); return 0; } -- Sindre Aamås

2 2