All of lore.kernel.org
 help / color / mirror / Atom feed
From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/5] ARM: mvebu: use clocks property for serial ports
Date: Fri, 18 Apr 2014 09:49:05 +0200	[thread overview]
Message-ID: <5350D8F1.5010602@free-electrons.com> (raw)
In-Reply-To: <1397806908-7550-4-git-send-email-thomas.petazzoni@free-electrons.com>

On 18/04/2014 09:41, Thomas Petazzoni wrote:
> Back when the Armada 370 and Armada XP initial support was introduced,
> the only way to pass the clock frequency to the of_serial driver was
> through a clock-frequency Device Tree property.
> 
> Thanks to 0bbeb3c3e84bc963d1c66661e082d207023b0e5c ('of serial port
> driver - add clk_get_rate() support'), it is possible to use the
> standard 'clocks' DT property to reference the clock used for a
> particular UART controller. This clock is then used by the of_serial
> driver to retrieve the clock rate.
> 
> This commit modifies the SoC-level Device Tree files of Armada 370,
> Armada XP, Armada 375 and Armada 38x to use this possibility. Since
> there is no gatable clock for the UART controllers, we simply
> reference the TCLK, which is the main SoC clock for the peripherals.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>


Thanks,

Gregory

> ---
>  arch/arm/boot/dts/armada-370-xp.dtsi | 2 ++
>  arch/arm/boot/dts/armada-375.dtsi    | 2 ++
>  arch/arm/boot/dts/armada-38x.dtsi    | 2 ++
>  arch/arm/boot/dts/armada-xp.dtsi     | 2 ++
>  4 files changed, 8 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/armada-370-xp.dtsi b/arch/arm/boot/dts/armada-370-xp.dtsi
> index bbb40f6..3bc1fa80 100644
> --- a/arch/arm/boot/dts/armada-370-xp.dtsi
> +++ b/arch/arm/boot/dts/armada-370-xp.dtsi
> @@ -157,6 +157,7 @@
>  				reg-shift = <2>;
>  				interrupts = <41>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  			serial at 12100 {
> @@ -165,6 +166,7 @@
>  				reg-shift = <2>;
>  				interrupts = <42>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  
> diff --git a/arch/arm/boot/dts/armada-375.dtsi b/arch/arm/boot/dts/armada-375.dtsi
> index 3877693..0c50c44 100644
> --- a/arch/arm/boot/dts/armada-375.dtsi
> +++ b/arch/arm/boot/dts/armada-375.dtsi
> @@ -194,6 +194,7 @@
>  				reg-shift = <2>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  
> @@ -203,6 +204,7 @@
>  				reg-shift = <2>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  
> diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi
> index a064f59..63fdf2d 100644
> --- a/arch/arm/boot/dts/armada-38x.dtsi
> +++ b/arch/arm/boot/dts/armada-38x.dtsi
> @@ -174,6 +174,7 @@
>  				reg-shift = <2>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  
> @@ -183,6 +184,7 @@
>  				reg-shift = <2>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  
> diff --git a/arch/arm/boot/dts/armada-xp.dtsi b/arch/arm/boot/dts/armada-xp.dtsi
> index abb9f9d..a9fff62 100644
> --- a/arch/arm/boot/dts/armada-xp.dtsi
> +++ b/arch/arm/boot/dts/armada-xp.dtsi
> @@ -58,6 +58,7 @@
>  				reg-shift = <2>;
>  				interrupts = <43>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  			serial at 12300 {
> @@ -66,6 +67,7 @@
>  				reg-shift = <2>;
>  				interrupts = <44>;
>  				reg-io-width = <1>;
> +				clocks = <&coreclk 0>;
>  				status = "disabled";
>  			};
>  
> 


-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  reply	other threads:[~2014-04-18  7:49 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-18  7:41 [PATCH 0/5] ARM: mvebu: misc Device Tree fixes and updates Thomas Petazzoni
2014-04-18  7:41 ` [PATCH 1/5] ARM: mvebu: specify I2C bus frequency on Armada 370 DB Thomas Petazzoni
2014-04-18  7:47   ` Gregory CLEMENT
2014-04-18  7:41 ` [PATCH 2/5] ARM: mvebu: enable the SATA interface on Armada 375 DB Thomas Petazzoni
2014-04-18  7:47   ` Gregory CLEMENT
2014-04-18  7:41 ` [PATCH 3/5] ARM: mvebu: use clocks property for serial ports Thomas Petazzoni
2014-04-18  7:49   ` Gregory CLEMENT [this message]
2014-04-18  7:41 ` [PATCH 4/5] ARM: mvebu: remove clock-frequency of serial port Device Tree nodes Thomas Petazzoni
2014-04-18  7:49   ` Gregory CLEMENT
2014-04-18  7:41 ` [PATCH 5/5] ARM: mvebu: don't use clocks property in UART node for Netgear RN2120 Thomas Petazzoni
2014-04-18  7:50   ` Gregory CLEMENT
2014-04-20 19:00 ` [PATCH 0/5] ARM: mvebu: misc Device Tree fixes and updates Jason Cooper
2014-04-26  0:37 ` Jason Cooper

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=5350D8F1.5010602@free-electrons.com \
    --to=gregory.clement@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.