On 06/25/2012 09:24 AM, Laxman Dewangan wrote: > On Monday 25 June 2012 08:42 PM, Stephen Warren wrote: >> On 06/25/2012 12:24 AM, Laxman Dewangan wrote: >>> On Saturday 23 June 2012 04:44 AM, Stephen Warren wrote: >>>> From: Stephen Warren<[email protected]> >>>> >>>> Seaboard uses a TPS6586x regulator. Instantiate this, and hook up a >>>> couple of fixed GPIO-controlled regulators too. ... >>>> + regulator@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. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
