April 2019 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH v2] ASoC: Mediatek: MT8183: Fix build error in mt8183_da7219_max98357_dev_probe
by Yue Haibing 03 Apr '19

03 Apr '19

From: YueHaibing <yuehaibing(a)huawei.com> When building CONFIG_SND_SOC_MT8183_DA7219_MAX98357A=m gcc warn this: sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c: In function mt8183_da7219_max98357_dev_probe: sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c:413:13: error: struct snd_soc_dai_link has no member named platform; did you mean platforms? dai_link->platform = NULL; ^~~~~~~~ platforms use 'dai_link->platforms' instead of 'dai_link->platform'. Fixes: 11c0269017b2 ("ASoC: Mediatek: MT8183: Add machine driver with TS3A227") Signed-off-by: YueHaibing <yuehaibing(a)huawei.com> --- v2: use correct build log --- sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c b/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c index 9353665..1e7e8ae 100644 --- a/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c +++ b/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c @@ -410,7 +410,7 @@ static int mt8183_da7219_max98357_dev_probe(struct platform_device *pdev) * the "platform" will not null when probe is trying * again. It's not expected normally. */ - dai_link->platform = NULL; + dai_link->platforms = NULL; if (dai_link->platform_name) continue; -- 2.7.0

1 0

[alsa-devel] [PATCH] ASoC: Mediatek: MT8183: Fix build error in mt8183_da7219_max98357_dev_probe
by Yue Haibing 03 Apr '19

03 Apr '19

From: YueHaibing <yuehaibing(a)huawei.com> When building CONFIG_SND_SOC_MT8183_DA7219_MAX98357A=m gcc warn this: sound/soc/mediatek/mt8183/mt8183-mt6358-ts3a227-max98357.c: In function mt8183_mt6358_ts3a227_max98357_dev_probe: sound/soc/mediatek/mt8183/mt8183-mt6358-ts3a227-max98357.c:325:13: error: struct snd_soc_dai_link has no member named platform; did you mean platforms? dai_link->platform = NULL; ^~~~~~~~ platforms use 'dai_link->platforms' instead of 'dai_link->platform'. Fixes: 11c0269017b2 ("ASoC: Mediatek: MT8183: Add machine driver with TS3A227") Signed-off-by: YueHaibing <yuehaibing(a)huawei.com> --- sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c b/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c index 9353665..1e7e8ae 100644 --- a/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c +++ b/sound/soc/mediatek/mt8183/mt8183-da7219-max98357.c @@ -410,7 +410,7 @@ static int mt8183_da7219_max98357_dev_probe(struct platform_device *pdev) * the "platform" will not null when probe is trying * again. It's not expected normally. */ - dai_link->platform = NULL; + dai_link->platforms = NULL; if (dai_link->platform_name) continue; -- 2.7.0

2 1

[alsa-devel] [PATCH] ASoC: soc-core: Fix probe deferral following prelink failure
by Jon Hunter 03 Apr '19

03 Apr '19

