Hi Chen-Yu, On 16/05/2017 07:18, Chen-Yu Tsai wrote: > On Thu, May 11, 2017 at 9:42 PM, Quentin Schulz > <quentin.schulz@xxxxxxxxxxxxxxxxxx> wrote: >> This adds support in X-Powers AXP20X and AXP22X battery driver for a >> fixed battery in DT. >> >> It will take the minimum supported voltage by the battery as defined in >> the battery DT node and set the V_OFF register to this value, telling >> the system to shut down if the supplied power is below this value. >> >> Signed-off-by: Quentin Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxx> >> --- >> drivers/power/supply/axp20x_battery.c | 10 ++++++++++ >> 1 file changed, 10 insertions(+) >> >> diff --git a/drivers/power/supply/axp20x_battery.c b/drivers/power/supply/axp20x_battery.c >> index 5d29b2eab8fc..66f530541735 100644 >> --- a/drivers/power/supply/axp20x_battery.c >> +++ b/drivers/power/supply/axp20x_battery.c >> @@ -433,6 +433,7 @@ static int axp20x_power_probe(struct platform_device *pdev) >> { >> struct axp20x_batt_ps *axp20x_batt; >> struct power_supply_config psy_cfg = {}; >> + struct power_supply_battery_info info; >> >> if (!of_device_is_available(pdev->dev.of_node)) >> return -ENODEV; >> @@ -484,6 +485,15 @@ static int axp20x_power_probe(struct platform_device *pdev) >> return PTR_ERR(axp20x_batt->batt); >> } >> >> + if (!power_supply_get_battery_info(axp20x_batt->batt, &info)) { > > This is not in -next. It would help if you mention any requisite patches. > I started working on this when next-20170503 was the latest and forgot to rebase before sending the patch series. It was actually in both next-20170502 and 20170503 but not in more recent ones. Liam told me in a private message he's soon sending a new version of his patch series, so I'll be waiting for the patches to reappear in next. Thanks, Quentin > ChenYu > >> + int vmin = info.voltage_min_design_uv; >> + >> + if (vmin > 0 && axp20x_set_voltage_min_design(axp20x_batt, >> + vmin)) >> + dev_err(&pdev->dev, >> + "couldn't set voltage_min_design\n"); >> + } >> + >> return 0; >> } >> >> -- >> 2.11.0 >> -- Quentin Schulz, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- 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