[linux-pm] [PATCH/RFC] Runtime PM: ARM: subarch-specific extensions of pdev_archdata

[Magnus Damm]

On Sun, Jul 25, 2010 at 5:24 AM, Grant Likely <grant.likely at secretlab.ca> wrote:
> "Magnus Damm" <magnus.damm at gmail.com> wrote:
>>On Tue, Oct 27, 2009 at 8:13 AM, Kevin Hilman
>><khilman at deeprootsystems.com> wrote:
>>> Eric Miao <eric.y.miao at gmail.com> writes:
>>>> On Thu, Sep 24, 2009 at 7:28 AM, Kevin Hilman
>>>> <khilman at deeprootsystems.com> wrote:
>>>>> Mikael Pettersson wrote:
>>>>>> Eric Miao writes:
>>>>>>  > On Wed, Sep 23, 2009 at 9:50 AM, Kevin Hilman
>>>>>>  > <khilman at deeprootsystems.com> wrote:
>>>>>>  > > On ARM platforms, power management can be very platform specific.
>>>>>>  > > This patch allows ARM subarches to extend the platform_device
>>>>>>  > > pdev_archdata for each subarch by creating a new struct pdev_machdata
>>>>>>  > > and allowing each subarch to customize it as needed.

>>Do you remember what happened with this patch?
>
> I don't have all the details in front of me because I'm on my phone, but I advised against pdev_archdata because it is multiplatform-unfriendly.

Ok, I did not expect that. =) But after thinking a bit it does make
sense. I wonder what my options are. I'm not so fond of the idea to
wrap the platform devices - that's not more multi-platform friendly,
is it?

How about using devres and platform bus notifiers?

For a Runtime PM prototype using devres (instead of pdev_archdata or
wrapping) look here:
https://patchwork.kernel.org/patch/113605/

To make it work with modules I propose adding a driver bind event:
https://patchwork.kernel.org/patch/113865/

Looks pretty multi-platform friendly to me. Any suggestions?

Thanks,

/ magnus