diff options
author | Peter Senna Tschudin <peter.senna@gmail.com> | 2012-10-03 08:40:44 -0400 |
---|---|---|
committer | Florian Tobias Schandinat <FlorianSchandinat@gmx.de> | 2012-10-09 22:06:18 -0400 |
commit | 36141e5692b42da5b010675b6e68e2292367eb9c (patch) | |
tree | 390305038a1433f6096e519b7c3e7e15829375f5 | |
parent | d63f2c532fb8885e481a7b626823a0c183694b53 (diff) |
drivers/video/exynos/exynos_mipi_dsi.c: fix error return code
Convert a nonnegative error return code to a negative one, as returned
elsewhere in the function.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// <smpl>
(
if@p1 (\(ret < 0\|ret != 0\))
{ ... return ret; }
|
ret@p1 = 0
)
... when != ret = e1
when != &ret
*if(...)
{
... when != ret = e2
when forall
return ret;
}
// </smpl>
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
Acked-by: Donghwa Lee <dh09.lee@samsung.com>
Signed-off-by: Florian Tobias Schandinat <FlorianSchandinat@gmx.de>
-rw-r--r-- | drivers/video/exynos/exynos_mipi_dsi.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/video/exynos/exynos_mipi_dsi.c b/drivers/video/exynos/exynos_mipi_dsi.c index ef68228cfd70..1f5de896bd3f 100644 --- a/drivers/video/exynos/exynos_mipi_dsi.c +++ b/drivers/video/exynos/exynos_mipi_dsi.c | |||
@@ -375,6 +375,7 @@ static int exynos_mipi_dsi_probe(struct platform_device *pdev) | |||
375 | dsim->clock = clk_get(&pdev->dev, "dsim0"); | 375 | dsim->clock = clk_get(&pdev->dev, "dsim0"); |
376 | if (IS_ERR(dsim->clock)) { | 376 | if (IS_ERR(dsim->clock)) { |
377 | dev_err(&pdev->dev, "failed to get dsim clock source\n"); | 377 | dev_err(&pdev->dev, "failed to get dsim clock source\n"); |
378 | ret = -ENODEV; | ||
378 | goto err_clock_get; | 379 | goto err_clock_get; |
379 | } | 380 | } |
380 | 381 | ||
@@ -383,6 +384,7 @@ static int exynos_mipi_dsi_probe(struct platform_device *pdev) | |||
383 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); | 384 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
384 | if (!res) { | 385 | if (!res) { |
385 | dev_err(&pdev->dev, "failed to get io memory region\n"); | 386 | dev_err(&pdev->dev, "failed to get io memory region\n"); |
387 | ret = -ENODEV; | ||
386 | goto err_platform_get; | 388 | goto err_platform_get; |
387 | } | 389 | } |
388 | 390 | ||
@@ -407,6 +409,7 @@ static int exynos_mipi_dsi_probe(struct platform_device *pdev) | |||
407 | dsim_ddi = exynos_mipi_dsi_bind_lcd_ddi(dsim, dsim_pd->lcd_panel_name); | 409 | dsim_ddi = exynos_mipi_dsi_bind_lcd_ddi(dsim, dsim_pd->lcd_panel_name); |
408 | if (!dsim_ddi) { | 410 | if (!dsim_ddi) { |
409 | dev_err(&pdev->dev, "mipi_dsim_ddi object not found.\n"); | 411 | dev_err(&pdev->dev, "mipi_dsim_ddi object not found.\n"); |
412 | ret = -EINVAL; | ||
410 | goto err_bind; | 413 | goto err_bind; |
411 | } | 414 | } |
412 | 415 | ||