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

[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

[alsa-devel] ALSA/ASOC new device and routing
by Eli Britstein 30 Nov '14

30 Nov '14

Hi I'm a newbie to ALSA. I'm targeting to use ALSA or ASoC solution in order to route capture/playback streams of a USB headset (that uses ALSA) to/from my own DSP device. I saw DPCM which is a part of ASoC manages dynamically connections between FE and BE streams. In my product, I just need them to be connected. No need for the "dynamically" part. My questions: 1. Can someone please recommend a step by step guide, or an example how to make my DSP "look like" a BE? Do I need to write an ALSA driver for it or a ASOC driver? 2. After I have it as a BE, how can I connect the streams? I want the transfers to be done in kernel level, without need for application interventions for each buffer. 3. I'm using kernel 2.6.10 (quite old, I know). I don't want to port my entire kernel to a new one. As my requirement is not very complex, I don't think I'd encounter too many issues. However, if someone can point the important parts I need to back port, I'd appreciate it. Alternatively, if someone sees a showstopper in this approach, I'd be grateful to be aware of it. Thanks, Best regards, Eli ________________________________ This email and any files transmitted with it are confidential material. They are intended solely for the use of the designated individual or entity to whom they are addressed. If the reader of this message is not the intended recipient, you are hereby notified that any dissemination, use, distribution or copying of this communication is strictly prohibited and may be unlawful. If you have received this email in error please immediately notify the sender and delete or destroy any copy of this message

2 4

[alsa-devel] AKAI EIE PRO
by Loki Harfagr 28 Nov '14

28 Nov '14

(hope that's correct to post that way through email since I'm just adding an answer to a specific question in a specific thread), here we go: > Daniel Mack daniel at zonque.org > Sun Sep 7 13:39:12 CEST 2014 > Previous message: [alsa-devel] AKAI EIE PRO > Next message: [alsa-devel] AKAI EIE PRO > Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] > On 09/07/2014 01:30 PM, humbert.olivier.1 at free.fr wrote: > > if that can help, we had an user on http://www.linuxmao.org with an EIE PRO. > > Here you have his lsusb -v : > > > > Bus 001 Device 006: ID 09e8:0010 AKAI Professional M.I. Corp. > > Couldn't open device, some information will be missing > > Device Descriptor: > > bLength 18 > > bDescriptorType 1 > > bcdUSB 2.00 > > bDeviceClass 255 Vendor Specific Class > > bDeviceSubClass 255 Vendor Specific Subclass > > bDeviceProtocol 255 Vendor Specific Protocol > > According to the tech specs, this should be a USB hub, but it isn't. Or > is this just one of the sub-devices? What does 'lsusb -t' say, and does > Linux see something like a mouse or mass storage device when connected > to one of the hub ports? To test accordingly to the question a USB device (Webcam C905) is plugged in one of the EIE Pro. hub ports 1) AKAI EIE Pro *OFF*: ------------------- $ lsusb Bus 002 Device 002: ID 152d:0551 JMicron Technology Corp. / JMicron USA Technology Corp. Bus 005 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 006 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 005 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 006 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 005 Device 003: ID 05e3:0608 Genesys Logic, Inc. USB-2.0 4-Port HUB Bus 005 Device 004: ID 174c:5106 ASMedia Technology Inc. Transcend StoreJet 25M3 Bus 006 Device 003: ID 058f:6362 Alcor Micro Corp. Flash Card Reader/Writer Bus 005 Device 006: ID 046d:c318 Logitech, Inc. Illuminated Keyboard Bus 005 Device 007: ID 047d:1020 Kensington Expert Mouse Trackball Bus 005 Device 012: ID 058f:9254 Alcor Micro Corp. Hub ------------------- 2) AKAI EIE Pro *ON*: ------------------- $ lsusb Bus 002 Device 002: ID 152d:0551 JMicron Technology Corp. / JMicron USA Technology Corp. Bus 005 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub ven. nov. 28 19:27:58 CET 2014 Bus 006 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 005 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 006 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 005 Device 003: ID 05e3:0608 Genesys Logic, Inc. USB-2.0 4-Port HUB Bus 005 Device 004: ID 174c:5106 ASMedia Technology Inc. Transcend StoreJet 25M3 Bus 005 Device 018: ID 0409:005a NEC Corp. HighSpeed Hub Bus 006 Device 003: ID 058f:6362 Alcor Micro Corp. Flash Card Reader/Writer Bus 005 Device 006: ID 046d:c318 Logitech, Inc. Illuminated Keyboard Bus 005 Device 007: ID 047d:1020 Kensington Expert Mouse Trackball Bus 005 Device 012: ID 058f:9254 Alcor Micro Corp. Hub Bus 005 Device 019: ID 046d:080a Logitech, Inc. Portable Webcam C905 Bus 005 Device 020: ID 09e8:0010 AKAI Professional M.I. Corp. ------------------- ------------------- $ lsusb -t /: Bus 06.Port 1: Dev 1, Class=root_hub, Driver=ehci-pci/2p, 480M |__ Port 1: Dev 2, If 0, Class=Hub, Driver=hub/8p, 480M |__ Port 7: Dev 3, If 0, Class=Mass Storage, Driver=usb-storage, 480M /: Bus 05.Port 1: Dev 1, Class=root_hub, Driver=ehci-pci/2p, 480M |__ Port 1: Dev 2, If 0, Class=Hub, Driver=hub/6p, 480M |__ Port 1: Dev 3, If 0, Class=Hub, Driver=hub/4p, 480M |__ Port 1: Dev 6, If 0, Class=Human Interface Device, Driver=usbhid, 12M |__ Port 1: Dev 6, If 1, Class=Human Interface Device, Driver=usbhid, 12M |__ Port 2: Dev 7, If 0, Class=Human Interface Device, Driver=usbhid, 1.5M |__ Port 3: Dev 12, If 0, Class=Hub, Driver=hub/4p, 12M |__ Port 2: Dev 4, If 0, Class=Mass Storage, Driver=usb-storage, 480M |__ Port 4: Dev 18, If 0, Class=Hub, Driver=hub/4p, 480M |__ Port 1: Dev 19, If 0, Class=Video, Driver=uvcvideo, 480M |__ Port 1: Dev 19, If 1, Class=Video, Driver=uvcvideo, 480M |__ Port 1: Dev 19, If 2, Class=Audio, Driver=snd-usb-audio, 480M |__ Port 1: Dev 19, If 3, Class=Audio, Driver=snd-usb-audio, 480M |__ Port 4: Dev 20, If 0, Class=Vendor Specific Class, Driver=, 480M |__ Port 4: Dev 20, If 1, Class=Vendor Specific Class, Driver=, 480M /: Bus 04.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 5000M /: Bus 03.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 480M /: Bus 02.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 5000M |__ Port 2: Dev 2, If 0, Class=Mass Storage, Driver=usb-storage, 5000M /: Bus 01.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 480M ------------------- 3) Then, the Webcam plugged in the EIE Pro. has been unplugged: ------------------- $ lsusb -t /: Bus 06.Port 1: Dev 1, Class=root_hub, Driver=ehci-pci/2p, 480M |__ Port 1: Dev 2, If 0, Class=Hub, Driver=hub/8p, 480M |__ Port 7: Dev 3, If 0, Class=Mass Storage, Driver=usb-storage, 480M /: Bus 05.Port 1: Dev 1, Class=root_hub, Driver=ehci-pci/2p, 480M |__ Port 1: Dev 2, If 0, Class=Hub, Driver=hub/6p, 480M |__ Port 1: Dev 3, If 0, Class=Hub, Driver=hub/4p, 480M |__ Port 1: Dev 6, If 0, Class=Human Interface Device, Driver=usbhid, 12M |__ Port 1: Dev 6, If 1, Class=Human Interface Device, Driver=usbhid, 12M |__ Port 2: Dev 7, If 0, Class=Human Interface Device, Driver=usbhid, 1.5M |__ Port 3: Dev 12, If 0, Class=Hub, Driver=hub/4p, 12M |__ Port 2: Dev 4, If 0, Class=Mass Storage, Driver=usb-storage, 480M |__ Port 4: Dev 18, If 0, Class=Hub, Driver=hub/4p, 480M |__ Port 4: Dev 20, If 0, Class=Vendor Specific Class, Driver=, 480M |__ Port 4: Dev 20, If 1, Class=Vendor Specific Class, Driver=, 480M /: Bus 04.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 5000M /: Bus 03.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 480M /: Bus 02.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 5000M |__ Port 2: Dev 2, If 0, Class=Mass Storage, Driver=usb-storage, 5000M /: Bus 01.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/2p, 480M ------------------- ------------------- $ lsusb Bus 002 Device 002: ID 152d:0551 JMicron Technology Corp. / JMicron USA Technology Corp. Bus 005 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 006 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 004 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 005 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 006 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub Bus 005 Device 003: ID 05e3:0608 Genesys Logic, Inc. USB-2.0 4-Port HUB Bus 005 Device 004: ID 174c:5106 ASMedia Technology Inc. Transcend StoreJet 25M3 Bus 005 Device 018: ID 0409:005a NEC Corp. HighSpeed Hub Bus 006 Device 003: ID 058f:6362 Alcor Micro Corp. Flash Card Reader/Writer Bus 005 Device 006: ID 046d:c318 Logitech, Inc. Illuminated Keyboard Bus 005 Device 007: ID 047d:1020 Kensington Expert Mouse Trackball Bus 005 Device 012: ID 058f:9254 Alcor Micro Corp. Hub Bus 005 Device 020: ID 09e8:0010 AKAI Professional M.I. Corp. ------------------- Sincerely hoping it'll help to include full support in ALSA for the AKAI EIE Pro as it's very good stuff, nicely built and convenient (for its hub) to USB ports eaters as we're all in the music phylum :-) Cheers! -- < °/_ /| ¹

