From: Lee Jones <lee.jones@linaro.org>
To: Priit Laes <plaes@plaes.org>
Cc: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>, Chen-Yu Tsai <wens@csie.org>,
Maxime Ripard <maxime.ripard@bootlin.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
Olliver Schinagl <oliver@schinagl.nl>,
linux-sunxi <linux-sunxi@googlegroups.com>
Subject: Re: [linux-sunxi] Re: [PATCH v2 1/8] mfd: axp20x: name voltage ramping define properly
Date: Fri, 21 Dec 2018 09:26:58 +0000 [thread overview]
Message-ID: <20181221092658.GB13248@dell> (raw)
In-Reply-To: <20181221084246.myrtv6n4jrv6lajl@plaes.org>
On Fri, 21 Dec 2018, Priit Laes wrote:
> On Fri, Dec 21, 2018 at 08:39:27AM +0000, Lee Jones wrote:
> > On Tue, 11 Dec 2018, Priit Laes wrote:
> >
> > > From: Olliver Schinagl <oliver@schinagl.nl>
> > >
> > > The current axp20x names the ramping register 'scal' which probably
> > > means scaling. Since the register really has nothing to do with
> > > scaling, but really is the voltage ramp we rename it appropriately.
> > >
> > > Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
> > > Signed-off-by: Priit Laes <plaes@plaes.org>
> > > Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org>
> > > ---
> > > include/linux/mfd/axp20x.h | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > I've lost track of what's happening with this set now.
> >
> > Mark,
> >
> > Did you take this patch in the end?
>
> Yes, this is in linux-next.
I have no idea why Mark's tracker isn't mailing me (this is not the
first time this has happened).
Could you please rebase this set and resubmit all of the patches which
have not yet been applied?
Thank you.
> > > diff --git a/include/linux/mfd/axp20x.h b/include/linux/mfd/axp20x.h
> > > index 2302b62..a353cd2 100644
> > > --- a/include/linux/mfd/axp20x.h
> > > +++ b/include/linux/mfd/axp20x.h
> > > @@ -35,7 +35,7 @@ enum axp20x_variants {
> > > #define AXP152_ALDO_OP_MODE 0x13
> > > #define AXP152_LDO0_CTRL 0x15
> > > #define AXP152_DCDC2_V_OUT 0x23
> > > -#define AXP152_DCDC2_V_SCAL 0x25
> > > +#define AXP152_DCDC2_V_RAMP 0x25
> > > #define AXP152_DCDC1_V_OUT 0x26
> > > #define AXP152_DCDC3_V_OUT 0x27
> > > #define AXP152_ALDO12_V_OUT 0x28
> > > @@ -53,7 +53,7 @@ enum axp20x_variants {
> > > #define AXP20X_USB_OTG_STATUS 0x02
> > > #define AXP20X_PWR_OUT_CTRL 0x12
> > > #define AXP20X_DCDC2_V_OUT 0x23
> > > -#define AXP20X_DCDC2_LDO3_V_SCAL 0x25
> > > +#define AXP20X_DCDC2_LDO3_V_RAMP 0x25
> > > #define AXP20X_DCDC3_V_OUT 0x27
> > > #define AXP20X_LDO24_V_OUT 0x28
> > > #define AXP20X_LDO3_V_OUT 0x29
> >
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2018-12-21 9:26 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-11 15:17 [PATCH v2 0/8] regulator: axp20x: Stop AXP209 from crashing when enabling LDO3 Priit Laes
2018-12-11 15:17 ` [PATCH v2 3/8] dt-bindings: mfd: axp20x: add support for regulator-ramp-delay for AXP209 Priit Laes
2018-12-11 15:17 ` [PATCH v2 4/8] regulator: axp20x: add software based soft_start for AXP209 LDO3 Priit Laes
2018-12-11 15:17 ` [PATCH v2 5/8] dt-bindings: mfd: axp20x: Add " Priit Laes
[not found] ` <25e67fe9b5397aa5f11bdd7026e4ae6be51dd18b.1544512919.git-series.plaes-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org>
2018-12-21 8:40 ` Lee Jones
2018-12-11 15:17 ` [PATCH v2 6/8] regulator: dts: enable soft-start and ramp delay for the OLinuXino Lime2 Priit Laes
[not found] ` <cover.923997dd7d7396bb4d71f8967698b58ca956b929.1544512919.git-series.plaes-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org>
2018-12-11 15:17 ` [PATCH v2 1/8] mfd: axp20x: name voltage ramping define properly Priit Laes
[not found] ` <01367c4f4f1e174fefbef9aac645f6792bda07e5.1544512919.git-series.plaes-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org>
2018-12-21 8:39 ` Lee Jones
2018-12-21 8:42 ` [linux-sunxi] " Priit Laes
2018-12-21 9:26 ` Lee Jones [this message]
2018-12-21 9:28 ` Priit Laes
[not found] ` <20181221092858.6koh5dkrlc7xhvbh-q/aMd4JkU83YtjvyW6yDsg@public.gmane.org>
2018-12-21 9:57 ` Lee Jones
2018-12-11 15:17 ` [PATCH v2 2/8] regulator: axp20x: add support for set_ramp_delay for AXP209 Priit Laes
2018-12-11 15:17 ` [PATCH v2 7/8] mfd: axp20x: Clean up included headers Priit Laes
2018-12-11 15:17 ` [PATCH v2 8/8] mfd: axp20x: use explicit bit defines Priit Laes
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=20181221092658.GB13248@dell \
--to=lee.jones@linaro.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sunxi@googlegroups.com \
--cc=mark.rutland@arm.com \
--cc=maxime.ripard@bootlin.com \
--cc=oliver@schinagl.nl \
--cc=plaes@plaes.org \
--cc=robh+dt@kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox