All of lore.kernel.org
 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 7/7] ASoC: dts: kirkwood-openrd-client: To support simple card newest style.
Date: Mon, 01 Sep 2014 04:29:41 +0000	[thread overview]
Message-ID: <1409545781-20530-8-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/kirkwood-openrd-client.dts | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/kirkwood-openrd-client.dts b/arch/arm/boot/dts/kirkwood-openrd-client.dts
index 887b9c1..3b6f169 100644
--- a/arch/arm/boot/dts/kirkwood-openrd-client.dts
+++ b/arch/arm/boot/dts/kirkwood-openrd-client.dts
@@ -33,15 +33,16 @@
 
 	sound {
 		compatible = "simple-audio-card";
-		simple-audio-card,format = "i2s";
 		simple-audio-card,mclk-fs = <256>;
 
-		simple-audio-card,cpu {
-			sound-dai = <&audio0>;
-		};
-
-		simple-audio-card,codec {
-			sound-dai = <&cs42l51>;
+		simple-audio-card,dai-link {
+			format = "i2s";
+			cpu {
+				sound-dai = <&audio0>;
+			};
+			codec {
+				sound-dai = <&cs42l51>;
+			};
 		};
 	};
 };
-- 
1.8.4


WARNING: multiple messages have this Message-ID (diff)
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 7/7] ASoC: dts: kirkwood-openrd-client: To support simple card newest style.
Date: Mon, 1 Sep 2014 12:29:41 +0800	[thread overview]
Message-ID: <1409545781-20530-8-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/kirkwood-openrd-client.dts | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/kirkwood-openrd-client.dts b/arch/arm/boot/dts/kirkwood-openrd-client.dts
index 887b9c1..3b6f169 100644
--- a/arch/arm/boot/dts/kirkwood-openrd-client.dts
+++ b/arch/arm/boot/dts/kirkwood-openrd-client.dts
@@ -33,15 +33,16 @@
 
 	sound {
 		compatible = "simple-audio-card";
-		simple-audio-card,format = "i2s";
 		simple-audio-card,mclk-fs = <256>;
 
-		simple-audio-card,cpu {
-			sound-dai = <&audio0>;
-		};
-
-		simple-audio-card,codec {
-			sound-dai = <&cs42l51>;
+		simple-audio-card,dai-link {
+			format = "i2s";
+			cpu {
+				sound-dai = <&audio0>;
+			};
+			codec {
+				sound-dai = <&cs42l51>;
+			};
 		};
 	};
 };
-- 
1.8.4


WARNING: multiple messages have this Message-ID (diff)
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 7/7] ASoC: dts: kirkwood-openrd-client: To support simple card newest style.
Date: Mon, 1 Sep 2014 12:29:41 +0800	[thread overview]
Message-ID: <1409545781-20530-8-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/kirkwood-openrd-client.dts | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/kirkwood-openrd-client.dts b/arch/arm/boot/dts/kirkwood-openrd-client.dts
index 887b9c1..3b6f169 100644
--- a/arch/arm/boot/dts/kirkwood-openrd-client.dts
+++ b/arch/arm/boot/dts/kirkwood-openrd-client.dts
@@ -33,15 +33,16 @@
 
 	sound {
 		compatible = "simple-audio-card";
-		simple-audio-card,format = "i2s";
 		simple-audio-card,mclk-fs = <256>;
 
-		simple-audio-card,cpu {
-			sound-dai = <&audio0>;
-		};
-
-		simple-audio-card,codec {
-			sound-dai = <&cs42l51>;
+		simple-audio-card,dai-link {
+			format = "i2s";
+			cpu {
+				sound-dai = <&audio0>;
+			};
+			codec {
+				sound-dai = <&cs42l51>;
+			};
 		};
 	};
 };
-- 
1.8.4

WARNING: multiple messages have this Message-ID (diff)
From: Li.Xiubo@freescale.com (Xiubo Li)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv1 7/7] ASoC: dts: kirkwood-openrd-client: To support simple card newest style.
Date: Mon, 1 Sep 2014 12:29:41 +0800	[thread overview]
Message-ID: <1409545781-20530-8-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/kirkwood-openrd-client.dts | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/kirkwood-openrd-client.dts b/arch/arm/boot/dts/kirkwood-openrd-client.dts
index 887b9c1..3b6f169 100644
--- a/arch/arm/boot/dts/kirkwood-openrd-client.dts
+++ b/arch/arm/boot/dts/kirkwood-openrd-client.dts
@@ -33,15 +33,16 @@
 
 	sound {
 		compatible = "simple-audio-card";
-		simple-audio-card,format = "i2s";
 		simple-audio-card,mclk-fs = <256>;
 
-		simple-audio-card,cpu {
-			sound-dai = <&audio0>;
-		};
-
-		simple-audio-card,codec {
-			sound-dai = <&cs42l51>;
+		simple-audio-card,dai-link {
+			format = "i2s";
+			cpu {
+				sound-dai = <&audio0>;
+			};
+			codec {
+				sound-dai = <&cs42l51>;
+			};
 		};
 	};
 };
-- 
1.8.4

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

Thread overview: 67+ 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 ` Xiubo Li
2014-09-01  4:29 ` Xiubo Li
2014-09-01  4:29 ` 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  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01 13:34   ` Andrew Lunn
2014-09-01 13:34     ` Andrew Lunn
2014-09-01 13:34     ` Andrew Lunn
2014-09-01 15:13     ` Mark Brown
2014-09-01 15:13       ` Mark Brown
2014-09-01 15:13       ` Mark Brown
     [not found]       ` <20140901151356.GB29327-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2014-09-02  3:22         ` Li.Xiubo
2014-09-02  3:22           ` Li.Xiubo at freescale.com
2014-09-02  3:22           ` Li.Xiubo-KZfg59tc24xl57MIdRCFDg
2014-09-02  3:22           ` Li.Xiubo
     [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:42       ` Kuninori Morimoto
2014-09-01  7:42       ` Kuninori Morimoto
2014-09-01  7:42       ` Kuninori Morimoto
2014-09-01  7:48       ` Li.Xiubo
2014-09-01  7:48         ` Li.Xiubo at freescale.com
2014-09-01  7:48         ` Li.Xiubo
2014-09-01  7:48         ` Li.Xiubo
2014-09-01  7:50         ` Kuninori Morimoto
2014-09-01  7:50           ` [alsa-devel] " Kuninori Morimoto
2014-09-02  9:15       ` Li.Xiubo
2014-09-02  9:15         ` Li.Xiubo at freescale.com
2014-09-02  9:15         ` Li.Xiubo
2014-09-02  9:15         ` Li.Xiubo
2014-09-01 15:05     ` Jason Cooper
2014-09-01 15:05       ` Jason Cooper
2014-09-01 15:05       ` Jason Cooper
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
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29 ` [PATCHv1 3/7] ASoC: dts: vf610-twr: To support simple card newest style Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29 ` [PATCHv1 4/7] ASoC: dts: kirkwood-t5325: " Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01 13:41   ` Andrew Lunn
2014-09-01 13:41     ` Andrew Lunn
2014-09-01 13:41     ` Andrew Lunn
     [not found]     ` <20140901134148.GB11312-g2DYL2Zd6BY@public.gmane.org>
2014-09-02  9:08       ` Li.Xiubo
2014-09-02  9:08         ` Li.Xiubo at freescale.com
2014-09-02  9:08         ` Li.Xiubo-KZfg59tc24xl57MIdRCFDg
2014-09-02  9:08         ` Li.Xiubo
2014-09-01  4:29 ` [PATCHv1 5/7] ASoC: dts: r8a7740-armadillo800eva-reference: " Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29 ` [PATCHv1 6/7] ASoC: dts: sh73a0-kzm9g-reference: " Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29 ` Xiubo Li [this message]
2014-09-01  4:29   ` [PATCHv1 7/7] ASoC: dts: kirkwood-openrd-client: " Xiubo Li
2014-09-01  4:29   ` Xiubo Li
2014-09-01  4:29   ` 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-8-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 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.