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

Re: [alsa-devel] Sound: BUG: KASAN: use-after-free in kill_fasync
by Baozeng Ding 21 Apr '16

21 Apr '16

On 2016/4/6 19:37, Baozeng Ding wrote: > > > On 2016/4/5 22:18, Takashi Iwai wrote: >> On Tue, 05 Apr 2016 15:51:30 +0200, >> Baozeng Ding wrote: >>> Hi all, >>> I've got the following report (use-after-free in kill_fasync) while >>> running syzkaller. >>> Unfortunately no reproducer.The kernel version is 4.5 (on Mar 16 commit >>> 09fd671ccb2475436bd5f597f751ca4a7d177aea). >>> >>> ================================================================== >>> BUG: KASAN: use-after-free in kill_fasync+0x3fb/0x420 at addr >>> ffff880067691d88 >>> Read of size 8 by task swapper/2/0 >>> ============================================================================= >>> >>> BUG kmalloc-2048 (Not tainted): kasan: bad access detected >>> ----------------------------------------------------------------------------- >>> >>> >>> Disabling lock debugging due to kernel taint >>> INFO: Allocated in 0xbbbbbbbbbbbbbbbb age=18446678412249576073 >>> cpu=2245704320 pid=-1 >>> [< inline >] kmalloc /kernel/include/linux/slab.h:472 >>> [< inline >] kzalloc /kernel/include/linux/slab.h:616 >>> [< none >] snd_pcm_attach_substream+0x3b4/0xb10 >>> /kernel/sound/core/pcm.c:966 >>> [< none >] ___slab_alloc+0x4c7/0x500 /kernel/mm/slub.c:2446 >>> [< none >] __slab_alloc+0x4c/0x90 /kernel/mm/slub.c:2475 >>> [< inline >] slab_alloc_node /kernel/mm/slub.c:2538 >>> [< inline >] slab_alloc /kernel/mm/slub.c:2580 >>> [< none >] kmem_cache_alloc_trace+0x262/0x300 >>> /kernel/mm/slub.c:2597 >>> [< inline >] kmalloc /kernel/include/linux/slab.h:472 >>> [< inline >] kzalloc /kernel/include/linux/slab.h:616 >>> [< none >] snd_pcm_attach_substream+0x3b4/0xb10 >>> /kernel/sound/core/pcm.c:966 >>> [< none >] snd_pcm_open_substream+0x84/0x450 >>> /kernel/sound/core/pcm_native.c:2262 >>> [< inline >] snd_pcm_oss_open_file >>> /kernel/sound/core/oss/pcm_oss.c:2346 >>> [< none >] snd_pcm_oss_open.part.17+0x5a4/0x1100 >>> /kernel/sound/core/oss/pcm_oss.c:2428 >>> [< none >] snd_pcm_oss_open+0x35/0x50 >>> /kernel/sound/core/oss/pcm_oss.c:2392 >>> [< none >] soundcore_open+0x30f/0x640 >>> /kernel/sound/sound_core.c:639 >>> [< none >] chrdev_open+0x22a/0x4c0 /kernel/fs/char_dev.c:388 >>> [< none >] do_dentry_open+0x6a2/0xcb0 /kernel/fs/open.c:736 >>> [< none >] vfs_open+0x17b/0x1f0 /kernel/fs/open.c:853 >>> [< inline >] do_last /kernel/fs/namei.c:3258 >>> [< none >] path_openat+0x4837/0x5830 /kernel/fs/namei.c:3394 >>> [< none >] do_filp_open+0x18e/0x250 /kernel/fs/namei.c:3429 >>> [< none >] do_sys_open+0x201/0x420 /kernel/fs/open.c:1022 >>> [< inline >] SYSC_open /kernel/fs/open.c:1040 >>> [< none >] SyS_open+0x2d/0x40 /kernel/fs/open.c:1035 >>> INFO: Freed in 0x10000b076 age=18446678416544543380 cpu=0 pid=0 >>> [< none >] snd_pcm_detach_substream+0x134/0x280 >>> /kernel/sound/core/pcm.c:1017 >>> [< none >] __slab_free+0x1e8/0x300 /kernel/mm/slub.c:2657 >>> [< inline >] slab_free /kernel/mm/slub.c:2810 >>> [< none >] kfree+0x24e/0x2d0 /kernel/mm/slub.c:3661 >>> [< none >] snd_pcm_detach_substream+0x134/0x280 >>> /kernel/sound/core/pcm.c:1017 >>> [< none >] snd_pcm_release_substream.part.38+0x219/0x2f0 >>> /kernel/sound/core/pcm_native.c:2250 >>> [< none >] snd_pcm_release_substream+0x59/0x70 >>> /kernel/sound/core/pcm_native.c:2251 >>> [< none >] snd_pcm_oss_release_file+0x45/0xb0 >>> /kernel/sound/core/oss/pcm_oss.c:2305 >>> [< none >] snd_pcm_oss_release+0xfa/0x250 >>> /kernel/sound/core/oss/pcm_oss.c:2485 >>> [< none >] __fput+0x236/0x780 /kernel/fs/file_table.c:208 >>> [< none >] ____fput+0x15/0x20 /kernel/fs/file_table.c:244 >>> [< none >] task_work_run+0x16b/0x200 >>> /kernel/kernel/task_work.c:115 >>> [< inline >] exit_task_work >>> /kernel/include/linux/task_work.h:21 >>> [< none >] do_exit+0x87f/0x2c90 /kernel/kernel/exit.c:748 >>> [< none >] do_group_exit+0x108/0x330 >>> /kernel/kernel/exit.c:878 >>> [< inline >] SYSC_exit_group /kernel/kernel/exit.c:889 >>> [< none >] SyS_exit_group+0x1d/0x20 /kernel/kernel/exit.c:887 >>> [< none >] entry_SYSCALL_64_fastpath+0x23/0xc1 >>> /kernel/arch/x86/entry/entry_64.S:207 >>> INFO: Slab 0xffffea00019da400 objects=13 used=8 fp=0xffff880067691be0 >>> flags=0x5fffc0000004080 >>> INFO: Object 0xffff880067691bd8 @offset=7128 fp=0xbbbbbbbbbbbbbbbb >>> Call Trace: >>> <IRQ> [< inline >] __dump_stack /kernel/lib/dump_stack.c:15 >>> <IRQ> [<ffffffff82945051>] dump_stack+0xb3/0x112 >>> /kernel/lib/dump_stack.c:51 >>> [<ffffffff817009ad>] print_trailer+0x10d/0x190 /kernel/mm/slub.c:668 >>> [<ffffffff817074af>] object_err+0x2f/0x40 /kernel/mm/slub.c:675 >>> [< inline >] print_address_description >>> /kernel/mm/kasan/report.c:138 >>> [<ffffffff81709ca5>] kasan_report_error+0x215/0x530 >>> /kernel/mm/kasan/report.c:236 >>> [< inline >] ? spin_lock >>> /kernel/include/linux/spinlock.h:302 >>> [<ffffffff84a1c550>] ? snd_pcm_stream_lock+0x80/0xd0 >>> /kernel/sound/core/pcm_native.c:104 >>> [< inline >] kasan_report /kernel/mm/kasan/report.c:259 >>> [<ffffffff8170a0be>] __asan_report_load8_noabort+0x3e/0x40 >>> /kernel/mm/kasan/report.c:280 >>> [<ffffffff81792a0b>] ? kill_fasync+0x3fb/0x420 >>> /kernel/fs/fcntl.c:729 >>> [<ffffffff81792a0b>] kill_fasync+0x3fb/0x420 /kernel/fs/fcntl.c:729 >>> [<ffffffff84a36e08>] snd_pcm_period_elapsed+0x1c8/0x230 >>> /kernel/sound/core/pcm_lib.c:1890 >>> [<ffffffff84b6e2e0>] ? get_bound_vga.isra.21.part.22+0x140/0x140 >>> /kernel/sound/pci/hda/hda_intel.c:1327 >>> [<ffffffff84b0637d>] stream_update+0xad/0xe0 >>> /kernel/sound/pci/hda/hda_controller.c:923 >>> [<ffffffff84c1583e>] snd_hdac_bus_handle_stream_irq+0x24e/0x350 >>> /kernel/sound/hda/hdac_controller.c:449 >>> [<ffffffff84b062d0>] ? azx_init_chip+0x100/0x100 >>> /kernel/sound/pci/hda/hda_controller.c:890 >>> [<ffffffff84b04d24>] ? azx_interrupt+0x14/0x4d0 >>> /kernel/sound/pci/hda/hda_controller.c:929 >>> [<ffffffff84b04eee>] azx_interrupt+0x1de/0x4d0 >>> /kernel/sound/pci/hda/hda_controller.c:954 >>> [<ffffffff84b04d10>] ? azx_stop_chip+0x20/0x20 >>> /kernel/sound/pci/hda/hda_controller.c:908 >>> [<ffffffff8143d9e3>] handle_irq_event_percpu+0xf3/0x790 >>> /kernel/kernel/irq/handle.c:145 >>> [<ffffffff8143e127>] handle_irq_event+0xa7/0x140 >>> /kernel/kernel/irq/handle.c:192 >>> [<ffffffff81447f81>] handle_edge_irq+0x1e1/0x8d0 >>> /kernel/kernel/irq/chip.c:623 >>> [< inline >] generic_handle_irq_desc >>> /kernel/include/linux/irqdesc.h:146 >>> [<ffffffff811ad739>] handle_irq+0x109/0x2a0 >>> /kernel/arch/x86/kernel/irq_64.c:78 >>> [< inline >] ? rcu_lock_release >>> /kernel/include/linux/rcupdate.h:491 >>> [< inline >] ? rcu_read_unlock >>> /kernel/include/linux/rcupdate.h:926 >>> [< inline >] ? __atomic_notifier_call_chain >>> /kernel/kernel/notifier.c:184 >>> [<ffffffff81369c4f>] ? atomic_notifier_call_chain+0xbf/0x140 >>> /kernel/kernel/notifier.c:193 >>> [<ffffffff81369b90>] ? __atomic_notifier_call_chain+0x150/0x150 >>> /kernel/include/linux/rcupdate.h:922 >>> [<ffffffff811ac36d>] do_IRQ+0x7d/0x1a0 >>> /kernel/arch/x86/kernel/irq.c:240 >>> [<ffffffff85dac40c>] common_interrupt+0x8c/0x8c >>> /kernel/arch/x86/entry/entry_64.S:454 >>> <EOI> [<ffffffff812245c6>] ? native_safe_halt+0x6/0x10 >>> /kernel/./arch/x86/include/asm/irqflags.h:49 >>> [<ffffffff81405ced>] ? trace_hardirqs_on+0xd/0x10 >>> /kernel/kernel/locking/lockdep.c:2635 >>> [< inline >] arch_safe_halt >>> /kernel/./arch/x86/include/asm/paravirt.h:117 >>> [<ffffffff811bf152>] default_idle+0x22/0x2d0 >>> /kernel/arch/x86/kernel/process.c:307 >>> [<ffffffff811c053a>] arch_cpu_idle+0xa/0x10 >>> /kernel/arch/x86/kernel/process.c:298 >>> [<ffffffff813eb618>] default_idle_call+0x48/0x70 >>> /kernel/kernel/sched/idle.c:93 >>> [< inline >] cpuidle_idle_call >>> /kernel/kernel/sched/idle.c:151 >>> [< inline >] cpu_idle_loop /kernel/kernel/sched/idle.c:242 >>> [<ffffffff813ebaa7>] cpu_startup_entry+0x467/0x600 >>> /kernel/kernel/sched/idle.c:291 >>> [<ffffffff81201432>] start_secondary+0x2b2/0x380 >>> /kernel/arch/x86/kernel/smpboot.c:259 >>> [<ffffffff81201180>] ? set_cpu_sibling_map+0x18a0/0x18a0 >>> /kernel/include/linux/topology.h:80 >>> Memory state around the buggy address: >>> ffff880067691c80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb >>> ffff880067691d00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb >>> >ffff880067691d80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb >>> ^ >>> ffff880067691e00: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb >>> ffff880067691e80: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb >>> ================================================================== >>> >>> It seems that in function snd_pcm_detach_substream, when freeing >>> runtime >>> structure, it did not handle its fasync >>> data pointer. Any ideas? Thanks. >> Well, fasync doesn't need a cleanup usually. Do you have a proper >> reproducer code? > Sorry, I cannot reproduce it. I will apply your patch and test it. I > will tell you the result whether it is okay a few days later. > Thanks. > >> In anyway, try the patch below. I'm not sure whether this would help, >> but if the path is via kill_fasync(), it'd be good to put in the >> protected context. >> During the last few weeks, I have not seen the bug any more after applying your patch. Thanks. >> >> Takashi >> >> --- >> diff --git a/sound/core/pcm_lib.c b/sound/core/pcm_lib.c >> index 3a9b66c6e09c..0aca39762ed0 100644 >> --- a/sound/core/pcm_lib.c >> +++ b/sound/core/pcm_lib.c >> @@ -1886,8 +1886,8 @@ void snd_pcm_period_elapsed(struct >> snd_pcm_substream *substream) >> snd_timer_interrupt(substream->timer, 1); >> #endif >> _end: >> - snd_pcm_stream_unlock_irqrestore(substream, flags); >> kill_fasync(&runtime->fasync, SIGIO, POLL_IN); >> + snd_pcm_stream_unlock_irqrestore(substream, flags); >> } >> EXPORT_SYMBOL(snd_pcm_period_elapsed); >

