From: Rob Herring <robh@kernel.org> To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Cc: Jose Abreu <Jose.Abreu@synopsys.com>, Linux-DT <devicetree@vger.kernel.org>, Linux-ALSA <alsa-devel@alsa-project.org>, Lars-Peter Clausen <lars@metafoo.de>, Mark Brown <broonie@kernel.org>, Liam Girdwood <lgirdwood@gmail.com>, Linux-Renesas <linux-renesas-soc@vger.kernel.org>, Simon <horms@verge.net.au>, Laurent <laurent.pinchart@ideasonboard.com> Subject: Re: [PATCH 0/4] ASoC: add graph base connection on simple-card Date: Mon, 18 Apr 2016 12:12:48 -0500 [thread overview] Message-ID: <20160418171248.GA18460@rob-hp-laptop> (raw) In-Reply-To: <871t68wxxe.wl%kuninori.morimoto.gx@renesas.com> On Thu, Apr 14, 2016 at 05:45:41AM +0000, Kuninori Morimoto wrote: > 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>; This is not valid usage of the graph binding. You are only using half of it. > }; > }; > > 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(-) > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org> To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Cc: Mark Brown <broonie@kernel.org>, Laurent <laurent.pinchart@ideasonboard.com>, Lars-Peter Clausen <lars@metafoo.de>, Simon <horms@verge.net.au>, Linux-Renesas <linux-renesas-soc@vger.kernel.org>, Linux-DT <devicetree@vger.kernel.org>, Linux-ALSA <alsa-devel@alsa-project.org>, Liam Girdwood <lgirdwood@gmail.com>, Jose Abreu <Jose.Abreu@synopsys.com> Subject: Re: [PATCH 0/4] ASoC: add graph base connection on simple-card Date: Mon, 18 Apr 2016 12:12:48 -0500 [thread overview] Message-ID: <20160418171248.GA18460@rob-hp-laptop> (raw) In-Reply-To: <871t68wxxe.wl%kuninori.morimoto.gx@renesas.com> On Thu, Apr 14, 2016 at 05:45:41AM +0000, Kuninori Morimoto wrote: > 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>; This is not valid usage of the graph binding. You are only using half of it. > }; > }; > > 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(-) > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-04-18 17:12 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-14 5:45 [PATCH 0/4] ASoC: add graph base connection on simple-card Kuninori Morimoto 2016-04-14 5:47 ` [PATCH 1/4] of_graph: add of_graph_get_endpoint_count() Kuninori Morimoto 2016-04-14 5:47 ` [PATCH 2/4] ASoC: add of_parse_snd_connection_with_args() for sound-dai/graph connection Kuninori Morimoto 2016-04-14 6:17 ` [alsa-devel] " kbuild test robot 2016-04-14 7:07 ` [PATCH] of: add missing const for of_parse_phandle_with_args() in !CONFIG_OF Kuninori Morimoto 2016-04-14 7:07 ` Kuninori Morimoto 2016-05-10 2:53 ` [PATCH][RESEND] " Kuninori Morimoto 2016-05-10 2:53 ` Kuninori Morimoto 2016-04-14 5:48 ` [PATCH 3/4] ASoC: snd_soc_of_get_dai_name() uses of_parse_snd_soc_connection_with_args() Kuninori Morimoto 2016-04-14 5:48 ` [PATCH 4/4] ASoC: simple-card: probe both sound-dai and remote-endpoint Kuninori Morimoto [not found] ` <871t68wxxe.wl%kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2016-04-14 9:01 ` [alsa-devel] [PATCH 0/4] ASoC: add graph base connection on simple-card Jean-Francois Moine 2016-04-14 9:01 ` Jean-Francois Moine [not found] ` <20160414110128.56a0da97d526cdcb8d5f1ddf-GANU6spQydw@public.gmane.org> 2016-04-15 7:25 ` Kuninori Morimoto 2016-04-15 7:25 ` Kuninori Morimoto 2016-04-15 9:26 ` Kuninori Morimoto 2016-04-15 11:31 ` Jean-Francois Moine 2016-04-15 11:31 ` Jean-Francois Moine 2016-04-18 2:43 ` Kuninori Morimoto 2016-04-18 2:43 ` [alsa-devel] " Kuninori Morimoto [not found] ` <87r3e3ej5t.wl%kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2016-04-18 7:38 ` Jean-Francois Moine 2016-04-18 7:38 ` Jean-Francois Moine 2016-04-20 2:55 ` Kuninori Morimoto 2016-04-20 16:18 ` Jean-Francois Moine 2016-04-20 16:18 ` Jean-Francois Moine 2016-04-18 17:12 ` Rob Herring [this message] 2016-04-18 17:12 ` Rob Herring
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160418171248.GA18460@rob-hp-laptop \ --to=robh@kernel.org \ --cc=Jose.Abreu@synopsys.com \ --cc=alsa-devel@alsa-project.org \ --cc=broonie@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=horms@verge.net.au \ --cc=kuninori.morimoto.gx@renesas.com \ --cc=lars@metafoo.de \ --cc=laurent.pinchart@ideasonboard.com \ --cc=lgirdwood@gmail.com \ --cc=linux-renesas-soc@vger.kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.