[PATCH v2 00/11] ARM: shmobile: r8a7740/armadillo800eva legacy PM domain support

[Geert Uytterhoeven]

Hi Simon, Magnus,

On Tue, Sep 30, 2014 at 10:13 AM, Simon Horman <horms at verge.net.au> wrote:
> On Tue, Sep 30, 2014 at 09:41:27AM +0200, Geert Uytterhoeven wrote:
>> On Tue, Sep 30, 2014 at 9:32 AM, Magnus Damm <magnus.damm at gmail.com> wrote:
>> >>> So what is a good way to proceed with this series I wonder?
>> >>
>> >> There are two options:
>> >>   1. Drop the two patches that introduce the D4 and A4MP PM domains,
>> >>   2. Amend the patches that introduce the D4 and A4MP PM domains to
>> >>      comment out the entries in the table in pm-r8a7740.c, like was done in
>> >>      the first submission.
>> >>
>> >> Both will cause conflicts when applying the successive patches.
>> >>
>> >> If you don't care about bisectability, you can implement option 2 as a
>> >> separate change, after applying the whole series.
>> >
>> > Thanks for leading the way! From my side I don't have any special
>> > feeling, so why don't you and Simon decide?
>>
>> OK.
>>
>> > If it was up to me only then I think option #1 that simply reduces the
>> > feature set seems rather natural, but perhaps it requires a lot of
>> > rework of the patches. If we go with option #1 we can always
>> > incrementally add D4 and A4MP later I assume.
>>
>> Yes we can.
>>
>> Please wait a bit, I'm looking into the !CONFIG_PM_RUNTIME case.
>> Mostly multi-platform, but it may affect legacy too, where there's interaction
>> with the drivers/sh/pm_runtime.c hack.

I posted a fix for the !CONFIG_PM_RUNTIME problem
"[PATCH] PM / clock_ops: Fix crash in clocks management code if
!CONFIG_PM_RUNTIME"
(https://lkml.org/lkml/2014/10/1/506). I hope to see that fix in v3.18-rc1.

> I also feel that option 1 is slightly nicer. But I'm happy
> to consider option 2 if its a better fit for you.
>
> As always I'm happy to wait and will now do so.

The fix for the HDMI-related interrupt storm, also affecting A4MP, is in
fbdev's next, so I expect to see that in v3.18-rc1.

For D4, a final solution seems to be a long-term plan...
I'll disable powering down the D4 power area, and will repost the patches
after v3.18-rc1, when the other prerequisites are in.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds