Re: [PATCHv1 0/3] N900 battery status fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 22/02/18 21:23, Sebastian Reichel wrote:
> Hi,
> 
> Merlijn Wajer reported an issue on N900, that battery status is reported
> "not charging" when the charger is not connected. This is a bug in
> bq27xxx driver, which does not properly handle error codes, like
> -ENODEV, for power_supply_am_i_supplied(). This is fixed in the first
> patch.
> 
> It also exposed, that the charger relationship is not described in
> the N900 DT. This is fixed by patch 3, which depends on patch 2
> (DT reference support for bq2415x).
> 
> Patch 1 and 2 has no dependencies. Patch 3 requires a kernel with
> patch 2 applied, otherwise bq27xxx will probe defer infinitely
> 
> -- Sebastian
> 
> Sebastian Reichel (3):
>   power: supply: bq27xxx: support missing supplier device
>   power: supply: bq2415x: add DT referencing support
>   ARM: dts: omap3-n900: Add link between battery and charger
> 
>  arch/arm/boot/dts/omap3-n900.dts       | 1 +
>  drivers/power/supply/bq2415x_charger.c | 5 ++++-
>  drivers/power/supply/bq27xxx_battery.c | 2 +-
>  3 files changed, 6 insertions(+), 2 deletions(-)
I have verified that these patches solve the problem:

Tested-by: Merlijn Wajer <merlijn@xxxxxxxxxx>

Thank you,
Cheers,
Merlijn

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux