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

[alsa-devel] snd-xfi programming, some guidelines.
by James Courtier-Dutton 19 Oct '08

19 Oct '08

Hi, I have not had time to review the current work on the snd-xfi, but I have provided input if I think it would be useful. I have one more piece of information that might help. The xfi has two ways to talk to all the registers. 1) ioports using ioport pointer offset write method as familiar with older emu10k1 cards. 2) iomem using simple memory writes. The same registers on the xfi card can be accessed via both methods. There is some guidance as to when to use each. I.e. Do not use iomem for everything. There are some registers that take longer to write to than others. Unfortunately, the datasheet does not tell me which these slow are, so trial and error might be required. Now, if one uses the iomem to write to these registers that take a long time to update, it ties up the entire PCI bus for the entire duration, so this is not a good idea. So, the recommendation is that one should use ioports to access these slow registers as this will not block the PCI bus. I suggest putting a timer round each ioport and iomem write to find the slow ones. Kind Regards James

1 0

[alsa-devel] [PATCH] snd-sbxfi: use SBXXXX model numbers instead of user-unfriendly names like UAA and ORIG.
by William Pitcock 19 Oct '08

19 Oct '08

This is just something that bothers me. There's no reason to accept or reject it, other than that it might be more informative about what hardware revision we are dealing with. Signed-off-by: William Pitcock <nenolod(a)sacredspiral.co.uk> --- sound/pci/sbxfi/sbxfi.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/pci/sbxfi/sbxfi.c b/sound/pci/sbxfi/sbxfi.c index 30ad25b..c006278 100644 --- a/sound/pci/sbxfi/sbxfi.c +++ b/sound/pci/sbxfi/sbxfi.c @@ -1702,10 +1702,10 @@ static int __devinit sbxfi_create_mixer(struct sbxfi *chip) */ static char *typenames[SBXFI_HW_TYPES] = { - [SBXFI_HW_ORIG] = "ORIG", - [SBXFI_HW_055x] = "055x", - [SBXFI_HW_073x] = "073x", - [SBXFI_HW_UAA] = "UAA" + [SBXFI_HW_ORIG] = "SB0760", + [SBXFI_HW_055x] = "SB0550", + [SBXFI_HW_073x] = "SB0730", + [SBXFI_HW_UAA] = "SB0886" }; #ifdef CONFIG_PROC_FS -- 1.5.5.4

2 1

[alsa-devel] [PATCH] snd-sbxfi: set up the DAC at card initialization, regardless of duplex mode
by William Pitcock 19 Oct '08

19 Oct '08

We should bring up the DAC at card initialization even in half-duplex mode, otherwise we have to deal with unfortunate glitches in the hardware. Constantly resetting the DAC can cause serious state desyncs, resulting in the driver eventually not working until a reboot occurs. Signed-off-by: William Pitcock <nenolod(a)sacredspiral.co.uk> --- sound/pci/sbxfi/sbxfi.c | 8 +------- 1 files changed, 1 insertions(+), 7 deletions(-) diff --git a/sound/pci/sbxfi/sbxfi.c b/sound/pci/sbxfi/sbxfi.c index d64d462..30ad25b 100644 --- a/sound/pci/sbxfi/sbxfi.c +++ b/sound/pci/sbxfi/sbxfi.c @@ -62,7 +62,7 @@ MODULE_PARM_DESC(debug, "Debug flag: 0 = disable"); /* continuous rate support */ #undef XXX_CONT_RATE -/* full-duplex support */ +/* full duplex support */ #undef XXX_FULL_DUPLEX /* multi playback - not implemented */ @@ -1364,10 +1364,6 @@ static int sbxfi_playback_prepare(struct snd_pcm_substream *substream) port->frag_count = runtime->period_size; port->need_update = 0; -#ifndef XXX_FULL_DUPLEX - sbxfi_init_dac(chip); - sbxfi_setup_i2s(chip); -#endif sbxfi_setup_play_pitch(chip, port); sbxfi_setup_play_mixer(chip, port); sbxfi_setup_play_mapper(chip, port); @@ -1924,10 +1920,8 @@ static int sbxfi_init_hw(struct sbxfi *chip) sbxfi_daoimap_clear(chip); -#ifdef XXX_FULL_DUPLEX sbxfi_init_dac(chip); sbxfi_setup_i2s(chip); -#endif return 0; } -- 1.5.5.4

2 1

[alsa-devel] [PATCH] snd-sbxfi: implement support for 192khz playback/capture.
by William Pitcock 19 Oct '08

19 Oct '08

Implements support for 192khz initialization for playback and capture. Signed-off-by: William Pitcock <nenolod(a)sacredspiral.co.uk> --- sound/pci/sbxfi/sbxfi.c | 18 ++++++++++++++---- 1 files changed, 14 insertions(+), 4 deletions(-) diff --git a/sound/pci/sbxfi/sbxfi.c b/sound/pci/sbxfi/sbxfi.c index 4c65125..d64d462 100644 --- a/sound/pci/sbxfi/sbxfi.c +++ b/sound/pci/sbxfi/sbxfi.c @@ -514,7 +514,7 @@ static void sbxfi_write_amop(struct sbxfi *chip, static void sbxfi_playback_start(struct sbxfi *chip, struct sbxfi_port *port) { unsigned int start, loop; - unsigned int ctrl, ratec; + unsigned int ctrl, ratec = 0; struct snd_pcm_runtime *runtime; if (port->tlb_index < 0) @@ -523,12 +523,19 @@ static void sbxfi_playback_start(struct sbxfi *chip, struct sbxfi_port *port) start = port->tlb_index * SBXFI_PAGE_SIZE; loop = start + frames_to_bytes(runtime, runtime->buffer_size); + /* + * ratec is the rate selection mode, 0x4c = 48000hz, 0x5c = 96000hz, + * 0x6c = 192000hz. + */ switch (runtime->rate) { case 48000: - ratec = 0x4c; /* XXX what meaning? */ + ratec = 0x4c; break; case 96000: - ratec = 0x5c; /* XXX what meaning? */ + ratec = 0x5c; + break; + case 192000: + ratec = 0x6c; break; } ctrl = ratec; @@ -559,7 +566,7 @@ static void sbxfi_playback_stop(struct sbxfi *chip, struct sbxfi_port *port) static void sbxfi_capture_start(struct sbxfi *chip, struct sbxfi_port *port) { unsigned int start, loop; - unsigned int ctrl, ratec; + unsigned int ctrl, ratec = 0; struct snd_pcm_runtime *runtime; if (port->tlb_index < 0) @@ -575,6 +582,9 @@ static void sbxfi_capture_start(struct sbxfi *chip, struct sbxfi_port *port) case 96000: ratec = 0x5d | SRCCTL_RUN_STATE; break; + case 192000: + ratec = 0x6d | SRCCTL_RUN_STATE; + break; } ctrl = ratec; -- 1.5.5.4

3 3

[alsa-devel] [PATCH] snd-sbxfi: rename erroneous Master mixer control to "PCM Playback Volume".
by William Pitcock 19 Oct '08

19 Oct '08

The OSS4-provided master mixer control is actually relative to EMU20K1 port volume settings. So, it should be named PCM. The Master mixer control will be shortly readded as a control which sets the master volume. Signed-off-by: William Pitcock <nenolod(a)petrie.sacredspiral.co.uk> --- sound/pci/sbxfi/sbxfi.c | 28 ++++++++++++++-------------- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/sound/pci/sbxfi/sbxfi.c b/sound/pci/sbxfi/sbxfi.c index c006278..3e99a97 100644 --- a/sound/pci/sbxfi/sbxfi.c +++ b/sound/pci/sbxfi/sbxfi.c @@ -155,7 +155,7 @@ struct sbxfi { int timer_reprogram; unsigned int base_rate; - int master_vol[2]; + int pcm_vol[2]; int capture_vol[2]; int capsrc; int micboost; @@ -1103,7 +1103,7 @@ static void sbxfi_setup_play_mixer(struct sbxfi *chip, int i, vol, audio_ch; for (i = 0; i < 2; i++) { - vol = port->vol[i] + chip->master_vol[i] - MAX_VOLUME; + vol = port->vol[i] + chip->pcm_vol[i] - MAX_VOLUME; if (vol < 0) vol = 0; vol = amop_vol_table[vol]; @@ -1124,9 +1124,9 @@ static int sbxfi_update_play_mixer(struct sbxfi *chip, int *vol) struct sbxfi_port *port; for (i = 0; i < 2; i++) { - if (vol[i] == chip->master_vol[i]) + if (vol[i] == chip->pcm_vol[i]) continue; - chip->master_vol[i] = vol[i]; + chip->pcm_vol[i] = vol[i]; spin_lock_irq(&chip->port_lock); list_for_each_entry(port, &chip->port_list, list) { if (port->substream->stream != @@ -1516,7 +1516,7 @@ static int __devinit sbxfi_create_pcm(struct sbxfi *chip) /* * Mixer interface */ -static int sbxfi_master_vol_info(struct snd_kcontrol *kcontrol, +static int sbxfi_pcm_vol_info(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_info *uinfo) { uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER; @@ -1526,16 +1526,16 @@ static int sbxfi_master_vol_info(struct snd_kcontrol *kcontrol, return 0; } -static int sbxfi_master_vol_get(struct snd_kcontrol *kcontrol, +static int sbxfi_pcm_vol_get(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol) { struct sbxfi *chip = snd_kcontrol_chip(kcontrol); - ucontrol->value.integer.value[0] = chip->master_vol[0]; - ucontrol->value.integer.value[1] = chip->master_vol[1]; + ucontrol->value.integer.value[0] = chip->pcm_vol[0]; + ucontrol->value.integer.value[1] = chip->pcm_vol[1]; return 0; } -static int sbxfi_master_vol_put(struct snd_kcontrol *kcontrol, +static int sbxfi_pcm_vol_put(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol) { struct sbxfi *chip = snd_kcontrol_chip(kcontrol); @@ -1543,7 +1543,7 @@ static int sbxfi_master_vol_put(struct snd_kcontrol *kcontrol, vol[0] = ucontrol->value.integer.value[0]; vol[1] = ucontrol->value.integer.value[1]; - if (!memcmp(vol, chip->master_vol, sizeof(vol))) + if (!memcmp(vol, chip->pcm_vol, sizeof(vol))) return 0; if (vol[0] < 0) @@ -1658,13 +1658,13 @@ static int sbxfi_micboost_put(struct snd_kcontrol *kcontrol, static struct snd_kcontrol_new mixers[] __devinitdata = { { - .name = "Master Playback Volume", + .name = "PCM Playback Volume", .iface = SNDRV_CTL_ELEM_IFACE_MIXER, .access = (SNDRV_CTL_ELEM_ACCESS_READWRITE /* | SNDRV_CTL_ELEM_ACCESS_TLV_READ*/), - .info = sbxfi_master_vol_info, - .get = sbxfi_master_vol_get, - .put = sbxfi_master_vol_put, + .info = sbxfi_pcm_vol_info, + .get = sbxfi_pcm_vol_get, + .put = sbxfi_pcm_vol_put, /*.tlv = { .p = db_master_vol }*/ }, { -- 1.5.5.4

1 0

[alsa-devel] alsa + resampler + jack: possible?
by Markus Grabner 19 Oct '08

19 Oct '08

Hi all! A while ago I started development of an ALSA driver for a device which natively only supports a sampling rate of 39062.5 Hz and 24 bit samples (see http://www.tanzband-scream.at/line6). The ALSA rate plugin does a perfect job for using this device with standard applications (such as xmms) to play audio files at "conventional" sample rates (e.g., 44.1 kHz) with a configuration like this: pcm.resampler { type rate slave { pcm hw:1 format S24_3LE rate 39062 } converter "samplerate" } However, this doesn't work for jack since jack requires a power-of-two period size, and ALSA can't find an integer period size for the raw device such that the target rate is met with sufficient precision. An example: the jack daemon can't be started with period size 512 and sample rate 48000 Hz, but it works for 47961 Hz since 512 * 39062.5 / 48000 = 416.666... ~= 417 bytes 512 / 417 * 39062.5 = 47961.63 ~= 47961 Hz When using this approach together with other audio data from a 48 kHz source, they would drift apart over time, which basically renders the alsa+jack setup useless for this device. I understand that resampling is easier to implement if the sample rate ratio equals the ratio of the (integer) period sizes of client and slave, but at least the underlying libsamplerate has no such restriction. I tried to find the relevant pieces in the ALSA source code (pcm_rate.c), but I'm not sure where these calculations are done and how much effort it would be to support arbitrary sample rate ratios for the resampler. Can this be done in a plugin, or does it require modifications in the ALSA library code base? Could the developers please give a few hints on that? Thanks & kind regards, Markus -- Markus Grabner - Computer Graphics and Vision Graz University of Technology, Inffeldgasse 16a/II, 8010 Graz, Austria Phone: +43/316/873-5041, Fax: +43/316/873-5050 WWW: http://www.icg.tugraz.at/Members/grabner

1 0

[alsa-devel] snd_pcm_sw_params_get_xfer_align deprecated?
by Erik de Castro Lopo 19 Oct '08

19 Oct '08

Hi all, I've got some code that gives me a waring that the above function is deprecated. What should I do: a) Call some other function? If so which one? b) Just drop the call to that function and everything will continue to work as it should? c) Something else? Cheers, Erik -- ----------------------------------------------------------------- Erik de Castro Lopo ----------------------------------------------------------------- "The very definition of hell is having to maintain someone else's Perl code." -- Anonymous

3 2

[alsa-devel] New BIOS, new vendor and device IDs
by Chris Pemberton 17 Oct '08

17 Oct '08

My machine is a Lenovo 3000 N100 laptop (0768 model). I had to recently update the BIOS. Old BIOS: lspci -> device id: [17aa:2066] which is a Realtek ALC861VD (found in patch_realtek.c) New BIOS: lspci -> device id: [8086:27d8] (found in hda_intel.c) In both cases, I use the module option "model=3stack" to get sound working: Laptop speakers work. Built-in mic works. External mic works Headphones work; but automatic speaker muting does not. I tried to debug why I needed this module option at all, several months ago (old BIOS), but I didn't get far and just became complacent with using the module option. If anyone is interested, I'll do what I can to help debug. It would be nice to be able to use Skype with headphones... Output of the alsa info script is at: http://www.alsa-project.org/db/?f=701ca6efa0500080dc78a555813f66c1d42096e6 That is with driver 1.0.17. I'll try to update to the latest snapshot and run the script again. Thanks, Chris

2 6

Re: [alsa-devel] Backported sbxfi driver (UNTESTED!)
by Takashi Iwai 17 Oct '08

17 Oct '08

At Thu, 16 Oct 2008 19:06:58 +0400, The Source wrote: > > Takashi Iwai пишет: > > At Thu, 16 Oct 2008 19:00:16 +0400, > > The Source wrote: > > > >> Takashi Iwai пишет: > >> > >>> At Thu, 16 Oct 2008 18:41:08 +0400, > >>> The Source wrote: > >>> > >>> > >>>> Takashi Iwai пишет: > >>>> > >>>> > >>>>> At Thu, 16 Oct 2008 18:15:45 +0400, > >>>>> The Source wrote: > >>>>> > >>>>> > >>>>> > >>>>>> Takashi Iwai пишет: > >>>>>> > >>>>>> > >>>>>> > >>>>>>> At Thu, 16 Oct 2008 17:36:04 +0400, > >>>>>>> The Source wrote: > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>>>>> And, which X-Fi model do you have? > >>>>>>>>>> Please show the lspci -nv output, too. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>> I've got the X-Fi Elite Pro. > >>>>>>>>> That's The one with the external In/Out box. > >>>>>>>>> > >>>>>>>>> Speaking of which, the headphone jack on it does not output a signal > >>>>>>>>> yet, the signal only goes to line out. > >>>>>>>>> > >>>>>>>>> There's some relais on the card that seem to switch these, they click > >>>>>>>>> multiple times with the windows driver and not all all with yours, I > >>>>>>>>> think that's the reason :) > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>> Original OSS driver doesn't output to external block also, so it > >>>>>>>> wouldn't be easy to make this support I think. > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>> The values for port->conv[0] and [1] values in sbxfi_playback_open() > >>>>>>> might play some role. It's I2SA_L and I2SA_R, alias DAI_CH_I2SAL > >>>>>>> and DAI_CH_I2SAR, as default. You can try other values, such as, > >>>>>>> DAI_CH_I2SBL, DAI_CH_I2SA1L, and so on. > >>>>>>> > >>>>>>> > >>>>>>> Takashi > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>> Latest snapshot has a bug: > >>>>>> make[3]: *** No rule to make target > >>>>>> `/mnt/e/temp/alsa-driver-unstable/acore/jack.o', needed by > >>>>>> `/mnt/e/temp/alsa-driver-unstable/acore/snd.o'. Stop. > >>>>>> > >>>>>> > >>>>>> > >>>>> Already fixed. > >>>>> > >>>>> > >>>>> Takashi > >>>>> > >>>>> > >>>>> > >>>>> > >>>> In file included from /mnt/e/temp/alsa-driver-unstable/acore/jack.c:3: > >>>> /mnt/e/temp/alsa-driver-unstable/acore/../alsa-kernel/core/jack.c: In > >>>> function ‘snd_jack_new’: > >>>> /mnt/e/temp/alsa-driver-unstable/acore/../alsa-kernel/core/jack.c:107: > >>>> error: ‘SW_MICROPHONE_INSERT’ undeclared (first use in this function) > >>>> /mnt/e/temp/alsa-driver-unstable/acore/../alsa-kernel/core/jack.c:107: > >>>> error: (Each undeclared identifier is reported only once > >>>> /mnt/e/temp/alsa-driver-unstable/acore/../alsa-kernel/core/jack.c:107: > >>>> error: for each function it appears in.) > >>>> /mnt/e/temp/alsa-driver-unstable/acore/../alsa-kernel/core/jack.c: In > >>>> function ‘snd_jack_report’: > >>>> /mnt/e/temp/alsa-driver-unstable/acore/../alsa-kernel/core/jack.c:157: > >>>> error: ‘SW_MICROPHONE_INSERT’ undeclared (first use in this function) > >>>> > >>>> > >>> Hmm, it seems broken for older kernels right now. > >>> The easy workaround is to pass --with-cards=sbxfi to configure. > >>> > >>> Anyway, I'll fix it now. > >>> > >>> thanks, > >>> > >>> Takashi > >>> > >>> > >>> > >> --Hey, man, this is cool! Plays just fine (volume ok, speed ok, no > >> --glitches) with 96000, 48000, 44100, 22050, 16/24 bit, Stereo and --Mono!! > >> --I didn't change anything in the source code, so I don't use system > >> --timer. Yes!! > >> > >> However oss (alsa-emulated) is unstable. I'll test more. > >> > > > > Could be due to 96kHz base-rate? Try base_rate=48000. > > If you get still problems, please show the kernel logs with debug=2. > > > > BTW, the jack.c compile error should have been fixed now (hopefully). > > Let me know if you still have the build errors. > > > > > > thanks, > > > > Takashi > > > > > Ok. OpenAL with alsa also seem to cause problems. In both cases, check the period_size and buffer_size values (shown in the kernel message, or /proc/asound/card0/pcm0p/sub0/hw_params). And, try to aplay with these parameters, whether you get the similar problem. % aplay -v --period-size=xxx --buffer-size=yyy foo.wav Takashi

5 22

Re: [alsa-devel] [PATCH] Fix debugfs_create_dir's error checking method for sound/soc/
by Mark Brown 17 Oct '08

17 Oct '08

On Fri, Oct 17, 2008 at 09:04:55PM +0800, Zhaolei wrote: > debugfs_create_dir() returns NULL if an error occurs, returns -ENODEV > when debugfs is not enabled in the kernel. ... > asoc_debugfs = debugfs_create_dir("asoc", NULL); > - if (!IS_ERR(asoc_debugfs)) > + if (!IS_ERR(asoc_debugfs) && asoc_debugfs) > debugfs_create_u32("dapm_pop_time", 0744, asoc_debugfs, > &pop_time); Hrmpf. This looks like something that should be fixed in debugfs - using both error reporting strategies is rather unhelpful and we're actually loosing information in the case where debugfs is built. I'll send a patch and see what people thing. Still in the meantime, Acked-by: Mark Brown <broonie(a)opensource.wolfsonmicro.com>

2 3