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

[alsa-devel] [PATCH V3 0/3] Add audio support on smdk5420
by Padmavathi Venna 08 Aug '13

08 Aug '13

Samsung has different versions of I2S introduced in different platforms. Each version has some new support added for multichannel, secondary fifo, s/w reset control, internal mux for rclk src clk and tdm support. Each newly added change has a quirk. So this patch adds all the required quirks as driver data and based on compatible string from dtsi fetches the quirks. This also adds i2s support on exynos5420. Changes since V2: - Seperated out driver side changes and dts changes in two patch sets. - Replaced samsung,s3c6410-i2s-v4 with samsung,s3c6410-i2s-multi for more clarity as suggested by Tomasz Figa. Changes since V1: - Pass quirks as driver data and fetch the quirks based on compatible string from dtsi file as suggested by Tomasz Figa and Mark Brown - Make the I2S driver more flexible with respect to register access as suggested by Tomasz Figa and Mark Brown - Add 5420 support in the driver. - Modify the dtsi files with the corresponding compatible strings and removed the i2s quirks from 5250 dtsi file. - Updated the i2s Documentation with relevent changes and i2s versioning info. - Add i2s nodes on exynos5420.dtsi - Enable sound support on smdk5420 This patch set is made based on Mark Brown for-next branch on sound.git. Padmavathi Venna (3): platform: Increase platform name size ASoC: Samsung: I2S: Add quirks as driver data in I2S ASoC: Samsung: I2S: Modify the I2S driver to support I2S on Exynos5420 .../devicetree/bindings/sound/samsung-i2s.txt | 25 ++-- include/linux/mod_devicetable.h | 2 +- include/linux/platform_data/asoc-s3c.h | 1 + sound/soc/samsung/i2s-regs.h | 15 ++ sound/soc/samsung/i2s.c | 163 +++++++++++++++----- 5 files changed, 158 insertions(+), 48 deletions(-) -- 1.7.4.4

3 5

[alsa-devel] Found a typo
by Torino 08 Aug '13

08 Aug '13

Dear Alsa developers, within Saucy (13.10), "alsa-utils", string nr. 124 it says "Channel %d doesn't match with hw_parmas" whereas in other strings like nr. 84 or 126 it says "hw_params". I hope it was correct informing you via this mailing list. Best Regads, Torino (from the German Übersetzer-Team)

2 1

[alsa-devel] [PATCH] ALSA: usb-audio: do not trust too-big wMaxPacketSize values
by Clemens Ladisch 08 Aug '13

08 Aug '13

The driver used to assume that the streaming endpoint's wMaxPacketSize value would be an indication of how much data the endpoint expects or sends, and compute the number of packets per URB using this value. However, the Focusrite Scarlett 2i4 declares a value of 1024 bytes, while only about 88 or 44 bytes are be actually used. This discrepancy would result in URBs with far too few packets, which would not work correctly on the EHCI driver. To get correct URBs, use wMaxPacketSize only as an upper limit on the packet size. Reported-by: James Stone <jamesmstone(a)gmail.com> Tested-by: James Stone <jamesmstone(a)gmail.com> Cc: <stable(a)vger.kernel.org> # 2.6.35+ Signed-off-by: Clemens Ladisch <clemens(a)ladisch.de> --- sound/usb/endpoint.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) --- a/sound/usb/endpoint.c +++ b/sound/usb/endpoint.c @@ -591,17 +591,16 @@ static int data_ep_set_params(struct snd_usb_endpoint *ep, ep->stride = frame_bits >> 3; ep->silence_value = pcm_format == SNDRV_PCM_FORMAT_U8 ? 0x80 : 0; - /* calculate max. frequency */ - if (ep->maxpacksize) { + /* assume max. frequency is 25% higher than nominal */ + ep->freqmax = ep->freqn + (ep->freqn >> 2); + maxsize = ((ep->freqmax + 0xffff) * (frame_bits >> 3)) + >> (16 - ep->datainterval); + /* but wMaxPacketSize might reduce this */ + if (ep->maxpacksize && ep->maxpacksize < maxsize) { /* whatever fits into a max. size packet */ maxsize = ep->maxpacksize; ep->freqmax = (maxsize / (frame_bits >> 3)) << (16 - ep->datainterval); - } else { - /* no max. packet size: just take 25% higher than nominal */ - ep->freqmax = ep->freqn + (ep->freqn >> 2); - maxsize = ((ep->freqmax + 0xffff) * (frame_bits >> 3)) - >> (16 - ep->datainterval); } if (ep->fill_max)

