Re: [PATCH V2] regulator: dt: fix the name of regulator supply

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

On Saturday 19 May 2012 04:13 PM, Mark Brown wrote:
* PGP Signed by an unknown key

On Sat, May 19, 2012 at 01:04:12AM +0530, Laxman Dewangan wrote:
If there is any input supply for regulator then the name
of supply need to provided by init_data->supply_regulator.
In dt case, the input supply is searched by the<name>-supply
and here it is require to fix the name so that correct device
node can be identified at the time of regulator registration.
Fixing the name for input supply as "regulator-supply".
No, supply_regulator is a legacy thing.  Regulators should set
supply_name in their regulator_desc to tell the framework what name
the parent has, allowing these supplies to behave in the same way
as other supplies as far as the binding goes.

So if there is two regulator ldo1 and ldo2 where ldo1 is supply for ldo2 then
ldo1: ldo1 {

ldo2: ldo2 {
    ldo2-supply = <&ldo1>;

and when we parse the regulator node, we should see the property name as the <name>-supply i.e. ldo1-supply or ldo2-supply and when it found then we set the
regulator_desc->supply_name = "ldo2"?

The descriptor is set on the regulator driver and hence it need to be parse this in the regulator driver. We need to set the name regulator_desc->supply_name and hence it needs to be in dt binding documentation of that device/regulators.

Is this what you recommend?

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at

[Index of Archives]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Photo]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux