May 2017 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH v7 0/9] ASoC: add OF-graph base simple-card
by Kuninori Morimoto 17 May '17

17 May '17

Hi Mark, Rob These are v7 of OF-graph sound card support. Main change from v6 "label" doesn't allow [prefix]label (in [6/9]) 1) - 4) : OF-graph base functions 5) - 9) : OF-graph + simple-card (= audio_graph_card) Kuninori Morimoto (9): of-graph: export symbol of_phandle_iterator_init/next of_graph: add of_graph_get_remote_endpoint() of_graph: add of_graph_get_port_parent() of_graph: add of_graph_get_endpoint_count() ASoC: soc-core: enable "dai-format" on snd_soc_of_parse_daifmt() ASoC: simple-card-utils: enable "label" on asoc_simple_card_parse_card_name ASoC: simple-card-utils: add asoc_simple_card_parse_graph_dai() ASoC: add audio-graph-card document ASoC: add audio-graph-card support .../devicetree/bindings/sound/audio-graph-card.txt | 124 +++++++++ drivers/of/base.c | 62 ++++- include/linux/of_graph.h | 21 ++ include/sound/simple_card_utils.h | 10 + sound/soc/generic/Kconfig | 8 + sound/soc/generic/Makefile | 2 + sound/soc/generic/audio-graph-card.c | 308 +++++++++++++++++++++ sound/soc/generic/simple-card-utils.c | 73 ++++- sound/soc/soc-core.c | 10 +- 9 files changed, 600 insertions(+), 18 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.txt create mode 100644 sound/soc/generic/audio-graph-card.c -- 1.9.1

3 16

[alsa-devel] [PATCH v5 0/9] ASoC: add OF-graph base simple-card
by Kuninori Morimoto 17 May '17

17 May '17

Hi Rob, Mark These are v5 of OF-graph base audio card patch-set. It fixed small things which was pointed by Rob. 1) - 3) : new OF-graph helper functions 4) - 5) : expand "label", "dai-format" 6) - 9) : OF-graph base simple-card Kuninori Morimoto (9): of_graph: add of_graph_get_remote_endpoint() of_graph: add of_graph_get_port_parent() of_graph: add of_graph_get_endpoint_count() ASoC: simple-card-utils: enable "label" on asoc_simple_card_parse_card_name ASoC: soc-core: enable "dai-name" on snd_soc_of_parse_daifmt() ASoC: add snd_soc_get_dai_id() ASoC: simple-card-utils: add asoc_simple_card_parse_graph_dai() ASoC: add audio-graph-card document ASoC: add audio-graph-card support .../devicetree/bindings/sound/audio-graph-card.txt | 124 +++++++++ drivers/of/base.c | 60 +++- include/linux/of_graph.h | 21 ++ include/sound/simple_card_utils.h | 10 + include/sound/soc.h | 1 + sound/soc/generic/Kconfig | 8 + sound/soc/generic/Makefile | 2 + sound/soc/generic/audio-graph-card.c | 308 +++++++++++++++++++++ sound/soc/generic/simple-card-utils.c | 54 +++- sound/soc/soc-core.c | 32 ++- 10 files changed, 604 insertions(+), 16 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.txt create mode 100644 sound/soc/generic/audio-graph-card.c -- 1.9.1

2 12

[alsa-devel] [PATCH 0/2] ASoC: rsnd: tidyup Document
by Kuninori Morimoto 17 May '17

17 May '17

Hi Mark Cc Geert These patches tidyup Document for Renesas sound driver Kuninori Morimoto (2): ASoC: rsnd: move "renesas,rsrc-card" to "simple-scu-audio-card" on Document ASoC: rsnd: add missing clocks/clock-names on Document .../devicetree/bindings/sound/renesas,rsnd.txt | 34 ++++++++++++---------- 1 file changed, 19 insertions(+), 15 deletions(-) -- 1.9.1

3 6

[alsa-devel] [PATCH] ASoC: hdmi-codec: fix spelling mistake: "deteced" -> "detected"
by Colin King 17 May '17

17 May '17

From: Colin Ian King <colin.king(a)canonical.com> Trivial fix to spelling mistake in dev_err message Signed-off-by: Colin Ian King <colin.king(a)canonical.com> --- sound/soc/codecs/hdmi-codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/hdmi-codec.c b/sound/soc/codecs/hdmi-codec.c index 8c5ae1fc23a9..a3f15149afcf 100644 --- a/sound/soc/codecs/hdmi-codec.c +++ b/sound/soc/codecs/hdmi-codec.c @@ -819,7 +819,7 @@ static int hdmi_codec_probe(struct platform_device *pdev) mutex_unlock(&hdmi_mutex); if (hd->cnt >= ARRAY_SIZE(hdmi_dai_name)) { - dev_err(dev, "too many hdmi codec are deteced\n"); + dev_err(dev, "too many hdmi codec are detected\n"); return -EINVAL; } -- 2.11.0

3 2

[alsa-devel] [PATCH] ASoC: rsnd: remove un-used OUF_SRCO/I macro
by Kuninori Morimoto 17 May '17

