mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 00/18] constify media pci_device_id
@ 2017-08-01 17:56 Arvind Yadav
  2017-08-01 17:56 ` [PATCH 01/18] [media] marvell-ccic: constify pci_device_id Arvind Yadav
                   ` (17 more replies)
  0 siblings, 18 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Arvind Yadav (18):
  [PATCH 01/18] [media] marvell-ccic: constify pci_device_id.
  [PATCH 02/18] [media] netup_unidvb: constify pci_device_id.
  [PATCH 03/18] [media] cx23885: constify pci_device_id.
  [PATCH 04/18] [media] meye: constify pci_device_id.
  [PATCH 05/18] [media] pluto2: constify pci_device_id.
  [PATCH 06/18] [media] dm1105: constify pci_device_id.
  [PATCH 07/18] [media] zoran: constify pci_device_id.
  [PATCH 08/18] [media] bt8xx: constify pci_device_id.
  [PATCH 09/18] [media] bt8xx: bttv: constify pci_device_id.
  [PATCH 10/18] [media] ivtv: constify pci_device_id.
  [PATCH 11/18] [media] cobalt: constify pci_device_id.
  [PATCH 12/18] [media] b2c2: constify pci_device_id.
  [PATCH 13/18] [media] saa7164: constify pci_device_id.
  [PATCH 14/18] [media] pt1: constify pci_device_id.
  [PATCH 15/18] [media] mantis: constify pci_device_id.
  [PATCH 16/18] [media] mantis: hopper_cards: constify pci_device_id.
  [PATCH 17/18] [media] cx18: constify pci_device_id.
  [PATCH 18/18] [media] radio: constify pci_device_id.

 drivers/media/pci/b2c2/flexcop-pci.c               | 2 +-
 drivers/media/pci/bt8xx/bt878.c                    | 2 +-
 drivers/media/pci/bt8xx/bttv-driver.c              | 2 +-
 drivers/media/pci/cobalt/cobalt-driver.c           | 2 +-
 drivers/media/pci/cx18/cx18-driver.c               | 2 +-
 drivers/media/pci/cx23885/cx23885-core.c           | 2 +-
 drivers/media/pci/dm1105/dm1105.c                  | 2 +-
 drivers/media/pci/ivtv/ivtv-driver.c               | 2 +-
 drivers/media/pci/mantis/hopper_cards.c            | 2 +-
 drivers/media/pci/mantis/mantis_cards.c            | 2 +-
 drivers/media/pci/meye/meye.c                      | 2 +-
 drivers/media/pci/netup_unidvb/netup_unidvb_core.c | 2 +-
 drivers/media/pci/pluto2/pluto2.c                  | 2 +-
 drivers/media/pci/pt1/pt1.c                        | 2 +-
 drivers/media/pci/saa7164/saa7164-core.c           | 2 +-
 drivers/media/pci/zoran/zoran_card.c               | 2 +-
 drivers/media/platform/marvell-ccic/cafe-driver.c  | 2 +-
 drivers/media/radio/radio-maxiradio.c              | 2 +-
 18 files changed, 18 insertions(+), 18 deletions(-)

