[media] v4l2: make vidioc_s_crop const

Write-only ioctls should have a const argument in the ioctl op.
Do this conversion for vidioc_s_crop.
Adding const for write-only ioctls was decided during the 2012 Media Workshop.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

3 files changed, 4 insertions, 4 deletions

diff --git a/include/media/soc_camera.h b/include/media/soc_camera.h
index 435e7b8ad1c3..6442edc2a151 100644
--- a/include/media/soc_camera.h
+++ b/include/media/soc_camera.h
@@ -85,14 +85,14 @@ struct soc_camera_host_ops {
         void (*put_formats)(struct soc_camera_device *);
         int (*cropcap)(struct soc_camera_device *, struct v4l2_cropcap *);
         int (*get_crop)(struct soc_camera_device *, struct v4l2_crop *);
-        int (*set_crop)(struct soc_camera_device *, struct v4l2_crop *);
+        int (*set_crop)(struct soc_camera_device *, const struct v4l2_crop *);
         int (*get_selection)(struct soc_camera_device *, struct v4l2_selection *);
         int (*set_selection)(struct soc_camera_device *, struct v4l2_selection *);
         /*
          * The difference to .set_crop() is, that .set_livecrop is not allowed
          * to change the output sizes
          */
-        int (*set_livecrop)(struct soc_camera_device *, struct v4l2_crop *);
+        int (*set_livecrop)(struct soc_camera_device *, const struct v4l2_crop *);
         int (*set_fmt)(struct soc_camera_device *, struct v4l2_format *);
         int (*try_fmt)(struct soc_camera_device *, struct v4l2_format *);
         void (*init_videobuf)(struct videobuf_queue *,
diff --git a/include/media/v4l2-ioctl.h b/include/media/v4l2-ioctl.h
index fbeb00e2c109..e48b571ca37d 100644
--- a/include/media/v4l2-ioctl.h
+++ b/include/media/v4l2-ioctl.h
@@ -186,7 +186,7 @@ struct v4l2_ioctl_ops {
         int (*vidioc_g_crop)           (struct file *file, void *fh,
                                         struct v4l2_crop *a);
         int (*vidioc_s_crop)           (struct file *file, void *fh,
-                                        struct v4l2_crop *a);
+                                        const struct v4l2_crop *a);
         int (*vidioc_g_selection)      (struct file *file, void *fh,
                                         struct v4l2_selection *s);
         int (*vidioc_s_selection)      (struct file *file, void *fh,
diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h
index e698f2cead4e..2ecd7377153b 100644
--- a/include/media/v4l2-subdev.h
+++ b/include/media/v4l2-subdev.h
@@ -286,7 +286,7 @@ struct v4l2_subdev_video_ops {
         int (*s_stream)(struct v4l2_subdev *sd, int enable);
         int (*cropcap)(struct v4l2_subdev *sd, struct v4l2_cropcap *cc);
         int (*g_crop)(struct v4l2_subdev *sd, struct v4l2_crop *crop);
-        int (*s_crop)(struct v4l2_subdev *sd, struct v4l2_crop *crop);
+        int (*s_crop)(struct v4l2_subdev *sd, const struct v4l2_crop *crop);
         int (*g_parm)(struct v4l2_subdev *sd, struct v4l2_streamparm *param);
         int (*s_parm)(struct v4l2_subdev *sd, struct v4l2_streamparm *param);
         int (*g_frame_interval)(struct v4l2_subdev *sd,