On Tue, Aug 20, 2013 at 06:35:53PM +0200, Sebastian Andrzej Siewior wrote:
> With enabled pm_runtime in the kernel the device won't work because it
> is not "on" during the probe function. This patch enables the device via
> pm_runtime on probe so it remains activated.
> 
> Cc: Vinod Koul <vinod.k...@intel.com>
> Signed-off-by: Sebastian Andrzej Siewior <bige...@linutronix.de>
Acked-by Vinod Koul <vinod.k...@intel.com>

~Vinod
> ---
>  drivers/dma/cppi41.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/dma/cppi41.c b/drivers/dma/cppi41.c
> index e696178..6c4e64f 100644
> --- a/drivers/dma/cppi41.c
> +++ b/drivers/dma/cppi41.c
> @@ -9,6 +9,7 @@
>  #include <linux/dmapool.h>
>  #include <linux/interrupt.h>
>  #include <linux/of_address.h>
> +#include <linux/pm_runtime.h>
>  #include "dmaengine.h"
>  
>  #define DESC_TYPE    27
> @@ -960,6 +961,11 @@ static int cppi41_dma_probe(struct platform_device *pdev)
>               goto err_remap;
>       }
>  
> +     pm_runtime_enable(&pdev->dev);
> +     ret = pm_runtime_get_sync(&pdev->dev);
> +     if (ret)
> +             goto err_get_sync;
> +
>       cdd->queues_rx = glue_info->queues_rx;
>       cdd->queues_tx = glue_info->queues_tx;
>       cdd->td_queue = glue_info->td_queue;
> @@ -1005,6 +1011,9 @@ static int cppi41_dma_probe(struct platform_device 
> *pdev)
>  err_chans:
>       deinit_cpii41(pdev, cdd);
>  err_init_cppi:
> +     pm_runtime_put(&pdev->dev);
> +err_get_sync:
> +     pm_runtime_disable(&pdev->dev);
>       iounmap(cdd->usbss_mem);
>       iounmap(cdd->ctrl_mem);
>       iounmap(cdd->sched_mem);
> @@ -1029,6 +1038,8 @@ static int cppi41_dma_remove(struct platform_device 
> *pdev)
>       iounmap(cdd->ctrl_mem);
>       iounmap(cdd->sched_mem);
>       iounmap(cdd->qmgr_mem);
> +     pm_runtime_put(&pdev->dev);
> +     pm_runtime_disable(&pdev->dev);
>       kfree(cdd);
>       return 0;
>  }
> -- 
> 1.8.4.rc2
> 

-- 
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to