Re: [PATCH v4 1/4] regulator: helper routine to extract regulator_init_data

Rajendra Nayak
2011-11-07T06:23:48Z

Hi Olof,

On Saturday 05 November 2011 01:59 AM, Olof Johansson wrote:
>> +- regulator-min-uV: smallest voltage consumers may set
>> > +- regulator-max-uV: largest voltage consumers may set
>> > +- regulator-uV-offset: Offset applied to voltages to compensate for voltage drops
>> > +- regulator-min-uA: smallest current consumers may set
>> > +- regulator-max-uA: largest current consumers may set
>> > +- regulator-always-on: boolean, regulator should never be disabled
>> > +- regulator-boot-on: bootloader/firmware enabled regulator
>
> Once you have a compatible field that can determine what kind of device node,
> and binding, this is, you can drop the regulator- prefix and save some space in
> the device tree. Properties are rarely prefixed by their subsystem. Only
> exception would/could be the regulator-name property where it could make
> sense to keep the prefix.

given that we decided not to have a compatible field like "regulator"
for the top-level bindings and instead have a device specific one,
and based on the discussions with Grant here[1], I hope its OK to keep
these the way they are.

regards,
Rajendra

[1]