* [PATCH V3 2/3] dt-bindings: mmc: sdhci-msm: Add entries for passing load values [not found] <1539004739-32060-1-git-send-email-vbadigan@codeaurora.org> @ 2018-10-08 13:18 ` Veerabhadrarao Badiganti 2018-10-12 14:55 ` Rob Herring 0 siblings, 1 reply; 3+ messages in thread From: Veerabhadrarao Badiganti @ 2018-10-08 13:18 UTC (permalink / raw) To: adrian.hunter, ulf.hansson, robh+dt, evgreen, dianders Cc: asutoshd, riteshh, stummala, sayalil, linux-mmc, linux-arm-msm, Vijay Viswanath, Veerabhadrarao Badiganti, Mark Rutland, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list From: Vijay Viswanath <vviswana@codeaurora.org> The load a particular sdhc controller should request from a regulator is device specific and hence each device should individually vote for the required load. Signed-off-by: Vijay Viswanath <vviswana@codeaurora.org> Signed-off-by: Veerabhadrarao Badiganti <vbadigan@codeaurora.org> --- Documentation/devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt index 502b3b8..cb22178 100644 --- a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt +++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt @@ -25,6 +25,10 @@ Required properties: "xo" - TCXO clock (optional) "cal" - reference clock for RCLK delay calibration (optional) "sleep" - sleep clock for RCLK delay calibration (optional) +- qcom,<supply>-current-level-microamp - specifies load levels for supply during BUS_ON and + BUS_OFF states in power irq. Should be specified in + pairs (lpm, hpm), for BUS_OFF and BUS_ON respectively. + Units uA. Example: @@ -36,7 +40,9 @@ Example: non-removable; vmmc-supply = <&pm8941_l20>; + qcom,vmmc-current-level-microamp = <200 570000>; vqmmc-supply = <&pm8941_s3>; + qcom,vqmmc-current-level-microamp = <200 325000>; pinctrl-names = "default"; pinctrl-0 = <&sdc1_clk &sdc1_cmd &sdc1_data>; @@ -53,7 +59,9 @@ Example: cd-gpios = <&msmgpio 62 0x1>; vmmc-supply = <&pm8941_l21>; + qcom,vmmc-current-level-microamp = <200 800000>; vqmmc-supply = <&pm8941_l13>; + qcom,vqmmc-current-level-microamp = <200 22000>; pinctrl-names = "default"; pinctrl-0 = <&sdc2_clk &sdc2_cmd &sdc2_data>; -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project. ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V3 2/3] dt-bindings: mmc: sdhci-msm: Add entries for passing load values 2018-10-08 13:18 ` [PATCH V3 2/3] dt-bindings: mmc: sdhci-msm: Add entries for passing load values Veerabhadrarao Badiganti @ 2018-10-12 14:55 ` Rob Herring 2018-11-08 14:13 ` Veerabhadrarao Badiganti 0 siblings, 1 reply; 3+ messages in thread From: Rob Herring @ 2018-10-12 14:55 UTC (permalink / raw) To: Veerabhadrarao Badiganti Cc: adrian.hunter, ulf.hansson, evgreen, dianders, asutoshd, riteshh, stummala, sayalil, linux-mmc, linux-arm-msm, Vijay Viswanath, Mark Rutland, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list On Mon, Oct 08, 2018 at 06:48:58PM +0530, Veerabhadrarao Badiganti wrote: > From: Vijay Viswanath <vviswana@codeaurora.org> > > The load a particular sdhc controller should request from a regulator > is device specific and hence each device should individually vote for > the required load. > > Signed-off-by: Vijay Viswanath <vviswana@codeaurora.org> > Signed-off-by: Veerabhadrarao Badiganti <vbadigan@codeaurora.org> > --- > Documentation/devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > index 502b3b8..cb22178 100644 > --- a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > +++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt > @@ -25,6 +25,10 @@ Required properties: > "xo" - TCXO clock (optional) > "cal" - reference clock for RCLK delay calibration (optional) > "sleep" - sleep clock for RCLK delay calibration (optional) > +- qcom,<supply>-current-level-microamp - specifies load levels for supply during BUS_ON and > + BUS_OFF states in power irq. Should be specified in > + pairs (lpm, hpm), for BUS_OFF and BUS_ON respectively. > + Units uA. Seems like something that should be common? ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V3 2/3] dt-bindings: mmc: sdhci-msm: Add entries for passing load values 2018-10-12 14:55 ` Rob Herring @ 2018-11-08 14:13 ` Veerabhadrarao Badiganti 0 siblings, 0 replies; 3+ messages in thread From: Veerabhadrarao Badiganti @ 2018-11-08 14:13 UTC (permalink / raw) To: Rob Herring Cc: adrian.hunter, ulf.hansson, evgreen, dianders, asutoshd, riteshh, stummala, sayalil, linux-mmc, linux-arm-msm, Vijay Viswanath, Mark Rutland, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, open list On 10/12/2018 8:25 PM, Rob Herring wrote: > On Mon, Oct 08, 2018 at 06:48:58PM +0530, Veerabhadrarao Badiganti wrote: >> From: Vijay Viswanath <vviswana@codeaurora.org> >> >> The load a particular sdhc controller should request from a regulator >> is device specific and hence each device should individually vote for >> the required load. >> >> Signed-off-by: Vijay Viswanath <vviswana@codeaurora.org> >> Signed-off-by: Veerabhadrarao Badiganti <vbadigan@codeaurora.org> >> --- >> Documentation/devicetree/bindings/mmc/sdhci-msm.txt | 8 ++++++++ >> 1 file changed, 8 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt >> index 502b3b8..cb22178 100644 >> --- a/Documentation/devicetree/bindings/mmc/sdhci-msm.txt >> +++ b/Documentation/devicetree/bindings/mmc/sdhci-msm.txt >> @@ -25,6 +25,10 @@ Required properties: >> "xo" - TCXO clock (optional) >> "cal" - reference clock for RCLK delay calibration (optional) >> "sleep" - sleep clock for RCLK delay calibration (optional) >> +- qcom,<supply>-current-level-microamp - specifies load levels for supply during BUS_ON and >> + BUS_OFF states in power irq. Should be specified in >> + pairs (lpm, hpm), for BUS_OFF and BUS_ON respectively. >> + Units uA. > Seems like something that should be common? Hi Rob, Can you please little elaborate your comment? Do mean this should be common for all vendors, not specific to qcom-mmc? Thanks, Veera ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-11-08 14:13 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <1539004739-32060-1-git-send-email-vbadigan@codeaurora.org> 2018-10-08 13:18 ` [PATCH V3 2/3] dt-bindings: mmc: sdhci-msm: Add entries for passing load values Veerabhadrarao Badiganti 2018-10-12 14:55 ` Rob Herring 2018-11-08 14:13 ` Veerabhadrarao Badiganti
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).