All of lore.kernel.org
 help / color / mirror / Atom feed
From: Quentin Schulz <quentin.schulz@free-electrons.com>
To: Sebastian Reichel <sre@kernel.org>
Cc: mark.rutland@arm.com, thomas.petazzoni@free-electrons.com,
	lars@metafoo.de, linux-pm@vger.kernel.org,
	linux-iio@vger.kernel.org, linux-sunxi@googlegroups.com,
	linux-kernel@vger.kernel.org, robh+dt@kernel.org, wens@csie.org,
	devicetree@vger.kernel.org, liam@networkimprov.net,
	icenowy@aosc.xyz, pmeerw@pmeerw.net, knaack.h@gmx.de,
	linux@armlinux.org.uk, maxime.ripard@free-electrons.com,
	bonbons@linux-vserver.org, Lee Jones <lee.jones@linaro.org>,
	jic23@kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v4 00/18] add support for AXP20X and AXP22X power supply drivers
Date: Mon, 20 Mar 2017 12:34:03 +0100	[thread overview]
Message-ID: <949439a1-2f66-bdb7-04ae-ff88cb388809@free-electrons.com> (raw)
In-Reply-To: <20170315224647.suhnptpczykuoffd@earth>


[-- Attachment #1.1: Type: text/plain, Size: 2330 bytes --]

Hi Sebastian and Liam,

On 15/03/2017 23:46, Sebastian Reichel wrote:
> On Wed, Mar 15, 2017 at 02:18:56PM +0100, Quentin Schulz wrote:
>> Hi Lee,
>>
>> On 15/03/2017 13:14, Lee Jones wrote:
>>> On Wed, 15 Mar 2017, Quentin Schulz wrote:
>>>
>>>> The X-Powers AXP20X and AXP22X PMICs have multiple ADCs. They expose
>>>> information and data of the various power supplies they support such as
>>>> ACIN, battery and VBUS. For example, they expose the current battery
>>>> voltage, charge or discharge, as well as ACIN and VBUS current voltages
>>>> and currents, internal PMIC temperature and ADC on 2 different GPIOs
>>>> when in the right mode (for the AXP209 only).
>>>
>>> What are you missing from this set now?  What Acks etc?  And what's
>>> the plan with regards to route to Mainline?  Where are the dependencies
>>> etc?
>>>
>>
>> Basically, a consensus on how we should named the DT property I added in
>> patch 01 and 02 (which is used afterwards in battery driver). All other
>> patches have ACKs.
>>
>> I added some logic to the battery driver so if anyone wants to review
>> that you're welcome. I already got several ACKs from the v3 though.
>>
>> No dependencies except for the battery driver (and the first two
>> patches) which needs the battery "framework" in the Power Supply
>> subsystem. A v9 of this framework has been sent a little bit more than a
>> week ago, I need this to be merged first or the battery driver will just
>> not compile. I think the other patches are ready to be merged.
> 
> FWIW I expect, that the axp battery patches go through my tree, since
> there doesn't seem to be a compile time dependency to the other
> patches from this series nor vice versa. I'm aware of the dependency
> to the bq27xxx series of course.
> 

As there are still some discussion going on Liam's patch series on which
my patch series depends, I propose to wait for it to be merged and then
I'll rework these battery driver patches. Is there anything I can do to
help the merging of Liam's patch series?

Could you ping me when it is merged? The easiest way would be to Cc me
to the next version though so I can follow the discussion :)

Thanks,
Quentin

