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

[alsa-devel] vsp1 BUG_ON() and crash (Re: [PATCH v9 03/12] media: Entities, pads and links)
by Geert Uytterhoeven 14 Dec '15

14 Dec '15

On Mon, Feb 14, 2011 at 1:20 PM, Laurent Pinchart <laurent.pinchart(a)ideasonboard.com> wrote: > +media_entity_create_link(struct media_entity *source, u16 source_pad, > + struct media_entity *sink, u16 sink_pad, u32 flags) > +{ > + struct media_link *link; > + struct media_link *backlink; > + > + BUG_ON(source == NULL || sink == NULL); > + BUG_ON(source_pad >= source->num_pads); > + BUG_ON(sink_pad >= sink->num_pads); This triggers on r8a7791/koelsch: vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 bru was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 hsi was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 hst was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 lut was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 rpf.0 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 rpf.1 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 rpf.2 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 rpf.3 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 rpf.4 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 sru was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 uds.0 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 uds.1 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 uds.2 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 wpf.0 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 wpf.1 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 wpf.2 was not initialized! vsp1 fe928000.vsp1: Entity type for entity fe928000.vsp1 wpf.3 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 bru was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 hsi was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 hst was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 lif was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 lut was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 rpf.0 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 rpf.1 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 rpf.2 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 rpf.3 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 uds.0 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 wpf.0 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 wpf.1 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 wpf.2 was not initialized! vsp1 fe930000.vsp1: Entity type for entity fe930000.vsp1 wpf.3 was not initialized! ------------[ cut here ]------------ kernel BUG at drivers/media/media-entity.c:521! Internal error: Oops - BUG: 0 [#1] SMP ARM CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.4.0-rc5-03225-g596de581263544d2 #400 Hardware name: Generic R8A7791 (Flattened Device Tree) task: ee84fb40 ti: ee850000 task.ti: ee850000 PC is at media_create_pad_link+0x50/0x13c LR is at vsp1_wpf_create_links+0x4c/0x54 pc : [<c034fd90>] lr : [<c03761cc>] psr: 60000013 sp : ee851d50 ip : ee851d80 fp : ee851d7c r10: 00000001 r9 : ee9e9600 r8 : 00000000 r7 : ee111c80 r6 : 00000000 r5 : ee114a40 r4 : ee114b18 r3 : 00000000 r2 : ee114b18 r1 : 00000001 r0 : ee114a40 Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none Control: 10c5307d Table: 4000406a DAC: 00000051 Process swapper/0 (pid: 1, stack limit = 0xee850210) Stack: (0xee851d50 to 0xee852000) 1d40: 00000000 ee111c10 ee114a10 ee111c90 1d60: ee111c80 ee9e9610 ee9e9600 ee111c98 ee851d94 ee851d80 c03761cc c034fd4c 1d80: 00000001 ee111c98 ee851de4 ee851d98 c03730c0 c037618c 00000000 ee9eac00 1da0: ee111c10 ee851db0 00000000 ee114840 ee114c40 00000000 c071b678 00000000 1dc0: ee9e9610 c06d3414 c071b678 c06d3414 00000000 00000000 ee851e04 ee851de8 1de0: c028f2b0 c0372bbc c028f258 ee9e9610 c071b668 00000000 ee851e2c ee851e08 1e00: c028dd4c c028f264 ee9e9610 ee9e9644 c06d3414 c06cb3a8 c069e010 c06e2000 1e20: ee851e4c ee851e30 c028df04 c028dc1c 00000001 00000000 c06d3414 c028de94 1e40: ee851e74 ee851e50 c028c47c c028dea0 ee884f5c ee9e7bb4 ee884f70 c06d3414 1e60: 00000000 eeaeb980 ee851e84 ee851e78 c028e138 c028c414 ee851eac ee851e88 1e80: c028cc18 c028e124 c0646a6b ee851e98 c06d3414 c0681dc8 00000000 c069e010 1ea0: ee851ec4 ee851eb0 c028e848 c028cb50 ee0bd380 c0681dc8 ee851ed4 ee851ec8 1ec0: c028fcbc c028e7b0 ee851ee4 ee851ed8 c0681de0 c028fc90 ee851f5c ee851ee8 1ee0: c0664df8 c0681dd4 ee851f0c ee851ef8 c0664604 c01c94bc efffcc00 efffcc10 1f00: ee851f5c ee851f10 c003f90c c06645f4 00000000 c0661740 c0661740 00000006 1f20: 00000006 000000b6 c0660908 efffcc38 ee851f5c 00000006 00000006 000000b6 1f40: c069e83c c06aad08 c06e2000 c06e2000 ee851f94 ee851f60 c0664fd4 c0664cfc 1f60: 00000006 00000006 00000000 c06645e8 00000000 c04e752c 00000000 00000000 1f80: 00000000 00000000 ee851fac ee851f98 c04e753c c0664eb8 ee850000 00000000 1fa0: 00000000 ee851fb0 c000fe98 c04e7538 00000000 00000000 00000000 00000000 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 Backtrace: [<c034fd40>] (media_create_pad_link) from [<c03761cc>] (vsp1_wpf_create_links+0x4c/0x54) r10:ee111c98 r9:ee9e9600 r8:ee9e9610 r7:ee111c80 r6:ee111c90 r5:ee114a10 r4:ee111c10 r3:00000000 [<c0376180>] (vsp1_wpf_create_links) from [<c03730c0>] (vsp1_probe+0x510/0x700) [<c0372bb0>] (vsp1_probe) from [<c028f2b0>] (platform_drv_probe+0x58/0xa8) r10:00000000 r9:00000000 r8:c06d3414 r7:c071b678 r6:c06d3414 r5:ee9e9610 r4:00000000 While bisecting, I arrived at a NULL pointer dereference: Unable to handle kernel NULL pointer dereference at virtual address 00000250 pgd = c0004000 [00000250] *pgd=00000000 Internal error: Oops: 5 [#1] SMP ARM CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.4.0-rc2-00170-gf9836d92a3c8c143 #417 ata1: SATA link down (SStatus 0 SControl 300) Hardware name: Generic R8A7791 (Flattened Device Tree) task: ee84fb40 ti: ee850000 task.ti: ee850000 PC is at media_gobj_init+0x20/0x68 LR is at media_entity_create_link+0xe4/0x108 pc : [<c0342428>] lr : [<c0342a80>] psr: 60000013 sp : ee851d10 ip : ee851d20 fp : ee851d1c r10: 00000000 r9 : ee103890 r8 : 00000000 r7 : ee159920 r6 : ee103740 r5 : ee103840 r4 : ee159840 r3 : 00000003 r2 : ee103840 r1 : 00000002 r0 : 00000000 Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none Control: 10c5307d Table: 4000406a DAC: 00000051 Process swapper/0 (pid: 1, stack limit = 0xee850210) Stack: (0xee851d10 to 0xee852000) 1d00: ee851d4c ee851d20 c0342a80 c0342414 1d20: 00000000 ee159810 00000000 ee159840 ee159910 00000000 ee9e6800 ee102898 1d40: ee851d94 ee851d50 c036506c c03429a8 00000003 00000000 000000ff 00000000 1d60: 00000001 00000000 000000ff 00000000 c0366740 ee102810 ee102890 00000000 1d80: ee102858 ee9e6810 ee851de4 ee851d98 c0362a44 c0364ef4 00000080 ee9e8f00 1da0: ee102810 ee851db0 c012bcac ee9e6810 ee9e6810 c06b1f88 c06e76b4 00000000 1dc0: ee9e6810 c06b1f88 c06e76b4 00000000 c06b1f88 00000000 ee851e04 ee851de8 1de0: c0285b14 c0362694 c0285abc ee9e6810 00000000 c06e76a8 ee851e2c ee851e08 1e00: c028457c c0285ac8 ee9e6810 ee9e6844 c06b1f88 c06aab38 c067d010 c06c1000 1e20: ee851e4c ee851e30 c0284770 c028446c 00000001 00000000 c06b1f88 c0284700 1e40: ee851e74 ee851e50 c0282d78 c028470c ee884f5c ee9e4cb4 ee884f70 c06b1f88 1e60: 00000000 ee9c4700 ee851e84 ee851e78 c028499c c0282d10 ee851eac ee851e88 1e80: c0283514 c0284988 c0627e00 ee851e98 c06b1f88 c06618a8 00000000 c067d010 1ea0: ee851ec4 ee851eb0 c02850ac c028344c eea9a200 c06618a8 ee851ed4 ee851ec8 1ec0: c02864e0 c0285014 ee851ee4 ee851ed8 c06618c0 c02864b4 ee851f5c ee851ee8 1ee0: c0644df8 c06618b4 ee851f0c ee851ef8 c0644604 c01c91bc efffcc00 efffcc10 1f00: ee851f5c ee851f10 c003f908 c06445f4 00000000 c0641c1c c0641c1c 00000006 1f20: 00000006 000000b1 c0640e48 efffcc38 ee851f5c 00000006 00000006 000000b1 1f40: c067d83c c0689c84 c06c1000 c06c1000 ee851f94 ee851f60 c0644fd4 c0644cfc 1f60: 00000006 00000006 00000000 c06445e8 00000000 c04d00ac 00000000 00000000 1f80: 00000000 00000000 ee851fac ee851f98 c04d00bc c0644eb8 ee850000 00000000 1fa0: 00000000 ee851fb0 c000fe98 c04d00b8 00000000 00000000 00000000 00000000 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 Backtrace: [<c0342408>] (media_gobj_init) from [<c0342a80>] (media_entity_create_link+0xe4/0x108) [<c034299c>] (media_entity_create_link) from [<c036506c>] (vsp1_rpf_create+0x184/0x1c8) r10:ee102898 r9:ee9e6800 r8:00000000 r7:ee159910 r6:ee159840 r5:00000000 r4:ee159810 r3:00000000 [<c0364ee8>] (vsp1_rpf_create) from [<c0362a44>] (vsp1_probe+0x3bc/0x6c0) r8:ee9e6810 r7:ee102858 r6:00000000 r5:ee102890 r4:ee102810 [<c0362688>] (vsp1_probe) from [<c0285b14>] (platform_drv_probe+0x58/0xa8) r10:00000000 r9:c06b1f88 r8:00000000 r7:c06e76b4 r6:c06b1f88 r5:ee9e6810 r4:00000000 The NULL pointer dereference was introduced by: commit f9836d92a3c8c143c6f5fe89c558c37767628fa6 Author: Mauro Carvalho Chehab <mchehab(a)osg.samsung.com> Date: Fri Aug 14 12:54:36 2015 -0300 [media] media: use media_gobj inside links and triggers until the following commit, after which the BUG_ON() takes over: commit eb9e998983338811c3328294ed2f093367161df1 Author: Javier Martinez Canillas <javier(a)osg.samsung.com> Date: Thu Sep 3 12:19:25 2015 -0300 [media] v4l: vsp1: separate links creation from entities init Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert(a)linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds

2 1

[alsa-devel] [PATCH v2] sound: codecs: pcm1792a: clean function exit
by Raphael Poggi 14 Dec '15

14 Dec '15

Signed-off-by: Raphael Poggi <poggi.raph(a)gmail.com> --- sound/soc/codecs/pcm1792a.c | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/sound/soc/codecs/pcm1792a.c b/sound/soc/codecs/pcm1792a.c index 08bb486..27d54bc 100644 --- a/sound/soc/codecs/pcm1792a.c +++ b/sound/soc/codecs/pcm1792a.c @@ -91,14 +91,9 @@ static int pcm1792a_digital_mute(struct snd_soc_dai *dai, int mute) { struct snd_soc_codec *codec = dai->codec; struct pcm1792a_private *priv = snd_soc_codec_get_drvdata(codec); - int ret; - ret = regmap_update_bits(priv->regmap, PCM1792A_SOFT_MUTE, + return regmap_update_bits(priv->regmap, PCM1792A_SOFT_MUTE, PCM1792A_MUTE_MASK, !!mute); - if (ret < 0) - return ret; - - return 0; } static int pcm1792a_hw_params(struct snd_pcm_substream *substream, @@ -107,7 +102,7 @@ static int pcm1792a_hw_params(struct snd_pcm_substream *substream, { struct snd_soc_codec *codec = dai->codec; struct pcm1792a_private *priv = snd_soc_codec_get_drvdata(codec); - int val = 0, ret; + int val = 0; priv->rate = params_rate(params); @@ -145,12 +140,8 @@ static int pcm1792a_hw_params(struct snd_pcm_substream *substream, val = val << PCM1792A_FMT_SHIFT | PCM1792A_ATLD_ENABLE; - ret = regmap_update_bits(priv->regmap, PCM1792A_FMT_CONTROL, + return regmap_update_bits(priv->regmap, PCM1792A_FMT_CONTROL, PCM1792A_FMT_MASK | PCM1792A_ATLD_ENABLE, val); - if (ret < 0) - return ret; - - return 0; } static const struct snd_soc_dai_ops pcm1792a_dai_ops = { -- 2.1.0

2 2

[alsa-devel] [PATCH 1/2] ALSA: usb-audio: Add a volume scale quirk for AudioQuest DragonFly
by Anssi Hannula 14 Dec '15

14 Dec '15

AudioQuest DragonFly DAC reports a volume control range of 0..50 (0x0000..0x0032) which in USB Audio means a range of 0 .. 0.2dB, which is obviously incorrect and causes software using the dB information in e.g. volume sliders to have a massive volume difference in 100..102% range. The actual volume mapping seems to be neither linear volume nor linear dB scale, but instead quite close to the cubic mapping e.g. alsamixer uses, with a range of -53...0 dB. Add a quirk for DragonFly to use a custom dB mapping, based on my measurements, using a 10-item range TLV (so it still fits in alsa-lib MAX_TLV_RANGE_SIZE). Tested on AudioQuest DragonFly HW v1.2. The quirk is only applied if the range is 0..50, so if this gets fixed/changed in later HW revisions it will no longer be applied. Signed-off-by: Anssi Hannula <anssi.hannula(a)iki.fi> Cc: <stable(a)vger.kernel.org> --- sound/usb/mixer_quirks.c | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c index 337c317ead6f..39d7f34e44e6 100644 --- a/sound/usb/mixer_quirks.c +++ b/sound/usb/mixer_quirks.c @@ -37,6 +37,7 @@ #include <sound/control.h> #include <sound/hwdep.h> #include <sound/info.h> +#include <sound/tlv.h> #include "usbaudio.h" #include "mixer.h" @@ -1733,6 +1734,38 @@ static int snd_microii_controls_create(struct usb_mixer_interface *mixer) return 0; } +static void snd_dragonfly_quirk_db_scale(struct usb_mixer_interface *mixer) +{ + struct usb_mixer_elem_list *list; + struct usb_mixer_elem_info *cval; + static const int unit_id = 7; + + /* approximation using 10 ranges based on output measurement on hw v1.2 */ + static const DECLARE_TLV_DB_RANGE(scale, + 0, 1, TLV_DB_MINMAX_ITEM(-5300, -4970), + 2, 5, TLV_DB_MINMAX_ITEM(-4710, -4160), + 6, 7, TLV_DB_MINMAX_ITEM(-3884, -3710), + 8, 14, TLV_DB_MINMAX_ITEM(-3443, -2560), + 15, 16, TLV_DB_MINMAX_ITEM(-2475, -2324), + 17, 19, TLV_DB_MINMAX_ITEM(-2228, -2031), + 20, 26, TLV_DB_MINMAX_ITEM(-1910, -1393), + 27, 31, TLV_DB_MINMAX_ITEM(-1322, -1032), + 32, 40, TLV_DB_MINMAX_ITEM(-968, -490), + 41, 50, TLV_DB_MINMAX_ITEM(-441, 0), + ); + + for (list = mixer->id_elems[unit_id]; list; list = list->next_id_elem) { + cval = (struct usb_mixer_elem_info *)list; + if (cval->control == UAC_FU_VOLUME && + cval->min == 0 && cval->max == 50) { + usb_audio_info(mixer->chip, "applying DragonFly dB scale quirk\n"); + list->kctl->tlv.p = scale; + list->kctl->vd[0].access |= SNDRV_CTL_ELEM_ACCESS_TLV_READ; + list->kctl->vd[0].access &= ~SNDRV_CTL_ELEM_ACCESS_TLV_CALLBACK; + } + } +} + int snd_usb_mixer_apply_create_quirk(struct usb_mixer_interface *mixer) { int err = 0; @@ -1810,6 +1843,10 @@ int snd_usb_mixer_apply_create_quirk(struct usb_mixer_interface *mixer) case USB_ID(0x1235, 0x800c): /* Focusrite Scarlett 18i20 */ err = snd_scarlett_controls_create(mixer); break; + + case USB_ID(0x21b4, 0x0081): /* AudioQuest DragonFly */ + snd_dragonfly_quirk_db_scale(mixer); + break; } return err; -- 2.3.8

3 9

[alsa-devel] [PATCH v3 0/5] ALSA: hda - hdmi support dynmaic pcm assignment
by libin.yang＠linux.intel.com 14 Dec '15

14 Dec '15

From: Libin Yang <libin.yang(a)linux.intel.com> The patches are trying to support dynamically binding PCM to pin in HDMI/DP audio. The pin will not bind to any PCM if there is not monitor connected. When there is a monitor connected to the port (pin), driver will try to find a proper PCM to assign to the port and setup the pin if necessary. Libin Yang (5): ALSA: hda - hdmi begin to support dynamic PCM assignment ALSA: hda - hdmi playback without monitor in dynamic pcm bind mode ALSA: hda - hdmi operate spdif based on pcm ALSA: hda - hdmi dynamically bind PCM to pin when monitor hotplug ALSA: hda - hdmi setup pin when monitor hotplug in pcm dynamic assignment mode sound/pci/hda/hda_codec.h | 1 + sound/pci/hda/patch_hdmi.c | 369 ++++++++++++++++++++++++++++++++++++++++++--- 2 files changed, 347 insertions(+), 23 deletions(-) -- 1.9.1

4 9

[alsa-devel] [PATCH v3 00/13] ASoC: Intel: Skylake: Update to SKL driver
by Vinod Koul 14 Dec '15

14 Dec '15

This update bring in new APIs in core and usage of those in SKL driver. The series is broadly of three parts - first two are fixes on the driver - then we add DMA resume capability in HDA core and use it to save restore DMA link values and add DMA resume capability in driver - lastly, we add support for MISCBDCGE configuration in core and use it while resetting and also for phrase detection per HW recommendation This should be merged thru ASoC tree due to obvious dependencies. Takashi would need your ack on core patches change is v3: - fix the missing export symbol, reported by lkp changes in v2: - move msicbdge api to driver and as a consequence sqash into into first use - fix lkp warning Dharageswari.R (1): ASoC: Intel: Skylake: Use CGCTL.MISCBDCGE for Phrase detection notification Jayachandran B (2): ALSA: hdac: Increase timeout value for link power check ASoC: Intel: Skylake: fix reset controller sequencing Jeeja KP (8): ASoC: Intel: Skylake: Clear stream registers before stream setup ASoC: Intel: Skylake: Fix to set pipe state to invalid when deleting ALSA: hdac: Add support for hda DMA Resume capability ALSA: hdac: couple the hda DMA stream in cleanup ASoC: Intel: Skylake: enable interrupt as wake source in active suspend ASoC: Intel: Skylake: Add DMA resume position in Trigger resume/suspend ASoC: Intel: Skylake: Reconfigure Link stream on suspend/resume ASoC: Intel: Skylake: Add Resume capability in PCM info. Vinod Koul (2): ALSA: hdac: add snd_hdac_ext_bus_link_power_up_all ASoC: Intel: Skylake: manage link power in active suspend include/sound/hda_register.h | 9 +++++ include/sound/hdaudio_ext.h | 15 +++++++ sound/hda/ext/hdac_ext_controller.c | 29 ++++++++++++- sound/hda/ext/hdac_ext_stream.c | 72 +++++++++++++++++++++++++++++++++ sound/soc/intel/skylake/skl-messages.c | 2 + sound/soc/intel/skylake/skl-pcm.c | 56 ++++++++++++++++++++----- sound/soc/intel/skylake/skl-sst-cldma.c | 38 ++++++++++------- sound/soc/intel/skylake/skl-sst-ipc.c | 15 +++++++ sound/soc/intel/skylake/skl-sst-ipc.h | 3 ++ sound/soc/intel/skylake/skl.c | 58 ++++++++++++++++++++++++-- sound/soc/intel/skylake/skl.h | 5 +++ 11 files changed, 272 insertions(+), 30 deletions(-) -- 1.9.1

1 13

Re: [alsa-devel] [PATCH v2 08/13] ASoC: Intel: Skylake: Use CGCTL.MISCBDCGE for Phrase detection notification
by Vinod Koul 14 Dec '15

14 Dec '15

On Sun, Dec 13, 2015 at 02:46:08AM +0800, kbuild test robot wrote: > Hi Dharageswari.R, > > [auto build test ERROR on asoc/for-next] > [also build test ERROR on next-20151211] > [cannot apply to v4.4-rc4] > > url: https://github.com/0day-ci/linux/commits/Vinod-Koul/ASoC-Intel-Skylake-Upda… > base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next > config: i386-allmodconfig (attached as .config) > reproduce: > # save the attached .config to linux build tree > make ARCH=i386 > > All errors (new ones prefixed by >>): > > >> ERROR: "skl_enable_miscbdcge" undefined! I will update this one, thanks for reporting -- ~Vinod

1 0

[alsa-devel] [PATCH] ASoC: es8328: Fix shifts for mixer switches
by John Keeping 13 Dec '15

13 Dec '15

These are all off by one; the playback and bypass switches are the top two bits of the registers, which are at shifts 7 and 6 not 8 and 7. Signed-off-by: John Keeping <john(a)metanate.com> --- sound/soc/codecs/es8328.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/sound/soc/codecs/es8328.c b/sound/soc/codecs/es8328.c index 969e337..84f5eb0 100644 --- a/sound/soc/codecs/es8328.c +++ b/sound/soc/codecs/es8328.c @@ -205,18 +205,18 @@ static const struct snd_kcontrol_new es8328_right_line_controls = /* Left Mixer */ static const struct snd_kcontrol_new es8328_left_mixer_controls[] = { - SOC_DAPM_SINGLE("Playback Switch", ES8328_DACCONTROL17, 8, 1, 0), - SOC_DAPM_SINGLE("Left Bypass Switch", ES8328_DACCONTROL17, 7, 1, 0), - SOC_DAPM_SINGLE("Right Playback Switch", ES8328_DACCONTROL18, 8, 1, 0), - SOC_DAPM_SINGLE("Right Bypass Switch", ES8328_DACCONTROL18, 7, 1, 0), + SOC_DAPM_SINGLE("Playback Switch", ES8328_DACCONTROL17, 7, 1, 0), + SOC_DAPM_SINGLE("Left Bypass Switch", ES8328_DACCONTROL17, 6, 1, 0), + SOC_DAPM_SINGLE("Right Playback Switch", ES8328_DACCONTROL18, 7, 1, 0), + SOC_DAPM_SINGLE("Right Bypass Switch", ES8328_DACCONTROL18, 6, 1, 0), }; /* Right Mixer */ static const struct snd_kcontrol_new es8328_right_mixer_controls[] = { - SOC_DAPM_SINGLE("Left Playback Switch", ES8328_DACCONTROL19, 8, 1, 0), - SOC_DAPM_SINGLE("Left Bypass Switch", ES8328_DACCONTROL19, 7, 1, 0), - SOC_DAPM_SINGLE("Playback Switch", ES8328_DACCONTROL20, 8, 1, 0), - SOC_DAPM_SINGLE("Right Bypass Switch", ES8328_DACCONTROL20, 7, 1, 0), + SOC_DAPM_SINGLE("Left Playback Switch", ES8328_DACCONTROL19, 7, 1, 0), + SOC_DAPM_SINGLE("Left Bypass Switch", ES8328_DACCONTROL19, 6, 1, 0), + SOC_DAPM_SINGLE("Playback Switch", ES8328_DACCONTROL20, 7, 1, 0), + SOC_DAPM_SINGLE("Right Bypass Switch", ES8328_DACCONTROL20, 6, 1, 0), }; static const char * const es8328_pga_sel[] = { -- 2.6.3.462.gbe2c914

2 1

[alsa-devel] [PATCH] ASoC: wm8974: set cache type for regmap
by Mans Rullgard 13 Dec '15

13 Dec '15

Attempting to use this codec driver triggers a BUG() in regcache_sync() since no cache type is set. The register map of this device is fairly small and has few holes so a flat cache is suitable. Signed-off-by: Mans Rullgard <mans(a)mansr.com> --- sound/soc/codecs/wm8974.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/codecs/wm8974.c b/sound/soc/codecs/wm8974.c index 45ba828..c284c7b 100644 --- a/sound/soc/codecs/wm8974.c +++ b/sound/soc/codecs/wm8974.c @@ -574,6 +574,7 @@ static const struct regmap_config wm8974_regmap = { .max_register = WM8974_MONOMIX, .reg_defaults = wm8974_reg_defaults, .num_reg_defaults = ARRAY_SIZE(wm8974_reg_defaults), + .cache_type = REGCACHE_FLAT, }; static int wm8974_probe(struct snd_soc_codec *codec) -- 2.6.3

3 2

[alsa-devel] [PATCH] ASoC: rockchip: spdif: Set transmit data level to 16 samples
by Sjoerd Simons 13 Dec '15

13 Dec '15

Explicitly set the transmit data level on the transceiver to 16 samples rather then the default 0. This matches both the level set in the vendor kernel and the (seemingly very similar) i2s engine. This fixes audio glitches when playing back at 192k rate. At the same time, fix a trivial typo in the TDL mask definition Signed-off-by: Sjoerd Simons <sjoerd.simons(a)collabora.co.uk> --- sound/soc/rockchip/rockchip_spdif.c | 6 ++++-- sound/soc/rockchip/rockchip_spdif.h | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/sound/soc/rockchip/rockchip_spdif.c b/sound/soc/rockchip/rockchip_spdif.c index ac72ff5..5a806da 100644 --- a/sound/soc/rockchip/rockchip_spdif.c +++ b/sound/soc/rockchip/rockchip_spdif.c @@ -152,8 +152,10 @@ static int rk_spdif_trigger(struct snd_pcm_substream *substream, case SNDRV_PCM_TRIGGER_RESUME: case SNDRV_PCM_TRIGGER_PAUSE_RELEASE: ret = regmap_update_bits(spdif->regmap, SPDIF_DMACR, - SPDIF_DMACR_TDE_ENABLE, - SPDIF_DMACR_TDE_ENABLE); + SPDIF_DMACR_TDE_ENABLE | + SPDIF_DMACR_TDL_MASK, + SPDIF_DMACR_TDE_ENABLE | + SPDIF_DMACR_TDL(16)); if (ret != 0) return ret; diff --git a/sound/soc/rockchip/rockchip_spdif.h b/sound/soc/rockchip/rockchip_spdif.h index 921b409..3ef1277 100644 --- a/sound/soc/rockchip/rockchip_spdif.h +++ b/sound/soc/rockchip/rockchip_spdif.h @@ -42,7 +42,7 @@ #define SPDIF_DMACR_TDL_SHIFT 0 #define SPDIF_DMACR_TDL(x) ((x) << SPDIF_DMACR_TDL_SHIFT) -#define SPDIF_DMACR_TDL_MASK (0x1f << SDPIF_DMACR_TDL_SHIFT) +#define SPDIF_DMACR_TDL_MASK (0x1f << SPDIF_DMACR_TDL_SHIFT) /* * XFER -- 2.6.2

2 1

[alsa-devel] [PATCH V2 0/3] Add support for Texas Instruments pcm3168a codec
by Damien Horsley 13 Dec '15

13 Dec '15

From: "Damien.Horsley" <Damien.Horsley(a)imgtec.com> Add SOC_DOUBLE_STS macro Add driver and binding docuemnt for Texas Instruments pcm3168a codec Changes in V2: Addressed comments from Mark Brown Damien.Horsley (3): ASoC: Add SOC_DOUBLE_STS macro ASoC: pcm3168a: Add binding document for pcm3168a codec ASoC: pcm3168a: Add driver for pcm3168a codec .../devicetree/bindings/sound/ti,pcm3168a.txt | 48 ++ include/sound/soc.h | 8 + sound/soc/codecs/Kconfig | 17 + sound/soc/codecs/Makefile | 6 + sound/soc/codecs/pcm3168a-i2c.c | 66 ++ sound/soc/codecs/pcm3168a-spi.c | 65 ++ sound/soc/codecs/pcm3168a.c | 767 +++++++++++++++++++++ sound/soc/codecs/pcm3168a.h | 100 +++ 8 files changed, 1077 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/ti,pcm3168a.txt create mode 100644 sound/soc/codecs/pcm3168a-i2c.c create mode 100644 sound/soc/codecs/pcm3168a-spi.c create mode 100644 sound/soc/codecs/pcm3168a.c create mode 100644 sound/soc/codecs/pcm3168a.h -- 2.1.4

2 5