[PATCH] clk: ti: dra7-atl-clock: Mark the device as pm_runtime_irq_safe

[PATCH] clk: ti: dra7-atl-clock: Mark the device as pm_runtime_irq_safe

[Peter Ujfalusi]

It is safe to call the pm sync calls in interrupt context in this driver.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
 drivers/clk/ti/clk-dra7-atl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c
index af29359677da..f18d149acd65 100644
--- a/drivers/clk/ti/clk-dra7-atl.c
+++ b/drivers/clk/ti/clk-dra7-atl.c
@@ -228,6 +228,7 @@ static int of_dra7_atl_clk_probe(struct platform_device *pdev)
 	cinfo->iobase = of_iomap(node, 0);
 	cinfo->dev = &pdev->dev;
 	pm_runtime_enable(cinfo->dev);
+	pm_runtime_irq_safe(cinfo->dev);
 
 	pm_runtime_get_sync(cinfo->dev);
 	atl_write(cinfo, DRA7_ATL_PCLKMUX_REG(0), DRA7_ATL_PCLKMUX);
-- 
2.1.0

[PATCH] clk: ti: dra7-atl-clock: Mark the device as pm_runtime_irq_safe

[Peter Ujfalusi]

It is safe to call the pm sync calls in interrupt context in this driver.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
 drivers/clk/ti/clk-dra7-atl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c
index af29359677da..f18d149acd65 100644
--- a/drivers/clk/ti/clk-dra7-atl.c
+++ b/drivers/clk/ti/clk-dra7-atl.c
@@ -228,6 +228,7 @@ static int of_dra7_atl_clk_probe(struct platform_device *pdev)
 	cinfo->iobase = of_iomap(node, 0);
 	cinfo->dev = &pdev->dev;
 	pm_runtime_enable(cinfo->dev);
+	pm_runtime_irq_safe(cinfo->dev);
 
 	pm_runtime_get_sync(cinfo->dev);
 	atl_write(cinfo, DRA7_ATL_PCLKMUX_REG(0), DRA7_ATL_PCLKMUX);
-- 
2.1.0

Re: [PATCH] clk: ti: dra7-atl-clock: Mark the device as pm_runtime_irq_safe

[Tero Kristo]

On 09/29/2014 11:10 AM, Peter Ujfalusi wrote:
> It is safe to call the pm sync calls in interrupt context in this driver.
>
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>

Thanks, applied to for-v3.18/ti-clk-drv.

-Tero

> ---
>   drivers/clk/ti/clk-dra7-atl.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c
> index af29359677da..f18d149acd65 100644
> --- a/drivers/clk/ti/clk-dra7-atl.c
> +++ b/drivers/clk/ti/clk-dra7-atl.c
> @@ -228,6 +228,7 @@ static int of_dra7_atl_clk_probe(struct platform_device *pdev)
>   	cinfo->iobase = of_iomap(node, 0);
>   	cinfo->dev = &pdev->dev;
>   	pm_runtime_enable(cinfo->dev);
> +	pm_runtime_irq_safe(cinfo->dev);
>
>   	pm_runtime_get_sync(cinfo->dev);
>   	atl_write(cinfo, DRA7_ATL_PCLKMUX_REG(0), DRA7_ATL_PCLKMUX);
>

Re: [PATCH] clk: ti: dra7-atl-clock: Mark the device as pm_runtime_irq_safe

[Tero Kristo]

On 09/29/2014 11:10 AM, Peter Ujfalusi wrote:
> It is safe to call the pm sync calls in interrupt context in this driver.
>
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>

Thanks, applied to for-v3.18/ti-clk-drv.

-Tero

> ---
>   drivers/clk/ti/clk-dra7-atl.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c
> index af29359677da..f18d149acd65 100644
> --- a/drivers/clk/ti/clk-dra7-atl.c
> +++ b/drivers/clk/ti/clk-dra7-atl.c
> @@ -228,6 +228,7 @@ static int of_dra7_atl_clk_probe(struct platform_device *pdev)
>   	cinfo->iobase = of_iomap(node, 0);
>   	cinfo->dev = &pdev->dev;
>   	pm_runtime_enable(cinfo->dev);
> +	pm_runtime_irq_safe(cinfo->dev);
>
>   	pm_runtime_get_sync(cinfo->dev);
>   	atl_write(cinfo, DRA7_ATL_PCLKMUX_REG(0), DRA7_ATL_PCLKMUX);
>