+ linux-arm-kernel Len Brown <[email protected]> writes:
> From: Len Brown <[email protected]> > > (pm_idle)() was originally used on x86 to vector bewteen > ACPI and APM. With the advent of CPU_IDLE, that reason > for pm_idle to exist vanished. > > But x86 APM still scribbled on pm_idle from a module, > so pm_idle didn't go away. Worse, it was declared in pm.h, > and so it spread to other architectures as dead code. > > But today, APM no longer scribbles on pm_idle, so > x86 no longer requires pm_idle to be visible to modules, > or global at all. > > Here we remove pm_idle from pm.h, to stop the mandate > that all architectures define it. > > This deletes dead code from most architectures, > while some continue using their own internal pm_idle. > > At the end of the day, pm_idle... > becomes static in sh, was global > becomes static in x86, was EXPORT_SYMBOL > becomes as global in sparc, was EXPORT_SYMBOL > continues as static in m32r (no pm.h use there) > and is deleted from all other places in the kernel. > > Signed-off-by: Len Brown <[email protected]> > Cc: [email protected] # x86 > Cc: Mike Frysinger <[email protected]> # blackfin > Cc: Jonas Bonn <[email protected]> # openrisc > Cc: Tony Luck <[email protected]> # ia64 > Cc: David Howells <[email protected]> # mn10300 > Cc: Mikael Starvik <[email protected]> # cris > Cc: Michal Simek <[email protected]> # microblaze > Cc: Paul Mundt <[email protected]> # sh > Cc: David S. Miller <[email protected]> # sparc > Cc: Russell King <[email protected]> # ARM > Cc: Rafael J. Wysocki <[email protected]> # PM > --- > arch/arm/kernel/process.c | 12 +++--------- Just looking at the ARM changes, it looks good to me. I tested it with and without CPUidle on my ARM-based TI/OMAP platforms (3430/n900, 4430/Panda) So for the arch/arm changes: Reviewed-by: Kevin Hilman <[email protected]> Tested-by: Kevin Hilman <[email protected]> Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
