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

[alsa-devel] [PATCH 00/30 v3] ALSA: Enhancement for existed FireWire drivers
by Takashi Sakamoto 03 Dec '14

03 Dec '14

This patchset is to enable ALSA Dice driver to handle more models, to improve ALSA firewire-speakers driver as OXFW driver. This patchset is for linux-next branch. == Updates from my previous post: [alsa-devel] [PATCH 00/29 v2] ALSA: Enhancement for existed FireWire drivers http://mailman.alsa-project.org/pipermail/alsa-devel/2014-October/082844.ht… - Add a patch to fix some minor bugs (dice) - Improve handling config-rom strings for loud models (oxfw) - Fix a bug of mutual exclusive lock for AMDTP streams (oxfw) - Obsolete atomic_t for reference counters (dice/oxfw) Takashi Sakamoto (30): ALSA: dice: suppress checkpatch.pl warnings ALSA: dice: Rename structure and its members ALSA: dice: Move file to its own directory ALSA: dice: Split transaction functionality into a file ALSA: dice: Split stream functionality into a file ALSA: dice: Split PCM functionality into a file ALSA: dice: Split hwdep functionality into a file ALSA: dice: Split proc interface into a file ALSA: dice: Add new functions for constraints of PCM parameters ALSA: dice: Change the way to start stream ALSA: dice: Add support for duplex streams with synchronization ALSA: dice: Support for non SYT-Match sampling clock source mode ALSA: dice: Add support for capturing PCM samples ALSA: dice: Add support for MIDI capture/playback ALSA: dice: remove experimental state ALSA: speakers: Rename to oxfw and rename some members ALSA: oxfw: Move to its own directory ALSA: oxfw: Split stream functionality to a new file and add a header file ALSA: oxfw: Split PCM functionality to a new file ALSA: oxfw: Split control functionality to a new file ALSA: oxfw: Change the way to name card ALSA: oxfw: Add support for AV/C stream format command to get/set supported stream formation ALSA: oxfw: Change the way to make PCM rules/constraints ALSA: oxfw: Add proc interface for debugging purpose ALSA: oxfw: Change the way to start stream ALSA: oxfw: Add support for Behringer/Mackie devices ALSA: oxfw: Add support AMDTP in-stream ALSA: oxfw: add support for capturing PCM samples ALSA: oxfw: Add support for capture/playback MIDI messages ALSA: oxfw: Add hwdep interface include/uapi/sound/asound.h | 3 +- include/uapi/sound/firewire.h | 3 +- sound/firewire/Kconfig | 26 +- sound/firewire/Makefile | 7 +- sound/firewire/dice.c | 1511 ---------------------------- sound/firewire/dice/Makefile | 3 + sound/firewire/dice/dice-hwdep.c | 190 ++++ sound/firewire/{ => dice}/dice-interface.h | 0 sound/firewire/dice/dice-midi.c | 194 ++++ sound/firewire/dice/dice-pcm.c | 424 ++++++++ sound/firewire/dice/dice-proc.c | 252 +++++ sound/firewire/dice/dice-stream.c | 407 ++++++++ sound/firewire/dice/dice-transaction.c | 382 +++++++ sound/firewire/dice/dice.c | 361 +++++++ sound/firewire/dice/dice.h | 189 ++++ sound/firewire/oxfw/Makefile | 3 + sound/firewire/oxfw/oxfw-command.c | 153 +++ sound/firewire/oxfw/oxfw-control.c | 283 ++++++ sound/firewire/oxfw/oxfw-hwdep.c | 190 ++++ sound/firewire/oxfw/oxfw-midi.c | 207 ++++ sound/firewire/oxfw/oxfw-pcm.c | 425 ++++++++ sound/firewire/oxfw/oxfw-proc.c | 113 +++ sound/firewire/oxfw/oxfw-stream.c | 685 +++++++++++++ sound/firewire/oxfw/oxfw.c | 315 ++++++ sound/firewire/oxfw/oxfw.h | 145 +++ sound/firewire/speakers.c | 792 --------------- 26 files changed, 4944 insertions(+), 2319 deletions(-) delete mode 100644 sound/firewire/dice.c create mode 100644 sound/firewire/dice/Makefile create mode 100644 sound/firewire/dice/dice-hwdep.c rename sound/firewire/{ => dice}/dice-interface.h (100%) create mode 100644 sound/firewire/dice/dice-midi.c create mode 100644 sound/firewire/dice/dice-pcm.c create mode 100644 sound/firewire/dice/dice-proc.c create mode 100644 sound/firewire/dice/dice-stream.c create mode 100644 sound/firewire/dice/dice-transaction.c create mode 100644 sound/firewire/dice/dice.c create mode 100644 sound/firewire/dice/dice.h create mode 100644 sound/firewire/oxfw/Makefile create mode 100644 sound/firewire/oxfw/oxfw-command.c create mode 100644 sound/firewire/oxfw/oxfw-control.c create mode 100644 sound/firewire/oxfw/oxfw-hwdep.c create mode 100644 sound/firewire/oxfw/oxfw-midi.c create mode 100644 sound/firewire/oxfw/oxfw-pcm.c create mode 100644 sound/firewire/oxfw/oxfw-proc.c create mode 100644 sound/firewire/oxfw/oxfw-stream.c create mode 100644 sound/firewire/oxfw/oxfw.c create mode 100644 sound/firewire/oxfw/oxfw.h delete mode 100644 sound/firewire/speakers.c -- 2.1.0

