On 12/10/17 18:12, Vinod Koul wrote:
> On Mon, Oct 09, 2017 at 03:40:27PM +0100, Ed Blake wrote:
>> +static int img_mdc_runtime_resume(struct device *dev)
>> +{
>> + struct mdc_dma *mdma = dev_get_drvdata(dev);
>> +
>> + return clk_prepare_enable(mdma->clk);
>> +}
> This one is same as...
>
> On Mon, Oct 09, 2017 at 03:40:26PM +0100, Ed Blake wrote:
>> +static int img_mdc_resume_early(struct device *dev)
>> +{
>> + struct mdc_dma *mdma = dev_get_drvdata(dev);
>> +
>> + return clk_prepare_enable(mdma->clk);
>> +}
> ... this one.
>
> why not use a same fn for both...
That is the img_mdc_resume_early() of patch 1/2. It changes in patch
2/2 so is no longer the same:
On 09/10/17 15:40, Ed Blake wrote:
> static int img_mdc_resume_early(struct device *dev)
> {
> - struct mdc_dma *mdma = dev_get_drvdata(dev);
> -
> - return clk_prepare_enable(mdma->clk);
> + return pm_runtime_force_resume(dev);
> }
---
Ed.
--
To unsubscribe from this list: send the line "unsubscribe dmaengine" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