17 May '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/sh/rcar/src.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/sound/soc/sh/rcar/src.c b/sound/soc/sh/rcar/src.c index 20b5b2e..a8ab26d 100644 --- a/sound/soc/sh/rcar/src.c +++ b/sound/soc/sh/rcar/src.c @@ -12,10 +12,6 @@ #define SRC_NAME "src" -/* SRCx_STATUS */ -#define OUF_SRCO ((1 << 12) | (1 << 13)) -#define OUF_SRCI ((1 << 9) | (1 << 8)) - /* SCU_SYSTEM_STATUS0/1 */ #define OUF_SRC(id) ((1 << (id + 16)) | (1 << id)) -- 1.9.1

2 1

[alsa-devel] [PATCH] ASoC: rsnd: remove mismatch explain comment from src.c
by Kuninori Morimoto 17 May '17

17 May '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Before, src.c cares SRC and SSIU, but now SSIU is cared by ssiu.c by commit c7f69ab5364d ("ASoC: rsnd: use mod base common method on SSIU"). This patch removes mismatched explanation for SRC/SSIU from src.c Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/sh/rcar/src.c | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/sound/soc/sh/rcar/src.c b/sound/soc/sh/rcar/src.c index a8ab26d..e4bde0c 100644 --- a/sound/soc/sh/rcar/src.c +++ b/sound/soc/sh/rcar/src.c @@ -51,20 +51,6 @@ struct rsnd_src { * */ -/* - * src.c is caring... - * - * Gen1 - * - * [mem] -> [SRU] -> [SSI] - * |--------| - * - * Gen2 - * - * [mem] -> [SRC] -> [SSIU] -> [SSI] - * |-----------------| - */ - static void rsnd_src_activation(struct rsnd_mod *mod) { rsnd_mod_write(mod, SRC_SWRSR, 0); -- 1.9.1

2 1

[alsa-devel] [PATCH] ASoC: rsnd: clkout-lr-asynchronous is AUDIO_CLKOUT related property
by Kuninori Morimoto 17 May '17

17 May '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> clkout-lr-asynchronous is AUDIO_CLKOUT related property, thus, it should be handled by rsnd_adg_get_clkout(). Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/sh/rcar/adg.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/sound/soc/sh/rcar/adg.c b/sound/soc/sh/rcar/adg.c index 66203d1..cb4eab0 100644 --- a/sound/soc/sh/rcar/adg.c +++ b/sound/soc/sh/rcar/adg.c @@ -480,6 +480,9 @@ static void rsnd_adg_get_clkout(struct rsnd_priv *priv, if (req_rate[0] % 48000 == 0) adg->flags = AUDIO_OUT_48; + if (of_get_property(np, "clkout-lr-asynchronous", NULL)) + adg->flags = LRCLK_ASYNC; + /* * This driver is assuming that AUDIO_CLKA/AUDIO_CLKB/AUDIO_CLKC * have 44.1kHz or 48kHz base clocks for now. @@ -578,7 +581,6 @@ int rsnd_adg_probe(struct rsnd_priv *priv) { struct rsnd_adg *adg; struct device *dev = rsnd_priv_to_dev(priv); - struct device_node *np = dev->of_node; int ret; adg = devm_kzalloc(dev, sizeof(*adg), GFP_KERNEL); @@ -595,9 +597,6 @@ int rsnd_adg_probe(struct rsnd_priv *priv) rsnd_adg_get_clkin(priv, adg); rsnd_adg_get_clkout(priv, adg); - if (of_get_property(np, "clkout-lr-asynchronous", NULL)) - adg->flags = LRCLK_ASYNC; - priv->adg = adg; rsnd_adg_clk_enable(priv); -- 1.9.1

2 1

[alsa-devel] [PATCH] ASoC: rsnd: remove error pointer for adg->clkout[i]
by Kuninori Morimoto 17 May '17

17 May '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> commit d7f298197a22f11b38059f257842dac7c30a564c ("ASoC: rcar: fixup of_clk_add_provider() usage for multi clkout") added adg->clkout[i] = ERR_PTR(-ENOENT), but, adg->clkout[i] are assumed NULL or clk pointer. This patch fix it. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> --- sound/soc/sh/rcar/adg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/sh/rcar/adg.c b/sound/soc/sh/rcar/adg.c index cb4eab0..36bece8 100644 --- a/sound/soc/sh/rcar/adg.c +++ b/sound/soc/sh/rcar/adg.c @@ -556,7 +556,6 @@ static void rsnd_adg_get_clkout(struct rsnd_priv *priv, clk = clk_register_fixed_rate(dev, clkout_name[i], parent_clk_name, 0, req_rate[0]); - adg->clkout[i] = ERR_PTR(-ENOENT); if (!IS_ERR(clk)) adg->clkout[i] = clk; } -- 1.9.1

2 1

[alsa-devel] [PATCH] ASoC: rsnd: need to call nolock_stop if nolock_start was failed
by Kuninori Morimoto 17 May '17

