On Sun 2013-11-24 17:49:31, Sebastian Reichel wrote: > Add devicetree binding documentation for bq2415x charger. > > Signed-off-by: Sebastian Reichel <sre@xxxxxxxxxx> Thanks! > +- reg: integer, i2c address of the device > +- ti,current-limit: integer, current limit in mA Does this need to be "ti," specific? Most of fields are likely to be needed for other li-ion chargers.... "Specifies maximum current charger can pull from power supply" ? > +- ti,charge-current: integer, charging current in mA ...why/how is it different from current-limit? Is the current-limit on 5V, while the charge-current relative to battery voltage... so that charge-current can be > current-limit when battery voltage is low? "Maximum current that will be supplied to the battery, as determined by voltage on current sense resistor"? > +- ti,weak-battery-voltage: integer, weak battery voltage threshold in mV It would be good to explain what this threshold means. Voltage so low that system needs to be shut down? Hmm, it looks to me like it is "as long as battery is below this voltage, fast charge is not started. Instead, slow 'precharge' is performed." > +- ti,battery-regulation-voltage: integer, battery regulation > voltage in mV "Selects maximum voltage for charging." > +- ti,termination-current: integer, termination current in mA "When current in constant-voltage phase drops below this value, charge is terminated". -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- 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