From: Thomas Petazzoni Date: Sun, 6 Jan 2013 10:10:44 +0000 (+0100) Subject: dma: mv_xor: fix error handling for clocks X-Git-Tag: upstream/snapshot3+hdmi~5604^2~61^2~12 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dab9206445952e64213582b2ab9077972850d65b;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git dma: mv_xor: fix error handling for clocks When a channel fails to initialize, we release all ressources, including clocks. However, a XOR unit is not necessarily associated to a clock (some variants of Marvell SoCs have a clock for XOR units, some don't), so we shouldn't unconditionally be releasing the clock. Instead, just like we do in the mv_xor_remove() function, we should check if one clock was found before releasing it. Signed-off-by: Thomas Petazzoni Signed-off-by: Jason Cooper --- diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c index cc5d23d..e17fad0 100644 --- a/drivers/dma/mv_xor.c +++ b/drivers/dma/mv_xor.c @@ -1366,8 +1366,11 @@ err_channel_add: irq_dispose_mapping(xordev->channels[i]->irq); } - clk_disable_unprepare(xordev->clk); - clk_put(xordev->clk); + if (!IS_ERR(xordev->clk)) { + clk_disable_unprepare(xordev->clk); + clk_put(xordev->clk); + } + return ret; }