[PATCH/RFT] ARM: dts: silk: Enable UHS-I SDR-50

[PATCH/RFT] ARM: dts: silk: Enable UHS-I SDR-50

[Simon Horman]

Add the "1v8" pinctrl state and sd-uhs-sdr50 property to SDHI1.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
* Compile tested only; no access to silk board
---
 arch/arm/boot/dts/r8a7794-silk.dts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/r8a7794-silk.dts b/arch/arm/boot/dts/r8a7794-silk.dts
index 4cb5278d104d..b0ef8ea75c29 100644
--- a/arch/arm/boot/dts/r8a7794-silk.dts
+++ b/arch/arm/boot/dts/r8a7794-silk.dts
@@ -226,6 +226,13 @@
 	du1_pins: du1 {
 		groups = "du1_rgb666", "du1_sync", "du1_disp", "du1_clk0_out";
 		function = "du1";
+		power-source = <3300>;
+	};
+
+	sdhi1_pins_uhs: sd1_uhs {
+		groups = "sdhi1_data4", "sdhi1_ctrl";
+		function = "sdhi1";
+		power-source = <1800>;
 	};
 
 	ssi_pins: sound {
@@ -343,6 +350,7 @@
 	vmmc-supply = <&vcc_sdhi1>;
 	vqmmc-supply = <&vccq_sdhi1>;
 	cd-gpios = <&gpio6 14 GPIO_ACTIVE_LOW>;
+	sd-uhs-sdr50;
 	status = "okay";
 };
 
-- 
2.7.0.rc3.207.g0ac5344

[PATCH/RFT] ARM: dts: silk: Enable UHS-I SDR-50

[Geert Uytterhoeven]

On Tue, Apr 18, 2017 at 7:27 AM, Simon Horman
<horms+renesas@verge.net.au> wrote:
> Add the "1v8" pinctrl state and sd-uhs-sdr50 property to SDHI1.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> * Compile tested only; no access to silk board
> ---
>  arch/arm/boot/dts/r8a7794-silk.dts | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/arch/arm/boot/dts/r8a7794-silk.dts b/arch/arm/boot/dts/r8a7794-silk.dts
> index 4cb5278d104d..b0ef8ea75c29 100644
> --- a/arch/arm/boot/dts/r8a7794-silk.dts
> +++ b/arch/arm/boot/dts/r8a7794-silk.dts
> @@ -226,6 +226,13 @@
>         du1_pins: du1 {
>                 groups = "du1_rgb666", "du1_sync", "du1_disp", "du1_clk0_out";
>                 function = "du1";
> +               power-source = <3300>;
> +       };
> +
> +       sdhi1_pins_uhs: sd1_uhs {
> +               groups = "sdhi1_data4", "sdhi1_ctrl";
> +               function = "sdhi1";
> +               power-source = <1800>;

Shouldn't this be inserted right after the existing sdhi1_pins?
Yes, the list is only partially sorted.

>         };

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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

[PATCH/RFT] ARM: dts: silk: Enable UHS-I SDR-50

[Simon Horman]

On Tue, Apr 18, 2017 at 09:22:24AM +0200, Geert Uytterhoeven wrote:
> On Tue, Apr 18, 2017 at 7:27 AM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > Add the "1v8" pinctrl state and sd-uhs-sdr50 property to SDHI1.
> >
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > * Compile tested only; no access to silk board
> > ---
> >  arch/arm/boot/dts/r8a7794-silk.dts | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/r8a7794-silk.dts b/arch/arm/boot/dts/r8a7794-silk.dts
> > index 4cb5278d104d..b0ef8ea75c29 100644
> > --- a/arch/arm/boot/dts/r8a7794-silk.dts
> > +++ b/arch/arm/boot/dts/r8a7794-silk.dts
> > @@ -226,6 +226,13 @@
> >         du1_pins: du1 {
> >                 groups = "du1_rgb666", "du1_sync", "du1_disp", "du1_clk0_out";
> >                 function = "du1";
> > +               power-source = <3300>;
> > +       };
> > +
> > +       sdhi1_pins_uhs: sd1_uhs {
> > +               groups = "sdhi1_data4", "sdhi1_ctrl";
> > +               function = "sdhi1";
> > +               power-source = <1800>;
> 
> Shouldn't this be inserted right after the existing sdhi1_pins?
> Yes, the list is only partially sorted.

Oh wow, I messed that up completely.
Let me post a v2.