No need to check whether unsigned variable is less than 0. CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> CC: ivtv-devel@xxxxxxxxxxxxxx CC: linux-media@xxxxxxxxxxxxxxx Signed-off-by: Tushar Behera <tushar.behera@xxxxxxxxxx> --- drivers/media/pci/ivtv/ivtv-ioctl.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/media/pci/ivtv/ivtv-ioctl.c b/drivers/media/pci/ivtv/ivtv-ioctl.c index 949ae23..4b47b5c 100644 --- a/drivers/media/pci/ivtv/ivtv-ioctl.c +++ b/drivers/media/pci/ivtv/ivtv-ioctl.c @@ -993,7 +993,7 @@ int ivtv_s_input(struct file *file, void *fh, unsigned int inp) v4l2_std_id std; int i; - if (inp < 0 || inp >= itv->nof_inputs) + if (inp >= itv->nof_inputs) return -EINVAL; if (inp == itv->active_input) { -- 1.7.4.1 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html