diff mbox

ARM: shmobile: r8a7790: fix max-frequency for SDHI

Message ID 1460897424-7256-1-git-send-email-wsa@the-dreams.de (mailing list archive)
State Superseded
Delegated to: Simon Horman
Headers show

Commit Message

Wolfram Sang April 17, 2016, 12:50 p.m. UTC 
From: Wolfram Sang <wsa+renesas@sang-engineering.com>

The wrong values come from an old datasheet. Fix them.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 arch/arm/boot/dts/r8a7790.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Geert Uytterhoeven April 18, 2016, 8:22 a.m. UTC | #1 
On Sun, Apr 17, 2016 at 2:50 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
>
> The wrong values come from an old datasheet. Fix them.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Simon Horman April 19, 2016, 11:06 p.m. UTC | #2 
On Mon, Apr 18, 2016 at 10:22:24AM +0200, Geert Uytterhoeven wrote:
> On Sun, Apr 17, 2016 at 2:50 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
> > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> >
> > The wrong values come from an old datasheet. Fix them.
> >
> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, I have squashed that into the patch which added the incorrect
values as I need to resubmit the relevant pull request anyway.
Simon Horman April 19, 2016, 11:07 p.m. UTC | #3 
On Wed, Apr 20, 2016 at 09:06:10AM +1000, Simon Horman wrote:
> On Mon, Apr 18, 2016 at 10:22:24AM +0200, Geert Uytterhoeven wrote:
> > On Sun, Apr 17, 2016 at 2:50 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
> > > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > >
> > > The wrong values come from an old datasheet. Fix them.
> > >
> > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > 
> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Thanks, I have squashed that into the patch which added the incorrect
> values as I need to resubmit the relevant pull request anyway.

I take that back, I have queued it up separately to preserve
the different authors.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/r8a7790.dtsi b/arch/arm/boot/dts/r8a7790.dtsi
index 1a3b6b08f1f3c9..e75baf369ccfb4 100644
--- a/arch/arm/boot/dts/r8a7790.dtsi
+++ b/arch/arm/boot/dts/r8a7790.dtsi
@@ -607,7 +607,7 @@ 
 		dmas = <&dmac1 0xcd>, <&dmac1 0xce>,
 		       <&dmac0 0xcd>, <&dmac0 0xce>;
 		dma-names = "tx", "rx", "tx", "rx";
-		max-frequency = <156000000>;
+		max-frequency = <195000000>;
 		power-domains = <&cpg_clocks>;
 		status = "disabled";
 	};
@@ -620,7 +620,7 @@ 
 		dmas = <&dmac1 0xc9>, <&dmac1 0xca>,
 		       <&dmac0 0xc9>, <&dmac0 0xca>;
 		dma-names = "tx", "rx", "tx", "rx";
-		max-frequency = <156000000>;
+		max-frequency = <195000000>;
 		power-domains = <&cpg_clocks>;
 		status = "disabled";
 	};