* [PATCH 00/10] [media] platform: make video_device const
@ 2017-08-26 10:20 Bhumika Goyal
2017-08-26 10:20 ` [PATCH 01/10] [media] cx88: " Bhumika Goyal
` (10 more replies)
0 siblings, 11 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make make video_device const.
Bhumika Goyal (10):
[media] cx88: make video_device const
[media] dt3155: make video_device const
[media]: marvell-ccic: make video_device const
[media] mx2-emmaprp: make video_device const
[media]: s5p-g2d: make video_device const
[media]: ti-vpe: make video_device const
[media] via-camera: make video_device const
[media]: fsl-viu: make video_device const
[media] m2m-deinterlace: make video_device const
[media] vim2m: make video_device const
drivers/media/pci/cx88/cx88-blackbird.c | 2 +-
drivers/media/pci/dt3155/dt3155.c | 2 +-
drivers/media/platform/fsl-viu.c | 2 +-
drivers/media/platform/m2m-deinterlace.c | 2 +-
drivers/media/platform/marvell-ccic/mcam-core.c | 2 +-
drivers/media/platform/mx2_emmaprp.c | 2 +-
drivers/media/platform/s5p-g2d/g2d.c | 2 +-
drivers/media/platform/ti-vpe/cal.c | 2 +-
drivers/media/platform/ti-vpe/vpe.c | 2 +-
drivers/media/platform/via-camera.c | 2 +-
drivers/media/platform/vim2m.c | 2 +-
11 files changed, 11 insertions(+), 11 deletions(-)
--
1.9.1
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 01/10] [media] cx88: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 02/10] [media] dt3155: " Bhumika Goyal
` (9 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only passed to the const argument of the
function cx88_vdev_init.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/pci/cx88/cx88-blackbird.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/cx88/cx88-blackbird.c b/drivers/media/pci/cx88/cx88-blackbird.c
index aa49c95..e3101f0 100644
--- a/drivers/media/pci/cx88/cx88-blackbird.c
+++ b/drivers/media/pci/cx88/cx88-blackbird.c
@@ -1075,7 +1075,7 @@ static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id id)
.vidioc_unsubscribe_event = v4l2_event_unsubscribe,
};
-static struct video_device cx8802_mpeg_template = {
+static const struct video_device cx8802_mpeg_template = {
.name = "cx8802",
.fops = &mpeg_fops,
.ioctl_ops = &mpeg_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 02/10] [media] dt3155: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
2017-08-26 10:20 ` [PATCH 01/10] [media] cx88: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 03/10] [media]: marvell-ccic: " Bhumika Goyal
` (8 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/pci/dt3155/dt3155.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/dt3155/dt3155.c b/drivers/media/pci/dt3155/dt3155.c
index 6a21969..1775c36 100644
--- a/drivers/media/pci/dt3155/dt3155.c
+++ b/drivers/media/pci/dt3155/dt3155.c
@@ -499,7 +499,7 @@ static int dt3155_init_board(struct dt3155_priv *pd)
return 0;
}
-static struct video_device dt3155_vdev = {
+static const struct video_device dt3155_vdev = {
.name = DT3155_NAME,
.fops = &dt3155_fops,
.ioctl_ops = &dt3155_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 03/10] [media]: marvell-ccic: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
2017-08-26 10:20 ` [PATCH 01/10] [media] cx88: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 02/10] [media] dt3155: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 04/10] [media] mx2-emmaprp: " Bhumika Goyal
` (7 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/marvell-ccic/mcam-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/marvell-ccic/mcam-core.c b/drivers/media/platform/marvell-ccic/mcam-core.c
index 8cac2f2..b07a251 100644
--- a/drivers/media/platform/marvell-ccic/mcam-core.c
+++ b/drivers/media/platform/marvell-ccic/mcam-core.c
@@ -1639,7 +1639,7 @@ static int mcam_v4l_release(struct file *filp)
* This template device holds all of those v4l2 methods; we
* clone it for specific real devices.
*/
-static struct video_device mcam_v4l_template = {
+static const struct video_device mcam_v4l_template = {
.name = "mcam",
.fops = &mcam_v4l_fops,
.ioctl_ops = &mcam_v4l_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 04/10] [media] mx2-emmaprp: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (2 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 03/10] [media]: marvell-ccic: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 05/10] [media]: s5p-g2d: " Bhumika Goyal
` (6 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/mx2_emmaprp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/mx2_emmaprp.c b/drivers/media/platform/mx2_emmaprp.c
index 7fd209e..3493d40 100644
--- a/drivers/media/platform/mx2_emmaprp.c
+++ b/drivers/media/platform/mx2_emmaprp.c
@@ -873,7 +873,7 @@ static int emmaprp_mmap(struct file *file, struct vm_area_struct *vma)
.mmap = emmaprp_mmap,
};
-static struct video_device emmaprp_videodev = {
+static const struct video_device emmaprp_videodev = {
.name = MEM2MEM_NAME,
.fops = &emmaprp_fops,
.ioctl_ops = &emmaprp_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 05/10] [media]: s5p-g2d: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (3 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 04/10] [media] mx2-emmaprp: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 06/10] [media]: ti-vpe: " Bhumika Goyal
` (5 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/s5p-g2d/g2d.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/s5p-g2d/g2d.c b/drivers/media/platform/s5p-g2d/g2d.c
index bd655b5..66aa8cf 100644
--- a/drivers/media/platform/s5p-g2d/g2d.c
+++ b/drivers/media/platform/s5p-g2d/g2d.c
@@ -602,7 +602,7 @@ static irqreturn_t g2d_isr(int irq, void *prv)
.vidioc_cropcap = vidioc_cropcap,
};
-static struct video_device g2d_videodev = {
+static const struct video_device g2d_videodev = {
.name = G2D_NAME,
.fops = &g2d_fops,
.ioctl_ops = &g2d_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 06/10] [media]: ti-vpe: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (4 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 05/10] [media]: s5p-g2d: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 07/10] [media] via-camera: " Bhumika Goyal
` (4 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/ti-vpe/cal.c | 2 +-
drivers/media/platform/ti-vpe/vpe.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/cal.c b/drivers/media/platform/ti-vpe/cal.c
index 0c7ddf8..42e383a 100644
--- a/drivers/media/platform/ti-vpe/cal.c
+++ b/drivers/media/platform/ti-vpe/cal.c
@@ -1420,7 +1420,7 @@ static void cal_stop_streaming(struct vb2_queue *vq)
.vidioc_unsubscribe_event = v4l2_event_unsubscribe,
};
-static struct video_device cal_videodev = {
+static const struct video_device cal_videodev = {
.name = CAL_MODULE_NAME,
.fops = &cal_fops,
.ioctl_ops = &cal_ioctl_ops,
diff --git a/drivers/media/platform/ti-vpe/vpe.c b/drivers/media/platform/ti-vpe/vpe.c
index 2873c22..45bd105 100644
--- a/drivers/media/platform/ti-vpe/vpe.c
+++ b/drivers/media/platform/ti-vpe/vpe.c
@@ -2421,7 +2421,7 @@ static int vpe_release(struct file *file)
.mmap = v4l2_m2m_fop_mmap,
};
-static struct video_device vpe_videodev = {
+static const struct video_device vpe_videodev = {
.name = VPE_MODULE_NAME,
.fops = &vpe_fops,
.ioctl_ops = &vpe_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 07/10] [media] via-camera: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (5 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 06/10] [media]: ti-vpe: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 08/10] [media]: fsl-viu: " Bhumika Goyal
` (3 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/via-camera.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/via-camera.c b/drivers/media/platform/via-camera.c
index e16f70a..805d4a8 100644
--- a/drivers/media/platform/via-camera.c
+++ b/drivers/media/platform/via-camera.c
@@ -1259,7 +1259,7 @@ static int viacam_resume(void *priv)
* Setup stuff.
*/
-static struct video_device viacam_v4l_template = {
+static const struct video_device viacam_v4l_template = {
.name = "via-camera",
.minor = -1,
.tvnorms = V4L2_STD_NTSC_M,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 08/10] [media]: fsl-viu: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (6 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 07/10] [media] via-camera: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 09/10] [media] m2m-deinterlace: " Bhumika Goyal
` (2 subsequent siblings)
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/fsl-viu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/fsl-viu.c b/drivers/media/platform/fsl-viu.c
index f7b88e5..b3b91cb 100644
--- a/drivers/media/platform/fsl-viu.c
+++ b/drivers/media/platform/fsl-viu.c
@@ -1380,7 +1380,7 @@ static int viu_mmap(struct file *file, struct vm_area_struct *vma)
.vidioc_unsubscribe_event = v4l2_event_unsubscribe,
};
-static struct video_device viu_template = {
+static const struct video_device viu_template = {
.name = "FSL viu",
.fops = &viu_fops,
.minor = -1,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 09/10] [media] m2m-deinterlace: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (7 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 08/10] [media]: fsl-viu: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 10:20 ` [PATCH 10/10] [media] vim2m: " Bhumika Goyal
2017-08-26 11:59 ` [PATCH 00/10] [media] platform: " Mauro Carvalho Chehab
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/m2m-deinterlace.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/m2m-deinterlace.c b/drivers/media/platform/m2m-deinterlace.c
index 98f6db2..c8a1249 100644
--- a/drivers/media/platform/m2m-deinterlace.c
+++ b/drivers/media/platform/m2m-deinterlace.c
@@ -979,7 +979,7 @@ static int deinterlace_mmap(struct file *file, struct vm_area_struct *vma)
.mmap = deinterlace_mmap,
};
-static struct video_device deinterlace_videodev = {
+static const struct video_device deinterlace_videodev = {
.name = MEM2MEM_NAME,
.fops = &deinterlace_fops,
.ioctl_ops = &deinterlace_ioctl_ops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 10/10] [media] vim2m: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (8 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 09/10] [media] m2m-deinterlace: " Bhumika Goyal
@ 2017-08-26 10:20 ` Bhumika Goyal
2017-08-26 11:59 ` [PATCH 00/10] [media] platform: " Mauro Carvalho Chehab
10 siblings, 0 replies; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 10:20 UTC (permalink / raw)
To: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Cc: Bhumika Goyal
Make this const as it is only used in a copy operation.
Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
drivers/media/platform/vim2m.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/vim2m.c b/drivers/media/platform/vim2m.c
index afbaa35..b01fba0 100644
--- a/drivers/media/platform/vim2m.c
+++ b/drivers/media/platform/vim2m.c
@@ -974,7 +974,7 @@ static int vim2m_release(struct file *file)
.mmap = v4l2_m2m_fop_mmap,
};
-static struct video_device vim2m_videodev = {
+static const struct video_device vim2m_videodev = {
.name = MEM2MEM_NAME,
.vfl_dir = VFL_DIR_M2M,
.fops = &vim2m_fops,
--
1.9.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 00/10] [media] platform: make video_device const
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
` (9 preceding siblings ...)
2017-08-26 10:20 ` [PATCH 10/10] [media] vim2m: " Bhumika Goyal
@ 2017-08-26 11:59 ` Mauro Carvalho Chehab
2017-08-26 12:12 ` Bhumika Goyal
10 siblings, 1 reply; 14+ messages in thread
From: Mauro Carvalho Chehab @ 2017-08-26 11:59 UTC (permalink / raw)
To: Bhumika Goyal
Cc: julia.lawall, mchehab, hverkuil, corbet, kyungmin.park, kamil,
a.hajda, bparrot, linux-media, linux-kernel, linux-arm-kernel
Em Sat, 26 Aug 2017 15:50:02 +0530
Bhumika Goyal <bhumirks@gmail.com> escreveu:
> Make make video_device const.
>
> Bhumika Goyal (10):
> [media] cx88: make video_device const
> [media] dt3155: make video_device const
> [media]: marvell-ccic: make video_device const
> [media] mx2-emmaprp: make video_device const
> [media]: s5p-g2d: make video_device const
> [media]: ti-vpe: make video_device const
> [media] via-camera: make video_device const
> [media]: fsl-viu: make video_device const
> [media] m2m-deinterlace: make video_device const
> [media] vim2m: make video_device const
>
> drivers/media/pci/cx88/cx88-blackbird.c | 2 +-
> drivers/media/pci/dt3155/dt3155.c | 2 +-
> drivers/media/platform/fsl-viu.c | 2 +-
> drivers/media/platform/m2m-deinterlace.c | 2 +-
> drivers/media/platform/marvell-ccic/mcam-core.c | 2 +-
> drivers/media/platform/mx2_emmaprp.c | 2 +-
> drivers/media/platform/s5p-g2d/g2d.c | 2 +-
> drivers/media/platform/ti-vpe/cal.c | 2 +-
> drivers/media/platform/ti-vpe/vpe.c | 2 +-
> drivers/media/platform/via-camera.c | 2 +-
> drivers/media/platform/vim2m.c | 2 +-
Please, don't do one such cleanup patch per file. Instead, group
it per subdirectory, e. g. on e patch for:
drivers/media/platform/
and another one for:
drivers/media/pci/
That makes a lot easier to review and apply.
Thanks,
Mauro
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 00/10] [media] platform: make video_device const
2017-08-26 11:59 ` [PATCH 00/10] [media] platform: " Mauro Carvalho Chehab
@ 2017-08-26 12:12 ` Bhumika Goyal
2017-08-26 12:28 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 14+ messages in thread
From: Bhumika Goyal @ 2017-08-26 12:12 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Julia Lawall, mchehab, hverkuil, Jonathan Corbet, Kyungmin Park,
kamil, a.hajda, bparrot, linux-media, Linux Kernel Mailing List,
linux-arm-kernel
On Sat, Aug 26, 2017 at 5:29 PM, Mauro Carvalho Chehab
<mchehab@s-opensource.com> wrote:
> Em Sat, 26 Aug 2017 15:50:02 +0530
> Bhumika Goyal <bhumirks@gmail.com> escreveu:
>
>> Make make video_device const.
>>
>> Bhumika Goyal (10):
>> [media] cx88: make video_device const
>> [media] dt3155: make video_device const
>> [media]: marvell-ccic: make video_device const
>> [media] mx2-emmaprp: make video_device const
>> [media]: s5p-g2d: make video_device const
>> [media]: ti-vpe: make video_device const
>> [media] via-camera: make video_device const
>> [media]: fsl-viu: make video_device const
>> [media] m2m-deinterlace: make video_device const
>> [media] vim2m: make video_device const
>>
>> drivers/media/pci/cx88/cx88-blackbird.c | 2 +-
>> drivers/media/pci/dt3155/dt3155.c | 2 +-
>> drivers/media/platform/fsl-viu.c | 2 +-
>> drivers/media/platform/m2m-deinterlace.c | 2 +-
>> drivers/media/platform/marvell-ccic/mcam-core.c | 2 +-
>> drivers/media/platform/mx2_emmaprp.c | 2 +-
>> drivers/media/platform/s5p-g2d/g2d.c | 2 +-
>> drivers/media/platform/ti-vpe/cal.c | 2 +-
>> drivers/media/platform/ti-vpe/vpe.c | 2 +-
>> drivers/media/platform/via-camera.c | 2 +-
>> drivers/media/platform/vim2m.c | 2 +-
>
> Please, don't do one such cleanup patch per file. Instead, group
> it per subdirectory, e. g. on e patch for:
> drivers/media/platform/
>
> and another one for:
> drivers/media/pci/
>
> That makes a lot easier to review and apply.
>
Okay, I will keep this in mind. Should I send a v2 for both the series?
Thanks,
Bhumika
> Thanks,
> Mauro
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 00/10] [media] platform: make video_device const
2017-08-26 12:12 ` Bhumika Goyal
@ 2017-08-26 12:28 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 14+ messages in thread
From: Mauro Carvalho Chehab @ 2017-08-26 12:28 UTC (permalink / raw)
To: Bhumika Goyal
Cc: Julia Lawall, mchehab, hverkuil, Jonathan Corbet, Kyungmin Park,
kamil, a.hajda, bparrot, linux-media, Linux Kernel Mailing List,
linux-arm-kernel
Em Sat, 26 Aug 2017 17:42:34 +0530
Bhumika Goyal <bhumirks@gmail.com> escreveu:
> On Sat, Aug 26, 2017 at 5:29 PM, Mauro Carvalho Chehab
> <mchehab@s-opensource.com> wrote:
> > Em Sat, 26 Aug 2017 15:50:02 +0530
> > Bhumika Goyal <bhumirks@gmail.com> escreveu:
> >
> >> Make make video_device const.
> >>
> >> Bhumika Goyal (10):
> >> [media] cx88: make video_device const
> >> [media] dt3155: make video_device const
> >> [media]: marvell-ccic: make video_device const
> >> [media] mx2-emmaprp: make video_device const
> >> [media]: s5p-g2d: make video_device const
> >> [media]: ti-vpe: make video_device const
> >> [media] via-camera: make video_device const
> >> [media]: fsl-viu: make video_device const
> >> [media] m2m-deinterlace: make video_device const
> >> [media] vim2m: make video_device const
> >>
> >> drivers/media/pci/cx88/cx88-blackbird.c | 2 +-
> >> drivers/media/pci/dt3155/dt3155.c | 2 +-
> >> drivers/media/platform/fsl-viu.c | 2 +-
> >> drivers/media/platform/m2m-deinterlace.c | 2 +-
> >> drivers/media/platform/marvell-ccic/mcam-core.c | 2 +-
> >> drivers/media/platform/mx2_emmaprp.c | 2 +-
> >> drivers/media/platform/s5p-g2d/g2d.c | 2 +-
> >> drivers/media/platform/ti-vpe/cal.c | 2 +-
> >> drivers/media/platform/ti-vpe/vpe.c | 2 +-
> >> drivers/media/platform/via-camera.c | 2 +-
> >> drivers/media/platform/vim2m.c | 2 +-
> >
> > Please, don't do one such cleanup patch per file. Instead, group
> > it per subdirectory, e. g. on e patch for:
> > drivers/media/platform/
> >
> > and another one for:
> > drivers/media/pci/
> >
> > That makes a lot easier to review and apply.
> >
>
> Okay, I will keep this in mind. Should I send a v2 for both the series?
Yes, please.
Thanks,
Mauro
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2017-08-26 12:29 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-26 10:20 [PATCH 00/10] [media] platform: make video_device const Bhumika Goyal
2017-08-26 10:20 ` [PATCH 01/10] [media] cx88: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 02/10] [media] dt3155: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 03/10] [media]: marvell-ccic: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 04/10] [media] mx2-emmaprp: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 05/10] [media]: s5p-g2d: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 06/10] [media]: ti-vpe: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 07/10] [media] via-camera: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 08/10] [media]: fsl-viu: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 09/10] [media] m2m-deinterlace: " Bhumika Goyal
2017-08-26 10:20 ` [PATCH 10/10] [media] vim2m: " Bhumika Goyal
2017-08-26 11:59 ` [PATCH 00/10] [media] platform: " Mauro Carvalho Chehab
2017-08-26 12:12 ` Bhumika Goyal
2017-08-26 12:28 ` Mauro Carvalho Chehab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).