-- 
Quentin Schulz, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Quentin Schulz <quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
To: Sebastian Reichel <sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: mark.rutland-5wv7dgnIgG8@public.gmane.org,
	thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org,
	lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org,
	linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	wens-jdAy2FN1RRM@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	liam-RYWXG+zxWwBdeoIcmNTgJF6hYfS7NtTn@public.gmane.org,
	icenowy-ymACFijhrKM@public.gmane.org,
	pmeerw-jW+XmwGofnusTnJN9+BGXg@public.gmane.org,
	knaack.h-Mmb7MZpHnFY@public.gmane.org,
	linux-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org,
	maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org,
	bonbons-ud5FBsm0p/xEiooADzr8i9i2O/JbrIOy@public.gmane.org,
	Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v4 00/18] add support for AXP20X and AXP22X power supply drivers
Date: Mon, 20 Mar 2017 12:34:03 +0100	[thread overview]
Message-ID: <949439a1-2f66-bdb7-04ae-ff88cb388809@free-electrons.com> (raw)
In-Reply-To: <20170315224647.suhnptpczykuoffd@earth>


[-- Attachment #1.1: Type: text/plain, Size: 2590 bytes --]

Hi Sebastian and Liam,

On 15/03/2017 23:46, Sebastian Reichel wrote:
> On Wed, Mar 15, 2017 at 02:18:56PM +0100, Quentin Schulz wrote:
>> Hi Lee,
>>
>> On 15/03/2017 13:14, Lee Jones wrote:
>>> On Wed, 15 Mar 2017, Quentin Schulz wrote:
>>>
>>>> The X-Powers AXP20X and AXP22X PMICs have multiple ADCs. They expose
>>>> information and data of the various power supplies they support such as
>>>> ACIN, battery and VBUS. For example, they expose the current battery
>>>> voltage, charge or discharge, as well as ACIN and VBUS current voltages
>>>> and currents, internal PMIC temperature and ADC on 2 different GPIOs
>>>> when in the right mode (for the AXP209 only).
>>>
>>> What are you missing from this set now?  What Acks etc?  And what's
>>> the plan with regards to route to Mainline?  Where are the dependencies
>>> etc?
>>>
>>
>> Basically, a consensus on how we should named the DT property I added in
>> patch 01 and 02 (which is used afterwards in battery driver). All other
>> patches have ACKs.
>>
>> I added some logic to the battery driver so if anyone wants to review
>> that you're welcome. I already got several ACKs from the v3 though.
>>
>> No dependencies except for the battery driver (and the first two
>> patches) which needs the battery "framework" in the Power Supply
>> subsystem. A v9 of this framework has been sent a little bit more than a
>> week ago, I need this to be merged first or the battery driver will just
>> not compile. I think the other patches are ready to be merged.
> 
> FWIW I expect, that the axp battery patches go through my tree, since
> there doesn't seem to be a compile time dependency to the other
> patches from this series nor vice versa. I'm aware of the dependency
> to the bq27xxx series of course.
> 

As there are still some discussion going on Liam's patch series on which
my patch series depends, I propose to wait for it to be merged and then
I'll rework these battery driver patches. Is there anything I can do to
help the merging of Liam's patch series?

Could you ping me when it is merged? The easiest way would be to Cc me
to the next version though so I can follow the discussion :)

Thanks,
Quentin

-- 
Quentin Schulz, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: quentin.schulz@free-electrons.com (Quentin Schulz)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 00/18] add support for AXP20X and AXP22X power supply drivers
Date: Mon, 20 Mar 2017 12:34:03 +0100	[thread overview]
Message-ID: <949439a1-2f66-bdb7-04ae-ff88cb388809@free-electrons.com> (raw)
In-Reply-To: <20170315224647.suhnptpczykuoffd@earth>

Hi Sebastian and Liam,