From: Jonathan Hunter <jonathanh(a)nvidia.com> Commit 78a24e10cd94 ("ASoC: soc-core: clear platform pointers on error") re-worked the clean-up of any platform pointers that may have been initialised by the function snd_soc_init_platform(). This commit missed one error path where if any of the prelinks for a soundcard failed to initialise, then these platform pointers would not be cleaned-up. This then prevents the soundcard from being initialised following a probe deferral when any of the soundcard prelinks cannot be found. Fix this by ensuring that soc_cleanup_platform() is called when initialising the soundcard prelinks fails. Fixes: 78a24e10cd94 ("ASoC: soc-core: clear platform pointers on error") Signed-off-by: Jonathan Hunter <jonathanh(a)nvidia.com> --- sound/soc/soc-core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/soc-core.c b/sound/soc/soc-core.c index 93d316d5bf8e..5a5764dba147 100644 --- a/sound/soc/soc-core.c +++ b/sound/soc/soc-core.c @@ -2797,6 +2797,7 @@ int snd_soc_register_card(struct snd_soc_card *card) ret = soc_init_dai_link(card, link); if (ret) { + soc_cleanup_platform(card); dev_err(card->dev, "ASoC: failed to init link %s\n", link->name); mutex_unlock(&client_mutex); -- 1.9.1

2 9

[alsa-devel] [asoc:for-5.2 105/107] ERROR: "sprd_platform_compr_ops" [sound/soc/sprd/sprd-pcm-dma.ko] undefined!
by kbuild test robot 03 Apr '19

03 Apr '19

Hi Wen, First bad commit (maybe != root cause): tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-5.2 head: 6d5e2bf9d203e4d9e08ca2e9420c6ff22ad190af commit: c705247136a523488eac806bd357c3e5d79a7acd [105/107] ASoC: fsl_utils: fix a leaked reference by adding missing of_node_put config: mips-allmodconfig (attached as .config) compiler: mips-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout c705247136a523488eac806bd357c3e5d79a7acd # save the attached .config to linux build tree GCC_VERSION=7.2.0 make.cross ARCH=mips All errors (new ones prefixed by >>): >> ERROR: "sprd_platform_compr_ops" [sound/soc/sprd/sprd-pcm-dma.ko] undefined! --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

3 2

[alsa-devel] [PATCH v15 0/6] Media Device Allocator API
by Shuah Khan 02 Apr '19

02 Apr '19

Media Device Allocator API to allows multiple drivers share a media device. This API solves a very common use-case for media devices where one physical device (an USB stick) provides both audio and video. When such media device exposes a standard USB Audio class, a proprietary Video class, two or more independent drivers will share a single physical USB bridge. In such cases, it is necessary to coordinate access to the shared resource. Using this API, drivers can allocate a media device with the shared struct device as the key. Once the media device is allocated by a driver, other drivers can get a reference to it. The media device is released when all the references are released. The primary focus for testing The patch series is making sure media device is released when both drivers release the media device with a series of unbind/binds on both drivers. - both au0828 and snd-usb-aduio as built-in - both au0828 and snd-usb-aduio as modules - au0828 as module and snd-usb-aduio as built-in - au0828 as built-in and snd-usb-aduio as module Test results can be found at: https://docs.google.com/document/d/1RMF8Rwj7xHJEoOx6_K2f-REgZJ63BMeAVEf-CV-… Changes since v14: - Fixed typos and Copyright date. - Made VBI shareable with audio and video. - Tested with Hans's patch to to fix second VBi stream stepping on active VBI stream. The patch didn't fix the problem. 1. Start qv4l2 -V /dev/vbi0 Start capture - Streaming active 2. Start qv4l2 -V /dev/vbi0 Start capture - Streaming active Streaming won't start and the first stream stops. There is still a problem in au0828. I am sending the patch series with audio, video, and VBI shareable with this known issue. Changes since v13: - Minor changes to variable names and other minor changes to copyright and typos suggested by Hans Verkuil. Changes since v12: - Patch 1: Fixed prototype warns from media_dev_allocator.c. Removed dependency on find_module() by adding struct module to input args. Still need module name to pass into media_device API. - Patch 2 & 4: Update media dev allocator api calls to pass in struct module pointer. - No changes to Patches 3 & 5. - Added patch 6 with a test. It can go in separately. Changes since v11: - Patch 1: Add CONFIG_MODULES dependency in media_dev_allocator files. to fix compile errors when CONFIG_MODULES is disabled. - Patch 2, 3: No changes. - Patch 4: Fix sparse error reported by Dan Carpenter. - Patch 5: Fix warns found by Hans Verkuil. - v11 was tested on 5.0-rc7 and addresses comments on v10 series from Hans Verkuil. Fixed problems found in resource sharing logic in au0828 adding a patch 5 to this series. The test plan used for testing resource sharing could serve as a regression test plan and the test results can be found at: - v10 was tested on 5.0-rc3 and addresses comments on v9 series from Hans Verkuil. Changes since v10: - Patch 1: Fixed SPDX tag and removed redundant IS_ENABLED(CONFIG_USB) around media_device_usb_allocate() - Sakari Ailus's review comment. - Patch 2 and 3: No changes - Patch 4: Fixed SPDX tag - Sakari Ailus's review comment. - Carried Reviewed-by tag from Takashi Iwai for the sound from v9. - Patch 5: This is a new patch added to fix resource sharing inconsistencies and problem found during testing using Han's tests. Changes since v9: - Patch 1: Fix mutex assert warning from find_module() calls. This code was written before the change to find_module() that requires callers to hold module_mutex. I missed this during my testing on 4.20-rc6. Hans Verkuil reported the problem. - Patch 4: sound/usb: Initializes all the entities it can before registering the device based on comments from Hans Verkuil - Carried Reviewed-by tag from Takashi Iwai for the sound from v9. - No changes to Patches 2 and 3. References: https://lkml.org/lkml/2018/11/2/169 https://www.mail-archive.com/linux-media@vger.kernel.org/msg105854.html Shuah Khan (6): media: Media Device Allocator API media: change au0828 to use Media Device Allocator API media: media.h: Enable ALSA MEDIA_INTF_T* interface types sound/usb: Use Media Controller API to share media resources au0828: fix enable and disable source audio and video inconsistencies selftests: media_dev_allocator api test Documentation/media/kapi/mc-core.rst | 41 +++ drivers/media/Makefile | 6 + drivers/media/media-dev-allocator.c | 135 ++++++++ drivers/media/usb/au0828/Kconfig | 2 + drivers/media/usb/au0828/au0828-core.c | 196 ++++++++--- drivers/media/usb/au0828/au0828.h | 6 +- include/media/media-dev-allocator.h | 63 ++++ include/uapi/linux/media.h | 25 +- sound/usb/Kconfig | 4 + sound/usb/Makefile | 2 + sound/usb/card.c | 14 + sound/usb/card.h | 3 + sound/usb/media.c | 327 ++++++++++++++++++ sound/usb/media.h | 74 ++++ sound/usb/mixer.h | 3 + sound/usb/pcm.c | 29 +- sound/usb/quirks-table.h | 1 + sound/usb/stream.c | 2 + sound/usb/usbaudio.h | 6 + .../media_tests/media_dev_allocator.sh | 85 +++++ 20 files changed, 963 insertions(+), 61 deletions(-) create mode 100644 drivers/media/media-dev-allocator.c create mode 100644 include/media/media-dev-allocator.h create mode 100644 sound/usb/media.c create mode 100644 sound/usb/media.h create mode 100755 tools/testing/selftests/media_tests/media_dev_allocator.sh -- 2.17.1

3 8

Re: [alsa-devel] WARNING in port_delete
by syzbot 02 Apr '19

02 Apr '19

syzbot has bisected this bug to: commit 63d86a7e85f84b8ac3b2f394570965aedbb03787 Author: Paul E. McKenney <paulmck(a)linux.vnet.ibm.com> Date: Tue May 1 20:08:46 2018 +0000 rcu: Convert rcu_grace_period_init tracepoint to gp_seq bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=17f86d2f200000 start commit: 091a1eaa Merge branch 'akpm' git tree: upstream final crash: https://syzkaller.appspot.com/x/report.txt?x=14046d2f200000 console output: https://syzkaller.appspot.com/x/log.txt?x=10046d2f200000 kernel config: https://syzkaller.appspot.com/x/.config?x=c0af03fe452b65fb dashboard link: https://syzkaller.appspot.com/bug?extid=e4c8abb920efa77bace9 syz repro: https://syzkaller.appspot.com/x/repro.syz?x=102e2c91400000 Reported-by: syzbot+e4c8abb920efa77bace9(a)syzkaller.appspotmail.com Fixes: 63d86a7e85f8 ("rcu: Convert rcu_grace_period_init tracepoint to gp_seq") For information about bisection process see: https://goo.gl/tpsmEJ#bisection

3 2

[alsa-devel] [PATCH v3 00/10] ACPI / utils: Replace leaky function
by Andy Shevchenko 02 Apr '19

02 Apr '19

The acpi_dev_get_first_match_name() is missing put_device() call and thus keeping reference counting unbalanced. In order to fix the issue introduce a new helper to convert existing users one-by-one to a better API. Since v3: - convert all existing users and drop old API (Mika) - add Hans' tag Cc: MyungJoo Ham <myungjoo.ham(a)samsung.com> Cc: Chanwoo Choi <cw00.choi(a)samsung.com> Cc: Chen-Yu Tsai <wens(a)csie.org> Cc: Linus Walleij <linus.walleij(a)linaro.org> Cc: Bartosz Golaszewski <bgolaszewski(a)baylibre.com> Andy Shevchenko (10): ACPI / utils: Introduce acpi_dev_get_first_match_dev() helper extcon: axp288: Convert to use acpi_dev_get_first_match_dev() gpio: merrifield: Convert to use acpi_dev_get_first_match_dev() ASoC: Intel: bytcht_da7213: Convert to use acpi_dev_get_first_match_dev() ASoC: Intel: bytcht_es8316: Convert to use acpi_dev_get_first_match_dev() ASoC: Intel: bytcr_rt5640: Convert to use acpi_dev_get_first_match_dev() ASoC: Intel: bytcr_rt5651: Convert to use acpi_dev_get_first_match_dev() ASoC: Intel: cht_bsw_rt5645: Convert to use acpi_dev_get_first_match_dev() ASoC: Intel: cht_bsw_rt5672: Convert to use acpi_dev_get_first_match_dev() ACPI / utils: Remove deprecated function since no user left drivers/acpi/utils.c | 16 ++++++++++------ drivers/extcon/extcon-axp288.c | 9 +++++---- drivers/gpio/gpio-merrifield.c | 18 ++++++++++++++---- include/acpi/acpi_bus.h | 4 ++-- include/linux/acpi.h | 4 ++-- sound/soc/intel/boards/bytcht_da7213.c | 9 +++++---- sound/soc/intel/boards/bytcht_es8316.c | 9 +++++---- sound/soc/intel/boards/bytcr_rt5640.c | 10 +++++----- sound/soc/intel/boards/bytcr_rt5651.c | 14 ++++++++------ sound/soc/intel/boards/cht_bsw_rt5645.c | 9 +++++---- sound/soc/intel/boards/cht_bsw_rt5672.c | 9 +++++---- 11 files changed, 66 insertions(+), 45 deletions(-) -- 2.20.1

8 26

[alsa-devel] [PATCH] ALSA: hda/realtek: Enable headset MIC of Acer TravelMate B114-21 with ALC233
by Jian-Hong Pan 02 Apr '19

02 Apr '19

The Acer TravelMate B114-21 laptop cannot detect and record sound from headset MIC. This patch adds the ALC233_FIXUP_ACER_HEADSET_MIC HDA verb quirk chained with ALC233_FIXUP_ASUS_MIC_NO_PRESENCE pin quirk to fix this issue. Signed-off-by: Jian-Hong Pan <jian-hong(a)endlessm.com> Signed-off-by: Daniel Drake <drake(a)endlessm.com> --- sound/pci/hda/patch_realtek.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index a3fb3d4c5730..bdb2227be4eb 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -5690,6 +5690,7 @@ enum { ALC286_FIXUP_ACER_AIO_HEADSET_MIC, ALC256_FIXUP_ASUS_MIC_NO_PRESENCE, ALC299_FIXUP_PREDATOR_SPK, + ALC233_FIXUP_ACER_HEADSET_MIC, }; static const struct hda_fixup alc269_fixups[] = { @@ -6713,6 +6714,15 @@ static const struct hda_fixup alc269_fixups[] = { { 0x21, 0x90170150 }, /* use as headset mic, without its own jack detect */ { } } + [ALC233_FIXUP_ACER_HEADSET_MIC] = { + .type = HDA_FIXUP_VERBS, + .v.verbs = (const struct hda_verb[]) { + { 0x20, AC_VERB_SET_COEF_INDEX, 0x45 }, + { 0x20, AC_VERB_SET_PROC_COEF, 0x5089 }, + { } + }, + .chained = true, + .chain_id = ALC233_FIXUP_ASUS_MIC_NO_PRESENCE }, }; @@ -6737,6 +6747,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x1025, 0x1290, "Acer Veriton Z4860G", ALC286_FIXUP_ACER_AIO_HEADSET_MIC), SND_PCI_QUIRK(0x1025, 0x1291, "Acer Veriton Z4660G", ALC286_FIXUP_ACER_AIO_HEADSET_MIC), SND_PCI_QUIRK(0x1025, 0x1308, "Acer Aspire Z24-890", ALC286_FIXUP_ACER_AIO_HEADSET_MIC), + SND_PCI_QUIRK(0x1025, 0x132a, "Acer TravelMate B114-21", ALC233_FIXUP_ACER_HEADSET_MIC), SND_PCI_QUIRK(0x1025, 0x1330, "Acer TravelMate X514-51T", ALC255_FIXUP_ACER_HEADSET_MIC), SND_PCI_QUIRK(0x1028, 0x0470, "Dell M101z", ALC269_FIXUP_DELL_M101Z), SND_PCI_QUIRK(0x1028, 0x054b, "Dell XPS one 2710", ALC275_FIXUP_DELL_XPS), -- 2.20.1