2 1

[alsa-devel] [RFC PATCH v2 0/5] ASoC: Mediatek: Add support for MT2701 SOC
by Garlic Tseng 21 Apr '16

21 Apr '16

Refactor and go v2 because of the 60KB limitation This patch adds basic support for Mediatek AFE for MT2701 SoC. The patch is based on broonie tree "for-next" branch. The interface to connect with BT chip is not finished yet, but as an upstream rookie I really need some comments. Any comments will be greatly appreciated. Garlic Tseng (5): ASoC: mediatek: Refine mt8173 driver ASoC: mediatek: add documents for mt2701 ASoC: mediatek: add clock and irq control for 2701 platform driver ASoC: mediatek: add mt2701 platform driver implementation. ASoC: mediatek: Add mt2701-cs42448 driver and config option. .../devicetree/bindings/sound/mt2701-cs42448.txt | 39 + .../devicetree/bindings/sound/mtk2701-afe-pcm.txt | 150 ++ sound/soc/mediatek/Kconfig | 21 + sound/soc/mediatek/Makefile | 15 +- sound/soc/mediatek/mt2701/Makefile | 20 + sound/soc/mediatek/mt2701/mt2701-cs42448.c | 376 +++++ sound/soc/mediatek/mt2701/mtk2701-afe-clock-ctrl.c | 259 +++ sound/soc/mediatek/mt2701/mtk2701-afe-clock-ctrl.h | 28 + sound/soc/mediatek/mt2701/mtk2701-afe-common.h | 217 +++ sound/soc/mediatek/mt2701/mtk2701-afe-pcm.c | 1752 ++++++++++++++++++++ sound/soc/mediatek/mt2701/mtk2701-irq.c | 109 ++ sound/soc/mediatek/mt2701/mtk2701-irq.h | 30 + sound/soc/mediatek/mt2701/mtk2701-reg.h | 165 ++ sound/soc/mediatek/mt8173/Makefile | 7 + sound/soc/mediatek/{ => mt8173}/mt8173-max98090.c | 2 +- .../mediatek/{ => mt8173}/mt8173-rt5650-rt5514.c | 2 +- .../mediatek/{ => mt8173}/mt8173-rt5650-rt5676.c | 4 +- sound/soc/mediatek/{ => mt8173}/mt8173-rt5650.c | 2 +- sound/soc/mediatek/{ => mt8173}/mtk-afe-common.h | 0 sound/soc/mediatek/{ => mt8173}/mtk-afe-pcm.c | 0 20 files changed, 3186 insertions(+), 12 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/mt2701-cs42448.txt create mode 100644 Documentation/devicetree/bindings/sound/mtk2701-afe-pcm.txt create mode 100644 sound/soc/mediatek/mt2701/Makefile create mode 100644 sound/soc/mediatek/mt2701/mt2701-cs42448.c create mode 100644 sound/soc/mediatek/mt2701/mtk2701-afe-clock-ctrl.c create mode 100644 sound/soc/mediatek/mt2701/mtk2701-afe-clock-ctrl.h create mode 100644 sound/soc/mediatek/mt2701/mtk2701-afe-common.h create mode 100644 sound/soc/mediatek/mt2701/mtk2701-afe-pcm.c create mode 100644 sound/soc/mediatek/mt2701/mtk2701-irq.c create mode 100644 sound/soc/mediatek/mt2701/mtk2701-irq.h create mode 100644 sound/soc/mediatek/mt2701/mtk2701-reg.h create mode 100644 sound/soc/mediatek/mt8173/Makefile rename sound/soc/mediatek/{ => mt8173}/mt8173-max98090.c (99%) rename sound/soc/mediatek/{ => mt8173}/mt8173-rt5650-rt5514.c (99%) rename sound/soc/mediatek/{ => mt8173}/mt8173-rt5650-rt5676.c (99%) rename sound/soc/mediatek/{ => mt8173}/mt8173-rt5650.c (99%) rename sound/soc/mediatek/{ => mt8173}/mtk-afe-common.h (100%) rename sound/soc/mediatek/{ => mt8173}/mtk-afe-pcm.c (100%) -- 1.9.1

