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

Re: [alsa-devel] [PATCH 10/10] wss_lib: use wss detection code instead of ad1848 one
by krzysztof.h1＠poczta.fm 25 Jul '08

25 Jul '08

Rene Herman napisał(a): > >>>> Is this now done in generic code? Not necessary anymore? Was no > comment > >>>> in the changelog. > >>> The MODE_RUNNING is not used at all in the cs4231 code. I wonder what > the purpose of it. > >> It was used by the AD1848 code though; snd_ad1848_trigger() set/reset > it > >> on start/stop and it was then tested by snd_ad1848_interrupt() to > decide > >> whether or not to call snd_pcm_period_elapsed(). > > > > It is not used by the cs4231. The only difference is that ad1848 does > not > > call the snd_pcm_period_elapsed() after calling the snd_ad1848_open() > > but before calling the snd_ad1848_trigger() (and similar restriction > > after the snd_ad1848_trigger() and snd_ad1848_close(). > > > > The cs4231 does not use such restriction. I decided it does not really > matter. > > The interrupts are not enabled before and after the > snd_ad1848_trigger(). > > If the cs4231 driver needed this it would be already causing problems. > > It seems we are talking at cross purposes. I'm not talking about cs4231. > I see this MODE_RUNNING thing disappear from ad1848_lib and, unless I've > missed it, not resurface in wss_lib -- that library that after these > patches is used to drive AD1848 chips that used to be driven by > ad1848_lib. > > The MODE_RUNNING looks as something someone will have once added upon > seeing spurious IRQs and as such, testing that it isn't needed on > chip/card A, B and C doesn't tell us much. The problem may have been > observed on type D, E or F and/or under condition foo or bar. > > Looking at snd_wss_interrupt() I dom't seem to be seeing a similar guard > after these patches. > > If it's not needed, all for slashing it but please be convincing. Note > that by now you should know a lot more about the innards of this code > than I do, so please be as verbose as needed. > I cannot be convincing as I am not convinced myself. I understand your reasoning. My situation was that I saw the guard in the ad1848 which was missing in the cs4231 while they should look like twins (in the area when MODE_RUNNING existed). I had two choices: 1. Add MODE_RUNNING to the cs4231 2. Remove MODE_RUNNING The first approach is a bloat to me as I tested on ad1848 and it was not needed. I don't know the conditions you called the "condition foo or bar" and I don't know if it ever exists. I have checked git history on the file, but this one was added earlier. Maybe there was a kind of approach in the old days then dropped/replaced later (like mutex_open which is not needed now). The second approach is more dangerous but with enough testing should be justified and provides us with cleaner code. Regards, Krzysztof ---------------------------------------------------------------------- Te newsy nakreca Cie na caly dzien! Sprawdz >>> http://link.interia.pl/f1e94

1 0

Re: [alsa-devel] [PATCH] [alsa-utils] Fix build with library
by Bryan Hanks 25 Jul '08

25 Jul '08

Takashi, Here is the patch for alsa-utils not being able to compile against alsa-lib if certain features were disabled in the library. It uses AC_CHECK_HEADERS as you suggested. It disables midi or seq and alsactl (if either are disabled, since it makes use of rawmidi.h and seq.h) from the build. Bryan

1 0

[alsa-devel] Help
by Sudeept Prusti 24 Jul '08

24 Jul '08

Hi all, As i am working on ALSA for last few weeks.Can anyone guide where can i get moe informative things on ALSA internals (Internally how alsa works). Could anyone suggest what are validation scenario on alsa driver flow. Thanks a lot. Awaiting for response soon. Regards, Sudeept The information contained in this electronic message and any attachments to this message are intended for the exclusive use of the addressee(s) and may contain proprietary, confidential or privileged information. If you are not the intended recipient, you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately and destroy all copies of this message and any attachments contained in it. Contact your Administrator for further information.

2 1

[alsa-devel] [Fwd: Filling "Struct snd_dma_buffer" manually]
by dinesh 24 Jul '08

24 Jul '08

-------- Forwarded Message -------- From: dinesh <dinesh.dua(a)coraltele.com> To: alsa-devel <alsa-devel(a)alsa-project.org> Subject: [alsa-devel] Filling "Struct snd_dma_buffer" manually Date: Thu, 24 Jul 2008 18:29:59 +0530 Hello, I am stuck with a problem please help me. I have allocated a buffer in ram and i want that if i playback something from user end then alsa library exchanges data with this buffer using memory mapping but for this i have to fill a structure "snd_dma_buffer" in driver code but as i have told i dont want to use dma so i have to fill it manually with address of ram buffer. Please tell me if it is possible if yes then help me. Thanks in advance.. Regards, Dinesh Dua _______________________________________________ Alsa-devel mailing list Alsa-devel(a)alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

2 1

Re: [alsa-devel] [PATCH 2/2] Update Poodle to current ASoC API
by Mark Brown 24 Jul '08

24 Jul '08

On Thu, Jul 24, 2008 at 04:54:48PM +0400, Dmitry Baryshkov wrote: > Signed-off-by: Dmitry Baryshkov <dbaryshkov(a)gmail.com> > Cc: Richard Purdie <rpurdie(a)rpsys.net> > Cc: Mark Brown <broonie(a)opensource.wolfsonmicro.com> > --- > This should probably go into 2.6.27, if possible I've queued this for Takashi, thanks (he's on holiday this week). I had thought that at least the second hunk was already fixed but it seems not. Please CC all ALSA patches to alsa-devel(a)alsa-project.org.

