Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1651979

Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current

Show all headers | View raw

Hi Liam,

2017-05-26 23:20 GMT+02:00 Liam Breck <liam@networkimprov.net>:
> Hi Enric,
>
> On Fri, May 26, 2017 at 4:04 AM, Enric Balletbo i Serra
> <enric.balletbo@collabora.com> wrote:
>> Add charging voltage and current characteristics to the battery DT for
>> proper handling of the battery by fuel-gauge and charger chips.
>>
>> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
>> ---
>> Changes since v2:
>>  - Requested by Sebastian Reichel
>>    - Move to its own patch and apply to simple-battery framework.
>> Changes since v1:
>>  - Requested by Rob Herring
>>    - Rename ti,charge-* to charge-* to be standard properties.
>>    - Use unit suffixes as per bindings/property-units.txt
>>
>>  Documentation/devicetree/bindings/power/supply/battery.txt | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/power/supply/battery.txt b/Documentation/devicetree/bindings/power/supply/battery.txt
>> index 63a7028..c87a439 100644
>> --- a/Documentation/devicetree/bindings/power/supply/battery.txt
>> +++ b/Documentation/devicetree/bindings/power/supply/battery.txt
>> @@ -12,6 +12,8 @@ Optional Properties:
>>   - voltage-min-design-microvolt: drained battery voltage
>>   - energy-full-design-microwatt-hours: battery design energy
>>   - charge-full-design-microamp-hours: battery design capacity
>> + - charge-voltage-microvolt: battery charging voltage
>> + - charge-current-microamp: battery charging current
>
> I think you mean constant-charge-* which is how you surface these
> properties in your tps65217_charger patch.
>

Yes, to be strict this is constant-charge-*

> I'll add these to v14 of my patchset which adds simple-battery
> support. Rob requested a single patch for this file.
>

Ok, I'll send the tps charger series without this patch, so please,
include this patch in your series.

> I've been waiting for feedback on v13.2 from Sebastian. If I don't
> hear from him within a few days, I'll post v14.
>
>>  Batteries must be referenced by chargers and/or fuel-gauges
>>  using a phandle. The phandle's property should be named
>> @@ -24,6 +26,8 @@ Example:
>>                 voltage-min-design-microvolt = <3200000>;
>>                 energy-full-design-microwatt-hours = <5290000>;
>>                 charge-full-design-microamp-hours = <1430000>;
>> +               charge-voltage-microvolt = <4100000>;
>> +               charge-current-microamp = <300000>;
>>         };
>>
>>         charger: charger@11 {
>> --
>> 2.9.3
>>

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-05-26 13:10 +0200
  [PATCH v3 4/4] power: tps65217_charger: add support for NTC type, voltage and current charge Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-05-26 13:10 +0200
    Re: [PATCH v3 4/4] power: tps65217_charger: add support for NTC type,  voltage and current charge Liam Breck <liam@networkimprov.net> - 2017-05-27 03:30 +0200
  [PATCH v3 2/4] power: supply: core: add charging voltage/current battery info Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-05-26 13:10 +0200
    Re: [PATCH v3 2/4] power: supply: core: add charging voltage/current  battery info Liam Breck <liam@networkimprov.net> - 2017-05-27 03:30 +0200
      Re: [PATCH v3 2/4] power: supply: core: add charging voltage/current  battery info Enric Balletbo Serra <eballetbo@gmail.com> - 2017-05-27 22:20 +0200
  [PATCH v3 3/4] dt-bindings: power: tps65217_charger: add NTC type for battery temperature measurement. Enric Balletbo i Serra <enric.balletbo@collabora.com> - 2017-05-26 13:10 +0200
    Re: [PATCH v3 3/4] dt-bindings: power: tps65217_charger: add NTC type  for battery temperature measurement. Liam Breck <liam@networkimprov.net> - 2017-05-27 03:30 +0200
      Re: [PATCH v3 3/4] dt-bindings: power: tps65217_charger: add NTC type  for battery temperature measurement. Enric Balletbo Serra <eballetbo@gmail.com> - 2017-05-27 22:20 +0200
  Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Liam Breck <liam@networkimprov.net> - 2017-05-27 03:30 +0200
    Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Enric Balletbo Serra <eballetbo@gmail.com> - 2017-05-27 22:20 +0200
      Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Liam Breck <liam@networkimprov.net> - 2017-06-01 09:10 +0200
        Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Enric Balletbo Serra <eballetbo@gmail.com> - 2017-06-01 10:20 +0200
          Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Liam Breck <liam@networkimprov.net> - 2017-06-01 12:30 +0200
            Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge voltage/current Enric Balletbo Serra <eballetbo@gmail.com> - 2017-06-01 12:40 +0200
  Re: [PATCH v3 1/4] dt-bindings: power: supply: add battery charge  voltage/current Rob Herring <robh@kernel.org> - 2017-05-31 21:00 +0200

csiph-web