On Tue, 2011-06-21 at 07:49 -0700, Kevin Hilman wrote: > Tomi Valkeinen <tomi.valkei...@ti.com> writes: > > > Use PM runtime and HWMOD support to handle enabling and disabling of DSS > > modules. > > > > Each DSS module will have get and put functions which can be used to > > enable and disable that module. The functions use pm_runtime and hwmod > > opt-clocks to enable the hardware. > > > > Signed-off-by: Tomi Valkeinen <tomi.valkei...@ti.com> > > Runtime PM usage is much better on this version, thanks! > > Like Paul, I'm not crazy about the helper functions, especially the ones > that don't do much other than call runtime PM with some debug code > around them since runtime PM already has useful debug. > > However, you're the maintainer and the one who has to debug this code > more than I do, so that decision is up to you.
I have to say I'm not familiar with runtime PM's debug (yet, I need to check it out), but if I'm debugging DSS, I'm not interested in runtime PM's debug concerning some other drivers or any verbose details about runtime PM. So by having the wrappers I can just observe DSS debug, and see when DSS enables/disables runtime PM etc. If runtime PM's debug allows me to easily only see debugs about DSS, then I agree that the wrappers are extra. Tomi -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
