Hi,

On Mon, Jun 28, 2021 at 01:45:14AM +0200, Linus Walleij wrote:

This adds a battery-type property and bindings for the different
"technologies" that are used in Linux. More types can be added.

Makes sense to have this.

This is needed to convert the custom ST-Ericsson AB8500 battery
properties over to the generic battery bindings.

Great :)

Signed-off-by: Linus Walleij <redacted>
---
I need a bunch of new bindings for switch the STE AB8500 custom
bindings out, but I need to start somewhere, this is as good as
any place to start.
---
 .../devicetree/bindings/power/supply/battery.yaml  | 14 ++++++++++++++

This should be CC'd to Rob Herring.

The smart battery standard names the property storing this information
"DeviceChemistry", which (as a non-native speaker) seems like a
better fitting name. Thoughts?

+    description: This describes the chemical technology of the battery.
+    oneOf:
+      - const: nickel-cadmium
+      - const: nickel-metal-hydride
+      - const: lithium-ion
+        description: This is a blanket type for all lithium-ion batteries,
+          including those below. If possible, a precise compatible string
+          from below should be used, but sometimes it is unknown which specific
+          lithium ion battery is employed and this wide compatible can be used.
+      - const: lithium-ion-polymer
+      - const: lithium-ion-iron-phosphate
+      - const: lithium-ion-manganese-oxide

The values look good to me.

-- Sebastian

   over-voltage-threshold-microvolt:
     description: battery over-voltage limit
 
-- 
2.31.1