linux.git/sound, branch v3.12

ALSA: fix oops in snd_pcm_info() caused by ASoC DPCM

2013-10-31T16:36:47+00:00

Unable to handle kernel NULL pointer dereference at virtual address 00000008
pgd = d5300000
[00000008] *pgd=0d265831, *pte=00000000, *ppte=00000000
Internal error: Oops: 17 [#1] PREEMPT ARM
CPU: 0 PID: 2295 Comm: vlc Not tainted 3.11.0+ #755
task: dee74800 ti: e213c000 task.ti: e213c000
PC is at snd_pcm_info+0xc8/0xd8
LR is at 0x30232065
pc : []    lr : [<30232065>]    psr: a0070013
sp : e213dea8  ip : d81cb0d0  fp : c05f7678
r10: c05f7770  r9 : fffffdfd  r8 : 00000000
r7 : d8a968a8  r6 : d8a96800  r5 : d8a96200  r4 : d81cb000
r3 : 00000000  r2 : d81cb000  r1 : 00000001  r0 : d8a96200
Flags: NzCv  IRQs on  FIQs on  Mode SVC_32  ISA ARM  Segment user
Control: 10c5387d  Table: 15300019  DAC: 00000015
Process vlc (pid: 2295, stack limit = 0xe213c248)
[] (snd_pcm_info) from [] (snd_pcm_info_user+0x34/0x9c)
[] (snd_pcm_info_user) from [] (snd_pcm_control_ioctl+0x274/0x280)
[] (snd_pcm_control_ioctl) from [] (snd_ctl_ioctl+0xc0/0x55c)
[] (snd_ctl_ioctl) from [] (do_vfs_ioctl+0x80/0x31c)
[] (do_vfs_ioctl) from [] (SyS_ioctl+0x3c/0x60)
[] (SyS_ioctl) from [] (ret_fast_syscall+0x0/0x48)
Code: e1a00005 e59530dc e3a01001 e1a02004 (e5933008)
---[ end trace cb3d9bdb8dfefb3c ]---

This is provoked when the ASoC front end is open along with its backend,
(which causes the backend to have a runtime assigned to it) and then the
SNDRV_CTL_IOCTL_PCM_INFO is requested for the (visible) backend device.

Resolve this by ensuring that ASoC internal backend devices are not
visible to userspace, just as the commentry for snd_pcm_new_internal()
says it should be.

Signed-off-by: Russell King 
Acked-by: Mark Brown 
Cc:  [v3.4+]
Signed-off-by: Takashi Iwai

Merge tag 'asoc-fix-v3.12-rc7' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound into for-linus

2013-10-30T17:42:13+00:00

ASoC: Fixes for v3.12

A few of the Coverity fixes from Takashi, one of which (the wm_hubs one)
is particularly noticable.

Merge remote-tracking branch 'asoc/fix/wm8994' into asoc-linus

2013-10-30T17:11:55+00:00

ASoC: wm_hubs: Add missing break in hp_supply_event()

2013-10-30T16:35:22+00:00

Spotted by coverity CID 115170.

Signed-off-by: Takashi Iwai 
Signed-off-by: Mark Brown 
Cc: stable@vger.kernel.org

ALSA: hda - Add a fixup for ASUS N76VZ

2013-10-30T11:31:35+00:00

ASUS N76VZ needs the same fixup as N56VZ for supporting the boost
speaker.

Bugzilla: https://bugzilla.novell.com/show_bug.cgi?id=846529
Cc: 
Signed-off-by: Takashi Iwai

ASoC: dapm: Return -ENOMEM in snd_soc_dapm_new_dai_widgets()

2013-10-28T16:33:10+00:00

... instead of NULL dereferences.

Spotted by coverity CID 402004.

Signed-off-by: Takashi Iwai 
Signed-off-by: Mark Brown

ASoC: dapm: Fix source list debugfs outputs

2013-10-28T16:32:57+00:00

... due to a copy & paste error.

Spotted by coverity CID 710923.

Signed-off-by: Takashi Iwai 
Signed-off-by: Mark Brown 
Cc: stable@vger.kernel.org

ALSA: hda - Fix silent headphone on Thinkpads with AD1984A codec

2013-10-25T22:30:32+00:00

AD1984A codec has a couple of pins with EAPD controls, and the generic
codec driver tries to turn each of them on/off depending on the pin
active state.  However, Thinkpads seem to use EAPD of the speaker pin
as a master EAPD for controlling the mute of all outputs, including
the headphone.  This results in the dead headphone output via the
headphone plugging because it mutes the speaker and turns off EAPD.

The fix is to simply add spec->gen.keep_on_eapd flag.

[This is a regression fix on 3.12 where we moved the AD codec parser
 to the generic parser.  3.11 and earlier didn't show this problem
 because still static quirks have been used.]

Reported-and-tested-by: Vito Caputo 
Signed-off-by: Takashi Iwai

ALSA: hda - Add missing initial vmaster hook at build_controls callback

2013-10-25T21:43:10+00:00

The generic parser has a support of vmaster hook, but this is
initialized only in the init callback with the check of the presence
of the corresponding kctl.  However, since kctl is NULL at the very
first init callback that is called before build_controls callback, the
vmaster hook sync is skipped there.  Eventually this leads to the
uninitialized state depending on the hook implementation.

This patch adds a simple workaround, just calling the sync function
explicitly at build_controls callback.

Cc: 
Signed-off-by: Takashi Iwai

ALSA: hda - Fix unbalanced runtime PM refcount after S3/S4

2013-10-24T07:21:45+00:00

When a machine goes to S3/S4 after power-save is enabled, the runtime
PM refcount might be incorrectly decreased because the power-down
triggered soon after resume assumes that the controller was already
powered up, and issues the pm_notify down.

This patch fixes the incorrect pm_notify call simply by checking the
current value properly.

Cc: 
Signed-off-by: Takashi Iwai