From patchwork Mon Feb 22 02:39:13 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Horman X-Patchwork-Id: 8370351 Return-Path: X-Original-To: patchwork-linux-mmc@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 0A9CAC0554 for ; Mon, 22 Feb 2016 02:39:26 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 26823202F8 for ; Mon, 22 Feb 2016 02:39:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E85692034C for ; Mon, 22 Feb 2016 02:39:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752595AbcBVCjW (ORCPT ); Sun, 21 Feb 2016 21:39:22 -0500 Received: from kirsty.vergenet.net ([202.4.237.240]:42457 "EHLO kirsty.vergenet.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752489AbcBVCjV (ORCPT ); Sun, 21 Feb 2016 21:39:21 -0500 Received: from reginn.isobedori.kobe.vergenet.net (p5023-ipbfp803kobeminato.hyogo.ocn.ne.jp [123.221.39.23]) by kirsty.vergenet.net (Postfix) with ESMTPA id 110D925B7C9; Mon, 22 Feb 2016 13:39:19 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=verge.net.au; s=mail; t=1456108759; bh=sDZjCTBTnSQcN/VQoffieDP75EqFA0Rc0Us5mjmWC8U=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=i9xU1N2XV3P+o9/veWOgcoTSLktT+x5DoImtU0V1rytRf+IhSM6gBuapUlrtemdTm v7zBLN0nSNdCfku9OuyiVE/T4DkZLPZBETgH1+8i8E1o/VH7ELe2xV3HxAhIo0ce/M vYMumuwFBMKy3zl1ybCaMtiGVoCpKKyN/R4ZB2aQ= Received: by reginn.isobedori.kobe.vergenet.net (Postfix, from userid 7100) id 102A19403D7; Mon, 22 Feb 2016 11:39:13 +0900 (JST) Date: Mon, 22 Feb 2016 11:39:13 +0900 From: Simon Horman To: Wolfram Sang Cc: linux-mmc@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Dirk Behme , Ulf Hansson , Geert Uytterhoeven Subject: Re: [PATCH RESEND 3/3] arm64: dts: r8a7795: salvator-x: enable SDHI0 & 3 Message-ID: <20160222023910.GA19765@verge.net.au> References: <1455548510-1690-1-git-send-email-wsa@the-dreams.de> <1455548510-1690-4-git-send-email-wsa@the-dreams.de> <20160219175823.GA1392@katana> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20160219175823.GA1392@katana> Organisation: Horms Solutions Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-mmc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Fri, Feb 19, 2016 at 06:58:23PM +0100, Wolfram Sang wrote: > On Mon, Feb 15, 2016 at 04:01:50PM +0100, Wolfram Sang wrote: > > From: Ai Kyuse > > > > Add the exposed SD card slots. The on-board eMMC needs to wait until we > > fixed the 8bit support (it works with 4bit if you really want it now). > > > > Signed-off-by: Ai Kyuse > > Signed-off-by: Wolfram Sang > > Simon, if you apply this, can you drop the sentence inside the brackets? > It is probably not true until I fix the timeout handling. Or do you want > me to resend? Sure, I have dropped the text in (). I have queued up the following after resolving a minor conflict. There was also some fuzz when I applied patch 2/2 so I'd appreciate it if you could double check that too. From: Ai Kyuse Date: Mon, 15 Feb 2016 16:01:50 +0100 Subject: [PATCH] arm64: dts: r8a7795: salvator-x: enable SDHI0 & 3 Add the exposed SD card slots. The on-board eMMC needs to wait until we fixed the 8bit support. Signed-off-by: Ai Kyuse Signed-off-by: Wolfram Sang Signed-off-by: Simon Horman --- arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 83 ++++++++++++++++++++++ 1 file changed, 83 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts index 31ace9c1f79d..53541351a56d 100644 --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts @@ -33,6 +33,7 @@ /dts-v1/; #include "r8a7795.dtsi" +#include / { model = "Renesas Salvator-X board based on r8a7795"; @@ -61,6 +62,54 @@ clock-frequency = <24576000>; }; + vcc_sdhi0: regulator@1 { + compatible = "regulator-fixed"; + + regulator-name = "SDHI0 Vcc"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + + gpio = <&gpio5 2 GPIO_ACTIVE_HIGH>; + enable-active-high; + }; + + vccq_sdhi0: regulator@2 { + compatible = "regulator-gpio"; + + regulator-name = "SDHI0 VccQ"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <3300000>; + + gpios = <&gpio5 1 GPIO_ACTIVE_HIGH>; + gpios-states = <1>; + states = <3300000 1 + 1800000 0>; + }; + + vcc_sdhi3: regulator@3 { + compatible = "regulator-fixed"; + + regulator-name = "SDHI3 Vcc"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + + gpio = <&gpio3 15 GPIO_ACTIVE_HIGH>; + enable-active-high; + }; + + vccq_sdhi3: regulator@4 { + compatible = "regulator-gpio"; + + regulator-name = "SDHI3 VccQ"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <3300000>; + + gpios = <&gpio3 14 GPIO_ACTIVE_HIGH>; + gpios-states = <1>; + states = <3300000 1 + 1800000 0>; + }; + audio_clkout: audio_clkout { /* * This is same as <&rcar_sound 0> @@ -119,6 +168,16 @@ renesas,function = "avb"; }; + sdhi0_pins: sd0 { + renesas,groups = "sdhi0_data4", "sdhi0_ctrl"; + renesas,function = "sdhi0"; + }; + + sdhi3_pins: sd3 { + renesas,groups = "sdhi3_data4", "sdhi3_ctrl"; + renesas,function = "sdhi3"; + }; + sound_pins: sound { renesas,groups = "ssi01239_ctrl", "ssi0_data", "ssi1_data_a"; renesas,function = "ssi"; @@ -226,6 +285,30 @@ &sata { status = "okay"; +; + +&sdhi0 { + pinctrl-0 = <&sdhi0_pins>; + pinctrl-names = "default"; + + vmmc-supply = <&vcc_sdhi0>; + vqmmc-supply = <&vccq_sdhi0>; + cd-gpios = <&gpio3 12 GPIO_ACTIVE_LOW>; + wp-gpios = <&gpio3 13 GPIO_ACTIVE_HIGH>; + bus-width = <4>; + status = "okay"; +}; + +&sdhi3 { + pinctrl-0 = <&sdhi3_pins>; + pinctrl-names = "default"; + + vmmc-supply = <&vcc_sdhi3>; + vqmmc-supply = <&vccq_sdhi3>; + cd-gpios = <&gpio4 15 GPIO_ACTIVE_LOW>; + wp-gpios = <&gpio4 16 GPIO_ACTIVE_HIGH>; + bus-width = <4>; + status = "okay"; }; &ssi1 {