Hi Nicola On Tue, Dec 20, 2016 at 9:01 PM, Nicolas Saenz Julienne <nicolas.saenz@xxxxxxxxxx> wrote: > This adds support for sbs-charger compilant chips as defined here: > http://sbs-forum.org/specs/sbc110.pdf > > This was tested on a arm board connected to an LTC4100 battery charger > chip. > > Signed-off-by: Nicolas Saenz Julienne <nicolas.saenz@xxxxxxxxxx> > --- > v3 -> v4 > - drop "lltc,ltc4100" compatible string for now > > v2 -> v3: > - add readable_reg() function to regmap config > - update compatible strings with part number > > v1 -> v2: > - add spec link in header > - use proper gpio/interrupt interface > - update regmap configuration (max register & endianness) > - dropped oldschool .supplied_to assignments > - use devm_* APIs > drivers/power/supply/Kconfig | 6 + > drivers/power/supply/Makefile | 1 + > drivers/power/supply/sbs-charger.c | 274 +++++++++++++++++++++++++++++++++++++ > 3 files changed, 281 insertions(+) > create mode 100644 drivers/power/supply/sbs-charger.c Just some general comment, Can you add some more properties here to know voltage and current? Also, can you add other properties present in charging status register like POWER_FAIL, VOLTAGE_OR, CURRENT_OR etc. Don't know weather it is feasible to add or not and Also, let me know, if it is already been covered in some part of the code. Thanks Manish Badarkhe -- 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
