Hello Simon, Geert thank you for your patience on this one. We found out that there is an HW issue on SDHI0 , SDHC_CD# and SDHC_CMD (on the carrier board) are pulled up to 3.3V, therefore we are dropping UHS on this interface, and as result we are dropping this patch. We will send a V2 shortly. Thanks, Fabrizio > -----Original Message----- > From: Simon Horman [mailto:horms@xxxxxxxxxxxx] > Sent: 15 September 2017 09:06 > To: Chris Paterson <Chris.Paterson2@xxxxxxxxxxx> > Cc: Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; Magnus Damm <magnus.damm@xxxxxxxxx>; > Russell King <linux@xxxxxxxxxxxxxxx>; Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; linux-renesas- > soc@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH 4/8] ARM: dts: iwg22d: Enable SDHI0 controller > > On Wed, Sep 13, 2017 at 06:05:37PM +0100, Chris Paterson wrote: > > From: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > Enable the SDHI0 controller on iWave RZ/G1E carrier board. > > ... > > > @@ -63,3 +88,15 @@ > > micrel,led-mode = <1>; > > }; > > }; > > + > > +&sdhi0 { > > +pinctrl-0 = <&sdhi0_pins>; > > +pinctrl-1 = <&sdhi0_pins_uhs>; > > +pinctrl-names = "default", "state_uhs"; > > + > > +vmmc-supply = <®_3p3v>; > > +vqmmc-supply = <&vccq_sdhi0>; > > +cd-gpios = <&gpio6 6 GPIO_ACTIVE_LOW>; > > I take that the absence of a wp-gpio means that that this is a µSD slot. > Could you help me by documenting this correctly on > http://elinux.org/index.php?title=Renesas-MMC-Enabled-Speeds ? > > For some reason I thought that SDHI0 wasn't exposed at all, so I guess > my reading of the documentation was incorrect. > > > I think you also want sd-uhs-sdr50 here too. > You can test it by removing the sd-uhs-sdr104 property. > > > +sd-uhs-sdr104; > > +status = "okay"; > > +}; > > -- > > 1.9.1 > > Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
