* linux-next: build failure after merge of the regulator tree
@ 2015-04-09 4:33 Stephen Rothwell
2015-04-09 9:35 ` Mark Brown
2015-04-09 15:00 ` Rob Clark
0 siblings, 2 replies; 9+ messages in thread
From: Stephen Rothwell @ 2015-04-09 4:33 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: linux-next, linux-kernel, Hai Li, Rob Clark, Dave Airlie,
Bjorn Andersson
[-- Attachment #1: Type: text/plain, Size: 2517 bytes --]
Hi all,
After merging the regulator tree, today's linux-next build (arm multi_v7_defconfig)
failed like this:
drivers/gpu/drm/msm/dsi/dsi_host.c: In function 'dsi_host_regulator_disable':
drivers/gpu/drm/msm/dsi/dsi_host.c:330:4: error: implicit declaration of function 'regulator_set_optimum_mode' [-Werror=implicit-function-declaration]
regulator_set_optimum_mode(s[i].consumer,
^
Caused by commit a689554ba6ed ("drm/msm: Initial add DSI connector
support") from the drm tree interacting with commit ae6e808f1574
("regulator: Drop temporary regulator_set_optimum_mode wrapper") from
the regulator tree.
This happens every time someone renames/changes an API :-(
I have applied the following merge fix patch. Someone needs to make
sure Linus finds out when the latter of these two trees is merged into
his.
From: Stephen Rothwell <sfr@canb•auug.org.au>
Date: Thu, 9 Apr 2015 14:28:42 +1000
Subject: [PATCH] drm/msm: fixes for regulator_set_optimum_mode name change
Signed-off-by: Stephen Rothwell <sfr@canb•auug.org.au>
---
drivers/gpu/drm/msm/dsi/dsi_host.c | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
index fdc54e3eff55..962cf545d24c 100644
--- a/drivers/gpu/drm/msm/dsi/dsi_host.c
+++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
@@ -327,8 +327,7 @@ static void dsi_host_regulator_disable(struct msm_dsi_host *msm_host)
DBG("");
for (i = num - 1; i >= 0; i--)
if (regs[i].disable_load >= 0)
- regulator_set_optimum_mode(s[i].consumer,
- regs[i].disable_load);
+ regulator_set_load(s[i].consumer, regs[i].disable_load);
regulator_bulk_disable(num, s);
}
@@ -343,8 +342,8 @@ static int dsi_host_regulator_enable(struct msm_dsi_host *msm_host)
DBG("");
for (i = 0; i < num; i++) {
if (regs[i].enable_load >= 0) {
- ret = regulator_set_optimum_mode(s[i].consumer,
- regs[i].enable_load);
+ ret = regulator_set_load(s[i].consumer,
+ regs[i].enable_load);
if (ret < 0) {
pr_err("regulator %d set op mode failed, %d\n",
i, ret);
@@ -363,7 +362,7 @@ static int dsi_host_regulator_enable(struct msm_dsi_host *msm_host)
fail:
for (i--; i >= 0; i--)
- regulator_set_optimum_mode(s[i].consumer, regs[i].disable_load);
+ regulator_set_load(s[i].consumer, regs[i].disable_load);
return ret;
}
--
2.1.4
--
Cheers,
Stephen Rothwell sfr@canb•auug.org.au
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: linux-next: build failure after merge of the regulator tree
2015-04-09 4:33 linux-next: build failure after merge of the regulator tree Stephen Rothwell
@ 2015-04-09 9:35 ` Mark Brown
2015-04-13 23:44 ` Stephen Rothwell
2015-04-09 15:00 ` Rob Clark
1 sibling, 1 reply; 9+ messages in thread
From: Mark Brown @ 2015-04-09 9:35 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Liam Girdwood, linux-next, linux-kernel, Hai Li, Rob Clark,
Dave Airlie, Bjorn Andersson
[-- Attachment #1: Type: text/plain, Size: 1032 bytes --]
On Thu, Apr 09, 2015 at 02:33:09PM +1000, Stephen Rothwell wrote:
> After merging the regulator tree, today's linux-next build (arm multi_v7_defconfig)
> failed like this:
> drivers/gpu/drm/msm/dsi/dsi_host.c: In function 'dsi_host_regulator_disable':
> drivers/gpu/drm/msm/dsi/dsi_host.c:330:4: error: implicit declaration of function 'regulator_set_optimum_mode' [-Werror=implicit-function-declaration]
> regulator_set_optimum_mode(s[i].consumer,
> ^
> Caused by commit a689554ba6ed ("drm/msm: Initial add DSI connector
> support") from the drm tree interacting with commit ae6e808f1574
> ("regulator: Drop temporary regulator_set_optimum_mode wrapper") from
> the regulator tree.
> This happens every time someone renames/changes an API :-(
It's *really* questionable if anyone should be using that API in the
first place, one of the reasons it's being removed is that it's really
unclear that anything would be able to successfully use it as-is. The
easist thing might be to drop the new usage for the time being.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: linux-next: build failure after merge of the regulator tree
2015-04-09 9:35 ` Mark Brown
@ 2015-04-13 23:44 ` Stephen Rothwell
2015-04-14 1:07 ` Bjorn Andersson
0 siblings, 1 reply; 9+ messages in thread
From: Stephen Rothwell @ 2015-04-13 23:44 UTC (permalink / raw)
To: Mark Brown
Cc: Liam Girdwood, linux-next, linux-kernel, Hai Li, Rob Clark,
Dave Airlie, Bjorn Andersson
[-- Attachment #1: Type: text/plain, Size: 1471 bytes --]
Hi all,
On Thu, 9 Apr 2015 10:35:56 +0100 Mark Brown <broonie@kernel•org> wrote:
>
> On Thu, Apr 09, 2015 at 02:33:09PM +1000, Stephen Rothwell wrote:
>
> > After merging the regulator tree, today's linux-next build (arm multi_v7_defconfig)
> > failed like this:
>
> > drivers/gpu/drm/msm/dsi/dsi_host.c: In function 'dsi_host_regulator_disable':
> > drivers/gpu/drm/msm/dsi/dsi_host.c:330:4: error: implicit declaration of function 'regulator_set_optimum_mode' [-Werror=implicit-function-declaration]
> > regulator_set_optimum_mode(s[i].consumer,
> > ^
>
> > Caused by commit a689554ba6ed ("drm/msm: Initial add DSI connector
> > support") from the drm tree interacting with commit ae6e808f1574
> > ("regulator: Drop temporary regulator_set_optimum_mode wrapper") from
> > the regulator tree.
>
> > This happens every time someone renames/changes an API :-(
>
> It's *really* questionable if anyone should be using that API in the
> first place, one of the reasons it's being removed is that it's really
> unclear that anything would be able to successfully use it as-is. The
> easist thing might be to drop the new usage for the time being.
This fix up patch is now needed when the drm tree is merged with Linus'
tree (the regulator part has been merged).
Mark, what did you above response mean? Should the drm tree be using
something different?
--
Cheers,
Stephen Rothwell sfr@canb•auug.org.au
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: linux-next: build failure after merge of the regulator tree
2015-04-13 23:44 ` Stephen Rothwell
@ 2015-04-14 1:07 ` Bjorn Andersson
2015-04-14 1:22 ` Stephen Rothwell
0 siblings, 1 reply; 9+ messages in thread
From: Bjorn Andersson @ 2015-04-14 1:07 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, linux-next@vger•kernel.org,
linux-kernel@vger•kernel.org, Hai Li, Rob Clark, Dave Airlie
On Mon 13 Apr 16:44 PDT 2015, Stephen Rothwell wrote:
> Hi all,
>
> On Thu, 9 Apr 2015 10:35:56 +0100 Mark Brown <broonie@kernel•org> wrote:
> >
> > On Thu, Apr 09, 2015 at 02:33:09PM +1000, Stephen Rothwell wrote:
> >
> > > After merging the regulator tree, today's linux-next build (arm multi_v7_defconfig)
> > > failed like this:
> >
> > > drivers/gpu/drm/msm/dsi/dsi_host.c: In function 'dsi_host_regulator_disable':
> > > drivers/gpu/drm/msm/dsi/dsi_host.c:330:4: error: implicit declaration of function 'regulator_set_optimum_mode' [-Werror=implicit-function-declaration]
> > > regulator_set_optimum_mode(s[i].consumer,
> > > ^
> >
> > > Caused by commit a689554ba6ed ("drm/msm: Initial add DSI connector
> > > support") from the drm tree interacting with commit ae6e808f1574
> > > ("regulator: Drop temporary regulator_set_optimum_mode wrapper") from
> > > the regulator tree.
> >
> > > This happens every time someone renames/changes an API :-(
> >
> > It's *really* questionable if anyone should be using that API in the
> > first place, one of the reasons it's being removed is that it's really
> > unclear that anything would be able to successfully use it as-is. The
> > easist thing might be to drop the new usage for the time being.
>
> This fix up patch is now needed when the drm tree is merged with Linus'
> tree (the regulator part has been merged).
>
> Mark, what did you above response mean? Should the drm tree be using
> something different?
>
What Mark mean is basically that there shouldn't be any users of
regulator_set_optimum_mode() for various reasons and we introduced
regulator_set_load() to solve these, before any users popped up.
Unfortunately we weren't fast enough and people got some code in that
uses the API that isn't implemented for any Qualcomm platform today...
Your patch looks correct and should preferrably be added to the drm
tree, or the last patch in my series that drops the API wrapper should
be held back until rc1(?)
Regards,
Bjorn
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: linux-next: build failure after merge of the regulator tree
2015-04-14 1:07 ` Bjorn Andersson
@ 2015-04-14 1:22 ` Stephen Rothwell
2015-04-14 9:40 ` Mark Brown
0 siblings, 1 reply; 9+ messages in thread
From: Stephen Rothwell @ 2015-04-14 1:22 UTC (permalink / raw)
To: Bjorn Andersson
Cc: Mark Brown, Liam Girdwood, linux-next@vger•kernel.org,
linux-kernel@vger•kernel.org, Hai Li, Rob Clark, Dave Airlie
[-- Attachment #1: Type: text/plain, Size: 802 bytes --]
Hi Bjorn,
On Mon, 13 Apr 2015 18:07:06 -0700 Bjorn Andersson <bjorn.andersson@sonymobile•com> wrote:
>
> What Mark mean is basically that there shouldn't be any users of
> regulator_set_optimum_mode() for various reasons and we introduced
> regulator_set_load() to solve these, before any users popped up.
>
> Unfortunately we weren't fast enough and people got some code in that
> uses the API that isn't implemented for any Qualcomm platform today...
Ah, OK.
> Your patch looks correct and should preferrably be added to the drm
> tree, or the last patch in my series that drops the API wrapper should
> be held back until rc1(?)
It needs to be sent to Linus as a merge fix when the drm tree is merged.
--
Cheers,
Stephen Rothwell sfr@canb•auug.org.au
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: linux-next: build failure after merge of the regulator tree
2015-04-14 1:22 ` Stephen Rothwell
@ 2015-04-14 9:40 ` Mark Brown
2015-04-15 3:03 ` Dave Airlie
0 siblings, 1 reply; 9+ messages in thread
From: Mark Brown @ 2015-04-14 9:40 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Bjorn Andersson, Liam Girdwood, linux-next@vger•kernel.org,
linux-kernel@vger•kernel.org, Hai Li, Rob Clark, Dave Airlie
[-- Attachment #1: Type: text/plain, Size: 529 bytes --]
On Tue, Apr 14, 2015 at 11:22:41AM +1000, Stephen Rothwell wrote:
> On Mon, 13 Apr 2015 18:07:06 -0700 Bjorn Andersson <bjorn.andersson@sonymobile•com> wrote:
> > Your patch looks correct and should preferrably be added to the drm
> > tree, or the last patch in my series that drops the API wrapper should
> > be held back until rc1(?)
> It needs to be sent to Linus as a merge fix when the drm tree is merged.
Or the DRM tree could pull my tree I guess - Rob/David, I can make a tag
specifically for this branch if you like?
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: linux-next: build failure after merge of the regulator tree
2015-04-14 9:40 ` Mark Brown
@ 2015-04-15 3:03 ` Dave Airlie
0 siblings, 0 replies; 9+ messages in thread
From: Dave Airlie @ 2015-04-15 3:03 UTC (permalink / raw)
To: Mark Brown
Cc: Stephen Rothwell, Bjorn Andersson, Liam Girdwood,
linux-next@vger•kernel.org, linux-kernel@vger•kernel.org, Hai Li,
Rob Clark, Dave Airlie
On 14 April 2015 at 19:40, Mark Brown <broonie@kernel•org> wrote:
> On Tue, Apr 14, 2015 at 11:22:41AM +1000, Stephen Rothwell wrote:
>> On Mon, 13 Apr 2015 18:07:06 -0700 Bjorn Andersson <bjorn.andersson@sonymobile•com> wrote:
>
>> > Your patch looks correct and should preferrably be added to the drm
>> > tree, or the last patch in my series that drops the API wrapper should
>> > be held back until rc1(?)
>
>> It needs to be sent to Linus as a merge fix when the drm tree is merged.
>
> Or the DRM tree could pull my tree I guess - Rob/David, I can make a tag
> specifically for this branch if you like?
I can just backmerge Linus's tree before I send it to him, and stick
this patch on top.
seems like the best answer.
Dave.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: linux-next: build failure after merge of the regulator tree
2015-04-09 4:33 linux-next: build failure after merge of the regulator tree Stephen Rothwell
2015-04-09 9:35 ` Mark Brown
@ 2015-04-09 15:00 ` Rob Clark
2015-04-09 23:49 ` drm-msm tree (Was: Re: linux-next: build failure after merge of the regulator tree) Stephen Rothwell
1 sibling, 1 reply; 9+ messages in thread
From: Rob Clark @ 2015-04-09 15:00 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Mark Brown, Liam Girdwood, linux-next, Linux Kernel Mailing List,
Hai Li, Dave Airlie, Bjorn Andersson
On Thu, Apr 9, 2015 at 12:33 AM, Stephen Rothwell <sfr@canb•auug.org.au> wrote:
> Hi all,
>
> After merging the regulator tree, today's linux-next build (arm multi_v7_defconfig)
> failed like this:
>
> drivers/gpu/drm/msm/dsi/dsi_host.c: In function 'dsi_host_regulator_disable':
> drivers/gpu/drm/msm/dsi/dsi_host.c:330:4: error: implicit declaration of function 'regulator_set_optimum_mode' [-Werror=implicit-function-declaration]
> regulator_set_optimum_mode(s[i].consumer,
> ^
>
> Caused by commit a689554ba6ed ("drm/msm: Initial add DSI connector
> support") from the drm tree interacting with commit ae6e808f1574
> ("regulator: Drop temporary regulator_set_optimum_mode wrapper") from
> the regulator tree.
Thanks.. btw, would it be possible to get my msm-next branch picked up
by linux-next? That seems like it would help to catch these earlier..
git://people.freedesktop.org/~robclark/linux msm-next
BR,
-R
> This happens every time someone renames/changes an API :-(
>
> I have applied the following merge fix patch. Someone needs to make
> sure Linus finds out when the latter of these two trees is merged into
> his.
>
> From: Stephen Rothwell <sfr@canb•auug.org.au>
> Date: Thu, 9 Apr 2015 14:28:42 +1000
> Subject: [PATCH] drm/msm: fixes for regulator_set_optimum_mode name change
>
> Signed-off-by: Stephen Rothwell <sfr@canb•auug.org.au>
> ---
> drivers/gpu/drm/msm/dsi/dsi_host.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c
> index fdc54e3eff55..962cf545d24c 100644
> --- a/drivers/gpu/drm/msm/dsi/dsi_host.c
> +++ b/drivers/gpu/drm/msm/dsi/dsi_host.c
> @@ -327,8 +327,7 @@ static void dsi_host_regulator_disable(struct msm_dsi_host *msm_host)
> DBG("");
> for (i = num - 1; i >= 0; i--)
> if (regs[i].disable_load >= 0)
> - regulator_set_optimum_mode(s[i].consumer,
> - regs[i].disable_load);
> + regulator_set_load(s[i].consumer, regs[i].disable_load);
>
> regulator_bulk_disable(num, s);
> }
> @@ -343,8 +342,8 @@ static int dsi_host_regulator_enable(struct msm_dsi_host *msm_host)
> DBG("");
> for (i = 0; i < num; i++) {
> if (regs[i].enable_load >= 0) {
> - ret = regulator_set_optimum_mode(s[i].consumer,
> - regs[i].enable_load);
> + ret = regulator_set_load(s[i].consumer,
> + regs[i].enable_load);
> if (ret < 0) {
> pr_err("regulator %d set op mode failed, %d\n",
> i, ret);
> @@ -363,7 +362,7 @@ static int dsi_host_regulator_enable(struct msm_dsi_host *msm_host)
>
> fail:
> for (i--; i >= 0; i--)
> - regulator_set_optimum_mode(s[i].consumer, regs[i].disable_load);
> + regulator_set_load(s[i].consumer, regs[i].disable_load);
> return ret;
> }
>
> --
> 2.1.4
>
> --
> Cheers,
> Stephen Rothwell sfr@canb•auug.org.au
^ permalink raw reply [flat|nested] 9+ messages in thread* drm-msm tree (Was: Re: linux-next: build failure after merge of the regulator tree)
2015-04-09 15:00 ` Rob Clark
@ 2015-04-09 23:49 ` Stephen Rothwell
0 siblings, 0 replies; 9+ messages in thread
From: Stephen Rothwell @ 2015-04-09 23:49 UTC (permalink / raw)
To: Rob Clark
Cc: Mark Brown, Liam Girdwood, linux-next, Linux Kernel Mailing List,
Hai Li, Dave Airlie, Bjorn Andersson
[-- Attachment #1: Type: text/plain, Size: 1183 bytes --]
Hi Rob,
On Thu, 9 Apr 2015 11:00:44 -0400 Rob Clark <robdclark@gmail•com> wrote:
>
> Thanks.. btw, would it be possible to get my msm-next branch picked up
> by linux-next? That seems like it would help to catch these earlier..
>
> git://people.freedesktop.org/~robclark/linux msm-next
Added from today.
Thanks for adding your subsystem tree as a participant of linux-next. As
you may know, this is not a judgment of your code. The purpose of
linux-next is for integration testing and to lower the impact of
conflicts between subsystems in the next merge window.
You will need to ensure that the patches/commits in your tree/series have
been:
* submitted under GPL v2 (or later) and include the Contributor's
Signed-off-by,
* posted to the relevant mailing list,
* reviewed by you (or another maintainer of your subsystem tree),
* successfully unit tested, and
* destined for the current or next Linux merge window.
Basically, this should be just what you would send to Linus (or ask him
to fetch). It is allowed to be rebased if you deem it necessary.
--
Cheers,
Stephen Rothwell
sfr@canb•auug.org.au
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-04-15 3:03 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-09 4:33 linux-next: build failure after merge of the regulator tree Stephen Rothwell
2015-04-09 9:35 ` Mark Brown
2015-04-13 23:44 ` Stephen Rothwell
2015-04-14 1:07 ` Bjorn Andersson
2015-04-14 1:22 ` Stephen Rothwell
2015-04-14 9:40 ` Mark Brown
2015-04-15 3:03 ` Dave Airlie
2015-04-09 15:00 ` Rob Clark
2015-04-09 23:49 ` drm-msm tree (Was: Re: linux-next: build failure after merge of the regulator tree) Stephen Rothwell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox