[PATCH 4/8] ARM: dts: iwg22d: Enable SDHI0 controller

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 at verge.net.au]
Sent: 15 September 2017 09:06
To: Chris Paterson <redacted>
Cc: Rob Herring <robh+dt@kernel.org>; Mark Rutland <mark.rutland@arm.com>; Magnus Damm <magnus.damm@gmail.com>;
Russell King [off-list ref]; Fabrizio Castro [off-list ref]; devicetree at vger.kernel.org; linux-renesas-
soc at vger.kernel.org; linux-arm-kernel at lists.infradead.org
Subject: Re: [PATCH 4/8] ARM: dts: iwg22d: Enable SDHI0 controller

On Wed, Sep 13, 2017 at 06:05:37PM +0100, Chris Paterson wrote:

quoted

From: Fabrizio Castro <redacted>

Enable the SDHI0 controller on iWave RZ/G1E carrier board.

...

quoted

@@ -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 = <&reg_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.

quoted

+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.