Commit 2d9fee6a authored by Liam Breck's avatar Liam Breck Committed by Sebastian Reichel

power: supply: bq24190_charger: Call power_supply_changed() for relevant component

We wrongly get uevents for bq24190-charger and bq24190-battery on every
register change.

Fix by checking the association with charger and battery before
emitting uevent(s).

Fixes: d7bf353f ("bq24190_charger: Add support for TI BQ24190 Battery Charger")
Signed-off-by: default avatarLiam Breck <kernel@networkimprov.net>
Acked-by: default avatarMark Greer <mgreer@animalcreek.com>
Acked-by: default avatarTony Lindgren <tony@atomide.com>
Signed-off-by: default avatarSebastian Reichel <sre@kernel.org>
parent d62acc5e
...@@ -159,7 +159,6 @@ struct bq24190_dev_info { ...@@ -159,7 +159,6 @@ struct bq24190_dev_info {
unsigned int gpio_int; unsigned int gpio_int;
unsigned int irq; unsigned int irq;
struct mutex f_reg_lock; struct mutex f_reg_lock;
bool first_time;
bool charger_health_valid; bool charger_health_valid;
bool battery_health_valid; bool battery_health_valid;
bool battery_status_valid; bool battery_status_valid;
...@@ -1197,7 +1196,10 @@ static const struct power_supply_desc bq24190_battery_desc = { ...@@ -1197,7 +1196,10 @@ static const struct power_supply_desc bq24190_battery_desc = {
static irqreturn_t bq24190_irq_handler_thread(int irq, void *data) static irqreturn_t bq24190_irq_handler_thread(int irq, void *data)
{ {
struct bq24190_dev_info *bdi = data; struct bq24190_dev_info *bdi = data;
bool alert_userspace = false; const u8 battery_mask_ss = BQ24190_REG_SS_CHRG_STAT_MASK;
const u8 battery_mask_f = BQ24190_REG_F_BAT_FAULT_MASK
| BQ24190_REG_F_NTC_FAULT_MASK;
bool alert_charger = false, alert_battery = false;
u8 ss_reg = 0, f_reg = 0; u8 ss_reg = 0, f_reg = 0;
int ret; int ret;
...@@ -1225,8 +1227,12 @@ static irqreturn_t bq24190_irq_handler_thread(int irq, void *data) ...@@ -1225,8 +1227,12 @@ static irqreturn_t bq24190_irq_handler_thread(int irq, void *data)
ret); ret);
} }
if ((bdi->ss_reg & battery_mask_ss) != (ss_reg & battery_mask_ss))
alert_battery = true;
if ((bdi->ss_reg & ~battery_mask_ss) != (ss_reg & ~battery_mask_ss))
alert_charger = true;
bdi->ss_reg = ss_reg; bdi->ss_reg = ss_reg;
alert_userspace = true;
} }
mutex_lock(&bdi->f_reg_lock); mutex_lock(&bdi->f_reg_lock);
...@@ -1239,33 +1245,23 @@ static irqreturn_t bq24190_irq_handler_thread(int irq, void *data) ...@@ -1239,33 +1245,23 @@ static irqreturn_t bq24190_irq_handler_thread(int irq, void *data)
} }
if (f_reg != bdi->f_reg) { if (f_reg != bdi->f_reg) {
if ((bdi->f_reg & battery_mask_f) != (f_reg & battery_mask_f))
alert_battery = true;
if ((bdi->f_reg & ~battery_mask_f) != (f_reg & ~battery_mask_f))
alert_charger = true;
bdi->f_reg = f_reg; bdi->f_reg = f_reg;
bdi->charger_health_valid = true; bdi->charger_health_valid = true;
bdi->battery_health_valid = true; bdi->battery_health_valid = true;
bdi->battery_status_valid = true; bdi->battery_status_valid = true;
alert_userspace = true;
} }
mutex_unlock(&bdi->f_reg_lock); mutex_unlock(&bdi->f_reg_lock);
/* if (alert_charger)
* Sometimes bq24190 gives a steady trickle of interrupts even power_supply_changed(bdi->charger);
* though the watchdog timer is turned off and neither the STATUS if (alert_battery)
* nor FAULT registers have changed. Weed out these sprurious power_supply_changed(bdi->battery);
* interrupts so userspace isn't alerted for no reason.
* In addition, the chip always generates an interrupt after
* register reset so we should ignore that one (the very first
* interrupt received).
*/
if (alert_userspace) {
if (!bdi->first_time) {
power_supply_changed(bdi->charger);
power_supply_changed(bdi->battery);
} else {
bdi->first_time = false;
}
}
out: out:
pm_runtime_put_sync(bdi->dev); pm_runtime_put_sync(bdi->dev);
...@@ -1300,6 +1296,10 @@ static int bq24190_hw_init(struct bq24190_dev_info *bdi) ...@@ -1300,6 +1296,10 @@ static int bq24190_hw_init(struct bq24190_dev_info *bdi)
goto out; goto out;
ret = bq24190_set_mode_host(bdi); ret = bq24190_set_mode_host(bdi);
if (ret < 0)
goto out;
ret = bq24190_read(bdi, BQ24190_REG_SS, &bdi->ss_reg);
out: out:
pm_runtime_put_sync(bdi->dev); pm_runtime_put_sync(bdi->dev);
return ret; return ret;
...@@ -1375,7 +1375,8 @@ static int bq24190_probe(struct i2c_client *client, ...@@ -1375,7 +1375,8 @@ static int bq24190_probe(struct i2c_client *client,
bdi->model = id->driver_data; bdi->model = id->driver_data;
strncpy(bdi->model_name, id->name, I2C_NAME_SIZE); strncpy(bdi->model_name, id->name, I2C_NAME_SIZE);
mutex_init(&bdi->f_reg_lock); mutex_init(&bdi->f_reg_lock);
bdi->first_time = true; bdi->f_reg = 0;
bdi->ss_reg = BQ24190_REG_SS_VBUS_STAT_MASK; /* impossible state */
bdi->charger_health_valid = false; bdi->charger_health_valid = false;
bdi->battery_health_valid = false; bdi->battery_health_valid = false;
bdi->battery_status_valid = false; bdi->battery_status_valid = false;
...@@ -1489,6 +1490,8 @@ static int bq24190_pm_resume(struct device *dev) ...@@ -1489,6 +1490,8 @@ static int bq24190_pm_resume(struct device *dev)
struct i2c_client *client = to_i2c_client(dev); struct i2c_client *client = to_i2c_client(dev);
struct bq24190_dev_info *bdi = i2c_get_clientdata(client); struct bq24190_dev_info *bdi = i2c_get_clientdata(client);
bdi->f_reg = 0;
bdi->ss_reg = BQ24190_REG_SS_VBUS_STAT_MASK; /* impossible state */
bdi->charger_health_valid = false; bdi->charger_health_valid = false;
bdi->battery_health_valid = false; bdi->battery_health_valid = false;
bdi->battery_status_valid = false; bdi->battery_status_valid = false;
...@@ -1496,6 +1499,7 @@ static int bq24190_pm_resume(struct device *dev) ...@@ -1496,6 +1499,7 @@ static int bq24190_pm_resume(struct device *dev)
pm_runtime_get_sync(bdi->dev); pm_runtime_get_sync(bdi->dev);
bq24190_register_reset(bdi); bq24190_register_reset(bdi);
bq24190_set_mode_host(bdi); bq24190_set_mode_host(bdi);
bq24190_read(bdi, BQ24190_REG_SS, &bdi->ss_reg);
pm_runtime_put_sync(bdi->dev); pm_runtime_put_sync(bdi->dev);
/* Things may have changed while suspended so alert upper layer */ /* Things may have changed while suspended so alert upper layer */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment