Re: [PATCH 1/3] amba: add no_pm_pclk_management flag to amba_driver

[Robert Baldyga]

On 04/16/2015 03:41 PM, Russell King - ARM Linux wrote:
> On Thu, Apr 16, 2015 at 03:39:12PM +0200, Robert Baldyga wrote:
>> This patch introduces no_pm_pclk_management flag in struct amba_driver.
>> It causes that pclk is not touched in pm_runtime callbacks. This allows
>> to manage clock in device driver independently of power domain management.
> 
> NAK.  We already have stuff in place to deal with the problem with
> DMA engine using runtime PM, which causes pclk management to fall
> back to the IRQ-safe clk_enable/clk_disable().
> 

What kind of stuff do you mean? If we want to manage pclk independently
form runtime PM, we need to prevent AMBA driver from touching pclk.
Otherwise our clock management inside of dmaengine driver is ineffective
because pclk refcount is incremented by AMBA driver.

Thanks,
Robert Baldyga
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/