[PATCH] ASoC: simple-card: Add device's module clock selection.

[PATCH] ASoC: simple-card: Add device's module clock selection.

[Xiubo Li]

Try to get the device's module clock if the dt has no clocks and
system-clock-frequency properties.

Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
---
 a.out                           |  0
 sound/soc/generic/simple-card.c | 21 ++++++++++++++++++---
 2 files changed, 18 insertions(+), 3 deletions(-)
 create mode 100644 a.out

diff --git a/a.out b/a.out
new file mode 100644
index 0000000..e69de29
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index 6ce9745..c0d92813 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -90,14 +90,29 @@ asoc_simple_card_sub_parse_of(struct device_node *np,
 	 * dai->sysclk come from
 	 *  "clocks = <&xxx>" (if system has common clock)
 	 *  or "system-clock-frequency = <xxx>"
+	 *  or device's module clock.
 	 */
-	clk = of_clk_get(np, 0);
-	if (IS_ERR(clk))
+	if (of_property_read_bool(np, "clocks")) {
+		clk = of_clk_get(np, 0);
+		if (IS_ERR(clk)) {
+			ret = PTR_ERR(clk);
+			goto parse_error;
+		}
+
+		dai->sysclk = clk_get_rate(clk);
+	} else if (of_property_read_bool(np, "system-clock-frequency")) {
 		of_property_read_u32(np,
 				     "system-clock-frequency",
 				     &dai->sysclk);
-	else
+	} else {
+		clk = of_clk_get(*node, 0);
+		if (IS_ERR(clk)) {
+			ret = PTR_ERR(clk);
+			goto parse_error;
+		}
+
 		dai->sysclk = clk_get_rate(clk);
+	}
 
 	ret = 0;
 
-- 
1.8.4

[PATCH] ASoC: simple-card: Add device's module clock selection.

[Xiubo Li]

Try to get the device's module clock if the dt has no clocks and
system-clock-frequency properties.

Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
---
 a.out                           |  0
 sound/soc/generic/simple-card.c | 21 ++++++++++++++++++---
 2 files changed, 18 insertions(+), 3 deletions(-)
 create mode 100644 a.out

diff --git a/a.out b/a.out
new file mode 100644
index 0000000..e69de29
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index 6ce9745..c0d92813 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -90,14 +90,29 @@ asoc_simple_card_sub_parse_of(struct device_node *np,
 	 * dai->sysclk come from
 	 *  "clocks = <&xxx>" (if system has common clock)
 	 *  or "system-clock-frequency = <xxx>"
+	 *  or device's module clock.
 	 */
-	clk = of_clk_get(np, 0);
-	if (IS_ERR(clk))
+	if (of_property_read_bool(np, "clocks")) {
+		clk = of_clk_get(np, 0);
+		if (IS_ERR(clk)) {
+			ret = PTR_ERR(clk);
+			goto parse_error;
+		}
+
+		dai->sysclk = clk_get_rate(clk);
+	} else if (of_property_read_bool(np, "system-clock-frequency")) {
 		of_property_read_u32(np,
 				     "system-clock-frequency",
 				     &dai->sysclk);
-	else
+	} else {
+		clk = of_clk_get(*node, 0);
+		if (IS_ERR(clk)) {
+			ret = PTR_ERR(clk);
+			goto parse_error;
+		}
+
 		dai->sysclk = clk_get_rate(clk);
+	}
 
 	ret = 0;
 
-- 
1.8.4

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 180 bytes --]

On Mon, Dec 23, 2013 at 03:25:38PM +0800, Xiubo Li wrote:
> Try to get the device's module clock if the dt has no clocks and
> system-clock-frequency properties.

Applied, thanks.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 180 bytes --]

On Mon, Dec 23, 2013 at 03:25:38PM +0800, Xiubo Li wrote:
> Try to get the device's module clock if the dt has no clocks and
> system-clock-frequency properties.

Applied, thanks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Lars-Peter Clausen]

