Re: [PATCH 1/6] [media] v4l: mt9t001: constify video_subdev structures

2017-08-08 Thread Laurent Pinchart
Hi Julia, Thank you for the patch. On Tuesday 08 Aug 2017 12:58:27 Julia Lawall wrote: > The v4l2_subdev_ops structure is only passed as the third argument of > v4l2_i2c_subdev_init, which is const, so the v4l2_subdev_ops structure > can be const as well. The other structures are only stored in

Re: [PATCH 1/6] [media] v4l: mt9t001: constify video_subdev structures

2017-08-08 Thread Laurent Pinchart
Hi Julia, Thank you for the patch. On Tuesday 08 Aug 2017 12:58:27 Julia Lawall wrote: > The v4l2_subdev_ops structure is only passed as the third argument of > v4l2_i2c_subdev_init, which is const, so the v4l2_subdev_ops structure > can be const as well. The other structures are only stored in

[PATCH 1/6] [media] v4l: mt9t001: constify video_subdev structures

2017-08-08 Thread Julia Lawall
The v4l2_subdev_ops structure is only passed as the third argument of v4l2_i2c_subdev_init, which is const, so the v4l2_subdev_ops structure can be const as well. The other structures are only stored in the v4l2_subdev_ops structure, all the fields of which are const, so these structures can also

[PATCH 1/6] [media] v4l: mt9t001: constify video_subdev structures

2017-08-08 Thread Julia Lawall
The v4l2_subdev_ops structure is only passed as the third argument of v4l2_i2c_subdev_init, which is const, so the v4l2_subdev_ops structure can be const as well. The other structures are only stored in the v4l2_subdev_ops structure, all the fields of which are const, so these structures can also