Skip to content

Commit cc49e8c

Browse files
committed
Merge tag 'for-v4.1-rc' of git://git.infradead.org/battery-2.6
Pull power supply and reset fixes from Sebastian Reichel: "misc fixes" * tag 'for-v4.1-rc' of git://git.infradead.org/battery-2.6: power: bq27x00_battery: Add missing MODULE_ALIAS power: reset: Add MFD_SYSCON depends for brcmstb power: reset: ltc2952: Remove bogus hrtimer_start() return value checks power_supply: fix oops in collie_battery driver power/reset: at91: fix return value check in at91_reset_platform_probe() MAINTAINERS: Add me as maintainer of Nokia N900 power supply drivers axp288_fuel_gauge: Add original author details
2 parents 4cfceaf + 8ebb7e9 commit cc49e8c

File tree

7 files changed

+27
-18
lines changed

7 files changed

+27
-18
lines changed

MAINTAINERS

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6959,6 +6959,17 @@ T: git git://git.rocketboards.org/linux-socfpga-next.git
69596959
S: Maintained
69606960
F: arch/nios2/
69616961

6962+
NOKIA N900 POWER SUPPLY DRIVERS
6963+
M: Pali Rohár <pali.rohar@gmail.com>
6964+
S: Maintained
6965+
F: include/linux/power/bq2415x_charger.h
6966+
F: include/linux/power/bq27x00_battery.h
6967+
F: include/linux/power/isp1704_charger.h
6968+
F: drivers/power/bq2415x_charger.c
6969+
F: drivers/power/bq27x00_battery.c
6970+
F: drivers/power/isp1704_charger.c
6971+
F: drivers/power/rx51_battery.c
6972+
69626973
NTB DRIVER
69636974
M: Jon Mason <jdmason@kudzu.us>
69646975
M: Dave Jiang <dave.jiang@intel.com>

drivers/power/axp288_fuel_gauge.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1149,6 +1149,7 @@ static struct platform_driver axp288_fuel_gauge_driver = {
11491149

11501150
module_platform_driver(axp288_fuel_gauge_driver);
11511151

1152+
MODULE_AUTHOR("Ramakrishna Pallala <ramakrishna.pallala@intel.com>");
11521153
MODULE_AUTHOR("Todd Brandt <todd.e.brandt@linux.intel.com>");
11531154
MODULE_DESCRIPTION("Xpower AXP288 Fuel Gauge Driver");
11541155
MODULE_LICENSE("GPL");

drivers/power/bq27x00_battery.c

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1109,6 +1109,14 @@ static void __exit bq27x00_battery_exit(void)
11091109
}
11101110
module_exit(bq27x00_battery_exit);
11111111

1112+
#ifdef CONFIG_BATTERY_BQ27X00_PLATFORM
1113+
MODULE_ALIAS("platform:bq27000-battery");
1114+
#endif
1115+
1116+
#ifdef CONFIG_BATTERY_BQ27X00_I2C
1117+
MODULE_ALIAS("i2c:bq27000-battery");
1118+
#endif
1119+
11121120
MODULE_AUTHOR("Rodolfo Giometti <giometti@linux.it>");
11131121
MODULE_DESCRIPTION("BQ27x00 battery monitor driver");
11141122
MODULE_LICENSE("GPL");

drivers/power/collie_battery.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ static int collie_bat_probe(struct ucb1x00_dev *dev)
347347
goto err_psy_reg_main;
348348
}
349349

350-
psy_main_cfg.drv_data = &collie_bat_bu;
350+
psy_bu_cfg.drv_data = &collie_bat_bu;
351351
collie_bat_bu.psy = power_supply_register(&dev->ucb->dev,
352352
&collie_bat_bu_desc,
353353
&psy_bu_cfg);

drivers/power/reset/Kconfig

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ config POWER_RESET_AXXIA
4141
config POWER_RESET_BRCMSTB
4242
bool "Broadcom STB reset driver"
4343
depends on ARM || MIPS || COMPILE_TEST
44+
depends on MFD_SYSCON
4445
default ARCH_BRCMSTB
4546
help
4647
This driver provides restart support for Broadcom STB boards.

drivers/power/reset/at91-reset.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -212,9 +212,9 @@ static int at91_reset_platform_probe(struct platform_device *pdev)
212212
res = platform_get_resource(pdev, IORESOURCE_MEM, idx + 1 );
213213
at91_ramc_base[idx] = devm_ioremap(&pdev->dev, res->start,
214214
resource_size(res));
215-
if (IS_ERR(at91_ramc_base[idx])) {
215+
if (!at91_ramc_base[idx]) {
216216
dev_err(&pdev->dev, "Could not map ram controller address\n");
217-
return PTR_ERR(at91_ramc_base[idx]);
217+
return -ENOMEM;
218218
}
219219
}
220220

drivers/power/reset/ltc2952-poweroff.c

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -120,18 +120,7 @@ static enum hrtimer_restart ltc2952_poweroff_timer_wde(struct hrtimer *timer)
120120

121121
static void ltc2952_poweroff_start_wde(struct ltc2952_poweroff *data)
122122
{
123-
if (hrtimer_start(&data->timer_wde, data->wde_interval,
124-
HRTIMER_MODE_REL)) {
125-
/*
126-
* The device will not toggle the watchdog reset,
127-
* thus shut down is only safe if the PowerPath controller
128-
* has a long enough time-off before triggering a hardware
129-
* power-off.
130-
*
131-
* Only sending a warning as the system will power-off anyway
132-
*/
133-
dev_err(data->dev, "unable to start the timer\n");
134-
}
123+
hrtimer_start(&data->timer_wde, data->wde_interval, HRTIMER_MODE_REL);
135124
}
136125

137126
static enum hrtimer_restart
@@ -165,9 +154,8 @@ static irqreturn_t ltc2952_poweroff_handler(int irq, void *dev_id)
165154
}
166155

167156
if (gpiod_get_value(data->gpio_trigger)) {
168-
if (hrtimer_start(&data->timer_trigger, data->trigger_delay,
169-
HRTIMER_MODE_REL))
170-
dev_err(data->dev, "unable to start the wait timer\n");
157+
hrtimer_start(&data->timer_trigger, data->trigger_delay,
158+
HRTIMER_MODE_REL);
171159
} else {
172160
hrtimer_cancel(&data->timer_trigger);
173161
/* omitting return value check, timer should have been valid */

0 commit comments

Comments
 (0)