All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastian Reichel <sre@kernel.org>
To: Peter Rosin <peda@axentia.se>
Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org
Subject: Re: [PATCH 0/3] power: supply: bq24735: handle AC adapter absence
Date: Thu, 22 Dec 2016 16:47:16 +0100	[thread overview]
Message-ID: <20161222154715.sowhh2kil4r7ovdx@earth> (raw)
In-Reply-To: <1482233582-29770-1-git-send-email-peda@axentia.se>

[-- Attachment #1: Type: text/plain, Size: 1538 bytes --]

Hi,

On Tue, Dec 20, 2016 at 12:32:59PM +0100, Peter Rosin wrote:
> There is provision in the driver for handling the case where the
> chargers are not responding to i2c requests when the AC adapter
> is absent. I think this has been used by some Tegra boards that
> leave the charger handling to some other component and that only
> wants the kernel to be aware of the charging status for reporting
> (or whatever).
> 
> However, in my case, the kernel should handle configuration and
> enable/disable the charging, but this is not working very well
> when the charger disappears (and is reset) on AC adapter absence.
> 
> This series fixes the issues I have found in this area so that I
> can boot without AC adapter and have the correct charging config
> every time I do attach the AC adapter.
> 
> Cheers,
> Peter
> 
> Peter Rosin (3):
>   power: supply: bq24735: move down bq24735_{en,dis}able_charging
>   power: supply: bq24735: configure the charger as part of enabling it
>   power: supply: bq24735: always check for AC adapter presence in probe
> 
>  drivers/power/supply/bq24735-charger.c | 56 ++++++++++++++++------------------
>  1 file changed, 26 insertions(+), 30 deletions(-)

Thanks for your patchset. We are currently in the merge
window and your patchset will appear in linux-next once
4.10-rc1 has been tagged by Linus Torvalds.

Until then I queued it into this branch:

https://git.kernel.org/cgit/linux/kernel/git/sre/linux-power-supply.git/log/?h=for-next-next

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      parent reply	other threads:[~2016-12-22 15:47 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-20 11:32 [PATCH 0/3] power: supply: bq24735: handle AC adapter absence Peter Rosin
2016-12-20 11:33 ` [PATCH 1/3] power: supply: bq24735: move down bq24735_{en,dis}able_charging Peter Rosin
2016-12-20 11:33 ` [PATCH 2/3] power: supply: bq24735: configure the charger as part of enabling it Peter Rosin
2016-12-20 11:33 ` [PATCH 3/3] power: supply: bq24735: always check for AC adapter presence in probe Peter Rosin
2016-12-22 15:47 ` Sebastian Reichel [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161222154715.sowhh2kil4r7ovdx@earth \
    --to=sre@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=peda@axentia.se \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.