diff options
author | Sakari Ailus <sakari.ailus@linux.intel.com> | 2019-02-18 19:33:54 +0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2019-05-29 18:18:01 +0300 |
commit | 0ace8734f40564ae9405df9f12f0fde6cd7fa7e8 (patch) | |
tree | ac4272df64b2d42bc332505e33a565a890743488 | |
parent | c9d52c114a9fcc61c30512c7f810247a9f2812af (diff) | |
download | linux-0ace8734f40564ae9405df9f12f0fde6cd7fa7e8.tar.xz |
media: staging: imgu: Remove redundant checks
Remove redundant checks for less than zero on unsigned variables.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Tested-by: Rajmohan Mani <rajmohan.mani@intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
-rw-r--r-- | drivers/staging/media/ipu3/ipu3-css-fw.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/media/ipu3/ipu3-css-fw.c b/drivers/staging/media/ipu3/ipu3-css-fw.c index 4122d4e42db6..45aff76198e2 100644 --- a/drivers/staging/media/ipu3/ipu3-css-fw.c +++ b/drivers/staging/media/ipu3/ipu3-css-fw.c @@ -200,13 +200,11 @@ int imgu_css_fw_init(struct imgu_css *css) goto bad_fw; for (j = 0; j < bi->info.isp.num_output_formats; j++) - if (bi->info.isp.output_formats[j] < 0 || - bi->info.isp.output_formats[j] >= + if (bi->info.isp.output_formats[j] >= IMGU_ABI_FRAME_FORMAT_NUM) goto bad_fw; for (j = 0; j < bi->info.isp.num_vf_formats; j++) - if (bi->info.isp.vf_formats[j] < 0 || - bi->info.isp.vf_formats[j] >= + if (bi->info.isp.vf_formats[j] >= IMGU_ABI_FRAME_FORMAT_NUM) goto bad_fw; |