| Message ID | 20191114101718.20619-1-peter.ujfalusi@ti.com |
|---|---|
| State | Accepted |
| Commit | ad931a2d56e35ff69902f5d7299775e80c35c322 |
| Headers | show |
| Series | clk: ti: dra7-atl: Remove pm_runtime_irq_safe() | expand |
* Peter Ujfalusi <peter.ujfalusi@ti.com> [191114 10:16]: > This is not needed for anything, and prevents proper PM transitions for > parent devices which is bad in case of ti-sysc; this effectively kills > PM completely. Thus, remove the flag. Good to hear and thanks for clearing this one: Reviewed-by: Tony Lindgren <tony@atomide.com>
diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c index f65e16c4f3c4..8d4c08b034bd 100644 --- a/drivers/clk/ti/clk-dra7-atl.c +++ b/drivers/clk/ti/clk-dra7-atl.c @@ -233,7 +233,6 @@ 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);
This is not needed for anything, and prevents proper PM transitions for parent devices which is bad in case of ti-sysc; this effectively kills PM completely. Thus, remove the flag. Suggested-by: Tony Lindgren <tony@atomide.com> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com> --- drivers/clk/ti/clk-dra7-atl.c | 1 - 1 file changed, 1 deletion(-) -- Peter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki