On 14 September 2012 08:45, Vinod Koul <vinod.koul@xxxxxxxxxxxxxxx> wrote: > On Fri, 2012-09-07 at 12:14 +0530, Inderpal Singh wrote: >> The controller clock is being managed at AMBA bus level probe/remove and >> pm_runtime/suspend functions. The existing driver does the clock enable/disable >> again in the same code paths, which unneccessarily increments the usage count of >> the clock for the same device. >> >> The following patches remove the redundant clock enable/disable from the driver. > Looks good, any tested by before I apply this.. Kukjin? if it matters, Tested-by: Chander Kashyap <chander.kashyap@xxxxxxxxxx> >> >> Inderpal Singh (2): >> DMA: PL330: Remove controller clock enable/disable >> DMA: PL330: Remove redundant runtime_suspend/resume functions >> >> drivers/dma/pl330.c | 73 ++++----------------------------------------------- >> 1 file changed, 5 insertions(+), 68 deletions(-) >> > > > -- > ~Vinod > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- with warm regards, Chander Kashyap -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html