diff options
author | Julia Lawall <julia@diku.dk> | 2011-06-01 17:10:11 +0000 |
---|---|---|
committer | Paul Mundt <lethal@linux-sh.org> | 2011-06-02 17:07:41 +0900 |
commit | 609d3bbf044a6766f6505d8a29d4ed48020c014c (patch) | |
tree | 2ee36cad27e52d6b28986b4db259a169acffcabd | |
parent | 70f289db31502ac030049450e63d380f3bc30cd2 (diff) | |
download | op-kernel-dev-609d3bbf044a6766f6505d8a29d4ed48020c014c.zip op-kernel-dev-609d3bbf044a6766f6505d8a29d4ed48020c014c.tar.gz |
drivers/video/imxfb.c: add missing clk_put
Reorder the labels at the end of the function to correspond to the order in
which the resources are allocated.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// <smpl>
@r exists@
expression e1,e2;
statement S;
@@
e1 = clk_get@p1(...);
... when != e1 = e2
when != clk_put(e1)
when any
if (...) { ... when != clk_put(e1)
when != if (...) { ... clk_put(e1) ... }
* return@p3 ...;
} else S
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Acked-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
-rw-r--r-- | drivers/video/imxfb.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/video/imxfb.c b/drivers/video/imxfb.c index d2ccfd6..f135dbe 100644 --- a/drivers/video/imxfb.c +++ b/drivers/video/imxfb.c @@ -856,10 +856,10 @@ failed_platform_init: dma_free_writecombine(&pdev->dev,fbi->map_size,fbi->map_cpu, fbi->map_dma); failed_map: - clk_put(fbi->clk); -failed_getclock: iounmap(fbi->regs); failed_ioremap: + clk_put(fbi->clk); +failed_getclock: release_mem_region(res->start, resource_size(res)); failed_req: kfree(info->pseudo_palette); |