[media] V4L2: drivers implementing vidioc_default should also return -ENOTTY
authorHans Verkuil <hans.verkuil@cisco.com>
Thu, 19 Apr 2012 15:36:03 +0000 (12:36 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 19 Apr 2012 20:23:55 +0000 (17:23 -0300)
If the vidioc_default implementation doesn't support the ioctl, then drivers
must return -ENOTTY instead of -EINVAL.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/cx18/cx18-ioctl.c
drivers/media/video/davinci/vpfe_capture.c
drivers/media/video/ivtv/ivtv-ioctl.c
drivers/media/video/meye.c
drivers/media/video/mxb.c

index be49f68..35fde4e 100644 (file)
@@ -1137,7 +1137,7 @@ static long cx18_default(struct file *file, void *fh, bool valid_prio,
        }
 
        default:
-               return -EINVAL;
+               return -ENOTTY;
        }
        return 0;
 }
index 20cf271..49a845f 100644 (file)
@@ -1761,7 +1761,7 @@ static long vpfe_param_handler(struct file *file, void *priv,
                }
                break;
        default:
-               ret = -EINVAL;
+               ret = -ENOTTY;
        }
 unlock_out:
        mutex_unlock(&vpfe_dev->lock);
index 989e556..a151271 100644 (file)
@@ -1827,7 +1827,7 @@ static long ivtv_default(struct file *file, void *fh, bool valid_prio,
                return ivtv_decoder_ioctls(file, cmd, (void *)arg);
 
        default:
-               return -EINVAL;
+               return -ENOTTY;
        }
        return 0;
 }
index b09a3c8..7bc7752 100644 (file)
@@ -1570,7 +1570,7 @@ static long vidioc_default(struct file *file, void *fh, bool valid_prio,
                return meyeioc_stilljcapt((int *) arg);
 
        default:
-               return -EINVAL;
+               return -ENOTTY;
        }
 
 }
index 2e41317..ca3f70f 100644 (file)
@@ -688,7 +688,7 @@ static long vidioc_default(struct file *file, void *fh, bool valid_prio,
 /*
                DEB2(pr_err("does not handle this ioctl\n"));
 */
-               return -ENOIOCTLCMD;
+               return -ENOTTY;
        }
        return 0;
 }