On Wed, Jan 25, 2017 at 10:19 PM, Liam Breck <liam@xxxxxxxxxxxxxxxxx> wrote: > On Sun, Jan 22, 2017 at 2:22 PM, Liam Breck <liam@xxxxxxxxxxxxxxxxx> wrote: > >> diff --git a/Documentation/devicetree/bindings/power/supply/battery.txt b/Documentation/devicetree/bindings/power/supply/battery.txt >> new file mode 100644 >> index 000000000000..398b4d622883 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/power/supply/battery.txt >> @@ -0,0 +1,34 @@ >> +Battery Characteristics >> + >> +Required Properties: >> + - compatible: Must be "fixed-battery" >> + >> +Optional Properties: >> + - termination-microvolt: dead battery voltage >> + - design-microwatt-hours: battery design energy >> + - design-microamp-hours: battery design capacity > > Also I suspect the members of struct power_supply_battery_info should > use the same names as appear in dts: These are internal and we can't be thinking just device tree.. ACPI and platform data is also an option. > > +struct power_supply_battery_info { > + int design_microwatt_hours; > + int design_microamp_hours; > + int termination_microvolt; > etc -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html