On 15/03/2017 23:46, Sebastian Reichel wrote:
> On Wed, Mar 15, 2017 at 02:18:56PM +0100, Quentin Schulz wrote:
>> Hi Lee,
>>
>> On 15/03/2017 13:14, Lee Jones wrote:
>>> On Wed, 15 Mar 2017, Quentin Schulz wrote:
>>>
>>>> The X-Powers AXP20X and AXP22X PMICs have multiple ADCs. They expose
>>>> information and data of the various power supplies they support such as
>>>> ACIN, battery and VBUS. For example, they expose the current battery
>>>> voltage, charge or discharge, as well as ACIN and VBUS current voltages
>>>> and currents, internal PMIC temperature and ADC on 2 different GPIOs
>>>> when in the right mode (for the AXP209 only).
>>>
>>> What are you missing from this set now?  What Acks etc?  And what's
>>> the plan with regards to route to Mainline?  Where are the dependencies
>>> etc?
>>>
>>
>> Basically, a consensus on how we should named the DT property I added in
>> patch 01 and 02 (which is used afterwards in battery driver). All other
>> patches have ACKs.
>>
>> I added some logic to the battery driver so if anyone wants to review
>> that you're welcome. I already got several ACKs from the v3 though.
>>
>> No dependencies except for the battery driver (and the first two
>> patches) which needs the battery "framework" in the Power Supply
>> subsystem. A v9 of this framework has been sent a little bit more than a
>> week ago, I need this to be merged first or the battery driver will just
>> not compile. I think the other patches are ready to be merged.
> 
> FWIW I expect, that the axp battery patches go through my tree, since
> there doesn't seem to be a compile time dependency to the other
> patches from this series nor vice versa. I'm aware of the dependency
> to the bq27xxx series of course.
> 

As there are still some discussion going on Liam's patch series on which
my patch series depends, I propose to wait for it to be merged and then
I'll rework these battery driver patches. Is there anything I can do to
help the merging of Liam's patch series?

Could you ping me when it is merged? The easiest way would be to Cc me
to the next version though so I can follow the discussion :)

Thanks,
Quentin

-- 
Quentin Schulz, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170320/3218d507/attachment.sig>

  reply	other threads:[~2017-03-20 11:34 UTC|newest]

Thread overview: 101+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-15 10:55 [PATCH v4 00/18] add support for AXP20X and AXP22X power supply drivers Quentin Schulz
2017-03-15 10:55 ` Quentin Schulz
2017-03-15 10:55 ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 01/18] dt-bindings: power: battery: add constant-charge-current property Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 12:08   ` Liam Breck
     [not found]     ` <CAKvHMgSnyPPMEZ1o70Ed2oBHGcMho=-jjFhfXGn4t+2bdALEew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-15 13:10       ` Quentin Schulz
     [not found]         ` <ce6f2ddb-f875-0370-ca45-293397d97495-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2017-03-15 22:23           ` Sebastian Reichel
2017-03-16  6:27           ` Liam Breck
     [not found]             ` <CAKvHMgR58sDnB-nn386Bd-H0Duevkeoc8eEfz=otLAumc8FD+w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-16  7:03               ` Quentin Schulz
2017-03-16  7:42                 ` Liam Breck
     [not found]                   ` <CAKvHMgRKroPFDdNMb=F=bA4bTwg9bYMAR-5Z4DKLzHJtCYz3_Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-29  6:55                     ` Liam Breck
     [not found]                       ` <CAKvHMgRtEnNbokYReu2vEs9UKH6qZnncdyQD=1JMDTZeN+aYCA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-29  7:09                         ` Quentin Schulz