4 11

[alsa-devel] [PATCH v7] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver
by PC Liao 20 Apr '16

20 Apr '16

This patch adds second I2S connection to rt5650 codec for capture path on mt8173-rt5650 machine driver. Signed-off-by: PC Liao <pc.liao(a)mediatek.com> --- Changes since v6: Use subnode to determine the capture path --- .../devicetree/bindings/sound/mt8173-rt5650.txt | 10 ++++ sound/soc/mediatek/mt8173-rt5650.c | 49 ++++++++++++++++++-- 2 files changed, 56 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/sound/mt8173-rt5650.txt b/Documentation/devicetree/bindings/sound/mt8173-rt5650.txt index fe5a5ef..5bfa6b6 100644 --- a/Documentation/devicetree/bindings/sound/mt8173-rt5650.txt +++ b/Documentation/devicetree/bindings/sound/mt8173-rt5650.txt @@ -5,11 +5,21 @@ Required properties: - mediatek,audio-codec: the phandles of rt5650 codecs - mediatek,platform: the phandle of MT8173 ASoC platform +Optional subnodes: +- codec-capture : the subnode of rt5650 codec capture +Required codec-capture subnode properties: +- sound-dai: audio codec dai name on capture path + <&rt5650 0> : Default setting. Connect rt5650 I2S1 for capture. (dai_name = rt5645-aif1) + <&rt5650 1> : Connect rt5650 I2S2 for capture. (dai_name = rt5645-aif2) + Example: sound { compatible = "mediatek,mt8173-rt5650"; mediatek,audio-codec = <&rt5650>; mediatek,platform = <&afe>; + codec-capture { + sound-dai = <&rt5650 1>; + }; }; diff --git a/sound/soc/mediatek/mt8173-rt5650.c b/sound/soc/mediatek/mt8173-rt5650.c index bb09bb1..b8ed965 100644 --- a/sound/soc/mediatek/mt8173-rt5650.c +++ b/sound/soc/mediatek/mt8173-rt5650.c @@ -85,12 +85,28 @@ static int mt8173_rt5650_init(struct snd_soc_pcm_runtime *runtime) { struct snd_soc_card *card = runtime->card; struct snd_soc_codec *codec = runtime->codec_dais[0]->codec; + const char *codec_capture_dai = runtime->codec_dais[1]->name; int ret; rt5645_sel_asrc_clk_src(codec, - RT5645_DA_STEREO_FILTER | - RT5645_AD_STEREO_FILTER, + RT5645_DA_STEREO_FILTER, RT5645_CLK_SEL_I2S1_ASRC); + + if (!strcmp(codec_capture_dai, "rt5645-aif1")) { + dev_warn(card->dev, + "Only one dai codec found in DTS, enabled AD filter\n"); + rt5645_sel_asrc_clk_src(codec, + RT5645_AD_STEREO_FILTER, + RT5645_CLK_SEL_I2S1_ASRC); + } else if (!strcmp(codec_capture_dai, "rt5645-aif2")) { + rt5645_sel_asrc_clk_src(codec, + RT5645_AD_STEREO_FILTER, + RT5645_CLK_SEL_I2S2_ASRC); + } else { + dev_err(card->dev, "Can't get the right codec dai\n"); + return -EINVAL; + } + /* enable jack detection */ ret = snd_soc_card_jack_new(card, "Headset Jack", SND_JACK_HEADPHONE | SND_JACK_MICROPHONE | @@ -110,6 +126,11 @@ static int mt8173_rt5650_init(struct snd_soc_pcm_runtime *runtime) static struct snd_soc_dai_link_component mt8173_rt5650_codecs[] = { { + /* Playback */ + .dai_name = "rt5645-aif1", + }, + { + /* Capture */ .dai_name = "rt5645-aif1", }, }; @@ -149,7 +170,7 @@ static struct snd_soc_dai_link mt8173_rt5650_dais[] = { .cpu_dai_name = "I2S", .no_pcm = 1, .codecs = mt8173_rt5650_codecs, - .num_codecs = 1, + .num_codecs = 2, .init = mt8173_rt5650_init, .dai_fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF | SND_SOC_DAIFMT_CBS_CFS, @@ -177,6 +198,8 @@ static int mt8173_rt5650_dev_probe(struct platform_device *pdev) { struct snd_soc_card *card = &mt8173_rt5650_card; struct device_node *platform_node; + struct device_node *np; + const char *codec_capture_dai; int i, ret; platform_node = of_parse_phandle(pdev->dev.of_node, @@ -199,6 +222,26 @@ static int mt8173_rt5650_dev_probe(struct platform_device *pdev) "Property 'audio-codec' missing or invalid\n"); return -EINVAL; } + mt8173_rt5650_codecs[1].of_node = mt8173_rt5650_codecs[0].of_node; + + if (of_find_node_by_name(platform_node, "codec-capture")) { + np = of_get_child_by_name(pdev->dev.of_node, "codec-capture"); + if (!np) { + dev_err(&pdev->dev, + "%s: Can't find codec-capture DT node\n", + __func__); + return -EINVAL; + } + ret = snd_soc_of_get_dai_name(np, &codec_capture_dai); + if (ret < 0) { + dev_err(&pdev->dev, + "%s codec_capture_dai name fail %d\n", + __func__, ret); + return ret; + } + mt8173_rt5650_codecs[1].dai_name = codec_capture_dai; + } + card->dev = &pdev->dev; platform_set_drvdata(pdev, card); -- 1.7.9.5

1 0

[alsa-devel] [PATCH] ASoC: ak4642: Remove CLK_IS_ROOT
by Stephen Boyd 20 Apr '16

20 Apr '16

This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate CLK_IS_ROOT", 2016-02-02) so remove it. Cc: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Signed-off-by: Stephen Boyd <sboyd(a)codeaurora.org> --- sound/soc/codecs/ak4642.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sound/soc/codecs/ak4642.c b/sound/soc/codecs/ak4642.c index cda27c22812a..1ee8506c06c7 100644 --- a/sound/soc/codecs/ak4642.c +++ b/sound/soc/codecs/ak4642.c @@ -608,9 +608,7 @@ static struct clk *ak4642_of_parse_mcko(struct device *dev) of_property_read_string(np, "clock-output-names", &clk_name); - clk = clk_register_fixed_rate(dev, clk_name, parent_clk_name, - (parent_clk_name) ? 0 : CLK_IS_ROOT, - rate); + clk = clk_register_fixed_rate(dev, clk_name, parent_clk_name, 0, rate); if (!IS_ERR(clk)) of_clk_add_provider(np, of_clk_src_simple_get, clk); -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project

3 2

[alsa-devel] [PATCH] ASoC: rsnd: Remove CLK_IS_ROOT
by Stephen Boyd 20 Apr '16

20 Apr '16

This flag is a no-op now (see commit 47b0eeb3dc8a "clk: Deprecate CLK_IS_ROOT", 2016-02-02) so remove it. Cc: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Signed-off-by: Stephen Boyd <sboyd(a)codeaurora.org> --- sound/soc/sh/rcar/adg.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/sound/soc/sh/rcar/adg.c b/sound/soc/sh/rcar/adg.c index 606399de684d..49354d17ea55 100644 --- a/sound/soc/sh/rcar/adg.c +++ b/sound/soc/sh/rcar/adg.c @@ -492,9 +492,7 @@ static void rsnd_adg_get_clkout(struct rsnd_priv *priv, */ if (!count) { clk = clk_register_fixed_rate(dev, clkout_name[CLKOUT], - parent_clk_name, - (parent_clk_name) ? - 0 : CLK_IS_ROOT, req_rate); + parent_clk_name, 0, req_rate); if (!IS_ERR(clk)) { adg->clkout[CLKOUT] = clk; of_clk_add_provider(np, of_clk_src_simple_get, clk); @@ -506,9 +504,7 @@ static void rsnd_adg_get_clkout(struct rsnd_priv *priv, else { for (i = 0; i < CLKOUTMAX; i++) { clk = clk_register_fixed_rate(dev, clkout_name[i], - parent_clk_name, - (parent_clk_name) ? - 0 : CLK_IS_ROOT, + parent_clk_name, 0, req_rate); if (!IS_ERR(clk)) { adg->onecell.clks = adg->clkout; -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project

3 2

[alsa-devel] [PATCH 0/4] ASoC: add graph base connection on simple-card
by Kuninori Morimoto 20 Apr '16

20 Apr '16

Hi Mark Current simple-card is using "sound-dai" base connection on DT, but V4L2 is using graph base connection. For example HDMI case, we would like to use both connection. To above confusable connection method, and to reuse current resource, this patch adds new function, and detect both "sound-dai" and "remote-endpoint" on simple-card. like this sound { compatible = "simple-audio-card"; ... sndcpu: simple-audio-card,cpu { sound-dai = <&xxx>; }; sndcodec: simple-audio-card,codec { => remote-endpoint = <&out_hdmi>; }; }; xxx { ... ports { ... port@0 { reg = <0>; out_rgb: endpoint { }; }; port@1 { reg = <1>; => out_hdmi: endpoint { }; }; ... }; }; Kuninori Morimoto (4): of_graph: add of_graph_get_endpoint_count() ASoC: add of_parse_snd_connection_with_args() for sound-dai/graph connection ASoC: snd_soc_of_get_dai_name() uses of_parse_snd_soc_connection_with_args() ASoC: simple-card: probe both sound-dai and remote-endpoint Documentation/devicetree/bindings/sound/simple-card.txt | 19 +++++++++++++++++++ include/linux/of_graph.h | 11 +++++++++++ include/sound/soc.h | 2 ++ sound/soc/generic/simple-card.c | 9 ++++++--- sound/soc/soc-core.c | 45 +++++++++++++++++++++++++++++++++++++++++++-- 5 files changed, 81 insertions(+), 5 deletions(-)

3 13

Re: [alsa-devel] Issues with ALSA:ice1724 sound cards on powerpc machines
by Julian Margetson 20 Apr '16

20 Apr '16

On 2/17/2016 2:29 PM, Julian Margetson wrote: > Hi > > Having an issue issue with ice1724 sound cards on powerpc motherboards. > Motherboards in question Acube Sam460ex . > The card appears as a valid device but there is no sound output on the > majority of boots of the computer. > Logs attached. > On the first set boot audio playback produced sound ok. > On the second set boot no sound was produce with audio playback > > > Regards > Julian Margetson > > > Anyone have any ideas ? Resubmitting again with links to logs instead of included as attachments. The issue also seen on A-Eon Cyrus P5020 motherboard . https://www.dropbox.com/s/72labcje9z1r1b7/1alsa-info.txt.7CLHMG0Gap?dl=0 https://www.dropbox.com/s/v1pn5kjqrvwu16f/boot_1.log?dl=0 https://www.dropbox.com/s/uuaa14rkpmmz1o0/2alsa-info.txt.VKiEVuytiJ?dl=0 https://www.dropbox.com/s/wwu2f5t93l8g26d/boot_2.log?dl=0 Thanks

3 7

[alsa-devel] GPIO control from ALSA
by Petr Kulhavy 20 Apr '16

20 Apr '16

Hello, I'm writing a driver for ASoC "sound card" hardware which uses CPU GPIO to mute/unmute an analogue amplifier and another GPIO to control an analogue switch for line/mic switching. Is there an abstraction in ALSA to model this kind of hardware? Thanks Petr

2 5

[alsa-devel] [PATCH 1/1] ALSA: hda - add PCI ID for Intel Broxton-T
by han.lu＠intel.com 20 Apr '16

20 Apr '16

From: "Lu, Han" <han.lu(a)intel.com> Add HD Audio Device PCI ID for the Intel Broxton-T platform. It is an HDA Intel PCH controller. Signed-off-by: Lu, Han <han.lu(a)intel.com> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index b680be0..637b8a0 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -2232,6 +2232,9 @@ static const struct pci_device_id azx_ids[] = { /* Broxton-P(Apollolake) */ { PCI_DEVICE(0x8086, 0x5a98), .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_BROXTON }, + /* Broxton-T */ + { PCI_DEVICE(0x8086, 0x1a98), + .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_BROXTON }, /* Haswell */ { PCI_DEVICE(0x8086, 0x0a0c), .driver_data = AZX_DRIVER_HDMI | AZX_DCAPS_INTEL_HASWELL }, -- 2.5.0

2 1

[alsa-devel] [asoc:topic/mtk 1/1] sound/soc/mediatek/mt8173-rt5650.c:236:3: error: implicit declaration of function 'snd_soc_get_dai_name'
by kbuild test robot 20 Apr '16

20 Apr '16

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/mtk head: 0beeb8d0b434b8b007ac8058fb2cac5997fff6df commit: 0beeb8d0b434b8b007ac8058fb2cac5997fff6df [1/1] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver config: arm-allmodconfig (attached as .config) reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/m… -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout 0beeb8d0b434b8b007ac8058fb2cac5997fff6df # save the attached .config to linux build tree make.cross ARCH=arm All errors (new ones prefixed by >>): sound/soc/mediatek/mt8173-rt5650.c: In function 'mt8173_rt5650_dev_probe': >> sound/soc/mediatek/mt8173-rt5650.c:236:3: error: implicit declaration of function 'snd_soc_get_dai_name' [-Werror=implicit-function-declaration] ret = snd_soc_get_dai_name(&args, &codec_dai_capture); ^ cc1: some warnings being treated as errors vim +/snd_soc_get_dai_name +236 sound/soc/mediatek/mt8173-rt5650.c 230 dev_err(&pdev->dev, 231 "%s capture-dai name fail %d\n", 232 __func__, ret); 233 return ret; 234 } 235 > 236 ret = snd_soc_get_dai_name(&args, &codec_dai_capture); 237 mt8173_rt5650_codecs[1].dai_name = codec_dai_capture; 238 } 239 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

1 0