devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Xiubo Li <Li.Xiubo@freescale.com>
To: broonie@kernel.org, lgirdwood@gmail.com, perex@perex.cz,
	tiwai@suse.de, kuninori.morimoto.gx@renesas.com, moinejf@free.fr,
	andrew@lunn.ch, jsarha@ti.com, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, linux-sh@vger.kernel.org,
	alsa-devel@alsa-project.org, shawn.guo@freescale.com
Cc: linux-kernel@vger.kernel.org, Xiubo Li <Li.Xiubo@freescale.com>
Subject: [PATCHv1 3/7] ASoC: dts: vf610-twr: To support simple card newest style.
Date: Mon, 1 Sep 2014 12:29:37 +0800	[thread overview]
Message-ID: <1409545781-20530-4-git-send-email-Li.Xiubo@freescale.com> (raw)
In-Reply-To: <1409545781-20530-1-git-send-email-Li.Xiubo@freescale.com>

This patch depends on the following simple card patch:
===
ASoC: simple-card: Merge single and muti DAI link code.

This patch merge single DAI link and muti-DAI links code together,
and simply the simple-card driver code.

And also do some other improvement:

Since from the DAI format micro SND_SOC_DAIFMT_CBx_CFx, the 'CBx'
mean Codec's bit clock is as master/slave and the 'CFx' mean Codec's
frame clock is as master/slave.

So these same DAI formats should be informed to CPU and CODE DAIs at
the same time. For the Codec driver will set the bit clock and frame
clock as the DAI formats said, but for the CPU driver, if the the
bit clock or frame clock is as Codec master, so it should be set CPU
DAI device as bit clock or frame clock as slave, and vice versa.

The old code will cause confusion, and we should be clear that the
letter 'C' here mean to Codec.
===

Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
---
 arch/arm/boot/dts/vf610-twr.dts | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/vf610-twr.dts b/arch/arm/boot/dts/vf610-twr.dts
index b8a5e8c..841a45a 100644
--- a/arch/arm/boot/dts/vf610-twr.dts
+++ b/arch/arm/boot/dts/vf610-twr.dts
@@ -61,7 +61,6 @@
 
 	sound {
 		compatible = "simple-audio-card";
-		simple-audio-card,format = "i2s";
 		simple-audio-card,widgets =
 			"Microphone", "Microphone Jack",
 			"Headphone", "Headphone Jack",
@@ -74,17 +73,17 @@
 			"Headphone Jack", "HP_OUT",
 			"Speaker Ext", "LINE_OUT";
 
-		simple-audio-card,cpu {
-			sound-dai = <&sai2>;
-			master-clkdir-out;
+		simple-audio-card,dai-link {
+			format = "i2s";
 			frame-master;
 			bitclock-master;
-		};
 
-		simple-audio-card,codec {
-			sound-dai = <&codec>;
-			frame-master;
-			bitclock-master;
+			cpu {
+				sound-dai = <&sai2>;
+			};
+			codec {
+				sound-dai = <&codec>;
+			};
 		};
 	};
 };
-- 
1.8.4

  parent reply	other threads:[~2014-09-01  4:29 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-01  4:29 [PATCHv1 0/7] simple-card: simplify the code Xiubo Li
2014-09-01  4:29 ` [PATCHv1 1/7] ASoC: simple-card: Merge single and muti DAI link code Xiubo Li
2014-09-01 13:34   ` Andrew Lunn
2014-09-01 15:13     ` Mark Brown
     [not found]       ` <20140901151356.GB29327-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2014-09-02  3:22         ` Li.Xiubo-KZfg59tc24xl57MIdRCFDg
     [not found]   ` <1409545781-20530-2-git-send-email-Li.Xiubo-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2014-09-01  7:42     ` [alsa-devel] " Kuninori Morimoto
2014-09-01  7:48       ` Li.Xiubo
2014-09-01  7:50         ` Kuninori Morimoto
2014-09-02  9:15       ` [alsa-devel] " Li.Xiubo
2014-09-01 15:05     ` Jason Cooper
2014-09-01  4:29 ` [PATCHv1 2/7] ASoC: simple-card: Adjust the comments of simple card Xiubo Li
2014-09-01  4:29 ` Xiubo Li [this message]
2014-09-01  4:29 ` [PATCHv1 4/7] ASoC: dts: kirkwood-t5325: To support simple card newest style Xiubo Li
2014-09-01 13:41   ` Andrew Lunn
     [not found]     ` <20140901134148.GB11312-g2DYL2Zd6BY@public.gmane.org>
2014-09-02  9:08       ` Li.Xiubo-KZfg59tc24xl57MIdRCFDg
2014-09-01  4:29 ` [PATCHv1 5/7] ASoC: dts: r8a7740-armadillo800eva-reference: " Xiubo Li
2014-09-01  4:29 ` [PATCHv1 6/7] ASoC: dts: sh73a0-kzm9g-reference: " Xiubo Li
2014-09-01  4:29 ` [PATCHv1 7/7] ASoC: dts: kirkwood-openrd-client: " Xiubo Li

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=1409545781-20530-4-git-send-email-Li.Xiubo@freescale.com \
    --to=li.xiubo@freescale.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=andrew@lunn.ch \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jsarha@ti.com \
    --cc=kuninori.morimoto.gx@renesas.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=moinejf@free.fr \
    --cc=perex@perex.cz \
    --cc=shawn.guo@freescale.com \
    --cc=tiwai@suse.de \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).