2017-08-26 11:13:47

by Bhumika Goyal

Subject: [PATCH 0/5] [media] PCI: make video_device const

Make video_device const.

Bhumika Goyal (5):
[media] meye: make video_device const
[media] saa7134: make video_device const
[media] solo6x10: make video_device const
[media] sta2x11: make video_device const
[media] tw68: make video_device const

drivers/media/pci/meye/meye.c | 2 +-
drivers/media/pci/saa7134/saa7134-empress.c | 2 +-
drivers/media/pci/solo6x10/solo6x10-v4l2.c | 2 +-
drivers/media/pci/sta2x11/sta2x11_vip.c | 2 +-
drivers/media/pci/tw68/tw68-video.c | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)

--
1.9.1

2017-08-26 11:13:57

by Bhumika Goyal

Subject: [PATCH 1/5] [media] meye: make video_device const

Make this const as it is only used in a copy operation.

Signed-off-by: Bhumika Goyal <[email protected]>
---
drivers/media/pci/meye/meye.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/meye/meye.c b/drivers/media/pci/meye/meye.c
index 0fe76be..49e047e 100644
--- a/drivers/media/pci/meye/meye.c
+++ b/drivers/media/pci/meye/meye.c
@@ -1533,7 +1533,7 @@ static int meye_mmap(struct file *file, struct vm_area_struct *vma)
.vidioc_default = vidioc_default,
};

-static struct video_device meye_template = {
+static const struct video_device meye_template = {
.name = "meye",
.fops = &meye_fops,
.ioctl_ops = &meye_ioctl_ops,
--
1.9.1

2017-08-26 11:14:04

by Bhumika Goyal

Subject: [PATCH 2/5] [media] saa7134: make video_device const

Make this const as it is only used in a copy operation.

Signed-off-by: Bhumika Goyal <[email protected]>
---
drivers/media/pci/saa7134/saa7134-empress.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/saa7134/saa7134-empress.c b/drivers/media/pci/saa7134/saa7134-empress.c
index b1d3648..66acfd3 100644
--- a/drivers/media/pci/saa7134/saa7134-empress.c
+++ b/drivers/media/pci/saa7134/saa7134-empress.c
@@ -205,7 +205,7 @@ static int empress_try_fmt_vid_cap(struct file *file, void *priv,

/* ----------------------------------------------------------- */

-static struct video_device saa7134_empress_template = {
+static const struct video_device saa7134_empress_template = {
.name = "saa7134-empress",
.fops = &ts_fops,
.ioctl_ops = &ts_ioctl_ops,
--
1.9.1

2017-08-26 11:14:13

by Bhumika Goyal

Subject: [PATCH 3/5] [media] solo6x10: make video_device const

Make this const as it is only used in a copy operation.

Signed-off-by: Bhumika Goyal <[email protected]>
---
drivers/media/pci/solo6x10/solo6x10-v4l2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/solo6x10/solo6x10-v4l2.c b/drivers/media/pci/solo6x10/solo6x10-v4l2.c
index 3266fc2..99ffd1e 100644
--- a/drivers/media/pci/solo6x10/solo6x10-v4l2.c
+++ b/drivers/media/pci/solo6x10/solo6x10-v4l2.c
@@ -630,7 +630,7 @@ static int solo_s_ctrl(struct v4l2_ctrl *ctrl)
.vidioc_unsubscribe_event = v4l2_event_unsubscribe,
};

-static struct video_device solo_v4l2_template = {
+static const struct video_device solo_v4l2_template = {
.name = SOLO6X10_NAME,
.fops = &solo_v4l2_fops,
.ioctl_ops = &solo_v4l2_ioctl_ops,
--
1.9.1

2017-08-26 11:14:20

by Bhumika Goyal

Subject: [PATCH 4/5] [media] sta2x11: make video_device const

Make this const as it is only used in a copy operation.

Signed-off-by: Bhumika Goyal <[email protected]>
---
drivers/media/pci/sta2x11/sta2x11_vip.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/sta2x11/sta2x11_vip.c b/drivers/media/pci/sta2x11/sta2x11_vip.c
index 6343d24..eb5a9ea 100644
--- a/drivers/media/pci/sta2x11/sta2x11_vip.c
+++ b/drivers/media/pci/sta2x11/sta2x11_vip.c
@@ -754,7 +754,7 @@ static int vidioc_g_fmt_vid_cap(struct file *file, void *priv,
.vidioc_unsubscribe_event = v4l2_event_unsubscribe,
};

-static struct video_device video_dev_template = {
+static const struct video_device video_dev_template = {
.name = KBUILD_MODNAME,
.release = video_device_release_empty,
.fops = &vip_fops,
--
1.9.1

2017-08-26 11:14:28

by Bhumika Goyal

Subject: [PATCH 5/5] [media] tw68: make video_device const

Make this const as it is only used in a copy operation.

Signed-off-by: Bhumika Goyal <[email protected]>
---
drivers/media/pci/tw68/tw68-video.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/tw68/tw68-video.c b/drivers/media/pci/tw68/tw68-video.c
index 58c4dd7..8c1f4a0 100644
--- a/drivers/media/pci/tw68/tw68-video.c
+++ b/drivers/media/pci/tw68/tw68-video.c
@@ -916,7 +916,7 @@ static int vidioc_s_register(struct file *file, void *priv,
#endif
};

-static struct video_device tw68_video_template = {
+static const struct video_device tw68_video_template = {
.name = "tw68_video",
.fops = &video_fops,
.ioctl_ops = &video_ioctl_ops,
--
1.9.1

2017-08-28 18:41:48

by Ismael Luceno

Subject: Re: [PATCH 3/5] [media] solo6x10: make video_device const

On 26/Aug/2017 16:43, Bhumika Goyal wrote:
> Make this const as it is only used in a copy operation.
>
> Signed-off-by: Bhumika Goyal <[email protected]>
> ---
> drivers/media/pci/solo6x10/solo6x10-v4l2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/pci/solo6x10/solo6x10-v4l2.c b/drivers/media/pci/solo6x10/solo6x10-v4l2.c
> index 3266fc2..99ffd1e 100644
> --- a/drivers/media/pci/solo6x10/solo6x10-v4l2.c
> +++ b/drivers/media/pci/solo6x10/solo6x10-v4l2.c
> @@ -630,7 +630,7 @@ static int solo_s_ctrl(struct v4l2_ctrl *ctrl)
> .vidioc_unsubscribe_event = v4l2_event_unsubscribe,
> };
>
> -static struct video_device solo_v4l2_template = {
> +static const struct video_device solo_v4l2_template = {
> .name = SOLO6X10_NAME,
> .fops = &solo_v4l2_fops,
> .ioctl_ops = &solo_v4l2_ioctl_ops,
> --
> 1.9.1
>

Signed-off-by: Ismael Luceno <[email protected]>