3 47

Re: [alsa-devel] ASoC: core: Call mute for cpu dais as well
by Dan Carpenter 02 Dec '14

02 Dec '14

Hello Ramesh Babu, This is a semi-automatic email about new static checker warnings. The patch ae11601b80b9: "ASoC: core: Call mute for cpu dais as well" from Oct 15, 2014, leads to the following Smatch complaint: sound/soc/soc-pcm.c:777 soc_pcm_prepare() error: we previously assumed 'cpu_dai->driver->ops' could be null (see line 755) sound/soc/soc-pcm.c 754 755 if (cpu_dai->driver->ops && cpu_dai->driver->ops->prepare) { ^^^^^^^^^^^^^^^^^^^^ I don't think this can be NULL. Probably we should remove this check. 756 ret = cpu_dai->driver->ops->prepare(substream, cpu_dai); 757 if (ret < 0) { 758 dev_err(cpu_dai->dev, "ASoC: DAI prepare error: %d\n", 759 ret); 760 goto out; 761 } 762 } 763 764 /* cancel any delayed stream shutdown that is pending */ 765 if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK && 766 rtd->pop_wait) { 767 rtd->pop_wait = 0; 768 cancel_delayed_work(&rtd->delayed_work); 769 } 770 771 snd_soc_dapm_stream_event(rtd, substream->stream, 772 SND_SOC_DAPM_STREAM_START); 773 774 for (i = 0; i < rtd->num_codecs; i++) 775 snd_soc_dai_digital_mute(rtd->codec_dais[i], 0, 776 substream->stream); 777 snd_soc_dai_digital_mute(cpu_dai, 0, substream->stream); ^^^^^^^ But if it were NULL then this newly introduced function call would Oops. 778 779 out: regards, dan carpenter

3 2

[alsa-devel] [PATCH 2/2] ARM: tegra: Enable the mic-detect gpio on Acer Chromebook 13
by Dylan Reid 02 Dec '14

02 Dec '14

Enables the gpio-base mic detection on the Acer Chromebook 13. This gpio is set by the jack-detection chip when it notices either of the TRRS type headsets with a microphone. Signed-off-by: Dylan Reid <dgreid(a)chromium.org> --- arch/arm/boot/dts/tegra124-nyan-big.dts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/boot/dts/tegra124-nyan-big.dts b/arch/arm/boot/dts/tegra124-nyan-big.dts index 7d0784c..1c68ff9 100644 --- a/arch/arm/boot/dts/tegra124-nyan-big.dts +++ b/arch/arm/boot/dts/tegra124-nyan-big.dts @@ -1130,6 +1130,8 @@ clock-names = "pll_a", "pll_a_out0", "mclk"; nvidia,hp-det-gpios = <&gpio TEGRA_GPIO(I, 7) GPIO_ACTIVE_HIGH>; + nvidia,mic-det-gpios = + <&gpio TEGRA_GPIO(R, 7) GPIO_ACTIVE_HIGH>; }; }; -- 2.1.2.330.g565301e

3 2

[alsa-devel] [PATCH v7.1 00/19] Rework OMAP4+ HDMI audio support
by Jyri Sarha 02 Dec '14

02 Dec '14

