Hi Rob, On Fri, Mar 24, 2017 at 8:55 AM, Rob Herring <robh@xxxxxxxxxx> wrote: > On Mon, Mar 20, 2017 at 02:43:26AM -0700, Liam Breck wrote: >> From: Liam Breck <kernel@xxxxxxxxxxxxxxxxx> >> >> Documentation of static battery characteristics that can be defined >> for batteries which cannot self-identify. This information is required >> by fuel-gauge and charger chips for proper handling of the battery. >> >> Cc: Rob Herring <robh@xxxxxxxxxx> >> Cc: devicetree@xxxxxxxxxxxxxxx >> Signed-off-by: Matt Ranostay <matt@ranostay.consulting> >> Signed-off-by: Liam Breck <kernel@xxxxxxxxxxxxxxxxx> >> --- >> .../devicetree/bindings/power/supply/battery.txt | 43 ++++++++++++++++++++++ >> 1 file changed, 43 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/power/supply/battery.txt > > While "simple" is not generally something I like to see in a compatible > string, we've beat this one to death. > > Acked-by: Rob Herring <robh@xxxxxxxxxx> Thanks. Are you in agreement with Sebastian on the following, or is this OK to include? On Thu, Mar 23, 2017 at 3:20 AM, Sebastian Reichel <sre@xxxxxxxxxx> wrote: >> +Battery properties are named, where possible, for the corresponding >> +elements in enum power_supply_property, defined in >> +https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/linux/power_supply.h#n86 > >The above paragraph does not belong into DT bindings. -- 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