| Message ID | 20170414125919.25771-5-hdegoede@redhat.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
On Fri, Apr 14, 2017 at 02:59:11PM +0200, Hans de Goede wrote: > The max17042 will typically be couple with some charger IC, almost all s/couple/coupled/ Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org> Best regards, Krzysztof > charger drivers contain: > > static char *xxx_charger_supplied_to[] = { > "main-battery", > }; > > probe() > { > struct power_supply_config cfg; > > cfg.supplied_to = xxx_charger_supplied_to; > cfg.num_supplicants = ARRAY_SIZE(xxx_charger_supplied_to); > xxx->charger = power_supply_register(dev, &x_charger_desc, &_cfg); > } > > Change the name in max17042's power_supply_desc to "main-battery" to > match, so that power_supply_am_i_supplied() can be used to implement > POWER_SUPPLY_PROP_STATUS. > > Signed-off-by: Hans de Goede <hdegoede@redhat.com> > --- > drivers/power/supply/max17042_battery.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >
diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c index 85a6bf3..f8a0384 100644 --- a/drivers/power/supply/max17042_battery.c +++ b/drivers/power/supply/max17042_battery.c @@ -866,7 +866,7 @@ static const struct regmap_config max17042_regmap_config = { }; static const struct power_supply_desc max17042_psy_desc = { - .name = "max170xx_battery", + .name = "main-battery", .type = POWER_SUPPLY_TYPE_BATTERY, .get_property = max17042_get_property, .set_property = max17042_set_property, @@ -876,7 +876,7 @@ static const struct power_supply_desc max17042_psy_desc = { }; static const struct power_supply_desc max17042_no_current_sense_psy_desc = { - .name = "max170xx_battery", + .name = "main-battery", .type = POWER_SUPPLY_TYPE_BATTERY, .get_property = max17042_get_property, .set_property = max17042_set_property,
The max17042 will typically be couple with some charger IC, almost all charger drivers contain: static char *xxx_charger_supplied_to[] = { "main-battery", }; probe() { struct power_supply_config cfg; cfg.supplied_to = xxx_charger_supplied_to; cfg.num_supplicants = ARRAY_SIZE(xxx_charger_supplied_to); xxx->charger = power_supply_register(dev, &x_charger_desc, &_cfg); } Change the name in max17042's power_supply_desc to "main-battery" to match, so that power_supply_am_i_supplied() can be used to implement POWER_SUPPLY_PROP_STATUS. Signed-off-by: Hans de Goede <hdegoede@redhat.com> --- drivers/power/supply/max17042_battery.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)