The patches are based on: git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git for-next The base, the patches, and couple of additional not-to-be-merged omap2plus_defconfig patches can be found here: https://github.com/jsarha/linux.git omap-hdmi-audio It would make the most sense to get these in trough fbdev tree. So it would be nice to get acked-bys (if the patches are Ok) for ASoC side changes from appropriate maintainers. Changes since v7: - Squash: - "OMAPDSS: omapdss.h: Remove audio_state member of struct omap_dss_device" - into "OMAPDSS: Remove all references to obsolete HDMI audio callbacks" Changes since v6: - "ASoC: omap-hdmi-audio: Add platform device for OMAP HDMI audio support" - Fix Kconfig help text - Remove #include <sound/simple_card.h> - Use snd-soc-dummy codec instead of hdmi-audio-codec - Add: OMAPDSS: hdmi_wp: Protect reserved bits in hdmi_wp_audio_config_format() - Add: OMAPDSS: hdmi5_core: Initialize mandatory sample_order parameter - Add: "OMAPDSS: omapdss.h: Remove audio_state member of struct omap_dss_device" - "OMAPDSS: hdmi4: Register ASoC platform device for omap hdmi audio" - Register "omap-hdmi-audio" with PLATFORM_DEVID_AUTO - "OMAPDSS: hdmi5: Register ASoC platform device for omap hdmi audio" - Register "omap-hdmi-audio" with PLATFORM_DEVID_AUTO - Add: "OMAPDSS: hdmi5: Change hdmi_wp idlemode to to no_idle for audio playback" Jyri Sarha (19): OMAPDSS: hdmi_wp: Protect reserved bits in hdmi_wp_audio_config_format() OMAPDSS: hdmi5_core: Initialize mandatory sample_order parameter OMAPDSS: hdmi.h: Add HDMI_AUDIO_LAYOUT_6CH enum value OMAPDSS: hdmi: Remove most of OMAP[45]_DSS_HDMI_AUDIO ifdefs OMAPDSS: hdmi4_core: Remove unused hdmi4_audio_get_dma_port() OMAPDSS: hdmi_wp: Add function for getting audio dma address OMAPDSS: hdmi: Make hdmi structure public OMAPDSS: hdmi: Add pdev pointer for audio_pdev in HDMI DRV data ASoC: omap-hdmi-audio: Add platform device for OMAP HDMI audio support OMAPDSS: Kconfig: Remove HDMI audio booleans from Kconfig OMAPDSS: hdmi: Make hdmi_mode_has_audio() more user friedly OMAPDSS: hdmi.h: Add members to hdmi drvdata for audio implementation OMAPDSS: hdmi4: Remove callbacks for the old ASoC DAI driver OMAPDSS: hdmi4: Register ASoC platform device for omap hdmi audio OMAPDSS: hdmi5: Remove callbacks for the old ASoC DAI driver OMAPDSS: hdmi5: Register ASoC platform device for omap hdmi audio ASoC: omap: Remove obsolete HDMI audio code and Kconfig options OMAPDSS: Remove all references to obsolete HDMI audio callbacks OMAPDSS: hdmi5: Change hdmi_wp idlemode to to no_idle for audio playback .../fbdev/omap2/displays-new/connector-hdmi.c | 99 ----- .../fbdev/omap2/displays-new/encoder-tpd12s015.c | 56 --- drivers/video/fbdev/omap2/dss/Kconfig | 7 - drivers/video/fbdev/omap2/dss/hdmi.h | 38 +- drivers/video/fbdev/omap2/dss/hdmi4.c | 269 +++++++------ drivers/video/fbdev/omap2/dss/hdmi4_core.c | 14 - drivers/video/fbdev/omap2/dss/hdmi4_core.h | 4 - drivers/video/fbdev/omap2/dss/hdmi5.c | 265 ++++++------- drivers/video/fbdev/omap2/dss/hdmi5_core.c | 9 +- drivers/video/fbdev/omap2/dss/hdmi5_core.h | 2 - drivers/video/fbdev/omap2/dss/hdmi_common.c | 2 - drivers/video/fbdev/omap2/dss/hdmi_wp.c | 16 +- include/sound/omap-hdmi-audio.h | 43 +++ include/video/omapdss.h | 40 -- sound/soc/omap/Kconfig | 26 +- sound/soc/omap/Makefile | 6 +- sound/soc/omap/omap-hdmi-audio.c | 407 ++++++++++++++++++++ sound/soc/omap/omap-hdmi-card.c | 87 ----- sound/soc/omap/omap-hdmi.c | 364 ----------------- sound/soc/omap/omap-hdmi.h | 38 -- 20 files changed, 781 insertions(+), 1011 deletions(-) create mode 100644 include/sound/omap-hdmi-audio.h create mode 100644 sound/soc/omap/omap-hdmi-audio.c delete mode 100644 sound/soc/omap/omap-hdmi-card.c delete mode 100644 sound/soc/omap/omap-hdmi.c delete mode 100644 sound/soc/omap/omap-hdmi.h -- 1.7.9.5

5 51

[alsa-devel] [PATCH v2] ASoC: Intel: Move capture PCM pin to PCM0 for Broadwell/Haswell
by Jie Yang 02 Dec '14

02 Dec '14