-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 01/18] [media] marvell-ccic: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 02/18] [media] netup_unidvb: " Arvind Yadav
                   ` (16 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/platform/marvell-ccic/cafe-driver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/marvell-ccic/cafe-driver.c b/drivers/media/platform/marvell-ccic/cafe-driver.c
index 77890bd..b1ad054 100644
--- a/drivers/media/platform/marvell-ccic/cafe-driver.c
+++ b/drivers/media/platform/marvell-ccic/cafe-driver.c
@@ -612,7 +612,7 @@ static int cafe_pci_resume(struct pci_dev *pdev)
 
 #endif  /* CONFIG_PM */
 
-static struct pci_device_id cafe_ids[] = {
+static const struct pci_device_id cafe_ids[] = {
 	{ PCI_DEVICE(PCI_VENDOR_ID_MARVELL,
 		     PCI_DEVICE_ID_MARVELL_88ALP01_CCIC) },
 	{ 0, }
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 02/18] [media] netup_unidvb: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
  2017-08-01 17:56 ` [PATCH 01/18] [media] marvell-ccic: constify pci_device_id Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 03/18] [media] cx23885: " Arvind Yadav
                   ` (15 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/netup_unidvb/netup_unidvb_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/netup_unidvb/netup_unidvb_core.c b/drivers/media/pci/netup_unidvb/netup_unidvb_core.c
index 5c0a4e6..60e6cd5 100644
--- a/drivers/media/pci/netup_unidvb/netup_unidvb_core.c
+++ b/drivers/media/pci/netup_unidvb/netup_unidvb_core.c
@@ -1014,7 +1014,7 @@ static void netup_unidvb_finidev(struct pci_dev *pci_dev)
 }
 
 
-static struct pci_device_id netup_unidvb_pci_tbl[] = {
+static const struct pci_device_id netup_unidvb_pci_tbl[] = {
 	{ PCI_DEVICE(0x1b55, 0x18f6) }, /* hw rev. 1.3 */
 	{ PCI_DEVICE(0x1b55, 0x18f7) }, /* hw rev. 1.4 */
 	{ 0, }
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 03/18] [media] cx23885: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
  2017-08-01 17:56 ` [PATCH 01/18] [media] marvell-ccic: constify pci_device_id Arvind Yadav
  2017-08-01 17:56 ` [PATCH 02/18] [media] netup_unidvb: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 04/18] [media] meye: " Arvind Yadav
                   ` (14 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/cx23885/cx23885-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/cx23885/cx23885-core.c b/drivers/media/pci/cx23885/cx23885-core.c
index 02b5ec5..8f63df1 100644
--- a/drivers/media/pci/cx23885/cx23885-core.c
+++ b/drivers/media/pci/cx23885/cx23885-core.c
@@ -2056,7 +2056,7 @@ static void cx23885_finidev(struct pci_dev *pci_dev)
 	kfree(dev);
 }
 
