diff options
author | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-08-21 16:56:16 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-08-26 17:51:59 -0400 |
commit | cff562137226d0f431e85733795edd271e946087 (patch) | |
tree | 8bd929c2004e04fffdc4398ff3842c1e7cc696e7 | |
parent | 0448056c7e97f3d8aef6777ffc8ed18569f973e3 (diff) |
[media] gsc-core: Remove useless test
drivers/media/platform/exynos-gsc/gsc-core.c: In function 'gsc_probe':
drivers/media/platform/exynos-gsc/gsc-core.c:1089:2: warning: comparison is alw
ays false due to limited range of data type [-Wtype-limits]
if (gsc->id < 0 || gsc->id >= drv_data->num_entities) {
^
gsc->id is declared as u16, so it should always be a positive
value.
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
-rw-r--r-- | drivers/media/platform/exynos-gsc/gsc-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/exynos-gsc/gsc-core.c b/drivers/media/platform/exynos-gsc/gsc-core.c index 9d0cc04d7ab7..8d8b3cff8212 100644 --- a/drivers/media/platform/exynos-gsc/gsc-core.c +++ b/drivers/media/platform/exynos-gsc/gsc-core.c | |||
@@ -1086,7 +1086,7 @@ static int gsc_probe(struct platform_device *pdev) | |||
1086 | else | 1086 | else |
1087 | gsc->id = pdev->id; | 1087 | gsc->id = pdev->id; |
1088 | 1088 | ||
1089 | if (gsc->id < 0 || gsc->id >= drv_data->num_entities) { | 1089 | if (gsc->id >= drv_data->num_entities) { |
1090 | dev_err(dev, "Invalid platform device id: %d\n", gsc->id); | 1090 | dev_err(dev, "Invalid platform device id: %d\n", gsc->id); |
1091 | return -EINVAL; | 1091 | return -EINVAL; |
1092 | } | 1092 | } |