On Mon, Mar 23, 2015 at 4:54 AM, Lee Jones <lee.jo...@linaro.org> wrote:
> On Tue, 10 Mar 2015, Chen-Yu Tsai wrote:
>
>> Add the list of regulators for AXP22x to the DT bindings.
>> This includes the names and supply names.
>>
>> Signed-off-by: Chen-Yu Tsai <w...@csie.org>
>> ---
>>  Documentation/devicetree/bindings/mfd/axp20x.txt | 27 
>> ++++++++++++++++++++++++
>>  1 file changed, 27 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/axp20x.txt 
>> b/Documentation/devicetree/bindings/mfd/axp20x.txt
>> index 3914a3f91ff6..0fc2f1f35cef 100644
>> --- a/Documentation/devicetree/bindings/mfd/axp20x.txt
>> +++ b/Documentation/devicetree/bindings/mfd/axp20x.txt
>> @@ -49,6 +49,33 @@ LDO3               : LDO           : ldo3in-supply
>>  LDO4         : LDO           : ldo24in-supply        : shared supply
>>  LDO5         : LDO           : ldo5in-supply
>>
>> +
>
> Nit: I'm not keen on double line spacing.
>
>> +AXP221 regulators, type, and corresponding input supply names:
>> +
>> +Regulator      Type            Supply Name             Notes
>> +---------      ----            -----------             -----
>> +DCDC1                : DC-DC buck    : vin1-supply
>> +DCDC2                : DC-DC buck    : vin2-supply
>> +DCDC3                : DC-DC buck    : vin3-supply
>> +DCDC4                : DC-DC buck    : vin4-supply
>> +DCDC5                : DC-DC buck    : vin5-supply
>> +DC1SW                : On/Off Switch : dcdc1-supply          : DCDC1 
>> secondary output
>> +DC5LDO               : LDO           : dcdc5-supply          : input from 
>> DCDC5
>> +ALDO1                : LDO           : aldoin-supply         : shared supply
>> +ALDO2                : LDO           : aldoin-supply         : shared supply
>> +ALDO3                : LDO           : aldoin-supply         : shared supply
>> +DLDO1                : LDO           : dldoin-supply         : shared supply
>> +DLDO2                : LDO           : dldoin-supply         : shared supply
>> +DLDO3                : LDO           : dldoin-supply         : shared supply
>> +DLDO4                : LDO           : dldoin-supply         : shared supply
>> +ELDO1                : LDO           : eldoin-supply         : shared supply
>> +ELDO2                : LDO           : eldoin-supply         : shared supply
>> +ELDO3                : LDO           : eldoin-supply         : shared supply
>> +LDO_IO0              : LDO           : ips-supply            : GPIO 0
>> +LDO_IO1              : LDO           : ips-supply            : GPIO 1
>> +RTC_LDO              : LDO           : ips-supply            : always on
>> +
>> +
>
> Nit: I'm not keen on double line spacing.

Ok. I can get rid of them. IMHO it makes different sections clearer though.
I assume you don't want them in the original axp20x patches either?

ChenYu

>>  Example:
>>
>>  axp209: pmic@34 {
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to