3 2

[alsa-devel] [PATCH 1/1] ASoC: s6105-ipcam: Fix incorrect placement of __initdata
by Sachin Kamat 08 Aug '13

08 Aug '13

__initdata should be placed between the variable name and equal sign for the variable to be placed in the intended section. Signed-off-by: Sachin Kamat <sachin.kamat(a)linaro.org> Cc: Daniel Gloeckner <dg(a)emlix.com> --- sound/soc/s6000/s6105-ipcam.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/s6000/s6105-ipcam.c b/sound/soc/s6000/s6105-ipcam.c index 58cfb1e..945e8ab 100644 --- a/sound/soc/s6000/s6105-ipcam.c +++ b/sound/soc/s6000/s6105-ipcam.c @@ -192,7 +192,7 @@ static struct snd_soc_card snd_soc_card_s6105 = { .num_links = 1, }; -static struct s6000_snd_platform_data __initdata s6105_snd_data = { +static struct s6000_snd_platform_data s6105_snd_data __initdata = { .wide = 0, .channel_in = 0, .channel_out = 1, -- 1.7.9.5

2 1

[alsa-devel] [PATCH 1/2] ASoC: ad73311: Add DAPM support
by Mark Brown 08 Aug '13

08 Aug '13

From: Mark Brown <broonie(a)linaro.org> This makes it possible to hook up other devices in boards and is required by removal of support for non-DAPM CODECs in the core. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/codecs/ad73311.c | 22 +++++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/sound/soc/codecs/ad73311.c b/sound/soc/codecs/ad73311.c index b1f2baf..5fac8ad 100644 --- a/sound/soc/codecs/ad73311.c +++ b/sound/soc/codecs/ad73311.c @@ -23,6 +23,21 @@ #include "ad73311.h" +static const struct snd_soc_dapm_widget ad73311_dapm_widgets[] = { +SND_SOC_DAPM_INPUT("VINP"), +SND_SOC_DAPM_INPUT("VINN"), +SND_SOC_DAPM_OUTPUT("VOUTN"), +SND_SOC_DAPM_OUTPUT("VOUTP"), +}; + +static const struct snd_soc_dapm_route ad73311_dapm_routes[] = { + { "Capture", NULL, "VINP" }, + { "Capture", NULL, "VINN" }, + + { "VOUTN", NULL, "Playback" }, + { "VOUTP", NULL, "Playback" }, +}; + static struct snd_soc_dai_driver ad73311_dai = { .name = "ad73311-hifi", .playback = { @@ -39,7 +54,12 @@ static struct snd_soc_dai_driver ad73311_dai = { .formats = SNDRV_PCM_FMTBIT_S16_LE, }, }; -static struct snd_soc_codec_driver soc_codec_dev_ad73311; +static struct snd_soc_codec_driver soc_codec_dev_ad73311 = { + .dapm_widgets = ad73311_dapm_widgets, + .num_dapm_widgets = ARRAY_SIZE(ad73311_dapm_widgets), + .dapm_routes = ad73311_dapm_routes, + .num_dapm_routes = ARRAY_SIZE(ad73311_dapm_routes), +}; static int ad73311_probe(struct platform_device *pdev) { -- 1.8.4.rc1

2 3

[alsa-devel] [PATCH] ASoC: pcm1681: Staticise DAI driver
by Mark Brown 08 Aug '13

08 Aug '13

From: Mark Brown <broonie(a)linaro.org> It is not exported so doesn't need to be in the global namespace and sparse warns on this. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/codecs/pcm1681.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/pcm1681.c b/sound/soc/codecs/pcm1681.c index 27da41b..51b1866 100644 --- a/sound/soc/codecs/pcm1681.c +++ b/sound/soc/codecs/pcm1681.c @@ -225,7 +225,7 @@ static const struct snd_kcontrol_new pcm1681_controls[] = { pcm1681_get_deemph, pcm1681_put_deemph), }; -struct snd_soc_dai_driver pcm1681_dai = { +static struct snd_soc_dai_driver pcm1681_dai = { .name = "pcm1681-hifi", .playback = { .stream_name = "Playback", -- 1.8.4.rc1

1 0

[alsa-devel] [PATCH v4 0/1] ASoc: kirkwood: extend the kirkwood audio subsystem for DT usage
by Jean-Francois Moine 08 Aug '13

08 Aug '13

This patch series adds DT support to the kirkwood/mvebu audio subsystem. History v4 - change the DT name to "mrvl,mvebu-audio" - remove previous patches 1..3 which have been applied by Russell King v3 - change the DT name from "kirkwood-i2s" to "mvebu-pcm-audio" - apply to linux-next - let NULL as the internal i2s clock for compatibility - change i2s external clock name to "extclk" instead of "external" - cancel previous patch 1/4. Russell's patch should be applied instead. v2 - fix compilation errors -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/

1 0

[alsa-devel] bug
by Mattia Bertoni 08 Aug '13

08 Aug '13

finally I can report a bug which I encountered some time ago; I send you the important part of the entire log, which is to much big to be posted: Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: snd_pcm_avail() returned a value that is exceptionally large: 420736 bytes (2191 ms). Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: Most likely this is a bug in the ALSA driver 'snd_intel8x0'. Please report this issue to the ALSA developers. Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: snd_pcm_dump(): Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: Hardware PCM card 0 'NVidia CK804' device 0 subdevice 0 Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: Its setup is: Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: stream : PLAYBACK Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: access : MMAP_INTERLEAVED Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: format : S16_LE Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: subformat : STD Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: channels : 2 Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: rate : 48000 Jun 15 18:18:00 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: exact rate : 48000 (48000/1) Jun 15 18:18:03 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: msbits : 16 Jun 15 18:18:03 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: buffer_size : 16384 Jun 15 18:18:03 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_size : 16384 Jun 15 18:18:03 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_time : 341333 Jun 15 18:18:03 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: tstamp_mode : ENABLE Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_step : 1 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: avail_min : 16384 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_event : 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: start_threshold : -1 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: stop_threshold : 4611686018427387904 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: silence_threshold: 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: silence_size : 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: boundary : 4611686018427387904 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: appl_ptr : 26406104 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: hw_ptr : 26494904 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: snd_pcm_delay() returned a value that is exceptionally large: -465056 bytes (-2422 ms). Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: Most likely this is a bug in the ALSA driver 'snd_intel8x0'. Please report this issue to the ALSA developers. Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: snd_pcm_dump(): Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: Hardware PCM card 0 'NVidia CK804' device 0 subdevice 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: Its setup is: Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: stream : PLAYBACK Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: access : MMAP_INTERLEAVED Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: format : S16_LE Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: subformat : STD Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: channels : 2 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: rate : 48000 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: exact rate : 48000 (48000/1) Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: msbits : 16 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: buffer_size : 16384 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_size : 16384 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_time : 341333 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: tstamp_mode : ENABLE Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_step : 1 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: avail_min : 16384 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: period_event : 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: start_threshold : -1 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: stop_threshold : 4611686018427387904 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: silence_threshold: 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: silence_size : 0 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: boundary : 4611686018427387904 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: appl_ptr : 26569944 Jun 15 18:18:04 arch-dekstop pulseaudio[776]: [alsa-sink-Intel ICH] alsa-util.c: hw_ptr : 26686208

2 6

[alsa-devel] [PATCH] [alsa-lib] UCM: Document some standard values
by Arun Raghavan 08 Aug '13

08 Aug '13

Playback/CaptureChannels has been around for a bit. Playback/CaptureRate is new and is intended to be used to specify the sample rate at which the Playback/CapturePCM should be opened. --- include/use-case.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/include/use-case.h b/include/use-case.h index 93b3456..4e13249 100644 --- a/include/use-case.h +++ b/include/use-case.h @@ -259,9 +259,13 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr, * TQ - Tone Quality * PlaybackPCM - full PCM playback device name * CapturePCM - full PCM capture device name + * PlaybackRate - playback device sample rate + * PlaybackChannels - playback device channel count * PlaybackCTL - playback control device name * PlaybackVolume - playback control volume ID string * PlaybackSwitch - playback control switch ID string + * CaptureRate - capture device sample rate + * CaptureChannels - capture device channel count * CaptureCTL - capture control device name * CaptureVolume - capture control volume ID string * CaptureSwitch - capture control switch ID string -- 1.8.2.1

2 1

Re: [alsa-devel] [PATCH V3 2/3] ASoC: Samsung: I2S: Add quirks as driver data in I2S
by Padma Venkat 08 Aug '13

08 Aug '13

Hi Tomasz, On Wed, Aug 7, 2013 at 4:43 PM, Tomasz Figa <t.figa(a)samsung.com> wrote: > Ahh, one more thing inline. > [snip] >> >> +static struct samsung_i2s_dai_data i2sv3_dai_type = { >> + .dai_type = TYPE_PRI, >> + .quirks = QUIRK_NO_MUXPSR, >> +}; >> + >> +static struct samsung_i2s_dai_data i2sv4_dai_type = { >> + .dai_type = TYPE_PRI, >> + .quirks = QUIRK_PRI_6CHAN | QUIRK_NO_MUXPSR, >> +}; >> + >> +static struct samsung_i2s_dai_data i2sv5_c100_dai_type = { >> + .dai_type = TYPE_PRI, >> + .quirks = QUIRK_PRI_6CHAN | QUIRK_NO_MUXPSR | QUIRK_SEC_DAI | >> + QUIRK_NEED_RSTCLR, >> +}; >> + >> +static struct samsung_i2s_dai_data i2sv5_dai_type = { >> + .dai_type = TYPE_PRI, >> + .quirks = QUIRK_PRI_6CHAN | QUIRK_SEC_DAI | QUIRK_NEED_RSTCLR, >> +}; >> + >> +static struct samsung_i2s_dai_data samsung_dai_type_sec = { >> + .dai_type = TYPE_SEC, >> +}; >> + >> static struct platform_device_id samsung_i2s_driver_ids[] = { >> { >> - .name = "samsung-i2s", >> - .driver_data = TYPE_PRI, >> + .name = "samsung,s3c6410-i2s", >> + .driver_data = (kernel_ulong_t)&i2sv3_dai_type, >> + }, { >> + .name = "samsung,s3c6410-i2s-multi", >> + .driver_data = (kernel_ulong_t)&i2sv4_dai_type, >> + }, { >> + .name = "samsung,s5pc100-i2s", >> + .driver_data = (kernel_ulong_t)&i2sv5_c100_dai_type, >> }, { >> - .name = "samsung-i2s-sec", >> - .driver_data = TYPE_SEC, >> + .name = "samsung,s5pv210-i2s", >> + .driver_data = (kernel_ulong_t)&i2sv5_dai_type, >> + }, { >> + .name = "samsung-i2s-sec", >> + .driver_data = (kernel_ulong_t)&samsung_dai_type_sec, > > I don't think you need to change the legacy platform IDs at all. If legacy platforms not required to change then I need to introduce a new samsung_i2s_dai_data structure which holds only dai_type for non-dt platforms. If I change legacy platforms it breaks the older platforms now. Is it okay adding a samsung_i2s_dai_data structure for non-dt platforms which will be removed later? > > IMHO it would be better to keep the old way of quirk retrieval from > platform_data (which I think this patch does anyway, because the probe path > without DT is unchanged), as it will be dropped in some point in time > anyway. > > This would also eliminate the need for patch 1. > > Best regards, > Tomasz > Thanks Padma

3 2