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.chauv...@neo-technologies.fr> --- 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/