On 12/23/2013 08:25 AM, Xiubo Li wrote:
> diff --git a/a.out b/a.out
> new file mode 100644
> index 0000000..e69de29

This patch did add a empty a.out file in the toplevel folder.

- Lars

Re: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock selection.

[Lars-Peter Clausen]

On 12/23/2013 08:25 AM, Xiubo Li wrote:
> diff --git a/a.out b/a.out
> new file mode 100644
> index 0000000..e69de29

This patch did add a empty a.out file in the toplevel folder.

- Lars

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 363 bytes --]

On Wed, Jan 01, 2014 at 01:11:09PM +0100, Lars-Peter Clausen wrote:
> On 12/23/2013 08:25 AM, Xiubo Li wrote:
> > diff --git a/a.out b/a.out
> > new file mode 100644
> > index 0000000..e69de29

> This patch did add a empty a.out file in the toplevel folder.

Removed now.  This is one of several patches from Xiubo that have done
this - Xiubo, please check this.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock selection.

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 363 bytes --]

On Wed, Jan 01, 2014 at 01:11:09PM +0100, Lars-Peter Clausen wrote:
> On 12/23/2013 08:25 AM, Xiubo Li wrote:
> > diff --git a/a.out b/a.out
> > new file mode 100644
> > index 0000000..e69de29

> This patch did add a empty a.out file in the toplevel folder.

Removed now.  This is one of several patches from Xiubo that have done
this - Xiubo, please check this.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

RE: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock selection.

[Li.Xiubo]

Hi Mark, Lars


> Subject: Re: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock
> selection.
> 
> On Wed, Jan 01, 2014 at 01:11:09PM +0100, Lars-Peter Clausen wrote:
> > On 12/23/2013 08:25 AM, Xiubo Li wrote:
> > > diff --git a/a.out b/a.out
> > > new file mode 100644
> > > index 0000000..e69de29
> 
> > This patch did add a empty a.out file in the toplevel folder.
> 
> Removed now.  This is one of several patches from Xiubo that have done
> this - Xiubo, please check this.
>

Yes, this had been removed by Mark already.

Thanks,

--
Best Regards,
Xiubo

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Li.Xiubo]

> Subject: Re: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock
> selection.
> 
> On 12/23/2013 08:25 AM, Xiubo Li wrote:
> > diff --git a/a.out b/a.out
> > new file mode 100644
> > index 0000000..e69de29
> 
> This patch did add a empty a.out file in the toplevel folder.
> 
> - Lars
> 

Yes, sorry, my mistake.

Thanks very much

--
Best regards,
Xiubo
 

RE: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock selection.

[Li.Xiubo]

> Subject: Re: [alsa-devel] [PATCH] ASoC: simple-card: Add device's module clock
> selection.
> 
> On 12/23/2013 08:25 AM, Xiubo Li wrote:
> > diff --git a/a.out b/a.out
> > new file mode 100644
> > index 0000000..e69de29
> 
> This patch did add a empty a.out file in the toplevel folder.
> 
> - Lars
> 

Yes, sorry, my mistake.

Thanks very much

--
Best regards,
Xiubo
 

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Jean-Francois Moine]

Li,

I am trying to have many DAI links in the simple-card, but I get an
error on getting the device clock: only the dma device needs a clock
and the spdif codec has none. What was the purpose of this default
device clock?

-- 
Ken ar c'hentañ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH] ASoC: simple-card: Add device's module clock selection.

[Li.Xiubo]

> Subject: Re: [PATCH] ASoC: simple-card: Add device's module clock selection.
> 
> Li,
> 
> I am trying to have many DAI links in the simple-card, but I get an
> error on getting the device clock: only the dma device needs a clock
> and the spdif codec has none. What was the purpose of this default
> device clock?
> 

Yes, for spdif there is no need to do the CPU/CODEC DAI devices' sysclk setting,
but for many others like SAI,SSI... need.

I will send another patch to fix this.

Thanks,

--
Best Regards,
Xiubo