On 06/25/2012 09:24 AM, Laxman Dewangan wrote:

On Monday 25 June 2012 08:42 PM, Stephen Warren wrote:

quoted

On 06/25/2012 12:24 AM, Laxman Dewangan wrote:

quoted

On Saturday 23 June 2012 04:44 AM, Stephen Warren wrote:

quoted

From: Stephen Warren<redacted>

Seaboard uses a TPS6586x regulator. Instantiate this, and hook up a
couple of fixed GPIO-controlled regulators too.

...

quoted

quoted

quoted

+                regulator at 3 {
+                    reg =<3>;
+                    regulator-compatible = "ldo0";
+                    regulator-name = "vdd_ldo0";
+                    regulator-min-microvolt =<1250000>;
+                    regulator-max-microvolt =<3300000>;
+                    vin-supply =<&sm2_reg>;

I think support for vin-supply is still not there for this regulator in
driver.

That's also true. I wonder if we shouldn't support this in the regulator
core bindings instead, since the existence of a parent regulator seems
likely to be common. Either way, I'd like to include the property to
document it for now.

I had detailed discussion with Mark on  this support and as per him
(based on my understanding), the input to different regulator is from
the pin of the chips and so the name should be the <pin-name>-supply
which should be part of chip-dt binding, not to the particular rail.

OK, that's fine. Can you please update the TPS6586x binding and driver
to allow this to be represented correctly then. Thanks.