3 5

[alsa-devel] alsa kernel panic
by Victor L 02 Apr '19

02 Apr '19

Hi , I have Linux panic when started play music with one of my USB audio device. first please see this logs: [ 71.684730] usbcore: registered new interface driver snd-usb-audio [ 73.835355] BUG: unable to handle kernel NULL pointer dereference at 0000000000000008 [ 73.835412] IP: [<ffffffffc0b45d3c>] snd_usb_endpoint_start+0x23c/0x2f0 [snd_usb_audio] [ 73.835414] PGD 0 [ 73.835415] [ 73.835418] Oops: 0000 [#1] PREEMPT SMP [ 73.835482] Modules linked in: snd_usb_audio snd_usbmidi_lib snd_hwdep snd_pcm_oss snd_mixer_oss snd_ pcm snd_seq_midi snd_seq_midi_event snd_rawmidi snd_seq snd_seq_device snd_timer snd soundcore coretemp iwl3945 iwlegacy i915 ehci_pci iTCO_wdt ehci_hcd pcmcia iTCO_vendor_support kvm usbcore mac80211 e1000e yenta_socket sr_mod pcmcia_rsrc cfg80211 pcmcia_core lpc_ich mfd_core drm_kms_helper cdrom rfkill ata_ge neric irqbypass ptp pps_core joydev sg pcspkr i2c_i801 drm battery i2c_algo_bit ata_piix i2c_smbus usb_c ommon shpchp rng_core ac acpi_cpufreq video button nls_utf8 cifs sha256_ssse3 cmac md4 des_generic arc4 dns_resolver fscache sunrpc loop ip_tables x_tables autofs4 ext4 crc16 jbd2 crc32c_generic fscrypto ecb glue_helper lrw gf128mul ablk_helper cryptd aes_x86_64 mbcache sd_mod ahci libahci [ 73.835486] evdev libata psmouse serio_raw scsi_mod thermal [ 73.835491] CPU: 1 PID: 1099 Comm: output:oops Not tainted 4.9.144-rt93 #1 [ 73.835492] Hardware name: LENOVO 636962M/636962M, BIOS 7IET38WW (1.19 ) 03/22/2011 [ 73.835494] task: ffff8beb97502c40 task.stack: ffffac1901068000 [ 73.835505] RIP: 0010:[<ffffffffc0b45d3c>] [<ffffffffc0b45d3c>] snd_usb_endpoint_start+0x23c/0x2f0 [ snd_usb_audio] [ 73.835507] RSP: 0000:ffffac190106bcc0 EFLAGS: 00010246 [ 73.835508] RAX: 00000000ffffffea RBX: ffff8beb96c00000 RCX: 00000000ffffffea [ 73.835509] RDX: 0000000000000000 RSI: ffffffffc0b5bf98 RDI: ffff8beb99a18000 [ 73.835511] RBP: 0000000000000002 R08: ffffffffc0b5a8d3 R09: 0000000000000000 [ 73.835512] R10: 00000000fffffe00 R11: 0000000000000200 R12: ffff8beb96c80070 [ 73.835513] R13: ffff8beb99a18000 R14: 0000000000000002 R15: 0000000000000000 [ 73.835516] FS: 00007fdfedc66700(0000) GS:ffff8bebbf300000(0000) knlGS:0000000000000000 [ 73.835518] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 73.835519] CR2: 0000000000000008 CR3: 00000000b4ab6000 CR4: 0000000000000670 [ 73.835520] Stack: [ 73.835525] ffff8bebb8888c18 ffff8beb96c00000 0000000000000000 0000000000008002 [ 73.835528] 00007fdfe40197d0 ffffffffc0b4e7cd ffff8bebb8888c18 ffff8beb974b2400 [ 73.835531] ffffffffc0b5088a ffff8beb0000ac44 ffff8bebb206d300 0000000000000000 [ 73.835531] Call Trace: [ 73.835545] [<ffffffffc0b4e7cd>] ? start_endpoints+0x3d/0x150 [snd_usb_audio] [ 73.835555] [<ffffffffc0b5088a>] ? snd_usb_pcm_prepare+0x2ea/0x520 [snd_usb_audio] [ 73.835577] [<ffffffffc0b0c176>] ? snd_pcm_sw_params_user+0x56/0x90 [snd_pcm] [ 73.835584] [<ffffffffc0b0b2d9>] ? snd_pcm_do_prepare+0x19/0x30 [snd_pcm] [ 73.835591] [<ffffffffc0b09ea6>] ? snd_pcm_action_single+0x36/0x80 [snd_pcm] [ 73.835598] [<ffffffffc0b0a311>] ? snd_pcm_action_nonatomic+0x61/0x70 [snd_pcm] [ 73.835606] [<ffffffffc0b0e219>] ? snd_pcm_common_ioctl1+0x3a9/0xc60 [snd_pcm] [ 73.835674] [<ffffffffc03a73ce>] ? ext4_file_write_iter+0x22e/0x370 [ext4] [ 73.835682] [<ffffffffc0b0ecaa>] ? snd_pcm_playback_ioctl1+0x1da/0x240 [snd_pcm] [ 73.835690] [<ffffffffb9736a38>] ? __seccomp_filter+0x78/0x2a0 [ 73.835698] [<ffffffffc0b0ed33>] ? snd_pcm_playback_ioctl+0x23/0x30 [snd_pcm] [ 73.835702] [<ffffffffb982c9c4>] ? do_vfs_ioctl+0xa4/0x620 [ 73.835706] [<ffffffffb96033b4>] ? syscall_trace_enter+0x144/0x300 [ 73.835709] [<ffffffffb982cfb4>] ? SyS_ioctl+0x74/0x80 [ 73.835711] [<ffffffffb9603c56>] ? do_syscall_64+0x76/0x100 [ 73.835717] [<ffffffffb9c38b4e>] ? entry_SYSCALL_64_after_swapgs+0x58/0xc6 [ 73.835747] Code: c0 d3 a8 b5 c0 83 f8 a6 74 15 83 f8 e0 49 c7 c0 8c a8 b5 c0 48 c7 c2 74 a8 b5 c0 4c 0f 45 c2 48 8b 13 89 c1 48 c7 c6 98 bf b5 c0 <48> 8b 7a 08 89 ea 48 81 c7 98 00 00 00 e8 e2 7b f4 f8 f0 80 63 [ 73.835757] RIP [<ffffffffc0b45d3c>] snd_usb_endpoint_start+0x23c/0x2f0 [snd_usb_audio] [ 73.835758] RSP <ffffac190106bcc0> [ 73.835759] CR2: 0000000000000008 I give you more detail, I have Debian Linux 4.9.144-rt93. I tested under Debian 4.19 and Ubuntu same panic present. Please help me to fix this problem, this device working fine under Windows. I believe this problem in Linux audio driver. I ready to provide any additional details if needs. -- Regards, Viktor L

1 2

[alsa-devel] [PATCH] ASoC: dpcm: skip missing substream while applying symmetry
by Jerome Brunet 02 Apr '19

02 Apr '19

If for any reason, the backend does not have the requested substream (like capture on a playback only backend), the BE will be skipped in dpcm_be_dai_startup(). However, dpcm_apply_symmetry() does not skip those BE and will dereference the be_substream (NULL) pointer anyway. Like in dpcm_be_dai_startup(), just skip those BE. Fixes: 906c7d690c3b ("ASoC: dpcm: Apply symmetry for DPCM") Signed-off-by: Jerome Brunet <jbrunet(a)baylibre.com> --- sound/soc/soc-pcm.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index 0d5ec68a1e50..99bbd724d2a6 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -1899,10 +1899,15 @@ static int dpcm_apply_symmetry(struct snd_pcm_substream *fe_substream, struct snd_soc_pcm_runtime *be = dpcm->be; struct snd_pcm_substream *be_substream = snd_soc_dpcm_get_substream(be, stream); - struct snd_soc_pcm_runtime *rtd = be_substream->private_data; + struct snd_soc_pcm_runtime *rtd; struct snd_soc_dai *codec_dai; int i; + /* A backend may not have the requested substream */ + if (!be_substream) + continue; + + rtd = be_substream->private_data; if (rtd->dai_link->be_hw_params_fixup) continue; -- 2.20.1

2 1