-static struct pci_device_id cx23885_pci_tbl[] = {
+static const struct pci_device_id cx23885_pci_tbl[] = {
 	{
 		/* CX23885 */
 		.vendor       = 0x14f1,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 04/18] [media] meye: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (2 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 03/18] [media] cx23885: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 05/18] [media] pluto2: " Arvind Yadav
                   ` (13 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 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 9c4a024..0fe76be 100644
--- a/drivers/media/pci/meye/meye.c
+++ b/drivers/media/pci/meye/meye.c
@@ -1801,7 +1801,7 @@ static void meye_remove(struct pci_dev *pcidev)
 	printk(KERN_INFO "meye: removed\n");
 }
 
-static struct pci_device_id meye_pci_tbl[] = {
+static const struct pci_device_id meye_pci_tbl[] = {
 	{ PCI_VDEVICE(KAWASAKI, PCI_DEVICE_ID_MCHIP_KL5A72002), 0 },
 	{ }
 };
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 05/18] [media] pluto2: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (3 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 04/18] [media] meye: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 06/18] [media] dm1105: " Arvind Yadav
                   ` (12 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/pluto2/pluto2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/pluto2/pluto2.c b/drivers/media/pci/pluto2/pluto2.c
index 7483810..39dcba2 100644
--- a/drivers/media/pci/pluto2/pluto2.c
+++ b/drivers/media/pci/pluto2/pluto2.c
@@ -770,7 +770,7 @@ static void pluto2_remove(struct pci_dev *pdev)
 #define PCI_DEVICE_ID_PLUTO2	0x0001
 #endif
 
-static struct pci_device_id pluto2_id_table[] = {
+static const struct pci_device_id pluto2_id_table[] = {
 	{
 		.vendor = PCI_VENDOR_ID_SCM,
 		.device = PCI_DEVICE_ID_PLUTO2,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 06/18] [media] dm1105: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (4 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 05/18] [media] pluto2: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 07/18] [media] zoran: " Arvind Yadav
                   ` (11 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/dm1105/dm1105.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/dm1105/dm1105.c b/drivers/media/pci/dm1105/dm1105.c
index 1d41934..0cd854f 100644
--- a/drivers/media/pci/dm1105/dm1105.c
+++ b/drivers/media/pci/dm1105/dm1105.c
@@ -1208,7 +1208,7 @@ static void dm1105_remove(struct pci_dev *pdev)
 	kfree(dev);
 }
 
-static struct pci_device_id dm1105_id_table[] = {
+static const struct pci_device_id dm1105_id_table[] = {
 	{
 		.vendor = PCI_VENDOR_ID_TRIGEM,
 		.device = PCI_DEVICE_ID_DM1105,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 07/18] [media] zoran: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (5 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 06/18] [media] dm1105: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 08/18] [media] bt8xx: " Arvind Yadav
                   ` (10 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/zoran/zoran_card.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/zoran/zoran_card.c b/drivers/media/pci/zoran/zoran_card.c
index 4680f00..a6b9ebd 100644
--- a/drivers/media/pci/zoran/zoran_card.c
+++ b/drivers/media/pci/zoran/zoran_card.c
@@ -130,7 +130,7 @@ MODULE_VERSION(ZORAN_VERSION);
 	.vendor = PCI_VENDOR_ID_ZORAN, .device = PCI_DEVICE_ID_ZORAN_36057, \
 	.subvendor = (subven), .subdevice = (subdev), .driver_data = (data) }
 
-static struct pci_device_id zr36067_pci_tbl[] = {
+static const struct pci_device_id zr36067_pci_tbl[] = {
 	ZR_DEVICE(PCI_VENDOR_ID_MIRO, PCI_DEVICE_ID_MIRO_DC10PLUS, DC10plus),
 	ZR_DEVICE(PCI_VENDOR_ID_MIRO, PCI_DEVICE_ID_MIRO_DC30PLUS, DC30plus),
 	ZR_DEVICE(PCI_VENDOR_ID_ELECTRONICDESIGNGMBH, PCI_DEVICE_ID_LML_33R10, LML33R10),
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 08/18] [media] bt8xx: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (6 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 07/18] [media] zoran: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 09/18] [media] bt8xx: bttv: " Arvind Yadav
                   ` (9 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/bt8xx/bt878.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/bt8xx/bt878.c b/drivers/media/pci/bt8xx/bt878.c
index 8aa7266..a5f5213 100644
--- a/drivers/media/pci/bt8xx/bt878.c
+++ b/drivers/media/pci/bt8xx/bt878.c
@@ -383,7 +383,7 @@ EXPORT_SYMBOL(bt878_device_control);
 		.driver_data = (unsigned long) name \
 	}
 
-static struct pci_device_id bt878_pci_tbl[] = {
+static const struct pci_device_id bt878_pci_tbl[] = {
 	BROOKTREE_878_DEVICE(0x0071, 0x0101, "Nebula Electronics DigiTV"),
 	BROOKTREE_878_DEVICE(0x1461, 0x0761, "AverMedia AverTV DVB-T 761"),
 	BROOKTREE_878_DEVICE(0x11bd, 0x001c, "Pinnacle PCTV Sat"),
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 09/18] [media] bt8xx: bttv: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (7 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 08/18] [media] bt8xx: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 10/18] [media] ivtv: " Arvind Yadav
                   ` (8 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/bt8xx/bttv-driver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/bt8xx/bttv-driver.c b/drivers/media/pci/bt8xx/bttv-driver.c
index ed319f1..40110be 100644
--- a/drivers/media/pci/bt8xx/bttv-driver.c
+++ b/drivers/media/pci/bt8xx/bttv-driver.c
@@ -4388,7 +4388,7 @@ static int bttv_resume(struct pci_dev *pci_dev)
 }
 #endif
 
-static struct pci_device_id bttv_pci_tbl[] = {
+static const struct pci_device_id bttv_pci_tbl[] = {
 	{PCI_VDEVICE(BROOKTREE, PCI_DEVICE_ID_BT848), 0},
 	{PCI_VDEVICE(BROOKTREE, PCI_DEVICE_ID_BT849), 0},
 	{PCI_VDEVICE(BROOKTREE, PCI_DEVICE_ID_BT878), 0},
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 10/18] [media] ivtv: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (8 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 09/18] [media] bt8xx: bttv: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 11/18] [media] cobalt: " Arvind Yadav
                   ` (7 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/ivtv/ivtv-driver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/ivtv/ivtv-driver.c b/drivers/media/pci/ivtv/ivtv-driver.c
index e8fa99b..54dcac4 100644
--- a/drivers/media/pci/ivtv/ivtv-driver.c
+++ b/drivers/media/pci/ivtv/ivtv-driver.c
@@ -73,7 +73,7 @@ int (*ivtv_ext_init)(struct ivtv *);
 EXPORT_SYMBOL(ivtv_ext_init);
 
 /* add your revision and whatnot here */
-static struct pci_device_id ivtv_pci_tbl[] = {
+static const struct pci_device_id ivtv_pci_tbl[] = {
 	{PCI_VENDOR_ID_ICOMP, PCI_DEVICE_ID_IVTV15,
 	 PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
 	{PCI_VENDOR_ID_ICOMP, PCI_DEVICE_ID_IVTV16,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 11/18] [media] cobalt: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (9 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 10/18] [media] ivtv: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 12/18] [media] b2c2: " Arvind Yadav
                   ` (6 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/cobalt/cobalt-driver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/cobalt/cobalt-driver.c b/drivers/media/pci/cobalt/cobalt-driver.c
index f8e173f..98b6cb9 100644
--- a/drivers/media/pci/cobalt/cobalt-driver.c
+++ b/drivers/media/pci/cobalt/cobalt-driver.c
@@ -36,7 +36,7 @@
 #include "cobalt-omnitek.h"
 
 /* add your revision and whatnot here */
-static struct pci_device_id cobalt_pci_tbl[] = {
+static const struct pci_device_id cobalt_pci_tbl[] = {
 	{PCI_VENDOR_ID_CISCO, PCI_DEVICE_ID_COBALT,
 	 PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
 	{0,}
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 12/18] [media] b2c2: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (10 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 11/18] [media] cobalt: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 13/18] [media] saa7164: " Arvind Yadav
                   ` (5 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/b2c2/flexcop-pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/b2c2/flexcop-pci.c b/drivers/media/pci/b2c2/flexcop-pci.c
index 6e60dec..cc6527e 100644
--- a/drivers/media/pci/b2c2/flexcop-pci.c
+++ b/drivers/media/pci/b2c2/flexcop-pci.c
@@ -415,7 +415,7 @@ static void flexcop_pci_remove(struct pci_dev *pdev)
 	flexcop_device_kfree(fc_pci->fc_dev);
 }
 
-static struct pci_device_id flexcop_pci_tbl[] = {
+static const struct pci_device_id flexcop_pci_tbl[] = {
 	{ PCI_DEVICE(0x13d0, 0x2103) },
 	{ },
 };
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 13/18] [media] saa7164: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (11 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 12/18] [media] b2c2: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 14/18] [media] pt1: " Arvind Yadav
                   ` (4 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/saa7164/saa7164-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/saa7164/saa7164-core.c b/drivers/media/pci/saa7164/saa7164-core.c
index 75eed4c..fca36a4 100644
--- a/drivers/media/pci/saa7164/saa7164-core.c
+++ b/drivers/media/pci/saa7164/saa7164-core.c
@@ -1490,7 +1490,7 @@ static void saa7164_finidev(struct pci_dev *pci_dev)
 	kfree(dev);
 }
 
-static struct pci_device_id saa7164_pci_tbl[] = {
+static const struct pci_device_id saa7164_pci_tbl[] = {
 	{
 		/* SAA7164 */
 		.vendor       = 0x1131,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 14/18] [media] pt1: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (12 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 13/18] [media] saa7164: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 15/18] [media] mantis: " Arvind Yadav
                   ` (3 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/pt1/pt1.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/pt1/pt1.c b/drivers/media/pci/pt1/pt1.c
index 3219d2f..b6b1a8d 100644
--- a/drivers/media/pci/pt1/pt1.c
+++ b/drivers/media/pci/pt1/pt1.c
@@ -1202,7 +1202,7 @@ static int pt1_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 
 }
 
-static struct pci_device_id pt1_id_table[] = {
+static const struct pci_device_id pt1_id_table[] = {
 	{ PCI_DEVICE(0x10ee, 0x211a) },
 	{ PCI_DEVICE(0x10ee, 0x222a) },
 	{ },
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 15/18] [media] mantis: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (13 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 14/18] [media] pt1: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 16/18] [media] mantis: hopper_cards: " Arvind Yadav
                   ` (2 subsequent siblings)
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/mantis/mantis_cards.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/mantis/mantis_cards.c b/drivers/media/pci/mantis/mantis_cards.c
index cdefffc..adc980d 100644
--- a/drivers/media/pci/mantis/mantis_cards.c
+++ b/drivers/media/pci/mantis/mantis_cards.c
@@ -281,7 +281,7 @@ static void mantis_pci_remove(struct pci_dev *pdev)
 	return;
 }
 
-static struct pci_device_id mantis_pci_table[] = {
+static const struct pci_device_id mantis_pci_table[] = {
 	MAKE_ENTRY(TECHNISAT, CABLESTAR_HD2, &vp2040_config,
 		   RC_MAP_TECHNISAT_TS35),
 	MAKE_ENTRY(TECHNISAT, SKYSTAR_HD2_10, &vp1041_config,
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 16/18] [media] mantis: hopper_cards: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (14 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 15/18] [media] mantis: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 17/18] [media] cx18: " Arvind Yadav
  2017-08-01 17:56 ` [PATCH 18/18] [media] radio: " Arvind Yadav
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/mantis/hopper_cards.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/mantis/hopper_cards.c b/drivers/media/pci/mantis/hopper_cards.c
index 68b5800..11e9878 100644
--- a/drivers/media/pci/mantis/hopper_cards.c
+++ b/drivers/media/pci/mantis/hopper_cards.c
@@ -255,7 +255,7 @@ static void hopper_pci_remove(struct pci_dev *pdev)
 
 }
 
-static struct pci_device_id hopper_pci_table[] = {
+static const struct pci_device_id hopper_pci_table[] = {
 	MAKE_ENTRY(TWINHAN_TECHNOLOGIES, MANTIS_VP_3028_DVB_T, &vp3028_config,
 		   NULL),
 	{ }
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 17/18] [media] cx18: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (15 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 16/18] [media] mantis: hopper_cards: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  2017-08-01 17:56 ` [PATCH 18/18] [media] radio: " Arvind Yadav
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/pci/cx18/cx18-driver.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/pci/cx18/cx18-driver.c b/drivers/media/pci/cx18/cx18-driver.c
index 8bce49c..8654710 100644
--- a/drivers/media/pci/cx18/cx18-driver.c
+++ b/drivers/media/pci/cx18/cx18-driver.c
@@ -48,7 +48,7 @@ int (*cx18_ext_init)(struct cx18 *);
 EXPORT_SYMBOL(cx18_ext_init);
 
 /* add your revision and whatnot here */
-static struct pci_device_id cx18_pci_tbl[] = {
+static const struct pci_device_id cx18_pci_tbl[] = {
 	{PCI_VENDOR_ID_CX, PCI_DEVICE_ID_CX23418,
 	 PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
 	{0,}
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 18/18] [media] radio: constify pci_device_id.
  2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
                   ` (16 preceding siblings ...)
  2017-08-01 17:56 ` [PATCH 17/18] [media] cx18: " Arvind Yadav
@ 2017-08-01 17:56 ` Arvind Yadav
  17 siblings, 0 replies; 19+ messages in thread
From: Arvind Yadav @ 2017-08-01 17:56 UTC (permalink / raw)
  To: corbet, mchehab, awalls, hverkuil, serjk, aospan, hans.verkuil
  Cc: linux-media, linux-kernel

pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <linux/pci.h> work with
const pci_device_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/media/radio/radio-maxiradio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/radio/radio-maxiradio.c b/drivers/media/radio/radio-maxiradio.c
index 8253f79..3aa5ad3 100644
--- a/drivers/media/radio/radio-maxiradio.c
+++ b/drivers/media/radio/radio-maxiradio.c
@@ -186,7 +186,7 @@ static void maxiradio_remove(struct pci_dev *pdev)
 	kfree(dev);
 }
 
-static struct pci_device_id maxiradio_pci_tbl[] = {
+static const struct pci_device_id maxiradio_pci_tbl[] = {
 	{ PCI_VENDOR_ID_GUILLEMOT, PCI_DEVICE_ID_GUILLEMOT_MAXIRADIO,
 		PCI_ANY_ID, PCI_ANY_ID, },
 	{ 0 }
-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2017-08-01 18:02 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-01 17:56 [PATCH 00/18] constify media pci_device_id Arvind Yadav
2017-08-01 17:56 ` [PATCH 01/18] [media] marvell-ccic: constify pci_device_id Arvind Yadav
2017-08-01 17:56 ` [PATCH 02/18] [media] netup_unidvb: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 03/18] [media] cx23885: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 04/18] [media] meye: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 05/18] [media] pluto2: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 06/18] [media] dm1105: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 07/18] [media] zoran: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 08/18] [media] bt8xx: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 09/18] [media] bt8xx: bttv: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 10/18] [media] ivtv: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 11/18] [media] cobalt: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 12/18] [media] b2c2: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 13/18] [media] saa7164: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 14/18] [media] pt1: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 15/18] [media] mantis: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 16/18] [media] mantis: hopper_cards: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 17/18] [media] cx18: " Arvind Yadav
2017-08-01 17:56 ` [PATCH 18/18] [media] radio: " Arvind Yadav

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®