1 0

[alsa-devel] [PATCH v3 0/2] usb: Add native DSD support for Denon/Marantz
by Jurgen Kramer 28 Nov '14

28 Nov '14

This patchset adds native DSD support for Denon/Marantz devices. A specific vendor command is needed to switch between PCM and DSD mode. This is implemented as a new quirk function. Changes from v2: - switch) -> switch ( - combine PCM and DSD mode select to a single USB ctrl msg Changes from v1: - fix compare in err check - remove unneeded brackets Jurgen Kramer (2): usb: Add native DSD support for Denon/Marantz DACs usb: Add mode select quirk for Denon/Marantz DACs sound/usb/pcm.c | 5 +++++ sound/usb/quirks.c | 47 +++++++++++++++++++++++++++++++++++++++++++++++ sound/usb/quirks.h | 3 +++ 3 files changed, 55 insertions(+) -- 1.9.3

2 3

[alsa-devel] [PATCH v2 0/2] usb: add native DSD support for Denon/Marantz
by Jurgen Kramer 28 Nov '14

28 Nov '14

This patchset adds native DSD support for Denon/Marantz devices. A specific vendor command is needed to switch between PCM and DSD mode. This is implemented as a new quirk function. Changes from v1: - fix compare in err check - remove unneeded brackets Jurgen Kramer (2): usb: Add native DSD support for Denon/Marantz DACs usb: Add mode select quirk for Denon/Marantz DACs sound/usb/pcm.c | 5 +++++ sound/usb/quirks.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++++ sound/usb/quirks.h | 3 +++ 3 files changed, 61 insertions(+) -- 1.9.3

2 4

[alsa-devel] [PATCH] ASoC: Intel: Move capture PCM pin to PCM0 for Broadwell
by Jie Yang 28 Nov '14

28 Nov '14

Move capture PCM pin from PCM4 to PCM0 for Broadwell. 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/sst-haswell-pcm.c | 34 ++++++++++++++++------------------ 2 files changed, 18 insertions(+), 31 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/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 3

[alsa-devel] [PATCH 0/2] usb: add native DSD support for Denon/Marantz
by Jurgen Kramer 28 Nov '14

28 Nov '14

This patchset adds native DSD support for Denon/Marantz devices. A specific vendor command is needed to switch between PCM and DSD mode. This is implemented as a new quirk function. Jurgen Kramer (2): Add native DSD support for Denon/Marantz DACs Add mode select quirk for Denon/Marantz DACs sound/usb/pcm.c | 6 ++++++ sound/usb/quirks.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++++ sound/usb/quirks.h | 3 +++ 3 files changed, 62 insertions(+) -- 1.9.3

3 5

[alsa-devel] [PATCH] ASoC: Intel: Correct the xmax volume
by Jie Yang 28 Nov '14

28 Nov '14

The xmax volume should be corrected to ARRAY_SIZE(volume_map)-1, otherwise, the xmax value will be mapped to 0 wrongly. Signed-off-by: Jie Yang <yang.jie(a)intel.com> --- sound/soc/intel/sst-haswell-pcm.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sound/soc/intel/sst-haswell-pcm.c b/sound/soc/intel/sst-haswell-pcm.c index 20561ca..5f70d81 100644 --- a/sound/soc/intel/sst-haswell-pcm.c +++ b/sound/soc/intel/sst-haswell-pcm.c @@ -298,19 +298,19 @@ static const DECLARE_TLV_DB_SCALE(hsw_vol_tlv, -9000, 300, 1); static const struct snd_kcontrol_new hsw_volume_controls[] = { /* Global DSP volume */ SOC_DOUBLE_EXT_TLV("Master Playback Volume", 0, 0, 8, - ARRAY_SIZE(volume_map) -1, 0, + ARRAY_SIZE(volume_map) - 1, 0, hsw_volume_get, hsw_volume_put, hsw_vol_tlv), /* Offload 0 volume */ SOC_DOUBLE_EXT_TLV("Media0 Playback Volume", 1, 0, 8, - ARRAY_SIZE(volume_map), 0, + ARRAY_SIZE(volume_map) - 1, 0, hsw_stream_volume_get, hsw_stream_volume_put, hsw_vol_tlv), /* Offload 1 volume */ SOC_DOUBLE_EXT_TLV("Media1 Playback Volume", 2, 0, 8, - ARRAY_SIZE(volume_map), 0, + 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, - ARRAY_SIZE(volume_map), 0, + ARRAY_SIZE(volume_map) - 1, 0, hsw_stream_volume_get, hsw_stream_volume_put, hsw_vol_tlv), }; -- 1.9.1

2 1

[alsa-devel] [PATCH 1/3] ASoC: mxs-sgtl5000: Remove MCLK restriction
by Fabio Estevam 27 Nov '14

27 Nov '14

From: Fabio Estevam <fabio.estevam(a)freescale.com> According to the sgtl5000 datasheet the MCLK frequency range restriction of 8 to 27 MHz only applies when the PLL is used - synchronous SYS_MCLK input mode. mxs-sgtl5000 machine sets the codec as slave, and mx28 generates MCLK in the range of 256*fs, 384*fs or 512*fs, which is called asynchronous SYS_MCLK input. In asynchronous SYS_MCLK we cannot have the 8 to 27 MHz check because if we want to play a 8KHz sample rate track, with a MCLK of 8k * 512 = 4.096MHz the current check would return -EINVAL, which is not correct. Remove the 8 to 27MHz frequency check, since this only applies to the synchronous SYS_MCLK input case. Signed-off-by: Fabio Estevam <fabio.estevam(a)freescale.com> --- sound/soc/mxs/mxs-sgtl5000.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/sound/soc/mxs/mxs-sgtl5000.c b/sound/soc/mxs/mxs-sgtl5000.c index 7515cd4..6f1916b 100644 --- a/sound/soc/mxs/mxs-sgtl5000.c +++ b/sound/soc/mxs/mxs-sgtl5000.c @@ -49,13 +49,6 @@ static int mxs_sgtl5000_hw_params(struct snd_pcm_substream *substream, break; } - /* Sgtl5000 sysclk should be >= 8MHz and <= 27M */ - if (mclk < 8000000 || mclk > 27000000) { - dev_err(codec_dai->dev, "Invalid mclk frequency: %u.%03uMHz\n", - mclk / 1000000, mclk / 1000 % 1000); - return -EINVAL; - } - /* Set SGTL5000's SYSCLK (provided by SAIF MCLK) */ ret = snd_soc_dai_set_sysclk(codec_dai, SGTL5000_SYSCLK, mclk, 0); if (ret) { -- 1.9.1

2 3

[alsa-devel] [PATCH 01/11] ASoC: cq93vc: Remove unused state struct
by Lars-Peter Clausen 27 Nov '14

27 Nov '14

While two of the fields in the cq93vc driver state struct are initialized none of them are ever acutally read again. So remove the whole struct. Signed-off-by: Lars-Peter Clausen <lars(a)metafoo.de> --- include/linux/mfd/davinci_voicecodec.h | 7 ------- sound/soc/codecs/cq93vc.c | 8 -------- 2 files changed, 15 deletions(-) diff --git a/include/linux/mfd/davinci_voicecodec.h b/include/linux/mfd/davinci_voicecodec.h index cb01496..8e1cdbe 100644 --- a/include/linux/mfd/davinci_voicecodec.h +++ b/include/linux/mfd/davinci_voicecodec.h @@ -99,12 +99,6 @@ struct davinci_vcif { dma_addr_t dma_rx_addr; }; -struct cq93vc { - struct platform_device *pdev; - struct snd_soc_codec *codec; - u32 sysclk; -}; - struct davinci_vc; struct davinci_vc { @@ -122,7 +116,6 @@ struct davinci_vc { /* Client devices */ struct davinci_vcif davinci_vcif; - struct cq93vc cq93vc; }; #endif diff --git a/sound/soc/codecs/cq93vc.c b/sound/soc/codecs/cq93vc.c index 537327c..036a877 100644 --- a/sound/soc/codecs/cq93vc.c +++ b/sound/soc/codecs/cq93vc.c @@ -62,14 +62,10 @@ static int cq93vc_mute(struct snd_soc_dai *dai, int mute) static int cq93vc_set_dai_sysclk(struct snd_soc_dai *codec_dai, int clk_id, unsigned int freq, int dir) { - struct snd_soc_codec *codec = codec_dai->codec; - struct davinci_vc *davinci_vc = codec->dev->platform_data; - switch (freq) { case 22579200: case 27000000: case 33868800: - davinci_vc->cq93vc.sysclk = freq; return 0; } @@ -135,10 +131,6 @@ static int cq93vc_resume(struct snd_soc_codec *codec) static int cq93vc_probe(struct snd_soc_codec *codec) { - struct davinci_vc *davinci_vc = codec->dev->platform_data; - - davinci_vc->cq93vc.codec = codec; - /* Off, with power on */ cq93vc_set_bias_level(codec, SND_SOC_BIAS_STANDBY); -- 1.8.0

3 22