Move capture PCM pin from PCM4 to PCM0 for Broadwell/Haswell. This will allow us to integrate with pulseaudio better for usually default device is set to 0. Signed-off-by: Jie Yang <yang.jie(a)intel.com> --- sound/soc/intel/broadwell.c | 15 ++------------- sound/soc/intel/haswell.c | 14 ++------------ sound/soc/intel/sst-haswell-pcm.c | 34 ++++++++++++++++------------------ 3 files changed, 20 insertions(+), 43 deletions(-) diff --git a/sound/soc/intel/broadwell.c b/sound/soc/intel/broadwell.c index 52cb764..c256764 100644 --- a/sound/soc/intel/broadwell.c +++ b/sound/soc/intel/broadwell.c @@ -171,7 +171,7 @@ static struct snd_soc_dai_link broadwell_rt286_dais[] = { /* Front End DAI links */ { .name = "System PCM", - .stream_name = "System Playback", + .stream_name = "System Playback/Capture", .cpu_dai_name = "System Pin", .platform_name = "haswell-pcm-audio", .dynamic = 1, @@ -180,6 +180,7 @@ static struct snd_soc_dai_link broadwell_rt286_dais[] = { .init = broadwell_rtd_init, .trigger = {SND_SOC_DPCM_TRIGGER_POST, SND_SOC_DPCM_TRIGGER_POST}, .dpcm_playback = 1, + .dpcm_capture = 1, }, { .name = "Offload0", @@ -214,18 +215,6 @@ static struct snd_soc_dai_link broadwell_rt286_dais[] = { .trigger = {SND_SOC_DPCM_TRIGGER_POST, SND_SOC_DPCM_TRIGGER_POST}, .dpcm_capture = 1, }, - { - .name = "Capture PCM", - .stream_name = "Capture", - .cpu_dai_name = "Capture Pin", - .platform_name = "haswell-pcm-audio", - .dynamic = 1, - .codec_name = "snd-soc-dummy", - .codec_dai_name = "snd-soc-dummy-dai", - .trigger = {SND_SOC_DPCM_TRIGGER_POST, SND_SOC_DPCM_TRIGGER_POST}, - .dpcm_capture = 1, - }, - /* Back End DAI links */ { /* SSP0 - Codec */ diff --git a/sound/soc/intel/haswell.c b/sound/soc/intel/haswell.c index 3981982..cb8a482 100644 --- a/sound/soc/intel/haswell.c +++ b/sound/soc/intel/haswell.c @@ -109,7 +109,7 @@ static struct snd_soc_dai_link haswell_rt5640_dais[] = { /* Front End DAI links */ { .name = "System", - .stream_name = "System Playback", + .stream_name = "System Playback/Capture", .cpu_dai_name = "System Pin", .platform_name = "haswell-pcm-audio", .dynamic = 1, @@ -118,6 +118,7 @@ static struct snd_soc_dai_link haswell_rt5640_dais[] = { .init = haswell_rtd_init, .trigger = {SND_SOC_DPCM_TRIGGER_POST, SND_SOC_DPCM_TRIGGER_POST}, .dpcm_playback = 1, + .dpcm_capture = 1, }, { .name = "Offload0", @@ -152,17 +153,6 @@ static struct snd_soc_dai_link haswell_rt5640_dais[] = { .trigger = {SND_SOC_DPCM_TRIGGER_POST, SND_SOC_DPCM_TRIGGER_POST}, .dpcm_capture = 1, }, - { - .name = "Capture", - .stream_name = "Capture", - .cpu_dai_name = "Capture Pin", - .platform_name = "haswell-pcm-audio", - .dynamic = 1, - .codec_name = "snd-soc-dummy", - .codec_dai_name = "snd-soc-dummy-dai", - .trigger = {SND_SOC_DPCM_TRIGGER_POST, SND_SOC_DPCM_TRIGGER_POST}, - .dpcm_capture = 1, - }, /* Back End DAI links */ { diff --git a/sound/soc/intel/sst-haswell-pcm.c b/sound/soc/intel/sst-haswell-pcm.c index 5f70d81..0180b38 100644 --- a/sound/soc/intel/sst-haswell-pcm.c +++ b/sound/soc/intel/sst-haswell-pcm.c @@ -309,7 +309,7 @@ static const struct snd_kcontrol_new hsw_volume_controls[] = { ARRAY_SIZE(volume_map) - 1, 0, hsw_stream_volume_get, hsw_stream_volume_put, hsw_vol_tlv), /* Mic Capture volume */ - SOC_DOUBLE_EXT_TLV("Mic Capture Volume", 4, 0, 8, + SOC_DOUBLE_EXT_TLV("Mic Capture Volume", 0, 0, 8, ARRAY_SIZE(volume_map) - 1, 0, hsw_stream_volume_get, hsw_stream_volume_put, hsw_vol_tlv), }; @@ -396,8 +396,14 @@ static int hsw_pcm_hw_params(struct snd_pcm_substream *substream, /* DSP stream type depends on DAI ID */ switch (rtd->cpu_dai->id) { case 0: - stream_type = SST_HSW_STREAM_TYPE_SYSTEM; - module_id = SST_HSW_MODULE_PCM_SYSTEM; + if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) { + stream_type = SST_HSW_STREAM_TYPE_SYSTEM; + module_id = SST_HSW_MODULE_PCM_SYSTEM; + } + else { + stream_type = SST_HSW_STREAM_TYPE_CAPTURE; + module_id = SST_HSW_MODULE_PCM_CAPTURE; + } break; case 1: case 2: @@ -410,10 +416,6 @@ static int hsw_pcm_hw_params(struct snd_pcm_substream *substream, path_id = SST_HSW_STREAM_PATH_SSP0_OUT; module_id = SST_HSW_MODULE_PCM_REFERENCE; break; - case 4: - stream_type = SST_HSW_STREAM_TYPE_CAPTURE; - module_id = SST_HSW_MODULE_PCM_CAPTURE; - break; default: dev_err(rtd->dev, "error: invalid DAI ID %d\n", rtd->cpu_dai->id); @@ -779,6 +781,13 @@ static struct snd_soc_dai_driver hsw_dais[] = { .rates = SNDRV_PCM_RATE_48000, .formats = SNDRV_PCM_FMTBIT_S24_LE | SNDRV_PCM_FMTBIT_S16_LE, }, + .capture = { + .stream_name = "Analog Capture", + .channels_min = 2, + .channels_max = 4, + .rates = SNDRV_PCM_RATE_48000, + .formats = SNDRV_PCM_FMTBIT_S24_LE | SNDRV_PCM_FMTBIT_S16_LE, + }, }, { /* PCM */ @@ -815,17 +824,6 @@ static struct snd_soc_dai_driver hsw_dais[] = { .formats = SNDRV_PCM_FMTBIT_S24_LE | SNDRV_PCM_FMTBIT_S16_LE, }, }, - { - .name = "Capture Pin", - .id = HSW_PCM_DAI_ID_CAPTURE, - .capture = { - .stream_name = "Analog Capture", - .channels_min = 2, - .channels_max = 4, - .rates = SNDRV_PCM_RATE_48000, - .formats = SNDRV_PCM_FMTBIT_S24_LE | SNDRV_PCM_FMTBIT_S16_LE, - }, - }, }; static const struct snd_soc_dapm_widget widgets[] = { -- 1.9.1

3 4

[alsa-devel] [PATCH] ALSA: usb: Add support for Zoom R16/24 capture and midi interfaces
by Panu Matilainen 01 Dec '14

01 Dec '14

This makes the midi interface and capture work out of the box with R16 (and presumably R24 too but untested). Playback stream would also seem to function fine except for one caveat: no sound is produced, so it is disabled for now. Signed-off-by: Panu Matilainen <pmatilai(a)laiskiainen.org> --- sound/usb/quirks-table.h | 34 ++++++++++++++++++++++++++++++++++ sound/usb/quirks.c | 8 ++++++++ 2 files changed, 42 insertions(+) diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h index c657752..73621c9 100644 --- a/sound/usb/quirks-table.h +++ b/sound/usb/quirks-table.h @@ -3202,6 +3202,40 @@ YAMAHA_DEVICE(0x7010, "UB99"), }, { + /* ZOOM R16/24 in audio interface mode */ + USB_DEVICE(0x1686, 0x00dd), + .driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) { + .ifnum = QUIRK_ANY_INTERFACE, + .type = QUIRK_COMPOSITE, + .data = (const struct snd_usb_audio_quirk[]) { + { + /* Control */ + .ifnum = 0, + .type = QUIRK_IGNORE_INTERFACE, + }, + { + /* Playback, sound is missing atm */ + .ifnum = 1, + .type = QUIRK_IGNORE_INTERFACE, + }, + { + /* Capture */ + .ifnum = 2, + .type = QUIRK_AUDIO_STANDARD_INTERFACE, + }, + { + /* Midi */ + .ifnum = 3, + .type = QUIRK_MIDI_STANDARD_INTERFACE + }, + { + .ifnum = -1 + }, + } + } +}, + +{ /* * Some USB MIDI devices don't have an audio control interface, * so we have to grab MIDI streaming interfaces here. diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index a5941f8..2e49193 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -1160,6 +1160,14 @@ void snd_usb_ctl_msg_quirk(struct usb_device *dev, unsigned int pipe, break; } } + + /* Zoom R16/24 needs a tiny delay here, otherwise requests like + * get/set frequency return as failed despite actually succeeding. + */ + if ((le16_to_cpu(dev->descriptor.idVendor) == 0x1686) && + (le16_to_cpu(dev->descriptor.idProduct) == 0x00dd) && + (requesttype & USB_TYPE_MASK) == USB_TYPE_CLASS) + mdelay(1); } /* -- 1.9.3

2 3

[alsa-devel] [PATCH v2] ALSA: usb: Add support for Zoom R16/24 capture and midi interfaces
by Panu Matilainen 01 Dec '14

01 Dec '14

This makes the midi interface and capture work out of the box with R16 (and presumably R24 too but untested). Playback stream would also seem to function fine except for one caveat: no sound is produced, so it is disabled for now. Mixer descriptors are garbage and will require further quirks to enable functionality, also disabled here. Signed-off-by: Panu Matilainen <pmatilai(a)laiskiainen.org> --- sound/usb/quirks-table.h | 40 ++++++++++++++++++++++++++++++++++++++++ sound/usb/quirks.c | 8 ++++++++ 2 files changed, 48 insertions(+) diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h index c657752..c955663 100644 --- a/sound/usb/quirks-table.h +++ b/sound/usb/quirks-table.h @@ -3203,6 +3203,46 @@ YAMAHA_DEVICE(0x7010, "UB99"), { /* + * ZOOM R16/24 in audio interface mode. + * Mixer descriptors are garbage, further quirks will be needed + * to make any of it functional, thus disabled for now. + * Playback stream appears to start and run fine but no sound + * is produced, so also disabled for now. + */ + USB_DEVICE(0x1686, 0x00dd), + .driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) { + .ifnum = QUIRK_ANY_INTERFACE, + .type = QUIRK_COMPOSITE, + .data = (const struct snd_usb_audio_quirk[]) { + { + /* Mixer */ + .ifnum = 0, + .type = QUIRK_IGNORE_INTERFACE, + }, + { + /* Playback */ + .ifnum = 1, + .type = QUIRK_IGNORE_INTERFACE, + }, + { + /* Capture */ + .ifnum = 2, + .type = QUIRK_AUDIO_STANDARD_INTERFACE, + }, + { + /* Midi */ + .ifnum = 3, + .type = QUIRK_MIDI_STANDARD_INTERFACE + }, + { + .ifnum = -1 + }, + } + } +}, + +{ + /* * Some USB MIDI devices don't have an audio control interface, * so we have to grab MIDI streaming interfaces here. */ diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index a5941f8..2e49193 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -1160,6 +1160,14 @@ void snd_usb_ctl_msg_quirk(struct usb_device *dev, unsigned int pipe, break; } } + + /* Zoom R16/24 needs a tiny delay here, otherwise requests like + * get/set frequency return as failed despite actually succeeding. + */ + if ((le16_to_cpu(dev->descriptor.idVendor) == 0x1686) && + (le16_to_cpu(dev->descriptor.idProduct) == 0x00dd) && + (requesttype & USB_TYPE_MASK) == USB_TYPE_CLASS) + mdelay(1); } /* -- 1.9.3

2 1

[alsa-devel] [GIT PULL] Add sst audio firmware for baytrail platforms
by Vinod Koul 01 Dec '14

01 Dec '14

Hello Linux firmware maintainers, Please PULL to receive the Intel audio firmware for audio DSP in Baytrail platforms. This firmware is using the same license as another version of the Baytrail firmware already available in linux-firmware git. The following changes since commit 0e5f63771d0df6d7859f7c4100a74d737c62ac88: are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/firmware.git byt Vinod Koul (1): linux-firmware: add sst audio firmware for baytrail platforms WHENCE | 8 ++++++++ intel/fw_sst_0f28.bin | Bin 0 -> 647078 bytes 2 files changed, 8 insertions(+), 0 deletions(-) create mode 100644 intel/fw_sst_0f28.bin Thanks -- ~Vinod

2 1

[alsa-devel] [PATCH] Sound: drivers: virmidi: fixed code style issues
by Kyle Chamberlin 30 Nov '14

30 Nov '14

Fixed some minor code style issues and also removed some assignments inside of if conditionals. Signed-off-by: Kyle Chamberlin <KyleChamberlin(a)project20million.org> --- sound/drivers/virmidi.c | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/sound/drivers/virmidi.c b/sound/drivers/virmidi.c index b17872429..0af88d9 100644 --- a/sound/drivers/virmidi.c +++ b/sound/drivers/virmidi.c @@ -99,30 +99,33 @@ static int snd_virmidi_probe(struct platform_device *devptr) if (midi_devs[dev] > MAX_MIDI_DEVICES) { snd_printk(KERN_WARNING - "too much midi devices for virmidi %d: " - "force to use %d\n", dev, MAX_MIDI_DEVICES); + "too much midi devices for virmidi %d: force to use %d\n", + dev, MAX_MIDI_DEVICES); midi_devs[dev] = MAX_MIDI_DEVICES; } for (idx = 0; idx < midi_devs[dev]; idx++) { struct snd_rawmidi *rmidi; struct snd_virmidi_dev *rdev; - if ((err = snd_virmidi_new(card, idx, &rmidi)) < 0) + + err = snd_virmidi_new(card, idx, &rmidi); + if (err < 0) goto __nodev; rdev = rmidi->private_data; vmidi->midi[idx] = rmidi; strcpy(rmidi->name, "Virtual Raw MIDI"); rdev->seq_mode = SNDRV_VIRMIDI_SEQ_DISPATCH; } - + strcpy(card->driver, "VirMIDI"); strcpy(card->shortname, "VirMIDI"); sprintf(card->longname, "Virtual MIDI Card %i", dev + 1); - if ((err = snd_card_register(card)) == 0) { + err = snd_card_register(card); + if (err) { platform_set_drvdata(devptr, card); return 0; } - __nodev: +__nodev: snd_card_free(card); return err; } @@ -157,13 +160,15 @@ static int __init alsa_card_virmidi_init(void) { int i, cards, err; - if ((err = platform_driver_register(&snd_virmidi_driver)) < 0) + err = platform_driver_register(&snd_virmidi_driver); + if (err < 0) return err; cards = 0; for (i = 0; i < SNDRV_CARDS; i++) { struct platform_device *device; - if (! enable[i]) + + if (!enable[i]) continue; device = platform_device_register_simple(SND_VIRMIDI_DRIVER, i, NULL, 0); -- 2.1.3

4 8

[alsa-devel] [alsa/virmidi] BUG: unable to handle kernel NULL pointer dereference at 000002a0
by Fengguang Wu 30 Nov '14

30 Nov '14

Hi Kyle, 0day kernel testing robot got the below dmesg and the first bad commit is git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next commit 316638a5030a04bb3259dcbca0632281001a4b24 Author: Kyle Chamberlin <kylechamberlin(a)project20million.org> AuthorDate: Fri Nov 28 13:59:56 2014 -0500 Commit: Takashi Iwai <tiwai(a)suse.de> CommitDate: Sat Nov 29 21:34:47 2014 +0100 ALSA: virmidi: fixed code style issues Fixed some minor code style issues and also removed some assignments inside of if conditionals. Signed-off-by: Kyle Chamberlin <kylechamberlin(a)project20million.org> Signed-off-by: Takashi Iwai <tiwai(a)suse.de> In particular, this chunk introduced the behavior change: - if ((err = snd_card_register(card)) == 0) { + err = snd_card_register(card); + if (err) { +-------------------------------------------------+------------+------------+------------+ | | 31514bfb4a | 316638a503 | 795172aaa6 | +-------------------------------------------------+------------+------------+------------+ | boot_successes | 67 | 0 | 0 | | boot_failures | 13 | 20 | 14 | | BUG:kernel_early_hang_without_any_printk_output | 13 | 0 | 2 | | BUG:unable_to_handle_kernel | 0 | 20 | 12 | | Oops | 0 | 20 | 12 | | EIP_is_at_snd_card_free | 0 | 20 | 12 | | Kernel_panic-not_syncing:Fatal_exception | 0 | 20 | 12 | | backtrace:platform_device_unregister | 0 | 20 | 12 | | backtrace:alsa_card_virmidi_init | 0 | 20 | 12 | | backtrace:kernel_init_freeable | 0 | 20 | 12 | +-------------------------------------------------+------------+------------+------------+ [ 1.688337] usbcore: registered new interface driver line6usb [ 1.689386] dgap: dgap-1.3-16, Digi International Part Number 40002347_C [ 1.690258] dgap: For the tools package please visit http://www.digi.com [ 1.694793] BUG: unable to handle kernel NULL pointer dereference at 000002a0 [ 1.695739] IP: [<7940555f>] snd_card_free+0x36/0x7f [ 1.696409] *pde = 00000000 [ 1.696686] Oops: 0002 [#1] [ 1.696686] Modules linked in: [ 1.696686] CPU: 0 PID: 1 Comm: swapper Not tainted 3.18.0-rc5-00179-g316638a #101 [ 1.696686] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.7.5-20140531_083030-gandalf 04/01/2014 [ 1.696686] task: 8a068000 ti: 8a064000 task.ti: 8a064000 [ 1.696686] EIP: 0060:[<7940555f>] EFLAGS: 00010246 CPU: 0 [ 1.696686] EIP is at snd_card_free+0x36/0x7f [ 1.696686] EAX: 00000000 EBX: 8a065e04 ECX: 7a354880 EDX: 798a5d21 [ 1.696686] ESI: 00000000 EDI: 00000000 EBP: 8a065e44 ESP: 8a065e04 [ 1.696686] DS: 007b ES: 007b FS: 0000 GS: 00e0 SS: 0068 [ 1.696686] CR0: 8005003b CR2: 000002a0 CR3: 01d49000 CR4: 000006d0 [ 1.696686] Stack: [ 1.696686] 00000000 00000001 dead4ead ffffffff ffffffff 7a354880 00000000 00000000 [ 1.696686] 798a5d21 00000000 00000000 8a065e30 8a065e30 006d61bc 7808440c 00000001 [ 1.696686] 8a065e4c 7943d967 8a065e54 792c0e93 8a065e68 792befa9 78084440 7808440c [ 1.696686] Call Trace: [ 1.696686] [<7943d967>] snd_virmidi_remove+0x13/0x17 [ 1.696686] [<792c0e93>] platform_drv_remove+0x14/0x16 [ 1.696686] [<792befa9>] __device_release_driver+0xc1/0x121 [ 1.696686] [<792bf028>] device_release_driver+0x1f/0x2a [ 1.696686] [<792be929>] bus_remove_device+0x166/0x17d [ 1.696686] [<792bb4ad>] device_del+0x14b/0x1d8 [ 1.696686] [<792bb4ad>] ? device_del+0x14b/0x1d8 [ 1.696686] [<792c1407>] platform_device_del+0x32/0xa3 [ 1.696686] [<792c1488>] platform_device_unregister+0x10/0x1a [ 1.696686] [<79cee7bf>] alsa_card_virmidi_init+0xd1/0x118 [ 1.696686] [<79cee6ee>] ? alsa_seq_midi_init+0x34/0x34 [ 1.696686] [<79000551>] do_one_initcall+0x186/0x288 [ 1.696686] [<79cb4400>] ? rdinit_setup+0x18/0x22 [ 1.696686] [<79060c87>] ? parse_args+0x4a6/0x579 [ 1.696686] [<7906be5f>] ? local_clock+0x2b/0x35 [ 1.696686] [<79cb44f6>] ? do_early_param+0xac/0xac [ 1.696686] [<79cb4f46>] kernel_init_freeable+0xfd/0x1bd [ 1.696686] [<79cb4f46>] ? kernel_init_freeable+0xfd/0x1bd [ 1.696686] [<7959bbfc>] kernel_init+0xe/0x13c [ 1.696686] [<795a7cc0>] ret_from_kernel_thread+0x20/0x30 [ 1.696686] [<7959bbee>] ? rest_init+0xa7/0xa7 [ 1.696686] Code: 26 00 89 c6 65 a1 14 00 00 00 89 45 f4 31 c0 c7 45 c0 00 00 00 00 8d 5d c0 b9 80 48 35 7a ba 21 5d 8a 79 8d 45 c4 e8 bc b2 c6 ff <89> 9e a0 02 00 00 89 f0 e8 80 fe ff ff 85 c0 0f 95 c2 0f b6 f2 [ 1.696686] EIP: [<7940555f>] snd_card_free+0x36/0x7f SS:ESP 0068:8a065e04 [ 1.696686] CR2: 00000000000002a0 [ 1.696686] ---[ end trace bc9b8dd3625e806d ]--- [ 1.696686] Kernel panic - not syncing: Fatal exception git bisect start 795172aaa6701c4b66b449e79b49c41e044156c6 5d01410fe4d92081f349b013a2e7a95429e4f2c9 -- git bisect bad 5d81516a74d44d9301e140ff97e06b37eaf5831d # 06:13 0- 13 Merge 'luto/userns/setgroups' into devel-lkp-hsx01-smoke-201411300558 git bisect good b5675b9e191a3981a38f5c94b6dd8b6fa4ba8091 # 06:23 20+ 3 0day base guard for 'devel-lkp-hsx01-smoke-201411300558' git bisect bad e929ef74cdd822177ad2322514a40127ad1e696c # 06:31 0- 4 Merge 'sound/for-next' into devel-lkp-hsx01-smoke-201411300558 git bisect good 2603fe21b764eb7412598c8c6cd6199fb8b1d9c5 # 06:41 20+ 10 ALSA: es18xx: Add GPO controls git bisect good 2bfb14c3b8fbc787ff4478f9d77ecee78cb922fe # 06:51 20+ 0 ALSA: usb-audio: Add Xonar U1 resume support git bisect good 37e661ee10c6d0d1310c62b3d29ae9a63073ac5d # 06:58 20+ 5 ALSA: hda - Add AZX_DCAPS_SNOOP_OFF (and refactor snoop setup) git bisect good 6eb6c81eee2a6270b39ca02a446f3ccece24b6f8 # 07:16 20+ 0 ALSA: dice: Split stream functionality into a file git bisect good 8832c5a74ba3506c51b6637ac78941fcd21afbef # 07:25 20+ 0 ALSA: speakers: Rename to oxfw and rename some members git bisect good 3713d93a6a12f8629c2660bb4a30d48b98105fca # 07:32 20+ 4 ALSA: oxfw: Split PCM functionality to a new file git bisect bad 316638a5030a04bb3259dcbca0632281001a4b24 # 07:40 0- 17 ALSA: virmidi: fixed code style issues git bisect good 31514bfb4ab8ba6f93b5ce5fcc543cb2ac4f96e5 # 07:46 20+ 1 ALSA: oxfw: Split control functionality to a new file # first bad commit: [316638a5030a04bb3259dcbca0632281001a4b24] ALSA: virmidi: fixed code style issues git bisect good 31514bfb4ab8ba6f93b5ce5fcc543cb2ac4f96e5 # 07:48 60+ 13 ALSA: oxfw: Split control functionality to a new file # extra tests on HEAD of linux-devel/devel-lkp-hsx01-smoke-201411300558 git bisect bad 795172aaa6701c4b66b449e79b49c41e044156c6 # 07:48 0- 14 0day head guard for 'devel-lkp-hsx01-smoke-201411300558' # extra tests on tree/branch sound/for-next git bisect bad 316638a5030a04bb3259dcbca0632281001a4b24 # 07:49 0- 20 ALSA: virmidi: fixed code style issues # extra tests on tree/branch linus/master git bisect good 7a5a4f978750756755dc839014e13d1b088ccc8e # 07:51 60+ 10 Merge branch 'irq-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip # extra tests on tree/branch next/master git bisect good 29340b8ee0b4def844aa8d6b09babcede80993d2 # 07:54 60+ 80 Add linux-next specific files for 20141128 This script may reproduce the error. ---------------------------------------------------------------------------- #!/bin/bash kernel=$1 initrd=quantal-core-i386.cgz wget --no-clobber https://github.com/fengguang/reproduce-kernel-bug/raw/master/initrd/$initrd kvm=( qemu-system-x86_64 -cpu kvm64 -enable-kvm -kernel $kernel -initrd $initrd -m 320 -smp 2 -net nic,vlan=1,model=e1000 -net user,vlan=1 -boot order=nc -no-reboot -watchdog i6300esb -rtc base=localtime -serial stdio -display none -monitor null ) append=( hung_task_panic=1 earlyprintk=ttyS0,115200 debug apic=debug sysrq_always_enabled rcupdate.rcu_cpu_stall_timeout=100 panic=-1 softlockup_panic=1 nmi_watchdog=panic oops=panic load_ramdisk=2 prompt_ramdisk=0 console=ttyS0,115200 console=tty0 vga=normal root=/dev/ram0 rw drbd.minor_count=8 ) "${kvm[@]}" --append "${append[*]}" ---------------------------------------------------------------------------- Thanks, Fengguang _______________________________________________ LKP mailing list LKP(a)linux.intel.com

2 1