From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Gross Subject: Re: [PATCH RFC 4/8] mmc: sdhci-msm: Add regulator DT props to sdhci-msm bindings Date: Wed, 29 Jun 2016 16:53:26 -0500 Message-ID: <20160629215326.GD16832@hector.attlocal.net> References: <1467199233-20506-1-git-send-email-riteshh@codeaurora.org> <1467199233-20506-5-git-send-email-riteshh@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-ob0-f169.google.com ([209.85.214.169]:34426 "EHLO mail-ob0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751587AbcF2Vx2 (ORCPT ); Wed, 29 Jun 2016 17:53:28 -0400 Received: by mail-ob0-f169.google.com with SMTP id ru5so44961931obc.1 for ; Wed, 29 Jun 2016 14:53:27 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1467199233-20506-5-git-send-email-riteshh@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Ritesh Harjani Cc: ulf.hansson@linaro.org, linux-mmc@vger.kernel.org, linux-arm-msm@vger.kernel.org, adrian.hunter@intel.com, asutoshd@codeaurora.org, kdorfman@codeaurora.org, david.griego@linaro.org, stummala@codeaurora.org, venkatg@codeaurora.org On Wed, Jun 29, 2016 at 04:50:29PM +0530, Ritesh Harjani wrote: > This patch adds the DT properties for voltage regulator nodes > for Qualcomm SDHCI driver. > > Signed-off-by: Ritesh Harjani > --- > Documentation/devicetree/bindings/mmc/sdhci-msm.txt | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > index 851e66d..32cea75 100644 > --- a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > +++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > @@ -17,7 +17,15 @@ Required properties: > "iface" - Main peripheral bus clock (PCLK/HCLK - AHB Bus clock) (required) > "core" - SDC MMC clock (MCLK) (required) > "bus" - SDCC bus voter clock (optional) > +- qcom,-voltage_level - specifies voltage levels for supply. Should be > + specified in pairs (min, max), units uV. > +- qcom,-current_level - specifies load levels for supply in lpm or > + high power mode (hpm). Should be specified in > + pairs (lpm, hpm), units uA. These seem like OPPs to me. Why use something non-standard? Check out Documentation/devicetree/bindings/opp/opp.txt > > +Optional Properties: > + - qcom,-always-on - specifies whether supply should be kept "on" always. Would this only be the base if mmc is used on this platform? You could specify this in the regulator binding itself if this is more of a global thing. > + - qcom,-lpm_sup - specifies whether supply can be kept in low power mode (lpm). > Example: > > sdhc_1: sdhci@f9824900 { > @@ -28,7 +36,13 @@ Example: > non-removable; > > vdd-supply = <&pm8941_l20>; > + qcom,vdd-voltage-level = <2950000 2950000>; > + qcom,vdd-current-level = <200 570000>; > + > vdd-io-supply = <&pm8941_s3>; > + qcom,vdd-io-always-on; > + qcom,vdd-io-voltage-level = <1800000 1800000>; > + qcom,vdd-io-current-level = <110 325000>; > > pinctrl-names = "default"; > pinctrl-0 = <&sdc1_clk &sdc1_cmd &sdc1_data>; > @@ -45,7 +59,12 @@ Example: > cd-gpios = <&msmgpio 62 0x1>; > > vdd-supply = <&pm8941_l21>; > + qcom,vdd-voltage-level = <2950000 2950000>; > + qcom,vdd-current-level = <200 800000>; > + > vdd-io-supply = <&pm8941_l13>; > + qcom,vdd-io-voltage-level = <1800000 2950000>; > + qcom,vdd-io-current-level = <200 22000>; > > pinctrl-names = "default"; > pinctrl-0 = <&sdc2_clk &sdc2_cmd &sdc2_data>; Regards, Andy