diff mbox

[RFC,6/7] ARM: shmobile: r8a7790-lager.dts: Set sdhi and mmcif clock rates

Message ID 1430397166.5802.45.camel@xylophone.i.decadent.org.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Ben Hutchings April 30, 2015, 12:32 p.m. UTC 
From: Ben Dooks <ben.dooks@codethink.co.uk>

[bwh: Fold in fix from Ian Molton]
Signed-off-by: Ben Hutchings <ben.hutchings@codethink.co.uk>
---
 arch/arm/boot/dts/r8a7790-lager.dts |   11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Sergei Shtylyov April 30, 2015, 4:06 p.m. UTC | #1 
On 04/30/2015 03:32 PM, Ben Hutchings wrote:

> From: Ben Dooks <ben.dooks@codethink.co.uk>

> [bwh: Fold in fix from Ian Molton]
> Signed-off-by: Ben Hutchings <ben.hutchings@codethink.co.uk>

    Not signed off by Ben Dooks?

[...]

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Simon Horman May 1, 2015, 12:59 a.m. UTC | #2 
Hi Ben,

Please use the following prefix for r8a7790-lager.dts patches:

ARM: shmobile: lager:

On Thu, Apr 30, 2015 at 01:32:46PM +0100, Ben Hutchings wrote:
> From: Ben Dooks <ben.dooks@codethink.co.uk>
> 
> [bwh: Fold in fix from Ian Molton]
> Signed-off-by: Ben Hutchings <ben.hutchings@codethink.co.uk>
> ---
>  arch/arm/boot/dts/r8a7790-lager.dts |   11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
> index aaa4f258e279..343ec0ccc8df 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -413,6 +413,11 @@
>  	vmmc-supply = <&fixedregulator3v3>;
>  	bus-width = <8>;
>  	non-removable;
> +
> +	assigned-clocks = <&mstp3_clks R8A7790_CLK_MMCIF1>;
> +	assigned-clock-rates = <97500000>;
> +	max-frequency = <50000000>;
> +
>  	status = "okay";
>  };
>  
> @@ -488,6 +493,9 @@
>  	pinctrl-0 = <&sdhi0_pins>;
>  	pinctrl-names = "default";
>  
> +	assigned-clocks = <&mstp3_clks R8A7790_CLK_SDHI0>;
> +	assigned-clock-rates = <156000000>;
> +
>  	vmmc-supply = <&vcc_sdhi0>;
>  	vqmmc-supply = <&vccq_sdhi0>;
>  	cd-gpios = <&gpio3 6 GPIO_ACTIVE_LOW>;
> @@ -498,6 +506,9 @@
>  	pinctrl-0 = <&sdhi2_pins>;
>  	pinctrl-names = "default";
>  
> +	assigned-clocks = <&mstp3_clks R8A7790_CLK_SDHI2>;
> +	assigned-clock-rates = <97500000>;
> +
>  	vmmc-supply = <&vcc_sdhi2>;
>  	vqmmc-supply = <&vccq_sdhi2>;
>  	cd-gpios = <&gpio3 22 GPIO_ACTIVE_LOW>;
> -- 
> 1.7.10.4
> 
> 
> 
> 
> --
> 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
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ben Hutchings May 6, 2015, 1:44 a.m. UTC | #3 
On Thu, 2015-04-30 at 19:06 +0300, Sergei Shtylyov wrote:
> On 04/30/2015 03:32 PM, Ben Hutchings wrote:
> 
> > From: Ben Dooks <ben.dooks@codethink.co.uk>
> 
> > [bwh: Fold in fix from Ian Molton]
> > Signed-off-by: Ben Hutchings <ben.hutchings@codethink.co.uk>
> 
>     Not signed off by Ben Dooks?

I'm quite capable of inferring that a colleague intended their work to
be released under GPL, thanks.

Ben.


--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ben Hutchings May 6, 2015, 1:46 a.m. UTC | #4 
On Fri, 2015-05-01 at 09:59 +0900, Simon Horman wrote:
> Hi Ben,
> 
> Please use the following prefix for r8a7790-lager.dts patches:
> 
> ARM: shmobile: lager:
[...]

OK, I'll do that in future.  I thought I'd seen the long name used more
often in the log, but I was mistaken.

Ben.


--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Simon Horman May 7, 2015, 4:29 a.m. UTC | #5 
On Wed, May 06, 2015 at 02:46:57AM +0100, Ben Hutchings wrote:
> On Fri, 2015-05-01 at 09:59 +0900, Simon Horman wrote:
> > Hi Ben,
> > 
> > Please use the following prefix for r8a7790-lager.dts patches:
> > 
> > ARM: shmobile: lager:
> [...]
> 
> OK, I'll do that in future.  I thought I'd seen the long name used more
> often in the log, but I was mistaken.

Possibly, I don't always make sure the prefixes are as consistent as I should.
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" 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-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index aaa4f258e279..343ec0ccc8df 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -413,6 +413,11 @@ 
 	vmmc-supply = <&fixedregulator3v3>;
 	bus-width = <8>;
 	non-removable;
+
+	assigned-clocks = <&mstp3_clks R8A7790_CLK_MMCIF1>;
+	assigned-clock-rates = <97500000>;
+	max-frequency = <50000000>;
+
 	status = "okay";
 };
 
@@ -488,6 +493,9 @@ 
 	pinctrl-0 = <&sdhi0_pins>;
 	pinctrl-names = "default";
 
+	assigned-clocks = <&mstp3_clks R8A7790_CLK_SDHI0>;
+	assigned-clock-rates = <156000000>;
+
 	vmmc-supply = <&vcc_sdhi0>;
 	vqmmc-supply = <&vccq_sdhi0>;
 	cd-gpios = <&gpio3 6 GPIO_ACTIVE_LOW>;
@@ -498,6 +506,9 @@ 
 	pinctrl-0 = <&sdhi2_pins>;
 	pinctrl-names = "default";
 
+	assigned-clocks = <&mstp3_clks R8A7790_CLK_SDHI2>;
+	assigned-clock-rates = <97500000>;
+
 	vmmc-supply = <&vcc_sdhi2>;
 	vqmmc-supply = <&vccq_sdhi2>;
 	cd-gpios = <&gpio3 22 GPIO_ACTIVE_LOW>;