On 11/27/2013 02:22 PM, Sekhar Nori wrote: > + Kevin > > On Monday 25 November 2013 11:04 PM, Joel Fernandes wrote: >> On 11/17/2013 04:19 PM, Daniel Mack wrote: >>> This patch makes the edma driver resume correctly after suspend. Tested >>> on an AM33xx platform with cyclic audio streams and omap_hsmmc. >>> >>> All information can be reconstructed by already known runtime >>> information. >>> >>> As we now use some functions that were previously only used from __init >>> context, annotations had to be dropped. >>> >>> [[email protected]: added error handling for runtime + suspend_late/early_resume] >>> Signed-off-by: Nishanth Menon <[email protected]> >>> Signed-off-by: Daniel Mack <[email protected]> >>> Tested-by: Joel Fernandes <[email protected]> >>> Acked-by: Joel Fernandes <[email protected]> >> >> Hi Sekhar, >> >> Can you consider pulling this patch? It has been tested and Acked. Thanks. > > Kevin had some inputs on previous version of this patch. Were you able > to make sure he is okay with this version being merged?
I had concerns about the feedback I got, and haven't got answers yet. In particular, I'm not convinced that using runtime PM to suspend channels would actually save any power during runtime, or have any other benefit. But I might be wrong - maybe someone at TI could comment on that? Thanks, Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
