[PATCH 1/2] ARM: at91/dt: sam9263: Fix typo: ac91_clk -> ac97_clk

[PATCH 1/2] ARM: at91/dt: sam9263: Fix typo: ac91_clk -> ac97_clk

[Alexander Stein]

That clock should be called ac97_clk.

Signed-off-by: Alexander Stein <alexanders83@web.de>
---
 arch/arm/boot/dts/at91sam9263.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
index 7989756..5ed2810 100644
--- a/arch/arm/boot/dts/at91sam9263.dtsi
+++ b/arch/arm/boot/dts/at91sam9263.dtsi
@@ -294,7 +294,7 @@
 						reg = <17>;
 					};
 
-					ac91_clk: ac97_clk {
+					ac97_clk: ac97_clk {
 						#clock-cells = <0>;
 						reg = <18>;
 					};
-- 
2.2.0

[PATCH 2/2] ARM: at91/dt: sam9263: Add missing clocks to ac97 node

[Alexander Stein]

The clock node specifying 'ac97_clk' was missing.

Signed-off-by: Alexander Stein <alexanders83@web.de>
---
 arch/arm/boot/dts/at91sam9263.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
index 5ed2810..2d9d0ad 100644
--- a/arch/arm/boot/dts/at91sam9263.dtsi
+++ b/arch/arm/boot/dts/at91sam9263.dtsi
@@ -821,6 +821,8 @@
 				interrupts = <18 IRQ_TYPE_LEVEL_HIGH 5>;
 				pinctrl-names = "default";
 				pinctrl-0 = <&pinctrl_ac97>;
+				clocks = <&ac97_clk>;
+				clock-names = "ac97_clk";
 				status = "disabled";
 			};
 
-- 
2.2.0

[PATCH 1/2] ARM: at91/dt: sam9263: Fix typo: ac91_clk -> ac97_clk

[Alexander Stein]

On Friday 05 December 2014, 15:16:55 wrote Alexander Stein:
> That clock should be called ac97_clk.
> 
> Signed-off-by: Alexander Stein <alexanders83@web.de>
> ---
>  arch/arm/boot/dts/at91sam9263.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> index 7989756..5ed2810 100644
> --- a/arch/arm/boot/dts/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> @@ -294,7 +294,7 @@
>  						reg = <17>;
>  					};
>  
> -					ac91_clk: ac97_clk {
> +					ac97_clk: ac97_clk {
>  						#clock-cells = <0>;
>  						reg = <18>;
>  					};
> 

Please ignore those 2 patches for now. I made a mistake during local rebase.

Best regards,
Alexander

[PATCH 1/2] ARM: at91/dt: sam9263: Fix typo: ac91_clk -> ac97_clk

[Nicolas Ferre]

Le 05/12/2014 16:00, Alexander Stein a ?crit :
> On Friday 05 December 2014, 15:16:55 wrote Alexander Stein:
>> That clock should be called ac97_clk.
>>
>> Signed-off-by: Alexander Stein <alexanders83@web.de>
>> ---
>>  arch/arm/boot/dts/at91sam9263.dtsi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
>> index 7989756..5ed2810 100644
>> --- a/arch/arm/boot/dts/at91sam9263.dtsi
>> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
>> @@ -294,7 +294,7 @@
>>  						reg = <17>;
>>  					};
>>  
>> -					ac91_clk: ac97_clk {
>> +					ac97_clk: ac97_clk {
>>  						#clock-cells = <0>;
>>  						reg = <18>;
>>  					};
>>
> 
> Please ignore those 2 patches for now. I made a mistake during local rebase.

Well at least this one (1/2) is valid and applies on my new
at91-3.20-dt. So I take it for now: tell me if you feel otherwise.

Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Bye,


> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 


-- 
Nicolas Ferre

[PATCH 2/2] ARM: at91/dt: sam9263: Add missing clocks to ac97 node

[Nicolas Ferre]

Le 05/12/2014 15:16, Alexander Stein a ?crit :
> The clock node specifying 'ac97_clk' was missing.
> 
> Signed-off-by: Alexander Stein <alexanders83@web.de>
> ---
>  arch/arm/boot/dts/at91sam9263.dtsi | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> index 5ed2810..2d9d0ad 100644
> --- a/arch/arm/boot/dts/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> @@ -821,6 +821,8 @@
>  				interrupts = <18 IRQ_TYPE_LEVEL_HIGH 5>;
>  				pinctrl-names = "default";
>  				pinctrl-0 = <&pinctrl_ac97>;
> +				clocks = <&ac97_clk>;
> +				clock-names = "ac97_clk";
>  				status = "disabled";
>  			};

I see, this one comes after the addition of the node itself. I ignore it.

Bye,
-- 
Nicolas Ferre