On Sun, Jun 07, 2020 at 05:41:06PM +0300, Dmitry Osipenko wrote: > Document generic battery temperature properties. > > Signed-off-by: Dmitry Osipenko <[email protected]> > --- > .../devicetree/bindings/power/supply/battery.txt | 10 ++++++++++ > 1 file changed, 10 insertions(+)
This is close to being converted to schema: https://lore.kernel.org/linux-pm/[email protected]/ > diff --git a/Documentation/devicetree/bindings/power/supply/battery.txt > b/Documentation/devicetree/bindings/power/supply/battery.txt > index 5e29595edd74..e0c35eff9d3f 100644 > --- a/Documentation/devicetree/bindings/power/supply/battery.txt > +++ b/Documentation/devicetree/bindings/power/supply/battery.txt > @@ -45,6 +45,16 @@ Optional Properties: > and corresponding battery internal resistance percent, which is used to > look > up the resistance percent according to current temperature to get a > accurate > batterty internal resistance in different temperatures. > + - temperature-ambient-min-alert-celsius: Alert when ambient temperature of a > + battery is lower than threshold value. > + - temperature-ambient-max-alert-celsius: Alert when ambient temperature of a > + battery is higher than threshold value. > + - temperature-min-alert-celsius: Alert when battery temperature is lower > + than threshold value. > + - temperature-max-alert-celsius: Alert when battery temperature is higher > + than threshold value. > + - temperature-min-celsius: minimum temperature at which battery can operate > + - temperature-max-celsius: maximum temperature at which battery can operate Perhaps 'temperature' is redundant since we have units. Perhaps do <min max> properties given specifying only min or max is probably not valid? Rob
