* [PATCH] ARM: ux500: update V-INTCORE constraints
@ 2013-03-27 15:21 Fabio Baltieri
2013-04-02 8:26 ` Linus Walleij
0 siblings, 1 reply; 9+ messages in thread
From: Fabio Baltieri @ 2013-03-27 15:21 UTC (permalink / raw)
To: linux-arm-kernel
Update V-INTCORE regulator constraints to allow voltage and
configuration changes from consumer drivers.
Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
---
Hi Linus,
this patch adds the regulators bits I'm missing for ux500-musb patches.
Without this I just get a runtime warning like:
V-INTCORE: operation not allowed
but the driver works anyway and there are no build dependencies, so I
think this can safely go in the arm-soc tree on its own, and the whole
thing should work happily in -next.
Thanks,
Fabio
arch/arm/mach-ux500/board-mop500-regulators.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-ux500/board-mop500-regulators.c b/arch/arm/mach-ux500/board-mop500-regulators.c
index 2a17bc5..0a428db 100644
--- a/arch/arm/mach-ux500/board-mop500-regulators.c
+++ b/arch/arm/mach-ux500/board-mop500-regulators.c
@@ -392,7 +392,15 @@ struct regulator_init_data ab8500_regulators[AB8500_NUM_REGULATORS] = {
[AB8500_LDO_INTCORE] = {
.constraints = {
.name = "V-INTCORE",
- .valid_ops_mask = REGULATOR_CHANGE_STATUS,
+ .min_uV = 1250000,
+ .max_uV = 1350000,
+ .input_uV = 1800000,
+ .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE |
+ REGULATOR_CHANGE_STATUS |
+ REGULATOR_CHANGE_MODE |
+ REGULATOR_CHANGE_DRMS,
+ .valid_modes_mask = REGULATOR_MODE_NORMAL |
+ REGULATOR_MODE_IDLE,
},
.num_consumer_supplies = ARRAY_SIZE(ab8500_vintcore_consumers),
.consumer_supplies = ab8500_vintcore_consumers,
--
1.8.1.3
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH] ARM: ux500: update V-INTCORE constraints
2013-03-27 15:21 [PATCH] ARM: ux500: update V-INTCORE constraints Fabio Baltieri
@ 2013-04-02 8:26 ` Linus Walleij
2013-04-02 8:38 ` Mark Brown
0 siblings, 1 reply; 9+ messages in thread
From: Linus Walleij @ 2013-04-02 8:26 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Mar 27, 2013 at 4:21 PM, Fabio Baltieri
<fabio.baltieri@linaro.org> wrote:
> Update V-INTCORE regulator constraints to allow voltage and
> configuration changes from consumer drivers.
>
> Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
> ---
> Hi Linus,
>
> this patch adds the regulators bits I'm missing for ux500-musb patches.
> Without this I just get a runtime warning like:
>
> V-INTCORE: operation not allowed
>
> but the driver works anyway and there are no build dependencies, so I
> think this can safely go in the arm-soc tree on its own, and the whole
> thing should work happily in -next.
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
I know that Lee's been sending off a large set of regulator patches
lately, some include alterations to this platform config.
Maybe it's better to take this patch into the regulator tree if these
machine configs will come in that way? Also like Lee to have a
look at this...
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: ux500: update V-INTCORE constraints
2013-04-02 8:26 ` Linus Walleij
@ 2013-04-02 8:38 ` Mark Brown
2013-04-02 8:44 ` Fabio Baltieri
0 siblings, 1 reply; 9+ messages in thread
From: Mark Brown @ 2013-04-02 8:38 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Apr 02, 2013 at 10:26:12AM +0200, Linus Walleij wrote:
> I know that Lee's been sending off a large set of regulator patches
> lately, some include alterations to this platform config.
> Maybe it's better to take this patch into the regulator tree if these
> machine configs will come in that way? Also like Lee to have a
> look at this...
If it's touching the board constraints it's likely to have conflicts
with Lee's stuff, yes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130402/1decce6c/attachment-0001.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: ux500: update V-INTCORE constraints
2013-04-02 8:38 ` Mark Brown
@ 2013-04-02 8:44 ` Fabio Baltieri
2013-04-02 8:46 ` Mark Brown
2013-04-02 9:24 ` [PATCH] " Lee Jones
0 siblings, 2 replies; 9+ messages in thread
From: Fabio Baltieri @ 2013-04-02 8:44 UTC (permalink / raw)
To: linux-arm-kernel
Hi Mark, Linus,
On Tue, Apr 02, 2013 at 09:38:30AM +0100, Mark Brown wrote:
> On Tue, Apr 02, 2013 at 10:26:12AM +0200, Linus Walleij wrote:
>
> > I know that Lee's been sending off a large set of regulator patches
> > lately, some include alterations to this platform config.
>
> > Maybe it's better to take this patch into the regulator tree if these
> > machine configs will come in that way? Also like Lee to have a
> > look at this...
>
> If it's touching the board constraints it's likely to have conflicts
> with Lee's stuff, yes.
Lee's patches are on -next and should not conflict with this one
(tested again on today's tree), but it may be nice anyway to have this
in the regulator framework with the others.
Mark, would you consider taking this one in your tree?
Thanks,
Fabio
--
Fabio Baltieri
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: ux500: update V-INTCORE constraints
2013-04-02 8:44 ` Fabio Baltieri
@ 2013-04-02 8:46 ` Mark Brown
2013-04-02 8:51 ` [PATCH RESEND] " Fabio Baltieri
2013-04-02 9:24 ` [PATCH] " Lee Jones
1 sibling, 1 reply; 9+ messages in thread
From: Mark Brown @ 2013-04-02 8:46 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Apr 02, 2013 at 10:44:46AM +0200, Fabio Baltieri wrote:
> Mark, would you consider taking this one in your tree?
If someone sends me the patch...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130402/f47d75a2/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH RESEND] ARM: ux500: update V-INTCORE constraints
2013-04-02 8:46 ` Mark Brown
@ 2013-04-02 8:51 ` Fabio Baltieri
2013-04-02 9:23 ` Lee Jones
0 siblings, 1 reply; 9+ messages in thread
From: Fabio Baltieri @ 2013-04-02 8:51 UTC (permalink / raw)
To: linux-arm-kernel
Update V-INTCORE regulator constraints to allow voltage and
configuration changes from consumer drivers.
Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
---
arch/arm/mach-ux500/board-mop500-regulators.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-ux500/board-mop500-regulators.c b/arch/arm/mach-ux500/board-mop500-regulators.c
index 2a17bc5..0a428db 100644
--- a/arch/arm/mach-ux500/board-mop500-regulators.c
+++ b/arch/arm/mach-ux500/board-mop500-regulators.c
@@ -392,7 +392,15 @@ struct regulator_init_data ab8500_regulators[AB8500_NUM_REGULATORS] = {
[AB8500_LDO_INTCORE] = {
.constraints = {
.name = "V-INTCORE",
- .valid_ops_mask = REGULATOR_CHANGE_STATUS,
+ .min_uV = 1250000,
+ .max_uV = 1350000,
+ .input_uV = 1800000,
+ .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE |
+ REGULATOR_CHANGE_STATUS |
+ REGULATOR_CHANGE_MODE |
+ REGULATOR_CHANGE_DRMS,
+ .valid_modes_mask = REGULATOR_MODE_NORMAL |
+ REGULATOR_MODE_IDLE,
},
.num_consumer_supplies = ARRAY_SIZE(ab8500_vintcore_consumers),
.consumer_supplies = ab8500_vintcore_consumers,
--
1.8.1.3
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH RESEND] ARM: ux500: update V-INTCORE constraints
2013-04-02 8:51 ` [PATCH RESEND] " Fabio Baltieri
@ 2013-04-02 9:23 ` Lee Jones
2013-04-02 9:34 ` Fabio Baltieri
0 siblings, 1 reply; 9+ messages in thread
From: Lee Jones @ 2013-04-02 9:23 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, 02 Apr 2013, Fabio Baltieri wrote:
> Update V-INTCORE regulator constraints to allow voltage and
> configuration changes from consumer drivers.
>
> Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
> ---
> arch/arm/mach-ux500/board-mop500-regulators.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
I already have this queued.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH RESEND] ARM: ux500: update V-INTCORE constraints
2013-04-02 9:23 ` Lee Jones
@ 2013-04-02 9:34 ` Fabio Baltieri
0 siblings, 0 replies; 9+ messages in thread
From: Fabio Baltieri @ 2013-04-02 9:34 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Apr 02, 2013 at 10:23:05AM +0100, Lee Jones wrote:
> On Tue, 02 Apr 2013, Fabio Baltieri wrote:
>
> > Update V-INTCORE regulator constraints to allow voltage and
> > configuration changes from consumer drivers.
> >
> > Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org>
> > ---
> > arch/arm/mach-ux500/board-mop500-regulators.c | 10 +++++++++-
> > 1 file changed, 9 insertions(+), 1 deletion(-)
>
> I already have this queued.
Ok, that was part of another yet to be merged series.
Mark, please ignore this one, sorry for the added noise.
Fabio
--
Fabio Baltieri
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: ux500: update V-INTCORE constraints
2013-04-02 8:44 ` Fabio Baltieri
2013-04-02 8:46 ` Mark Brown
@ 2013-04-02 9:24 ` Lee Jones
1 sibling, 0 replies; 9+ messages in thread
From: Lee Jones @ 2013-04-02 9:24 UTC (permalink / raw)
To: linux-arm-kernel
> > > I know that Lee's been sending off a large set of regulator patches
> > > lately, some include alterations to this platform config.
> >
> > > Maybe it's better to take this patch into the regulator tree if these
> > > machine configs will come in that way? Also like Lee to have a
> > > look at this...
> >
> > If it's touching the board constraints it's likely to have conflicts
> > with Lee's stuff, yes.
>
> Lee's patches are on -next
Not all of them are on -next, but all are on the MLs.
I'm currently fixing up some of Mark's review comments in order to
re-submit the remainder.
> and should not conflict with this one
> (tested again on today's tree), but it may be nice anyway to have this
> in the regulator framework with the others.
> Mark, would you consider taking this one in your tree?
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2013-04-02 9:34 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-27 15:21 [PATCH] ARM: ux500: update V-INTCORE constraints Fabio Baltieri
2013-04-02 8:26 ` Linus Walleij
2013-04-02 8:38 ` Mark Brown
2013-04-02 8:44 ` Fabio Baltieri
2013-04-02 8:46 ` Mark Brown
2013-04-02 8:51 ` [PATCH RESEND] " Fabio Baltieri
2013-04-02 9:23 ` Lee Jones
2013-04-02 9:34 ` Fabio Baltieri
2013-04-02 9:24 ` [PATCH] " Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).