3 6

[alsa-devel] Filling "Struct snd_dma_buffer" manually
by dinesh 24 Jul '08

24 Jul '08

Hello, I am stuck with a problem please help me. I have allocated a buffer in ram and i want that if i playback something from user end then alsa library exchanges data with this buffer using memory mapping but for this i have to fill a structure "snd_dma_buffer" in driver code but as i have told i dont want to use dma so i have to fill it manually with address of ram buffer. Please tell me if it is possible if yes then help me. Thanks in advance.. Regards, Dinesh Dua

1 0

[alsa-devel] [PATCH] ASOC, rename mask to max where appropriate
by Jon Smirl 24 Jul '08

24 Jul '08

Where appropriate, rename ASOC mask variables to max to reflect their use Signed-off-by: Jon Smirl <jonsmirl(a)gmail.com> --- include/sound/soc.h | 22 +++++++++++----------- sound/soc/soc-core.c | 20 ++++++++++---------- sound/soc/soc-dapm.c | 14 +++++++------- 3 files changed, 28 insertions(+), 28 deletions(-) diff --git a/include/sound/soc.h b/include/sound/soc.h index cc64b04..412cd83 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -88,25 +88,25 @@ .put = snd_soc_put_volsw_s8, \ .private_value = (unsigned long)&(struct soc_mixer_control) \ {.reg = xreg, .min = xmin, .max = xmax} } -#define SOC_ENUM_DOUBLE(xreg, xshift_l, xshift_r, xmask, xtexts) \ +#define SOC_ENUM_DOUBLE(xreg, xshift_l, xshift_r, xmax, xtexts) \ { .reg = xreg, .shift_l = xshift_l, .shift_r = xshift_r, \ - .mask = xmask, .texts = xtexts } -#define SOC_ENUM_SINGLE(xreg, xshift, xmask, xtexts) \ - SOC_ENUM_DOUBLE(xreg, xshift, xshift, xmask, xtexts) -#define SOC_ENUM_SINGLE_EXT(xmask, xtexts) \ -{ .mask = xmask, .texts = xtexts } + .max = xmax, .texts = xtexts } +#define SOC_ENUM_SINGLE(xreg, xshift, xmax, xtexts) \ + SOC_ENUM_DOUBLE(xreg, xshift, xshift, xmax, xtexts) +#define SOC_ENUM_SINGLE_EXT(xmax, xtexts) \ +{ .max = xmax, .texts = xtexts } #define SOC_ENUM(xname, xenum) \ { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, .name = xname,\ .info = snd_soc_info_enum_double, \ .get = snd_soc_get_enum_double, .put = snd_soc_put_enum_double, \ .private_value = (unsigned long)&xenum } -#define SOC_SINGLE_EXT(xname, xreg, xshift, xmask, xinvert,\ +#define SOC_SINGLE_EXT(xname, xreg, xshift, xmax, xinvert,\ xhandler_get, xhandler_put) \ { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, .name = xname, \ .info = snd_soc_info_volsw, \ .get = xhandler_get, .put = xhandler_put, \ - .private_value = SOC_SINGLE_VALUE(xreg, xshift, xmask, xinvert) } -#define SOC_SINGLE_EXT_TLV(xname, xreg, xshift, xmask, xinvert,\ + .private_value = SOC_SINGLE_VALUE(xreg, xshift, xmax, xinvert) } +#define SOC_SINGLE_EXT_TLV(xname, xreg, xshift, xmax, xinvert,\ xhandler_get, xhandler_put, tlv_array) \ { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, .name = xname, \ .access = SNDRV_CTL_ELEM_ACCESS_TLV_READ |\ @@ -114,7 +114,7 @@ .tlv.p = (tlv_array), \ .info = snd_soc_info_volsw, \ .get = xhandler_get, .put = xhandler_put, \ - .private_value = SOC_SINGLE_VALUE(xreg, xshift, xmask, xinvert) } + .private_value = SOC_SINGLE_VALUE(xreg, xshift, xmax, xinvert) } #define SOC_SINGLE_BOOL_EXT(xname, xdata, xhandler_get, xhandler_put) \ { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, .name = xname, \ .info = snd_soc_info_bool_ext, \ @@ -534,7 +534,7 @@ struct soc_enum { unsigned short reg2; unsigned char shift_l; unsigned char shift_r; - unsigned int mask; + unsigned int max; const char **texts; void *dapm; }; diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 7c59665..c0e6ab4 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -1296,10 +1296,10 @@ int snd_soc_info_enum_double(struct snd_kcontrol *kcontrol, uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED; uinfo->count = e->shift_l == e->shift_r ? 1 : 2; - uinfo->value.enumerated.items = e->mask; + uinfo->value.enumerated.items = e->max; - if (uinfo->value.enumerated.item > e->mask - 1) - uinfo->value.enumerated.item = e->mask - 1; + if (uinfo->value.enumerated.item > e->max - 1) + uinfo->value.enumerated.item = e->max - 1; strcpy(uinfo->value.enumerated.name, e->texts[uinfo->value.enumerated.item]); return 0; @@ -1322,7 +1322,7 @@ int snd_soc_get_enum_double(struct snd_kcontrol *kcontrol, struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; unsigned short val, bitmask; - for (bitmask = 1; bitmask < e->mask; bitmask <<= 1) + for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; val = snd_soc_read(codec, e->reg); ucontrol->value.enumerated.item[0] @@ -1352,14 +1352,14 @@ int snd_soc_put_enum_double(struct snd_kcontrol *kcontrol, unsigned short val; unsigned short mask, bitmask; - for (bitmask = 1; bitmask < e->mask; bitmask <<= 1) + for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; - if (ucontrol->value.enumerated.item[0] > e->mask - 1) + if (ucontrol->value.enumerated.item[0] > e->max - 1) return -EINVAL; val = ucontrol->value.enumerated.item[0] << e->shift_l; mask = (bitmask - 1) << e->shift_l; if (e->shift_l != e->shift_r) { - if (ucontrol->value.enumerated.item[1] > e->mask - 1) + if (ucontrol->value.enumerated.item[1] > e->max - 1) return -EINVAL; val |= ucontrol->value.enumerated.item[1] << e->shift_r; mask |= (bitmask - 1) << e->shift_r; @@ -1386,10 +1386,10 @@ int snd_soc_info_enum_ext(struct snd_kcontrol *kcontrol, uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED; uinfo->count = 1; - uinfo->value.enumerated.items = e->mask; + uinfo->value.enumerated.items = e->max; - if (uinfo->value.enumerated.item > e->mask - 1) - uinfo->value.enumerated.item = e->mask - 1; + if (uinfo->value.enumerated.item > e->max - 1) + uinfo->value.enumerated.item = e->max - 1; strcpy(uinfo->value.enumerated.name, e->texts[uinfo->value.enumerated.item]); return 0; diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c index b956771..424e4e8 100644 --- a/sound/soc/soc-dapm.c +++ b/sound/soc/soc-dapm.c @@ -125,13 +125,13 @@ static void dapm_set_path_status(struct snd_soc_dapm_widget *w, struct soc_enum *e = (struct soc_enum *)w->kcontrols[i].private_value; int val, item, bitmask; - for (bitmask = 1; bitmask < e->mask; bitmask <<= 1) + for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; val = snd_soc_read(w->codec, e->reg); item = (val >> e->shift_l) & (bitmask - 1); p->connect = 0; - for (i = 0; i < e->mask; i++) { + for (i = 0; i < e->max; i++) { if (!(strcmp(p->name, e->texts[i])) && item == i) p->connect = 1; } @@ -168,7 +168,7 @@ static int dapm_connect_mux(struct snd_soc_codec *codec, struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; int i; - for (i = 0; i < e->mask; i++) { + for (i = 0; i < e->max; i++) { if (!(strcmp(control_name, e->texts[i]))) { list_add(&path->list, &codec->dapm_paths); list_add(&path->list_sink, &dest->sources); @@ -1264,7 +1264,7 @@ int snd_soc_dapm_get_enum_double(struct snd_kcontrol *kcontrol, struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; unsigned short val, bitmask; - for (bitmask = 1; bitmask < e->mask; bitmask <<= 1) + for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; val = snd_soc_read(widget->codec, e->reg); ucontrol->value.enumerated.item[0] = (val >> e->shift_l) & (bitmask - 1); @@ -1294,15 +1294,15 @@ int snd_soc_dapm_put_enum_double(struct snd_kcontrol *kcontrol, unsigned short mask, bitmask; int ret = 0; - for (bitmask = 1; bitmask < e->mask; bitmask <<= 1) + for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; - if (ucontrol->value.enumerated.item[0] > e->mask - 1) + if (ucontrol->value.enumerated.item[0] > e->max - 1) return -EINVAL; mux = ucontrol->value.enumerated.item[0]; val = mux << e->shift_l; mask = (bitmask - 1) << e->shift_l; if (e->shift_l != e->shift_r) { - if (ucontrol->value.enumerated.item[1] > e->mask - 1) + if (ucontrol->value.enumerated.item[1] > e->max - 1) return -EINVAL; val |= ucontrol->value.enumerated.item[1] << e->shift_r; mask |= (bitmask - 1) << e->shift_r;

2 1

[alsa-devel] [PATCH] Clean up ASOC functions to support 32b values instead of 16b
by Jon Smirl 24 Jul '08

24 Jul '08

Clean up ASOC functions to support 32b values instead of 16b Signed-off-by: Jon Smirl <jonsmirl(a)gmail.com> --- include/sound/soc.h | 22 +++++++++++----------- sound/soc/soc-core.c | 35 +++++++++++++++++------------------ sound/soc/soc-dapm.c | 32 ++++++++++++++++---------------- 3 files changed, 44 insertions(+), 45 deletions(-) diff --git a/include/sound/soc.h b/include/sound/soc.h index 412cd83..ac59506 100644 --- a/include/sound/soc.h +++ b/include/sound/soc.h @@ -253,10 +253,10 @@ int snd_soc_set_runtime_hwparams(struct snd_pcm_substream *substream, #define snd_soc_write(codec, reg, value) codec->write(codec, reg, value) /* codec register bit access */ -int snd_soc_update_bits(struct snd_soc_codec *codec, unsigned short reg, - unsigned short mask, unsigned short value); -int snd_soc_test_bits(struct snd_soc_codec *codec, unsigned short reg, - unsigned short mask, unsigned short value); +int snd_soc_update_bits(struct snd_soc_codec *codec, uint reg, + uint mask, uint value); +int snd_soc_test_bits(struct snd_soc_codec *codec, uint reg, + uint mask, uint value); int snd_soc_new_ac97_codec(struct snd_soc_codec *codec, struct snd_ac97_bus_ops *ops, int num); @@ -419,8 +419,8 @@ struct snd_soc_codec { hw_write_t hw_write; hw_read_t hw_read; void *reg_cache; - short reg_cache_size; - short reg_cache_step; + uint reg_cache_size; + uint reg_cache_step; /* dapm */ struct list_head dapm_widgets; @@ -530,11 +530,11 @@ struct soc_mixer_control { /* enumerated kcontrol */ struct soc_enum { - unsigned short reg; - unsigned short reg2; - unsigned char shift_l; - unsigned char shift_r; - unsigned int max; + uint reg; + uint reg2; + uint shift_l; + uint shift_r; + uint max; const char **texts; void *dapm; }; diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index c0e6ab4..358aa95 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -961,7 +961,7 @@ static ssize_t codec_reg_show(struct device *dev, { struct snd_soc_device *devdata = dev_get_drvdata(dev); struct snd_soc_codec *codec = devdata->codec; - int i, step = 1, count = 0; + uint i, step = 1, count = 0; if (!codec->reg_cache_size) return 0; @@ -1039,11 +1039,11 @@ EXPORT_SYMBOL_GPL(snd_soc_free_ac97_codec); * * Returns 1 for change else 0. */ -int snd_soc_update_bits(struct snd_soc_codec *codec, unsigned short reg, - unsigned short mask, unsigned short value) +int snd_soc_update_bits(struct snd_soc_codec *codec, u32 reg, + u32 mask, u32 value) { int change; - unsigned short old, new; + u32 old, new; mutex_lock(&io_mutex); old = snd_soc_read(codec, reg); @@ -1069,11 +1069,11 @@ EXPORT_SYMBOL_GPL(snd_soc_update_bits); * * Returns 1 for change else 0. */ -int snd_soc_test_bits(struct snd_soc_codec *codec, unsigned short reg, - unsigned short mask, unsigned short value) +int snd_soc_test_bits(struct snd_soc_codec *codec, u32 reg, + u32 mask, u32 value) { int change; - unsigned short old, new; + u32 old, new; mutex_lock(&io_mutex); old = snd_soc_read(codec, reg); @@ -1320,7 +1320,7 @@ int snd_soc_get_enum_double(struct snd_kcontrol *kcontrol, { struct snd_soc_codec *codec = snd_kcontrol_chip(kcontrol); struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; - unsigned short val, bitmask; + u32 val, bitmask; for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; @@ -1349,8 +1349,7 @@ int snd_soc_put_enum_double(struct snd_kcontrol *kcontrol, { struct snd_soc_codec *codec = snd_kcontrol_chip(kcontrol); struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; - unsigned short val; - unsigned short mask, bitmask; + u32 val, mask, bitmask; for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; @@ -1512,7 +1511,7 @@ int snd_soc_put_volsw(struct snd_kcontrol *kcontrol, int max = mc->max; uint mask = (1 << fls(max)) - 1; uint invert = mc->invert; - unsigned short val, val2, val_mask; + u32 val, val2, val_mask; val = (ucontrol->value.integer.value[0] & mask); if (invert) @@ -1618,7 +1617,7 @@ int snd_soc_put_volsw_2r(struct snd_kcontrol *kcontrol, uint mask = (1 << fls(max)) - 1; uint invert = mc->invert; int err; - unsigned short val, val2, val_mask; + u32 val, val2, val_mask; val_mask = mask << shift; val = (ucontrol->value.integer.value[0] & mask); @@ -1683,12 +1682,12 @@ int snd_soc_get_volsw_s8(struct snd_kcontrol *kcontrol, struct snd_soc_codec *codec = snd_kcontrol_chip(kcontrol); uint reg = mc->reg; int min = mc->min; - int val = snd_soc_read(codec, reg); + uint val = snd_soc_read(codec, reg); ucontrol->value.integer.value[0] = - ((signed char)(val & 0xff))-min; + ((signed char)(val & 0xff)) - min; ucontrol->value.integer.value[1] = - ((signed char)((val >> 8) & 0xff))-min; + ((signed char)((val >> 8) & 0xff)) - min; return 0; } EXPORT_SYMBOL_GPL(snd_soc_get_volsw_s8); @@ -1710,10 +1709,10 @@ int snd_soc_put_volsw_s8(struct snd_kcontrol *kcontrol, struct snd_soc_codec *codec = snd_kcontrol_chip(kcontrol); uint reg = mc->reg; int min = mc->min; - unsigned short val; + uint val; - val = (ucontrol->value.integer.value[0]+min) & 0xff; - val |= ((ucontrol->value.integer.value[1]+min) & 0xff) << 8; + val = (ucontrol->value.integer.value[0] + min) & 0xff; + val |= ((ucontrol->value.integer.value[1] + min) & 0xff) << 8; return snd_soc_update_bits(codec, reg, 0xffff, val); } diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c index 424e4e8..d88b21a 100644 --- a/sound/soc/soc-dapm.c +++ b/sound/soc/soc-dapm.c @@ -103,7 +103,7 @@ static void dapm_set_path_status(struct snd_soc_dapm_widget *w, switch (w->id) { case snd_soc_dapm_switch: case snd_soc_dapm_mixer: { - int val; + uint val; struct soc_mixer_control *mc = (struct soc_mixer_control *) w->kcontrols[i].private_value; uint reg = mc->reg; @@ -123,7 +123,7 @@ static void dapm_set_path_status(struct snd_soc_dapm_widget *w, break; case snd_soc_dapm_mux: { struct soc_enum *e = (struct soc_enum *)w->kcontrols[i].private_value; - int val, item, bitmask; + uint val, item, bitmask; for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; @@ -206,8 +206,8 @@ static int dapm_connect_mixer(struct snd_soc_codec *codec, /* update dapm codec register bits */ static int dapm_update_bits(struct snd_soc_dapm_widget *widget) { - int change, power; - unsigned short old, new; + uint change, power; + uint old, new; struct snd_soc_codec *codec = widget->codec; /* check for valid widgets */ @@ -221,7 +221,7 @@ static int dapm_update_bits(struct snd_soc_dapm_widget *widget) power = widget->power; if (widget->invert) - power = (power ? 0:1); + power = (power ? 0 : 1); old = snd_soc_read(codec, widget->reg); new = (old & ~(0x1 << widget->shift)) | (power << widget->shift); @@ -464,7 +464,7 @@ static int is_connected_input_ep(struct snd_soc_dapm_widget *widget) int dapm_reg_event(struct snd_soc_dapm_widget *w, struct snd_kcontrol *kcontrol, int event) { - unsigned int val; + uint val; if (SND_SOC_DAPM_EVENT_ON(event)) val = w->on_val; @@ -695,8 +695,8 @@ static void dbg_dump_dapm(struct snd_soc_codec* codec, const char *action) /* test and update the power status of a mux widget */ static int dapm_mux_update_power(struct snd_soc_dapm_widget *widget, - struct snd_kcontrol *kcontrol, int mask, - int val, struct soc_enum* e) + struct snd_kcontrol *kcontrol, uint mask, + uint val, struct soc_enum *e) { struct snd_soc_dapm_path *path; int found = 0; @@ -733,8 +733,8 @@ static int dapm_mux_update_power(struct snd_soc_dapm_widget *widget, /* test and update the power status of a mixer or switch widget */ static int dapm_mixer_update_power(struct snd_soc_dapm_widget *widget, - struct snd_kcontrol *kcontrol, int reg, - int val_mask, int val, int invert) + struct snd_kcontrol *kcontrol, uint reg, + uint val_mask, uint val, uint invert) { struct snd_soc_dapm_path *path; int found = 0; @@ -755,10 +755,10 @@ static int dapm_mixer_update_power(struct snd_soc_dapm_widget *widget, found = 1; if (val) /* new connection */ - path->connect = invert ? 0:1; + path->connect = invert ? 0 : 1; else /* old connection must be powered down */ - path->connect = invert ? 1:0; + path->connect = invert ? 1 : 0; break; } @@ -1198,7 +1198,7 @@ int snd_soc_dapm_put_volsw(struct snd_kcontrol *kcontrol, int max = mc->max; uint mask = (1 << fls(max)) - 1; uint invert = mc->invert; - unsigned short val, val2, val_mask; + uint val, val2, val_mask; int ret; val = (ucontrol->value.integer.value[0] & mask); @@ -1262,7 +1262,7 @@ int snd_soc_dapm_get_enum_double(struct snd_kcontrol *kcontrol, { struct snd_soc_dapm_widget *widget = snd_kcontrol_chip(kcontrol); struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; - unsigned short val, bitmask; + uint val, bitmask; for (bitmask = 1; bitmask < e->max; bitmask <<= 1) ; @@ -1290,8 +1290,8 @@ int snd_soc_dapm_put_enum_double(struct snd_kcontrol *kcontrol, { struct snd_soc_dapm_widget *widget = snd_kcontrol_chip(kcontrol); struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; - unsigned short val, mux; - unsigned short mask, bitmask; + uint val, mux; + uint mask, bitmask; int ret = 0; for (bitmask = 1; bitmask < e->max; bitmask <<= 1)

3 6

[alsa-devel] Roland/Edirol M-16DX
by Lasse Kärkkäinen 24 Jul '08

24 Jul '08

Repost, as I got no responses to the original. This device doesn't seem to be supported yet. Does Roland make the specifications available, etc? The device is not compatible with USB Audio, but rather uses Vendor Specific Class. lsusb -vvv is attached below and if you need more information, I can debug the device on Linux and on Vista. Bus 007 Device 025: ID 0582:00c4 Roland Corp. Device Descriptor: bLength 18 bDescriptorType 1 bcdUSB 2.00 bDeviceClass 255 Vendor Specific Class bDeviceSubClass 0 bDeviceProtocol 255 bMaxPacketSize0 64 idVendor 0x0582 Roland Corp. idProduct 0x00c4 bcdDevice 0.00 iManufacturer 1 EDIROL iProduct 2 M-16DX iSerial 0 bNumConfigurations 1 Configuration Descriptor: bLength 9 bDescriptorType 2 wTotalLength 167 bNumInterfaces 3 bConfigurationValue 1 iConfiguration 0 bmAttributes 0xc0 Self Powered MaxPower 0mA Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 0 bAlternateSetting 0 bNumEndpoints 0 bInterfaceClass 255 Vendor Specific Class bInterfaceSubClass 1 bInterfaceProtocol 2 iInterface 0 ** UNRECOGNIZED: 06 24 f1 01 00 00 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 0 bAlternateSetting 1 bNumEndpoints 1 bInterfaceClass 255 Vendor Specific Class bInterfaceSubClass 2 bInterfaceProtocol 2 iInterface 0 ** UNRECOGNIZED: 07 24 01 01 00 01 00 ** UNRECOGNIZED: 0b 24 02 01 02 04 18 01 80 bb 00 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x02 EP 2 OUT bmAttributes 5 Transfer Type Isochronous Synch Type Asynchronous Usage Type Data wMaxPacketSize 0x0038 1x 56 bytes bInterval 1 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 1 bAlternateSetting 0 bNumEndpoints 0 bInterfaceClass 255 Vendor Specific Class bInterfaceSubClass 1 bInterfaceProtocol 2 iInterface 0 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 1 bAlternateSetting 1 bNumEndpoints 1 bInterfaceClass 255 Vendor Specific Class bInterfaceSubClass 2 bInterfaceProtocol 1 iInterface 0 ** UNRECOGNIZED: 07 24 01 07 00 01 00 ** UNRECOGNIZED: 0b 24 02 01 12 04 18 01 80 bb 00 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x81 EP 1 IN bmAttributes 37 Transfer Type Isochronous Synch Type Asynchronous Usage Type Implicit feedback Data wMaxPacketSize 0x01f8 1x 504 bytes bInterval 1 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 2 bAlternateSetting 0 bNumEndpoints 2 bInterfaceClass 255 Vendor Specific Class bInterfaceSubClass 1 bInterfaceProtocol 3 iInterface 0 ** UNRECOGNIZED: 06 24 f1 02 01 01 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x04 EP 4 OUT bmAttributes 2 Transfer Type Bulk Synch Type None Usage Type Data wMaxPacketSize 0x0200 1x 512 bytes bInterval 1 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x83 EP 3 IN bmAttributes 2 Transfer Type Bulk Synch Type None Usage Type Data wMaxPacketSize 0x0200 1x 512 bytes bInterval 1 Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber 2 bAlternateSetting 1 bNumEndpoints 2 bInterfaceClass 255 Vendor Specific Class bInterfaceSubClass 1 bInterfaceProtocol 3 iInterface 0 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x04 EP 4 OUT bmAttributes 2 Transfer Type Bulk Synch Type None Usage Type Data wMaxPacketSize 0x0200 1x 512 bytes bInterval 1 Endpoint Descriptor: bLength 7 bDescriptorType 5 bEndpointAddress 0x83 EP 3 IN bmAttributes 3 Transfer Type Interrupt Synch Type None Usage Type Data wMaxPacketSize 0x0200 1x 512 bytes bInterval 1

2 1

[alsa-devel] Setting format to SND_PCM_FORMAT_MU_LAW does not let me apply hardware parameters
by Mitul Sen (misen) 24 Jul '08

24 Jul '08

I am a newbie to ALSA and any help will be much appreciated. I have an application which sets up the signaling between an IP phone and my desktop and then sets up the audio path between the two. On my desktop application, I am able to receive RTP packets from IP phone. I use an RTP stack to parse the data and after going through the RTP stack, I try to play back the audio using ALSA. When I use the ALSA code to play back (in real time) using sound card of my device, there is only noise, I cannot hear the audio that I speak into the IP phone. However, if I take the raw data coming from the RTP stack and write it to a file, I can play back the file successfully. Since my data from IP phone is G.711 encoded, I have set the sampling rate within ALSA to 8000. Also I am using one source (mono) and non-interleaved data option for preparing ALSA for playback. When I set the format to SND_PCM_FORMAT_MU_LAW, at runtime it lets me set that format ie. snd_pcm_hw_params_set_format (for SND_PCM_FORMAT_MU_LAW) is successful. However I get a runtime error while applying the hardware parameters using snd_pcm_hw_params(..) if the format set earlier is SND_PCM_FORMAT_MU_LAW. Using any other format like SND_PCM_FORMAT_U8 or SND_PCM_FORMAT_S8, lets me apply the hardware parameters but it gives the problem of the noise (no "audible" voice) that I described earlier. This is what my ALSA code looks like ***** beginning of code snippet ***************** int err; snd_pcm_hw_params_t *hw_params; // Try to open the default device err = snd_pcm_open( &_soundDevice, "plughw:0,0", SND_PCM_STREAM_PLAYBACK, 0 ); // Check for error on open. if( err < 0 ) { printf("Init: cannot open audio device\n"); return -1; } // Allocate the hardware parameter structure. if ((err = snd_pcm_hw_params_malloc (&hw_params)) < 0) { printf("Init: cannot allocate hardware parameter structure\n"); return -1; } if ((err = snd_pcm_hw_params_any (_soundDevice, hw_params)) < 0) { printf("Init: cannot initialize hardware parameter structure\n"); return -1; } // Set access to RW interleaved. if ((err = snd_pcm_hw_params_set_access (_soundDevice, hw_params, SND_PCM_ACCESS_RW_NONINTERLEAVED)) < 0) { printf("Init: cannot set access type\n"); return -1; } if ((err = snd_pcm_hw_params_set_format (_soundDevice, hw_params, SND_PCM_FORMAT_MU_LAW)) < 0) { return -1; } // Set channels to mono (1) if ((err = snd_pcm_hw_params_set_channels (_soundDevice, hw_params, 1)) < 0) { return -1; } // Set sample rate. unsigned int actualRate = 8000; if ((err = snd_pcm_hw_params_set_rate_near (_soundDevice, hw_params, &actualRate, 0)) < 0) { return -1; } // Apply the hardware parameters that we've set. if ((err = snd_pcm_hw_params (_soundDevice, hw_params)) < 0) { printf("Audio device parameters have not been set successfully.(%s)(%s)\n", strerror (err), snd_strerror (err)); return -1; } // Free the hardware parameters now that we're done with them. snd_pcm_hw_params_free (hw_params); /* Prepare interface for use. if ((err = snd_pcm_prepare (_soundDevice)) < 0) { printf("Audio device not prepared for use\n"); return -1; } // If the frames are received... /* find out how much space is available for playback data if ((frames_to_deliver = snd_pcm_avail_update (_soundDevice)) < 0) { printf("Error returned by snd_pcm_avail_update\n"); } frames_to_deliver = frames_to_deliver > 4096 ? 4096 : frames_to_deliver; printf("frames to deliver is %d\n",(int)frames_to_deliver); */ frames_to_deliver = 172; bufs[1] = (void*)buffer; if ((err = snd_pcm_writen( _soundDevice, bufs, frames_to_deliver)) < 0) { printf ("write failed (%s)\n", snd_strerror (err)); } ****************** end of code snippet ********************************** The error I get is as follows:- Audio device parameters have not been set successfully.(Unknown error 4294967274)(Invalid argument) Any pointers on what I could be doing wrong? Thanks and regards, Mitul

3 14