diff options
| author | Tomeu Vizoso <[email protected]> | 2019-10-23 14:21:57 +0200 |
|---|---|---|
| committer | Rob Herring <[email protected]> | 2019-10-23 12:50:47 -0500 |
| commit | 4cad2a574d342cd9b658db6e32e9024f559383ed (patch) | |
| tree | de30587dccb05942858295e61a1df626f86bbd52 | |
| parent | 8ae501e295cce9bc6e0dd82d5204a1d5faef44f8 (diff) | |
panfrost: Properly undo pm_runtime_enable when deferring a probe
When deferring the probe because of a missing regulator, we were calling
pm_runtime_disable even if pm_runtime_enable wasn't called.
Move the call to pm_runtime_disable to the right place.
Fixes: 635430797d3f ("drm/panfrost: Rework runtime PM initialization")
Reported-by: Chen-Yu Tsai <[email protected]>
Cc: Robin Murphy <[email protected]>
Signed-off-by: Tomeu Vizoso <[email protected]>
Reviewed-by: Robin Murphy <[email protected]>
Reviewed-by: Steven Price <[email protected]>
Signed-off-by: Rob Herring <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
| -rw-r--r-- | drivers/gpu/drm/panfrost/panfrost_drv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/panfrost/panfrost_drv.c b/drivers/gpu/drm/panfrost/panfrost_drv.c index bc2ddeb55f5d..f21bc8a7ee3a 100644 --- a/drivers/gpu/drm/panfrost/panfrost_drv.c +++ b/drivers/gpu/drm/panfrost/panfrost_drv.c @@ -556,11 +556,11 @@ static int panfrost_probe(struct platform_device *pdev) return 0; err_out2: + pm_runtime_disable(pfdev->dev); panfrost_devfreq_fini(pfdev); err_out1: panfrost_device_fini(pfdev); err_out0: - pm_runtime_disable(pfdev->dev); drm_dev_put(ddev); return err; } |