[PATCH 5/7] at91 : fix dirty hack for the selfrefresh function

Russell King - ARM Linux linux at arm.linux.org.uk
Fri Jan 13 05:22:16 EST 2012


On Fri, Jan 13, 2012 at 10:29:49AM +0100, Daniel Lezcano wrote:
> I was asking me the same question. I am wondering if that makes sense to
> create an arch/arm/include/asm/pm.h file where we move from the system.h
> file these two functions:
> 
> extern void (*arm_pm_restart)(char str, const char *cmd);

Ideally, I want this one to go away - which it can do once its usage in
the AT91 code has been fixed up.  I don't know how to do that because
I don't know which machines/platforms end up with which SoCs.

The idea of exporting arm_pm_standby() is one I thought about, but I'm
not sure if it makes sense for all SoCs to call their standby function
from the CPU idle code.

I'm coming to the conclusion that moving the AT91 CPU idle driver out
of arch/arm/mach-at91 is the wrong thing to do - it seems that the CPU
idle driver is intimately linked to core AT91 code, and moving it out
just makes those linkages a lot more complicated to handle.  And for
what benefit - just to have this located under the drivers/ subtree
because it's now seen to be a driver ?

I think this is one of the times where the 'solution' is worse than the
problem.



More information about the linux-arm-kernel mailing list