2017-03-29  7:39                           ` Liam Breck
2017-03-29  7:54                             ` Quentin Schulz
2017-03-29  9:26                               ` Liam Breck
     [not found]                                 ` <CAKvHMgQf=Zod-F9SMPDDj5Eq3v=JHGh7uns-T_=ynQM0dMtEvw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-03-30  6:41                                   ` Quentin Schulz
2017-03-30  7:30                                     ` Liam Breck
2017-03-15 22:45         ` Liam Breck
2017-03-15 10:55 ` [PATCH v4 02/18] power: supply: power_supply_core: add constant-charge-current optional property Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 22:24   ` Sebastian Reichel
2017-03-15 22:24     ` Sebastian Reichel
2017-03-15 22:24     ` Sebastian Reichel
2017-03-15 10:55 ` [PATCH v4 03/18] mfd: axp20x: correct name of temperature data ADC registers Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 04/18] iio: adc: add support for X-Powers AXP20X and AXP22X PMICs ADCs Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 05/18] mfd: axp20x: add ADC cells for AXP20X and AXP22X PMICs Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 06/18] mfd: axp20x: add AC power supply cells for " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 07/18] ARM: dtsi: axp209: add AC power supply subnode Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 08/18] ARM: dtsi: axp22x: " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 09/18] ARM: dts: sun8i: sina33: enable ACIN " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 10/18] ARM: sun5i: chip: " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 11/18] dt-bindings: power: supply: add AXP20X/AXP22X battery DT binding Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 22:28   ` Sebastian Reichel
2017-03-15 22:28     ` Sebastian Reichel
2017-03-15 22:28     ` Sebastian Reichel
2017-03-15 22:41     ` Quentin Schulz
2017-03-15 22:41       ` Quentin Schulz
2017-03-15 22:41       ` Quentin Schulz
2017-03-15 23:02       ` Sebastian Reichel
2017-03-15 23:02         ` Sebastian Reichel
2017-03-15 23:02         ` Sebastian Reichel
2017-03-15 10:55 ` [PATCH v4 12/18] mfd: axp20x: add CHRG_CTRL1/2/3 to writeable regs for AXP20X/AXP22X Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 13/18] power: supply: add battery driver for AXP20X and AXP22X PMICs Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 22:38   ` Sebastian Reichel
2017-03-15 22:38     ` Sebastian Reichel
2017-03-15 22:38     ` Sebastian Reichel
2017-03-15 10:55 ` [PATCH v4 14/18] mfd: axp20x: add MFD cells for AXP20X and AXP22X battery driver Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 15/18] ARM: dtsi: axp209: add battery power supply subnode Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 16/18] ARM: dtsi: axp22x: " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 17/18] ARM: dts: sun8i: sina33: enable " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55 ` [PATCH v4 18/18] ARM: sun5i: chip: " Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 10:55   ` Quentin Schulz
2017-03-15 12:14 ` [PATCH v4 00/18] add support for AXP20X and AXP22X power supply drivers Lee Jones
2017-03-15 12:14   ` Lee Jones
2017-03-15 12:14   ` Lee Jones
2017-03-15 13:18   ` Quentin Schulz
2017-03-15 13:18     ` Quentin Schulz
2017-03-15 13:18     ` Quentin Schulz
2017-03-15 22:46     ` Sebastian Reichel
2017-03-15 22:46       ` Sebastian Reichel
2017-03-15 22:46       ` Sebastian Reichel
2017-03-20 11:34       ` Quentin Schulz [this message]
2017-03-20 11:34         ` Quentin Schulz
2017-03-20 11:34         ` Quentin Schulz
2017-03-20 19:11         ` Sebastian Reichel
2017-03-20 19:11           ` Sebastian Reichel
2017-03-20 19:11           ` Sebastian Reichel

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=949439a1-2f66-bdb7-04ae-ff88cb388809@free-electrons.com \
    --to=quentin.schulz@free-electrons.com \
    --cc=bonbons@linux-vserver.org \
    --cc=devicetree@vger.kernel.org \
    --cc=icenowy@aosc.xyz \
    --cc=jic23@kernel.org \
    --cc=knaack.h@gmx.de \
    --cc=lars@metafoo.de \
    --cc=lee.jones@linaro.org \
    --cc=liam@networkimprov.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux-sunxi@googlegroups.com \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=maxime.ripard@free-electrons.com \
    --cc=pmeerw@pmeerw.net \
    --cc=robh+dt@kernel.org \
    --cc=sre@kernel.org \
    --cc=thomas.petazzoni@free-electrons.com \
    --cc=wens@csie.org \
    /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.