17 May '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> rsnd_dai_call() is counting the called number of start/stop functions. nolock_stop should be called if nolock_start was failed. Otherwise, nolock_start doesn't called in 2nd play. This patch solved this issue. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Tested-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx(a)renesas.com> --- sound/soc/sh/rcar/core.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/sound/soc/sh/rcar/core.c b/sound/soc/sh/rcar/core.c index 8c1f4e2..e16f723 100644 --- a/sound/soc/sh/rcar/core.c +++ b/sound/soc/sh/rcar/core.c @@ -766,11 +766,16 @@ static int rsnd_soc_dai_startup(struct snd_pcm_substream *substream, { struct rsnd_dai *rdai = rsnd_dai_to_rdai(dai); struct rsnd_dai_stream *io = rsnd_rdai_to_io(rdai, substream); + int ret; /* * call rsnd_dai_call without spinlock */ - return rsnd_dai_call(nolock_start, io, priv); + ret = rsnd_dai_call(nolock_start, io, priv); + if (ret < 0) + rsnd_dai_call(nolock_stop, io, priv); + + return ret; } static void rsnd_soc_dai_shutdown(struct snd_pcm_substream *substream, -- 1.9.1

2 2

Re: [alsa-devel] ASoC: rsnd: fix sound route path when using SRC6/SRC9
by Geert Uytterhoeven 17 May '17

17 May '17

Hi Morimoto-san, On Fri, Mar 31, 2017 at 9:32 PM, Linux Kernel Mailing List <linux-kernel(a)vger.kernel.org> wrote: > Web: https://git.kernel.org/torvalds/c/a1c2ff53726907aff5feb37e4cfd45c1ff626431 > Commit: a1c2ff53726907aff5feb37e4cfd45c1ff626431 > Parent: 4b30eebfc35c67771b5f58d9274d3e321b72d7a8 > Refname: refs/heads/master > Author: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx(a)renesas.com> > AuthorDate: Wed Mar 1 03:51:00 2017 +0000 > Committer: Mark Brown <broonie(a)kernel.org> > CommitDate: Mon Mar 13 12:58:07 2017 +0000 > > ASoC: rsnd: fix sound route path when using SRC6/SRC9 > > This patch fixes the problem that the missing value of the route path > setting table and incorrect values are set in the CMD_ROUTE_SELECT > register. > > Signed-off-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx(a)renesas.com> > [Kuninori: shared data on MIX and non-MIX case] > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> > Signed-off-by: Mark Brown <broonie(a)kernel.org> > --- > sound/soc/sh/rcar/cmd.c | 36 ++++++++++++++++++++---------------- > 1 file changed, 20 insertions(+), 16 deletions(-) > > diff --git a/sound/soc/sh/rcar/cmd.c b/sound/soc/sh/rcar/cmd.c > index abb5eaac854a..7d92a24b7cfa 100644 > --- a/sound/soc/sh/rcar/cmd.c > +++ b/sound/soc/sh/rcar/cmd.c > @@ -31,23 +31,24 @@ static int rsnd_cmd_init(struct rsnd_mod *mod, > struct rsnd_mod *mix = rsnd_io_to_mod_mix(io); > struct device *dev = rsnd_priv_to_dev(priv); > u32 data; > + u32 path[] = { > + [1] = 1 << 0, > + [5] = 1 << 8, > + [6] = 1 << 12, > + [9] = 1 << 15, > + }; > > if (!mix && !dvc) > return 0; > > + if (ARRAY_SIZE(path) < rsnd_mod_id(mod) + 1) > + return -ENXIO; > + > if (mix) { > struct rsnd_dai *rdai; > struct rsnd_mod *src; > struct rsnd_dai_stream *tio; > int i; > - u32 path[] = { > - [0] = 0, > - [1] = 1 << 0, > - [2] = 0, > - [3] = 0, > - [4] = 0, > - [5] = 1 << 8 > - }; > > /* > * it is assuming that integrater is well understanding about > @@ -70,16 +71,19 @@ static int rsnd_cmd_init(struct rsnd_mod *mod, > } else { > struct rsnd_mod *src = rsnd_io_to_mod_src(io); > > - u32 path[] = { > - [0] = 0x30000, > - [1] = 0x30001, > - [2] = 0x40000, > - [3] = 0x10000, > - [4] = 0x20000, > - [5] = 0x40100 > + u8 cmd_case[] = { > + [0] = 0x3, > + [1] = 0x3, > + [2] = 0x4, > + [3] = 0x1, > + [4] = 0x2, > + [5] = 0x4, > + [6] = 0x1, > + [9] = 0x2, > }; > > - data = path[rsnd_mod_id(src)]; > + data = path[rsnd_mod_id(src)] | With gcc 4.9.0: sound/soc/sh/rcar/cmd.c: In function 'rsnd_cmd_init': sound/soc/sh/rcar/cmd.c:85:14: warning: array subscript is below array bounds [-Warray-bounds] data = path[rsnd_mod_id(src)] | > + cmd_case[rsnd_mod_id(src)] << 16; > } > > dev_dbg(dev, "ctu/mix path = 0x%08x", data); 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

3 7