Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1507832 > unrolled thread
| Started by | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| First post | 2016-10-25 01:50 +0200 |
| Last post | 2016-10-25 05:40 +0200 |
| Articles | 4 — 4 participants |
Back to article view | Back to linux.kernel
linux-next: manual merge of the pm tree with the imx-mxs tree Stephen Rothwell <sfr@canb.auug.org.au> - 2016-10-25 01:50 +0200
Re: linux-next: manual merge of the pm tree with the imx-mxs tree Shawn Guo <shawnguo@kernel.org> - 2016-10-25 02:00 +0200
Re: linux-next: manual merge of the pm tree with the imx-mxs tree "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-10-25 02:00 +0200
Re: linux-next: manual merge of the pm tree with the imx-mxs tree Lina Iyer <lina.iyer@linaro.org> - 2016-10-25 05:40 +0200
| From | Stephen Rothwell <sfr@canb.auug.org.au> |
|---|---|
| Date | 2016-10-25 01:50 +0200 |
| Subject | linux-next: manual merge of the pm tree with the imx-mxs tree |
| Message-ID | <svXa9-61a-1@gated-at.bofh.it> |
Hi Rafael,
Today's linux-next merge of the pm tree got a conflict in:
arch/arm/mach-imx/gpc.c
between commits:
eef0b282bb58 ("ARM: imx: gpc: Initialize all power domains")
f9d1f7a7ad91 ("ARM: imx: gpc: Fix the imx_gpc_genpd_init() error path")
from the imx-mxs tree and commit:
59d65b73a23c ("PM / Domains: Make genpd state allocation dynamic")
from the pm tree.
I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging. You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.
--
Cheers,
Stephen Rothwell
diff --cc arch/arm/mach-imx/gpc.c
index b54db47f6f32,57a410bbb6a2..000000000000
--- a/arch/arm/mach-imx/gpc.c
+++ b/arch/arm/mach-imx/gpc.c
@@@ -430,18 -423,20 +423,28 @@@ static int imx_gpc_genpd_init(struct de
if (!IS_ENABLED(CONFIG_PM_GENERIC_DOMAINS))
return 0;
+ imx6q_pu_domain.base.states = devm_kzalloc(dev,
+ sizeof(*imx6q_pu_domain.base.states),
+ GFP_KERNEL);
+ if (!imx6q_pu_domain.base.states)
+ return -ENOMEM;
+
+ imx6q_pu_domain.base.states[0].power_off_latency_ns = 25000;
+ imx6q_pu_domain.base.states[0].power_on_latency_ns = 2000000;
+ imx6q_pu_domain.base.state_count = 1;
+
- pm_genpd_init(&imx6q_pu_domain.base, NULL, false);
- return of_genpd_add_provider_onecell(dev->of_node,
+ for (i = 0; i < ARRAY_SIZE(imx_gpc_domains); i++)
+ pm_genpd_init(imx_gpc_domains[i], NULL, false);
+
+ ret = of_genpd_add_provider_onecell(dev->of_node,
&imx_gpc_onecell_data);
+ if (ret)
+ goto power_off;
+
+ return 0;
+power_off:
+ imx6q_pm_pu_power_off(&imx6q_pu_domain.base);
clk_err:
while (i--)
clk_put(imx6q_pu_domain.clk[i]);
[toc] | [next] | [standalone]
| From | Shawn Guo <shawnguo@kernel.org> |
|---|---|
| Date | 2016-10-25 02:00 +0200 |
| Message-ID | <svXjP-64L-3@gated-at.bofh.it> |
| In reply to | #1507832 |
On Tue, Oct 25, 2016 at 01:58:25AM +0200, Rafael J. Wysocki wrote:
> On Tuesday, October 25, 2016 10:47:29 AM Stephen Rothwell wrote:
> > Hi Rafael,
> >
> > Today's linux-next merge of the pm tree got a conflict in:
> >
> > arch/arm/mach-imx/gpc.c
> >
> > between commits:
> >
> > eef0b282bb58 ("ARM: imx: gpc: Initialize all power domains")
> > f9d1f7a7ad91 ("ARM: imx: gpc: Fix the imx_gpc_genpd_init() error path")
> >
> > from the imx-mxs tree and commit:
> >
> > 59d65b73a23c ("PM / Domains: Make genpd state allocation dynamic")
> >
> > from the pm tree.
> >
> > I fixed it up (see below) and can carry the fix as necessary.
>
> Thanks Stephen!
>
> Lina, please have a look at the Stephen's fix and let me know if that
> conflict should be resolved in a different way.
FYI. We target the following two fixes 4.9-rc.
eef0b282bb58 ("ARM: imx: gpc: Initialize all power domains")
f9d1f7a7ad91 ("ARM: imx: gpc: Fix the imx_gpc_genpd_init() error path")
Shawn
[toc] | [prev] | [next] | [standalone]
| From | "Rafael J. Wysocki" <rjw@rjwysocki.net> |
|---|---|
| Date | 2016-10-25 02:00 +0200 |
| Message-ID | <svXjP-64L-5@gated-at.bofh.it> |
| In reply to | #1507832 |
On Tuesday, October 25, 2016 10:47:29 AM Stephen Rothwell wrote:
> Hi Rafael,
>
> Today's linux-next merge of the pm tree got a conflict in:
>
> arch/arm/mach-imx/gpc.c
>
> between commits:
>
> eef0b282bb58 ("ARM: imx: gpc: Initialize all power domains")
> f9d1f7a7ad91 ("ARM: imx: gpc: Fix the imx_gpc_genpd_init() error path")
>
> from the imx-mxs tree and commit:
>
> 59d65b73a23c ("PM / Domains: Make genpd state allocation dynamic")
>
> from the pm tree.
>
> I fixed it up (see below) and can carry the fix as necessary.
Thanks Stephen!
Lina, please have a look at the Stephen's fix and let me know if that
conflict should be resolved in a different way.
Thanks,
Rafael
[toc] | [prev] | [next] | [standalone]
| From | Lina Iyer <lina.iyer@linaro.org> |
|---|---|
| Date | 2016-10-25 05:40 +0200 |
| Message-ID | <sw0KJ-8uY-1@gated-at.bofh.it> |
| In reply to | #1507836 |
On Tue, Oct 25 2016 at 17:51 -0600, Rafael J. Wysocki wrote:
>On Tuesday, October 25, 2016 10:47:29 AM Stephen Rothwell wrote:
>> Hi Rafael,
>>
>> Today's linux-next merge of the pm tree got a conflict in:
>>
>> arch/arm/mach-imx/gpc.c
>>
>> between commits:
>>
>> eef0b282bb58 ("ARM: imx: gpc: Initialize all power domains")
>> f9d1f7a7ad91 ("ARM: imx: gpc: Fix the imx_gpc_genpd_init() error path")
>>
>> from the imx-mxs tree and commit:
>>
>> 59d65b73a23c ("PM / Domains: Make genpd state allocation dynamic")
>>
>> from the pm tree.
>>
>> I fixed it up (see below) and can carry the fix as necessary.
>
>Thanks Stephen!
>
>Lina, please have a look at the Stephen's fix and let me know if that
>conflict should be resolved in a different way.
>
Hi Rafael,
Stephen's conflict resolution seems correct. Thanks Stephen.
-- Lina
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web