diff options
author | Dmitry Torokhov <dtor@chromium.org> | 2014-11-14 17:09:50 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-11-25 09:47:13 -0500 |
commit | 9a7a848df1995f05f750b4ed491ac80a459c912f (patch) | |
tree | 0c7dcef36238ac33caaee4e11361e2ef876c023a /drivers/media/platform | |
parent | 460c8a7c25fef0712abdeeefd07bf02a8e8cb9e1 (diff) |
[media] exynos4-is: fix error handling of irq_of_parse_and_map
Return value of irq_of_parse_and_map() is unsigned int, with 0
indicating failure, so testing for negative result never works.
Signed-off-by: Dmitry Torokhov <dtor@chromium.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/platform')
-rw-r--r-- | drivers/media/platform/exynos4-is/fimc-is.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/platform/exynos4-is/fimc-is.c b/drivers/media/platform/exynos4-is/fimc-is.c index 22162b2567da..3d49ce03b176 100644 --- a/drivers/media/platform/exynos4-is/fimc-is.c +++ b/drivers/media/platform/exynos4-is/fimc-is.c | |||
@@ -814,9 +814,9 @@ static int fimc_is_probe(struct platform_device *pdev) | |||
814 | return -ENOMEM; | 814 | return -ENOMEM; |
815 | 815 | ||
816 | is->irq = irq_of_parse_and_map(dev->of_node, 0); | 816 | is->irq = irq_of_parse_and_map(dev->of_node, 0); |
817 | if (is->irq < 0) { | 817 | if (!is->irq) { |
818 | dev_err(dev, "no irq found\n"); | 818 | dev_err(dev, "no irq found\n"); |
819 | return is->irq; | 819 | return -EINVAL; |
820 | } | 820 | } |
821 | 821 | ||
822 | ret = fimc_is_get_clocks(is); | 822 | ret = fimc_is_get_clocks(is); |