Re: [PATCH V3 2/3] dt-bindings: mmc: sdhci-msm: Add entries for passing load values

[Veerabhadrarao Badiganti]

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@xxxxxxxxxxxxxx>
> >
> > 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@xxxxxxxxxxxxxx>
> > Signed-off-by: Veerabhadrarao Badiganti <vbadigan@xxxxxxxxxxxxxx>
> > ---
> >   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