We need to call clk_put on display clock, in the same way as functional clock. Also remove the check for priv->clk, since clk_put() is safe against NULL pointers. Signed-off-by: Julien CHAUVEAU <julien.chauveau@xxxxxxxxxxxxxxxxxxx> --- Changes since v1: - remove the check for priv->clk, as suggested by Felipe Balbi drivers/gpu/drm/tilcdc/tilcdc_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c b/drivers/gpu/drm/tilcdc/tilcdc_drv.c index 79a34cb..b0c9687 100644 --- a/drivers/gpu/drm/tilcdc/tilcdc_drv.c +++ b/drivers/gpu/drm/tilcdc/tilcdc_drv.c @@ -136,8 +136,8 @@ static int tilcdc_unload(struct drm_device *dev) CPUFREQ_TRANSITION_NOTIFIER); #endif - if (priv->clk) - clk_put(priv->clk); + clk_put(priv->disp_clk); + clk_put(priv->clk); if (priv->mmio) iounmap(priv->mmio); -- 2.1.0 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
