+ linux-arm-kernel Len Brown <l...@kernel.org> writes:
> From: Len Brown <len.br...@intel.com> > > (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 <len.br...@intel.com> > Cc: x...@kernel.org # x86 > Cc: Mike Frysinger <vap...@gentoo.org> # blackfin > Cc: Jonas Bonn <jo...@southpole.se> # openrisc > Cc: Tony Luck <tony.l...@intel.com> # ia64 > Cc: David Howells <dhowe...@redhat.com> # mn10300 > Cc: Mikael Starvik <star...@axis.com> # cris > Cc: Michal Simek <mon...@monstr.eu> # microblaze > Cc: Paul Mundt <let...@linux-sh.org> # sh > Cc: David S. Miller <da...@davemloft.net> # sparc > Cc: Russell King <li...@arm.linux.org.uk> # ARM > Cc: Rafael J. Wysocki <r...@sisk.pl> # 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 <khil...@ti.com> Tested-by: Kevin Hilman <khil...@ti.com> Kevin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
