On Thursday, April 07, 2011, Kevin Hilman wrote: > This series aims to consolidate OMAP and SH-mobile runtime PM > implementations around the new device power domains. > > In 2.6.39, device power domains were added (commit > 7538e3db6e015e890825fbd9f8659952896ddd5b, PM: add support for device > power domains). In converting both OMAP and SH-mobile to use device > power domains, the overlap between implementations was almost 100%. > > To share the runtime PM implementation based on simple clock gating, > move it to arch/arm/common and initialize it from OMAP and SH-mobile. > > Also, OMAP was the only user of platform_bus_set_pm_ops(). Now that > it has been converted to device power domains, remove > platform_bus_set_pm_ops(). Please, don't do it this way. First, we'll still need platform_bus_set_pm_ops() on shmobile and the reason is that we want to override the platform bus type PM ops for _all_ devices on that platform, which power domains are not really suitable for. Second, we're going to introduce code for handling real power domains for shmobile that would conflict with the way you're using power domains for overriding the default PM ops. Besides, the way you've used power domains appears to assume that drivers will not define their own runtime PM callbacks, because if they do, those callbacks will be called _in_ _addition_ to the power domain callbacks you're trying to add (from the default platform bus type callbacks). Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
