On 23/09/14 11:53, Adam Thomson wrote: > This patch adds support for DA9150 Charger & Fuel-Gauge IC Charger. > > Signed-off-by: Adam Thomson <Adam.Thomson.Opensource@xxxxxxxxxxx> The IIO elements of this look fine to me. I'm not failiar enough with the charger framework to give more than a superficial opinion on that. I do note that we have a lot of attributes added to the device after registration which we have spent time elsewhere in the kernel avoiding because of notification issues... http://kroah.com/log/blog/2013/06/26/how-to-create-a-sysfs-file-correctly/ (convient summary of this from GregKH) Still probably not an issue with this driver it self... > --- > drivers/power/Kconfig | 12 + > drivers/power/Makefile | 1 + > drivers/power/da9150-charger.c | 740 +++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 753 insertions(+) > create mode 100644 drivers/power/da9150-charger.c > > diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig > index 73cfcdf..622884d 100644 > --- a/drivers/power/Kconfig > +++ b/drivers/power/Kconfig > @@ -192,6 +192,18 @@ config BATTERY_DA9052 > Say Y here to enable support for batteries charger integrated into > DA9052 PMIC. > > +config CHARGER_DA9150 > + tristate "Dialog Semiconductor DA9150 Charger support" > + depends on MFD_DA9150 > + depends on DA9150_GPADC > + depends on IIO > + help > + Say Y here to enable support for charger unit of the DA9150 > + Integrated Charger & Fuel-Gauge IC. > + > + This driver can also be built as a module. If so, the module will be > + called da9150-charger. > + > config BATTERY_MAX17040 > tristate "Maxim MAX17040 Fuel Gauge" > depends on I2C > diff --git a/drivers/power/Makefile b/drivers/power/Makefile > index dfa8942..0c1896d 100644 > --- a/drivers/power/Makefile > +++ b/drivers/power/Makefile > @@ -31,6 +31,7 @@ obj-$(CONFIG_BATTERY_SBS) += sbs-battery.o > obj-$(CONFIG_BATTERY_BQ27x00) += bq27x00_battery.o > obj-$(CONFIG_BATTERY_DA9030) += da9030_battery.o > obj-$(CONFIG_BATTERY_DA9052) += da9052-battery.o > +obj-$(CONFIG_CHARGER_DA9150) += da9150-charger.o > obj-$(CONFIG_BATTERY_MAX17040) += max17040_battery.o > obj-$(CONFIG_BATTERY_MAX17042) += max17042_battery.o > obj-$(CONFIG_BATTERY_Z2) += z2_battery.o > diff --git a/drivers/power/da9150-charger.c b/drivers/power/da9150-charger.c > new file mode 100644 > index 0000000..38345cf > --- /dev/null > +++ b/drivers/power/da9150-charger.c > @@ -0,0 +1,740 @@ > +/* > + * DA9150 Charger Driver > + * > + * Copyright (c) 2014 Dialog Semiconductor > + * > + * Author: Adam Thomson <Adam.Thomson.Opensource@xxxxxxxxxxx> > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or (at your > + * option) any later version. > + */ > + > +#include <linux/kernel.h> > +#include <linux/slab.h> > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/of.h> > +#include <linux/of_platform.h> > +#include <linux/interrupt.h> > +#include <linux/power_supply.h> > +#include <linux/notifier.h> > +#include <linux/usb/phy.h> > +#include <linux/iio/consumer.h> > +#include <linux/sysfs.h> > +#include <linux/mfd/da9150/core.h> > +#include <linux/mfd/da9150/registers.h> > + > +/* Private data */ > +struct da9150_charger_attr_map { > + struct device_attribute attr; > + u16 reg; > + u8 shift; > + u8 mask; > +}; > + > +struct da9150_charger { > + struct da9150 *da9150; > + struct device *dev; > + > + struct power_supply usb; > + struct power_supply battery; > + struct power_supply *supply_online; > + > + struct usb_phy *usb_phy; > + struct notifier_block otg_nb; > + struct work_struct otg_work; > + unsigned long usb_event; > + > + struct iio_channel *ibus_chan; > + struct iio_channel *vbus_chan; > + struct iio_channel *tjunc_chan; > + struct iio_channel *vbat_chan; > +}; > + > +static inline int da9150_charger_supply_online(struct da9150_charger *charger, > + struct power_supply *psy, > + union power_supply_propval *val) > +{ > + val->intval = (psy == charger->supply_online) ? 1 : 0; > + > + return 0; > +} > + > +/* Charger Properties */ > +static int da9150_charger_vbus_voltage_now(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + int v_val, ret; > + > + /* Read processed value - mV units */ > + ret = iio_read_channel_processed(charger->vbus_chan, &v_val); > + if (ret < 0) > + return ret; > + > + /* Convert voltage to expected uV units */ > + val->intval = v_val * 1000; > + > + return 0; > +} > + > +static int da9150_charger_ibus_current_avg(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + int i_val, ret; > + > + /* Read processed value - mA units */ > + ret = iio_read_channel_processed(charger->ibus_chan, &i_val); > + if (ret < 0) > + return ret; > + > + /* Convert current to expected uA units */ > + val->intval = i_val * 1000; > + > + return 0; > +} > + > +static int da9150_charger_tjunc_temp(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + int t_val, ret; > + > + /* Read processed value - 0.001 degrees C units */ > + ret = iio_read_channel_processed(charger->tjunc_chan, &t_val); > + if (ret < 0) > + return ret; > + > + /* Convert temp to expect 0.1 degrees C units */ > + val->intval = t_val / 100; > + > + return 0; > +} > + > +static enum power_supply_property da9150_charger_props[] = { > + POWER_SUPPLY_PROP_ONLINE, > + POWER_SUPPLY_PROP_VOLTAGE_NOW, > + POWER_SUPPLY_PROP_CURRENT_AVG, > + POWER_SUPPLY_PROP_TEMP, > +}; > + > +static int da9150_charger_get_prop(struct power_supply *psy, > + enum power_supply_property psp, > + union power_supply_propval *val) > +{ > + struct da9150_charger *charger = dev_get_drvdata(psy->dev->parent); > + int ret; > + > + switch (psp) { > + case POWER_SUPPLY_PROP_ONLINE: > + ret = da9150_charger_supply_online(charger, psy, val); > + break; > + case POWER_SUPPLY_PROP_VOLTAGE_NOW: > + ret = da9150_charger_vbus_voltage_now(charger, val); > + break; > + case POWER_SUPPLY_PROP_CURRENT_AVG: > + ret = da9150_charger_ibus_current_avg(charger, val); > + break; > + case POWER_SUPPLY_PROP_TEMP: > + ret = da9150_charger_tjunc_temp(charger, val); > + break; > + default: > + ret = -EINVAL; > + break; > + } > + > + return ret; > +} > + > +/* Battery Properties */ > +static int da9150_charger_battery_status(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + u8 reg; > + > + /* Check to see if battery is discharging */ > + reg = da9150_reg_read(charger->da9150, DA9150_STATUS_H); > + > + if (((reg & DA9150_VBUS_STAT_MASK) == DA9150_VBUS_STAT_OFF) || > + ((reg & DA9150_VBUS_STAT_MASK) == DA9150_VBUS_STAT_WAIT)) { > + val->intval = POWER_SUPPLY_STATUS_DISCHARGING; > + > + return 0; > + } > + > + reg = da9150_reg_read(charger->da9150, DA9150_STATUS_J); > + > + /* Now check for other states */ > + switch (reg & DA9150_CHG_STAT_MASK) { > + case DA9150_CHG_STAT_ACT: > + case DA9150_CHG_STAT_PRE: > + case DA9150_CHG_STAT_CC: > + case DA9150_CHG_STAT_CV: > + val->intval = POWER_SUPPLY_STATUS_CHARGING; > + break; > + case DA9150_CHG_STAT_OFF: > + case DA9150_CHG_STAT_SUSP: > + case DA9150_CHG_STAT_TEMP: > + case DA9150_CHG_STAT_TIME: > + case DA9150_CHG_STAT_BAT: > + val->intval = POWER_SUPPLY_STATUS_NOT_CHARGING; > + break; > + case DA9150_CHG_STAT_FULL: > + val->intval = POWER_SUPPLY_STATUS_FULL; > + break; > + default: > + val->intval = POWER_SUPPLY_STATUS_UNKNOWN; > + break; > + } > + > + return 0; > +} > + > +static int da9150_charger_battery_health(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + u8 reg; > + > + reg = da9150_reg_read(charger->da9150, DA9150_STATUS_J); > + > + /* Check if temperature limit reached */ > + switch (reg & DA9150_CHG_TEMP_MASK) { > + case DA9150_CHG_TEMP_UNDER: > + val->intval = POWER_SUPPLY_HEALTH_COLD; > + return 0; > + case DA9150_CHG_TEMP_OVER: > + val->intval = POWER_SUPPLY_HEALTH_OVERHEAT; > + return 0; > + default: > + break; > + } > + > + /* Check for other health states */ > + switch (reg & DA9150_CHG_STAT_MASK) { > + case DA9150_CHG_STAT_ACT: > + case DA9150_CHG_STAT_PRE: > + val->intval = POWER_SUPPLY_HEALTH_DEAD; > + break; > + case DA9150_CHG_STAT_TIME: > + val->intval = POWER_SUPPLY_HEALTH_UNSPEC_FAILURE; > + break; > + default: > + val->intval = POWER_SUPPLY_HEALTH_GOOD; > + break; > + } > + > + return 0; > +} > + > +static int da9150_charger_battery_present(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + u8 reg; > + > + /* Check if battery present or removed */ > + reg = da9150_reg_read(charger->da9150, DA9150_STATUS_J); > + if ((reg & DA9150_CHG_STAT_MASK) == DA9150_CHG_STAT_BAT) > + val->intval = 0; > + else > + val->intval = 1; > + > + return 0; > +} > + > +static int da9150_charger_battery_charge_type(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + u8 reg; > + > + reg = da9150_reg_read(charger->da9150, DA9150_STATUS_J); > + > + switch (reg & DA9150_CHG_STAT_MASK) { > + case DA9150_CHG_STAT_CC: > + val->intval = POWER_SUPPLY_CHARGE_TYPE_FAST; > + break; > + case DA9150_CHG_STAT_ACT: > + case DA9150_CHG_STAT_PRE: > + case DA9150_CHG_STAT_CV: > + val->intval = POWER_SUPPLY_CHARGE_TYPE_TRICKLE; > + break; > + default: > + val->intval = POWER_SUPPLY_CHARGE_TYPE_NONE; > + break; > + } > + > + return 0; > +} > + > +static int da9150_charger_battery_voltage_min(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + u8 reg; > + > + reg = da9150_reg_read(charger->da9150, DA9150_PPR_CHGCTRL_C); > + > + /* Value starts at 2500 mV, 50 mV increments, presented in uV */ > + val->intval = ((reg & DA9150_CHG_VFAULT_MASK) * 50000) + 2500000; > + > + return 0; > +} > + > +static int da9150_charger_battery_voltage_now(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + int v_val, ret; > + > + /* Read processed value - mV units */ > + ret = iio_read_channel_processed(charger->vbat_chan, &v_val); > + if (ret < 0) > + return ret; > + > + val->intval = v_val * 1000; > + > + return 0; > +} > + > +static int da9150_charger_battery_current_max(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + int reg; > + > + reg = da9150_reg_read(charger->da9150, DA9150_PPR_CHGCTRL_D); > + > + /* 25mA increments */ > + val->intval = reg * 25000; > + > + return 0; > +} > + > +static int da9150_charger_battery_voltage_max(struct da9150_charger *charger, > + union power_supply_propval *val) > +{ > + u8 reg; > + > + reg = da9150_reg_read(charger->da9150, DA9150_PPR_CHGCTRL_B); > + > + /* Value starts at 3650 mV, 25 mV increments, presented in uV */ > + val->intval = ((reg & DA9150_CHG_VBAT_MASK) * 25000) + 3650000; > + return 0; > +} > + > +static enum power_supply_property da9150_charger_bat_props[] = { > + POWER_SUPPLY_PROP_STATUS, > + POWER_SUPPLY_PROP_ONLINE, > + POWER_SUPPLY_PROP_HEALTH, > + POWER_SUPPLY_PROP_PRESENT, > + POWER_SUPPLY_PROP_CHARGE_TYPE, > + POWER_SUPPLY_PROP_VOLTAGE_MIN_DESIGN, > + POWER_SUPPLY_PROP_VOLTAGE_NOW, > + POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX, > + POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX, > +}; > + > +static int da9150_charger_battery_get_prop(struct power_supply *psy, > + enum power_supply_property psp, > + union power_supply_propval *val) > +{ > + struct da9150_charger *charger = dev_get_drvdata(psy->dev->parent); > + int ret; > + > + switch (psp) { > + case POWER_SUPPLY_PROP_STATUS: > + ret = da9150_charger_battery_status(charger, val); > + break; > + case POWER_SUPPLY_PROP_ONLINE: > + ret = da9150_charger_supply_online(charger, psy, val); > + break; > + case POWER_SUPPLY_PROP_HEALTH: > + ret = da9150_charger_battery_health(charger, val); > + break; > + case POWER_SUPPLY_PROP_PRESENT: > + ret = da9150_charger_battery_present(charger, val); > + break; > + case POWER_SUPPLY_PROP_CHARGE_TYPE: > + ret = da9150_charger_battery_charge_type(charger, val); > + break; > + case POWER_SUPPLY_PROP_VOLTAGE_MIN_DESIGN: > + ret = da9150_charger_battery_voltage_min(charger, val); > + break; > + case POWER_SUPPLY_PROP_VOLTAGE_NOW: > + ret = da9150_charger_battery_voltage_now(charger, val); > + break; > + case POWER_SUPPLY_PROP_CONSTANT_CHARGE_CURRENT_MAX: > + ret = da9150_charger_battery_current_max(charger, val); > + break; > + case POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX: > + ret = da9150_charger_battery_voltage_max(charger, val); > + break; > + default: > + ret = -EINVAL; > + break; > + } > + > + return ret; > +} > + > +/* Other properties */ > +static ssize_t da9150_charger_attr_show(struct device *dev, > + struct device_attribute *attr, > + char *buf); > + > +#define DA9150_CHARGER_ATTR(_name, _reg, _shift, _mask) \ > + { \ > + .attr = __ATTR(_name, S_IRUGO, \ > + da9150_charger_attr_show, \ > + NULL), \ > + .reg = _reg, \ > + .shift = _shift, \ > + .mask = _mask, \ > + } > + > +static struct da9150_charger_attr_map da9150_charger_attrs[] = { > + DA9150_CHARGER_ATTR(chg_en, DA9150_PPR_CHGCTRL_A, > + DA9150_CHG_EN_SHIFT, DA9150_CHG_EN_MASK), > + DA9150_CHARGER_ATTR(chg_ipre, DA9150_PPR_CHGCTRL_C, > + DA9150_CHG_IPRE_SHIFT, DA9150_CHG_IPRE_MASK), > + DA9150_CHARGER_ATTR(chg_iend, DA9150_PPR_CHGCTRL_E, > + DA9150_CHG_IEND_SHIFT, DA9150_CHG_IEND_MASK), > + DA9150_CHARGER_ATTR(chg_temp, DA9150_STATUS_J, > + DA9150_CHG_TEMP_SHIFT, DA9150_CHG_TEMP_MASK), > + DA9150_CHARGER_ATTR(chg_vdrop, DA9150_PPR_CHGCTRL_B, > + DA9150_CHG_VDROP_SHIFT, DA9150_CHG_VDROP_MASK), > + DA9150_CHARGER_ATTR(tbat_upper, DA9150_PPR_THYST_E, > + DA9150_TBAT_T5_SHIFT, DA9150_TBAT_T5_MASK), > + DA9150_CHARGER_ATTR(tbat_lower, DA9150_PPR_THYST_A, > + DA9150_TBAT_T1_SHIFT, DA9150_TBAT_T1_MASK), > + DA9150_CHARGER_ATTR(vbus_mode, DA9150_PPR_BKCTRL_A, > + DA9150_VBUS_MODE_SHIFT, DA9150_VBUS_MODE_MASK), > + DA9150_CHARGER_ATTR(vbus_tred, DA9150_STATUS_H, > + DA9150_VBUS_TRED_SHIFT, DA9150_VBUS_TRED_MASK), > + DA9150_CHARGER_ATTR(vbus_drop_stat, DA9150_STATUS_H, > + DA9150_VBUS_DROP_STAT_SHIFT, > + DA9150_VBUS_DROP_STAT_MASK), > +}; > + > +static ssize_t da9150_charger_attr_show(struct device *dev, > + struct device_attribute *attr, > + char *buf) > +{ > + struct da9150_charger *charger = dev_get_drvdata(dev); > + struct da9150_charger_attr_map *charger_attrs; > + int i; > + > + /* Find attribute, and print out register field contents. */ > + for (i = 0, charger_attrs = da9150_charger_attrs; > + i < ARRAY_SIZE(da9150_charger_attrs); > + ++i, ++charger_attrs) { > + if (attr == &charger_attrs->attr) { > + u8 val; > + > + val = da9150_reg_read(charger->da9150, > + charger_attrs->reg); > + val &= charger_attrs->mask; > + val = val >> charger_attrs->shift; > + > + return sprintf(buf, "0x%x\n", val); > + } > + } > + > + return 0; > +} > + > +static irqreturn_t da9150_charger_chg_irq(int irq, void *data) > +{ > + struct da9150_charger *charger = data; > + > + power_supply_changed(&charger->battery); > + > + return IRQ_HANDLED; > +} > + > +static irqreturn_t da9150_charger_tjunc_irq(int irq, void *data) > +{ > + struct da9150_charger *charger = data; > + > + /* Nothing we can really do except report this. */ > + dev_crit(charger->dev, "TJunc over temperature!!!\n"); > + power_supply_changed(&charger->usb); > + > + return IRQ_HANDLED; > +} > + > +static irqreturn_t da9150_charger_vfault_irq(int irq, void *data) > +{ > + struct da9150_charger *charger = data; > + > + /* Nothing we can really do except report this. */ > + dev_crit(charger->dev, "VSYS under voltage!!!\n"); > + power_supply_changed(&charger->usb); > + power_supply_changed(&charger->battery); > + > + return IRQ_HANDLED; > +} > + > +static irqreturn_t da9150_charger_vbus_irq(int irq, void *data) > +{ > + struct da9150_charger *charger = data; > + u8 reg; > + > + reg = da9150_reg_read(charger->da9150, DA9150_STATUS_H); > + > + /* Charger plugged in or battery only */ > + switch (reg & DA9150_VBUS_STAT_MASK) { > + case DA9150_VBUS_STAT_OFF: > + case DA9150_VBUS_STAT_WAIT: > + charger->supply_online = &charger->battery; > + break; > + case DA9150_VBUS_STAT_CHG: > + charger->supply_online = &charger->usb; > + break; > + default: > + dev_warn(charger->dev, "Unknown VBUS state - reg = 0x%x\n", > + reg); > + charger->supply_online = NULL; > + break; > + } > + > + power_supply_changed(&charger->usb); > + power_supply_changed(&charger->battery); > + > + return IRQ_HANDLED; > +} > + > +static void da9150_charger_otg_work(struct work_struct *data) > +{ > + struct da9150_charger *charger = > + container_of(data, struct da9150_charger, otg_work); > + > + switch (charger->usb_event) { > + case USB_EVENT_ID: > + /* Enable OTG Boost */ > + da9150_set_bits(charger->da9150, DA9150_PPR_BKCTRL_A, > + DA9150_VBUS_MODE_MASK, DA9150_VBUS_MODE_OTG); > + break; > + case USB_EVENT_NONE: > + /* Revert to charge mode */ > + power_supply_changed(&charger->usb); > + power_supply_changed(&charger->battery); > + da9150_set_bits(charger->da9150, DA9150_PPR_BKCTRL_A, > + DA9150_VBUS_MODE_MASK, DA9150_VBUS_MODE_CHG); > + break; > + } > +} > + > +static int da9150_charger_otg_ncb(struct notifier_block *nb, unsigned long val, > + void *priv) > +{ > + struct da9150_charger *charger = > + container_of(nb, struct da9150_charger, otg_nb); > + > + dev_dbg(charger->dev, "DA9150 OTG notify %lu\n", val); > + > + charger->usb_event = val; > + schedule_work(&charger->otg_work); > + > + return NOTIFY_OK; > +} > + > +static int da9150_charger_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct da9150 *da9150 = dev_get_drvdata(dev->parent); > + struct da9150_charger *charger; > + struct power_supply *usb, *battery; > + u8 reg; > + int i, irq, ret; > + > + charger = devm_kzalloc(dev, sizeof(struct da9150_charger), GFP_KERNEL); > + if (charger == NULL) > + return -ENOMEM; > + > + platform_set_drvdata(pdev, charger); > + charger->da9150 = da9150; > + charger->dev = dev; > + > + /* Acquire ADC channels */ > + charger->ibus_chan = iio_channel_get(dev, "CHAN_IBUS"); > + if (IS_ERR(charger->ibus_chan)) { > + ret = PTR_ERR(charger->ibus_chan); > + goto ibus_chan_fail; > + } > + > + charger->vbus_chan = iio_channel_get(dev, "CHAN_VBUS"); > + if (IS_ERR(charger->vbus_chan)) { > + ret = PTR_ERR(charger->vbus_chan); > + goto vbus_chan_fail; > + } > + > + charger->tjunc_chan = iio_channel_get(dev, "CHAN_TJUNC"); > + if (IS_ERR(charger->tjunc_chan)) { > + ret = PTR_ERR(charger->tjunc_chan); > + goto tjunc_chan_fail; > + } > + > + charger->vbat_chan = iio_channel_get(dev, "CHAN_VBAT"); > + if (IS_ERR(charger->vbat_chan)) { > + ret = PTR_ERR(charger->vbat_chan); > + goto vbat_chan_fail; > + } > + > + /* Register power supplies */ > + usb = &charger->usb; > + battery = &charger->battery; > + > + usb->name = "da9150-usb", > + usb->type = POWER_SUPPLY_TYPE_USB; > + usb->properties = da9150_charger_props; > + usb->num_properties = ARRAY_SIZE(da9150_charger_props); > + usb->get_property = da9150_charger_get_prop; > + ret = power_supply_register(dev, usb); > + if (ret) > + goto usb_fail; > + > + battery->name = "da9150-battery"; > + battery->type = POWER_SUPPLY_TYPE_BATTERY; > + battery->properties = da9150_charger_bat_props; > + battery->num_properties = ARRAY_SIZE(da9150_charger_bat_props); > + battery->get_property = da9150_charger_battery_get_prop; > + ret = power_supply_register(dev, battery); > + if (ret) > + goto battery_fail; > + > + /* Create additional sysfs attributes */ > + for (i = 0; i < ARRAY_SIZE(da9150_charger_attrs); ++i) { > + ret = device_create_file(dev, &da9150_charger_attrs[i].attr); > + if (ret) > + goto sysfs_fail; > + } > + > + /* Get initial online supply */ > + reg = da9150_reg_read(da9150, DA9150_STATUS_H); > + > + switch (reg & DA9150_VBUS_STAT_MASK) { > + case DA9150_VBUS_STAT_OFF: > + case DA9150_VBUS_STAT_WAIT: > + charger->supply_online = &charger->battery; > + break; > + case DA9150_VBUS_STAT_CHG: > + charger->supply_online = &charger->usb; > + break; > + default: > + dev_warn(dev, "Unknown VBUS state - reg = 0x%x\n", reg); > + charger->supply_online = NULL; > + break; > + } > + > + /* Setup OTG reporting & configuration */ > + charger->usb_phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2); > + if (!IS_ERR_OR_NULL(charger->usb_phy)) { > + INIT_WORK(&charger->otg_work, da9150_charger_otg_work); > + charger->otg_nb.notifier_call = da9150_charger_otg_ncb; > + usb_register_notifier(charger->usb_phy, &charger->otg_nb); > + } > + > + /* Register IRQs */ > + irq = platform_get_irq_byname(pdev, "CHG_STATUS"); > + ret = devm_request_threaded_irq(dev, irq, NULL, da9150_charger_chg_irq, > + IRQF_ONESHOT, "CHG_STATUS", charger); > + if (ret) > + goto irq_fail; > + > + irq = platform_get_irq_byname(pdev, "CHG_TJUNC"); > + ret = devm_request_threaded_irq(dev, irq, NULL, > + da9150_charger_tjunc_irq, > + IRQF_ONESHOT, "CHG_TJUNC", charger); > + if (ret) > + goto irq_fail; > + > + irq = platform_get_irq_byname(pdev, "CHG_VFAULT"); > + ret = devm_request_threaded_irq(dev, irq, NULL, > + da9150_charger_vfault_irq, > + IRQF_ONESHOT, "CHG_VFAULT", charger); > + if (ret) > + goto irq_fail; > + > + irq = platform_get_irq_byname(pdev, "CHG_VBUS"); > + ret = devm_request_threaded_irq(dev, irq, NULL, da9150_charger_vbus_irq, > + IRQF_ONESHOT, "CHG_VBUS", charger); > + if (ret) > + goto irq_fail; > + > + return 0; > + > +irq_fail: > + dev_err(&pdev->dev, "Failed to request IRQ %d: %d\n", irq, ret); > + if (!IS_ERR_OR_NULL(charger->usb_phy)) > + usb_unregister_notifier(charger->usb_phy, &charger->otg_nb); > +sysfs_fail: > + while (--i >= 0) > + device_remove_file(&pdev->dev, &da9150_charger_attrs[i].attr); > + power_supply_unregister(battery); > + > +battery_fail: > + power_supply_unregister(usb); > + > +usb_fail: > + iio_channel_release(charger->vbat_chan); > + > +vbat_chan_fail: > + iio_channel_release(charger->tjunc_chan); > + > +tjunc_chan_fail: > + iio_channel_release(charger->vbus_chan); > + > +vbus_chan_fail: > + iio_channel_release(charger->ibus_chan); > + > +ibus_chan_fail: > + return ret; > +} > + > +static int da9150_charger_remove(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct da9150_charger *charger = platform_get_drvdata(pdev); > + int i, irq; > + > + /* Make sure IRQs are released before unregistering power supplies */ > + irq = platform_get_irq_byname(pdev, "CHG_VBUS"); > + devm_free_irq(dev, irq, charger); > + > + irq = platform_get_irq_byname(pdev, "CHG_VFAULT"); > + devm_free_irq(dev, irq, charger); > + > + irq = platform_get_irq_byname(pdev, "CHG_TJUNC"); > + devm_free_irq(dev, irq, charger); > + > + irq = platform_get_irq_byname(pdev, "CHG_STATUS"); > + devm_free_irq(dev, irq, charger); > + > + if (!IS_ERR_OR_NULL(charger->usb_phy)) > + usb_unregister_notifier(charger->usb_phy, &charger->otg_nb); > + > + power_supply_unregister(&charger->battery); > + power_supply_unregister(&charger->usb); > + > + for (i = 0; i < ARRAY_SIZE(da9150_charger_attrs); ++i) > + device_remove_file(&pdev->dev, &da9150_charger_attrs[i].attr); > + > + /* Release ADC channels */ > + iio_channel_release(charger->ibus_chan); > + iio_channel_release(charger->vbus_chan); > + iio_channel_release(charger->tjunc_chan); > + iio_channel_release(charger->vbat_chan); > + > + return 0; > +} > + > +static struct platform_driver da9150_charger_driver = { > + .driver = { > + .name = "da9150-charger", > + }, > + .probe = da9150_charger_probe, > + .remove = da9150_charger_remove, > +}; > + > +module_platform_driver(da9150_charger_driver); > + > +MODULE_DESCRIPTION("Charger Driver for DA9150"); > +MODULE_AUTHOR("Adam Thomson <Adam.Thomson.Opensource@xxxxxxxxxxx"); > +MODULE_LICENSE("GPL"); > -- > 1.9.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html