Re: [PATCH 1/5] dt-bindings: power: Introduce one property to describe the battery resistance with temperature changes

[Rob Herring <robh@kernel.org>]

On Mon, Oct 28, 2019 at 03:18:57PM +0800, Baolin Wang wrote:
> Since the battery internal resistance can be changed as the temperature
> changes, thus add one table to describe the battery resistance percent
> in different temperature to get a accurate battery internal resistance.
> 
> Signed-off-by: Baolin Wang <baolin.wang@linaro.org>
> ---
>  .../devicetree/bindings/power/supply/battery.txt   |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/battery.txt b/Documentation/devicetree/bindings/power/supply/battery.txt
> index 5c913d4c..1a6f951 100644
> --- a/Documentation/devicetree/bindings/power/supply/battery.txt
> +++ b/Documentation/devicetree/bindings/power/supply/battery.txt
> @@ -35,6 +35,10 @@ Optional Properties:
>     for each of the battery capacity lookup table. The first temperature value
>     specifies the OCV table 0, and the second temperature value specifies the
>     OCV table 1, and so on.
> + - resistance-temp-table: An array providing the resistance percent and
> +   corresponding temperature in degree Celsius, which is used to look up the
> +   resistance percent according to current temperature to get a accurate
> +   batterty internal resistance.

What's the order of values? The description and example don't seem to 
agree unless negative percent is a thing.

>  
>  Battery properties are named, where possible, for the corresponding
>  elements in enum power_supply_property, defined in
> @@ -61,6 +65,7 @@ Example:
>  		ocv-capacity-table-0 = <4185000 100>, <4113000 95>, <4066000 90>, ...;
>  		ocv-capacity-table-1 = <4200000 100>, <4185000 95>, <4113000 90>, ...;
>  		ocv-capacity-table-2 = <4250000 100>, <4200000 95>, <4185000 90>, ...;
> +		resistance-temp-table = <20 100>, <10 90>, <0 80>, <(-10) 60>;
>  	};
>  
>  	charger: charger@11 {
> -- 
> 1.7.9.5
>