On Mon, Apr 03, 2017 at 06:44:46PM +0200, Enric Balletbo i Serra wrote: > Allow the possibility to configure the charge and the current voltage of > the charger and also the NTC type for battery temperature measurement. > > Signed-off-by: Enric Balletbo i Serra <enric.balle...@collabora.com> > --- > .../bindings/power/supply/tps65217_charger.txt | 15 ++ > drivers/power/supply/tps65217_charger.c | 187 > +++++++++++++++++++-- > include/linux/mfd/tps65217.h | 2 + > 3 files changed, 192 insertions(+), 12 deletions(-) > > diff --git > a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt > b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt > index a11072c..600da6e 100644 > --- a/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt > +++ b/Documentation/devicetree/bindings/power/supply/tps65217_charger.txt > @@ -6,6 +6,18 @@ Required Properties: > Should be <0> for the USB charger and <1> for the AC adapter. > -interrupt-names: Should be "USB" and "AC" > > +Optional properties: > +-ti,charge-voltage-uvolt: set the charge voltage. The value can be: 4100000, > + 4150000, 4200000, 4250000; default: 4100000 > + > +-ti,charge-current-uamp: set the charging current. The value can be: 300000, > + 400000, 500000, 700000; default: 500000
These should be standard properties. (Also, you are not using standard unit suffixes. See bindings/property-units.txt.) > + > +-ti,ntc-type: set the NTC type for battery temperature measurement. The value > + must be 0 or 1, where: > + 0 – 100k (curve 1, B = 3960) > + 1 – 10k (curve 2, B = 3480) (default) > + > This node is a subnode of the tps65217 PMIC. > > Example: > @@ -14,4 +26,7 @@ Example: > compatible = "ti,tps65217-charger"; > interrupts = <0>, <1>; > interrupt-names = "USB", "AC"; > + ti,charge-voltage-uvolt = <4100000>; > + ti,charge-current-uamp = <500000>; > + ti,ntc-type = <1>; > };
