diff mbox

[1/2] ARM: shmobile: r8a7790 dtsi: Add CPG/MSTP Clock Domain for sound

Message ID 6507c4efd2ca4a3dccedbc5b4724f9faabf97fca.1443752733.git.horms+renesas@verge.net.au (mailing list archive)
State Accepted
Commit 6507c4efd2ca4a3dccedbc5b4724f9faabf97fca
Headers show

Commit Message

Simon Horman Oct. 2, 2015, 2:29 a.m. UTC
From: Geert Uytterhoeven <geert+renesas@glider.be>

484adb005886 ("ARM: shmobile: r8a7790 dtsi: Add CPG/MSTP Clock Domain")
added CPG/MSTP clock-cells domain support, but it was missing sound
support. This patch adds it.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Tested-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
[horms: Updated commit id referred to in changelog]
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/r8a7790.dtsi | 1 +
 1 file changed, 1 insertion(+)

Comments

Sergei Shtylyov Oct. 2, 2015, 11:29 a.m. UTC | #1
Hello.

On 10/2/2015 5:29 AM, Simon Horman wrote:

> From: Geert Uytterhoeven <geert+renesas@glider.be>
>
> 484adb005886 ("ARM: shmobile: r8a7790 dtsi: Add CPG/MSTP Clock Domain")
> added CPG/MSTP clock-cells domain support, but it was missing sound
> support. This patch adds it.

    Really? I'm only seeing "power-domains" prop added...

> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Tested-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> [horms: Updated commit id referred to in changelog]
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>   arch/arm/boot/dts/r8a7790.dtsi | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
> index a0b2a79cbfbd..4624d0f2a754 100644
> --- a/arch/arm/boot/dts/r8a7790.dtsi
> +++ b/arch/arm/boot/dts/r8a7790.dtsi
> @@ -1627,6 +1627,7 @@
>   				"mix.0", "mix.1",
>   				"dvc.0", "dvc.1",
>   				"clk_a", "clk_b", "clk_c", "clk_i";
> +		power-domains = <&cpg_clocks>;
>
>   		status = "disabled";
>

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sergei Shtylyov Oct. 2, 2015, 11:32 a.m. UTC | #2
On 10/2/2015 2:29 PM, Sergei Shtylyov wrote:

>> From: Geert Uytterhoeven <geert+renesas@glider.be>
>>
>> 484adb005886 ("ARM: shmobile: r8a7790 dtsi: Add CPG/MSTP Clock Domain")
>> added CPG/MSTP clock-cells domain support, but it was missing sound
>> support. This patch adds it.
>
>     Really? I'm only seeing "power-domains" prop added...

    Oh, I understand now. The log seems somewhat poorly worded though...

>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> Tested-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>> [horms: Updated commit id referred to in changelog]
>> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
[...]

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Arnd Bergmann Oct. 6, 2015, 12:37 p.m. UTC | #3
On Friday 02 October 2015 14:32:49 Sergei Shtylyov wrote:
> On 10/2/2015 2:29 PM, Sergei Shtylyov wrote:
> 
> >> From: Geert Uytterhoeven <geert+renesas@glider.be>
> >>
> >> 484adb005886 ("ARM: shmobile: r8a7790 dtsi: Add CPG/MSTP Clock Domain")
> >> added CPG/MSTP clock-cells domain support, but it was missing sound
> >> support. This patch adds it.
> >
> >     Really? I'm only seeing "power-domains" prop added...
> 
>     Oh, I understand now. The log seems somewhat poorly worded though...

I understood it right and merged it as is, but I agree this could
have been clearer.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index a0b2a79cbfbd..4624d0f2a754 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -1627,6 +1627,7 @@ 
 				"mix.0", "mix.1",
 				"dvc.0", "dvc.1",
 				"clk_a", "clk_b", "clk_c", "clk_i";
+		power-domains = <&cpg_clocks>;
 
 		status = "disabled";