* [PATCH 0/9] constify media pci_device_id/pci_tbl.
@ 2017-08-02 17:14 Arvind Yadav
2017-08-02 17:14 ` [PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id Arvind Yadav
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
SAA7146 DVD card base pci device id const.
Arvind Yadav (9):
[PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id.
[PATCH 2/9] [media] ttpci: budget: constify pci_device_id.
[PATCH 3/9] [media] ttpci: budget-patch: constify pci_device_id.
[PATCH 4/9] [media] ttpci: budget-ci: constify pci_device_id.
[PATCH 5/9] [media] ttpci: budget-av: constify pci_device_id.
[PATCH 6/9] [media] ttpci: av7110: constify pci_device_id.
[PATCH 7/9] [media] saa7146: mxb: constify pci_device_id.
[PATCH 8/9] [media] saa7146: hexium_orion: constify pci_device_id.
[PATCH 9/9] [media] saa7146: hexium_gemini: constify pci_device_id.
drivers/media/pci/saa7146/hexium_gemini.c | 2 +-
drivers/media/pci/saa7146/hexium_orion.c | 2 +-
drivers/media/pci/saa7146/mxb.c | 2 +-
drivers/media/pci/ttpci/av7110.c | 2 +-
drivers/media/pci/ttpci/budget-av.c | 2 +-
drivers/media/pci/ttpci/budget-ci.c | 2 +-
drivers/media/pci/ttpci/budget-patch.c | 2 +-
drivers/media/pci/ttpci/budget.c | 2 +-
include/media/drv-intf/saa7146.h | 2 +-
9 files changed, 9 insertions(+), 9 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 2/9] [media] ttpci: budget: " Arvind Yadav
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +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.
So making 'pci_tbl' as const member of 'struct saa7146_extension'.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
include/media/drv-intf/saa7146.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/media/drv-intf/saa7146.h b/include/media/drv-intf/saa7146.h
index 96058a5..4529432 100644
--- a/include/media/drv-intf/saa7146.h
+++ b/include/media/drv-intf/saa7146.h
@@ -96,7 +96,7 @@ struct saa7146_extension
supported devices, last entry 0xffff, 0xfff */
struct module *module;
struct pci_driver driver;
- struct pci_device_id *pci_tbl;
+ const struct pci_device_id *pci_tbl;
/* extension functions */
int (*probe)(struct saa7146_dev *);
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/9] [media] ttpci: budget: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
2017-08-02 17:14 ` [PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 3/9] [media] ttpci: budget-patch: " Arvind Yadav
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/ttpci/budget.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ttpci/budget.c b/drivers/media/pci/ttpci/budget.c
index 81fe35c..f59eadb 100644
--- a/drivers/media/pci/ttpci/budget.c
+++ b/drivers/media/pci/ttpci/budget.c
@@ -845,7 +845,7 @@ MAKE_BUDGET_INFO(fsact1, "Fujitsu Siemens Activy Budget-T PCI (rev AL/ALPS TDHD1
MAKE_BUDGET_INFO(omicom, "Omicom S2 PCI", BUDGET_TT);
MAKE_BUDGET_INFO(sylt, "Philips Semi Sylt PCI", BUDGET_TT_HW_DISEQC);
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
MAKE_EXTENSION_PCI(ttbs, 0x13c2, 0x1003),
MAKE_EXTENSION_PCI(ttbc, 0x13c2, 0x1004),
MAKE_EXTENSION_PCI(ttbt, 0x13c2, 0x1005),
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/9] [media] ttpci: budget-patch: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
2017-08-02 17:14 ` [PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id Arvind Yadav
2017-08-02 17:14 ` [PATCH 2/9] [media] ttpci: budget: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 4/9] [media] ttpci: budget-ci: " Arvind Yadav
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/ttpci/budget-patch.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ttpci/budget-patch.c b/drivers/media/pci/ttpci/budget-patch.c
index 4429923..a738018 100644
--- a/drivers/media/pci/ttpci/budget-patch.c
+++ b/drivers/media/pci/ttpci/budget-patch.c
@@ -45,7 +45,7 @@ static struct saa7146_extension budget_extension;
MAKE_BUDGET_INFO(ttbp, "TT-Budget/Patch DVB-S 1.x PCI", BUDGET_PATCH);
//MAKE_BUDGET_INFO(satel,"TT-Budget/Patch SATELCO PCI", BUDGET_TT_HW_DISEQC);
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
MAKE_EXTENSION_PCI(ttbp,0x13c2, 0x0000),
// MAKE_EXTENSION_PCI(satel, 0x13c2, 0x1013),
{
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/9] [media] ttpci: budget-ci: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
` (2 preceding siblings ...)
2017-08-02 17:14 ` [PATCH 3/9] [media] ttpci: budget-patch: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 5/9] [media] ttpci: budget-av: " Arvind Yadav
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/ttpci/budget-ci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ttpci/budget-ci.c b/drivers/media/pci/ttpci/budget-ci.c
index 11b9227..5b8aab4 100644
--- a/drivers/media/pci/ttpci/budget-ci.c
+++ b/drivers/media/pci/ttpci/budget-ci.c
@@ -1538,7 +1538,7 @@ MAKE_BUDGET_INFO(ttc1501, "TT-Budget C-1501 PCI", BUDGET_TT);
MAKE_BUDGET_INFO(tt3200, "TT-Budget S2-3200 PCI", BUDGET_TT);
MAKE_BUDGET_INFO(ttbs1500b, "TT-Budget S-1500B PCI", BUDGET_TT);
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
MAKE_EXTENSION_PCI(ttbci, 0x13c2, 0x100c),
MAKE_EXTENSION_PCI(ttbci, 0x13c2, 0x100f),
MAKE_EXTENSION_PCI(ttbcci, 0x13c2, 0x1010),
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 5/9] [media] ttpci: budget-av: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
` (3 preceding siblings ...)
2017-08-02 17:14 ` [PATCH 4/9] [media] ttpci: budget-ci: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 6/9] [media] ttpci: av7110: " Arvind Yadav
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/ttpci/budget-av.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ttpci/budget-av.c b/drivers/media/pci/ttpci/budget-av.c
index dc7be8f..ac83fff 100644
--- a/drivers/media/pci/ttpci/budget-av.c
+++ b/drivers/media/pci/ttpci/budget-av.c
@@ -1567,7 +1567,7 @@ MAKE_BUDGET_INFO(cin1200c, "Terratec Cinergy 1200 DVB-C", BUDGET_CIN1200C);
MAKE_BUDGET_INFO(cin1200cmk3, "Terratec Cinergy 1200 DVB-C MK3", BUDGET_CIN1200C_MK3);
MAKE_BUDGET_INFO(cin1200t, "Terratec Cinergy 1200 DVB-T", BUDGET_CIN1200T);
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
MAKE_EXTENSION_PCI(knc1s, 0x1131, 0x4f56),
MAKE_EXTENSION_PCI(knc1s, 0x1131, 0x0010),
MAKE_EXTENSION_PCI(knc1s, 0x1894, 0x0010),
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 6/9] [media] ttpci: av7110: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
` (4 preceding siblings ...)
2017-08-02 17:14 ` [PATCH 5/9] [media] ttpci: budget-av: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 7/9] [media] saa7146: mxb: " Arvind Yadav
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/ttpci/av7110.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ttpci/av7110.c b/drivers/media/pci/ttpci/av7110.c
index f2905bd..f46947d 100644
--- a/drivers/media/pci/ttpci/av7110.c
+++ b/drivers/media/pci/ttpci/av7110.c
@@ -2872,7 +2872,7 @@ MAKE_AV7110_INFO(fsc, "Fujitsu Siemens DVB-C");
MAKE_AV7110_INFO(fss, "Fujitsu Siemens DVB-S rev1.6");
MAKE_AV7110_INFO(gxs_1_3, "Galaxis DVB-S rev1.3");
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
MAKE_EXTENSION_PCI(fsc, 0x110a, 0x0000),
MAKE_EXTENSION_PCI(tts_1_X_fsc, 0x13c2, 0x0000),
MAKE_EXTENSION_PCI(ttt_1_X, 0x13c2, 0x0001),
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 7/9] [media] saa7146: mxb: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
` (5 preceding siblings ...)
2017-08-02 17:14 ` [PATCH 6/9] [media] ttpci: av7110: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 8/9] [media] saa7146: hexium_orion: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 9/9] [media] saa7146: hexium_gemini: " Arvind Yadav
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/saa7146/mxb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/saa7146/mxb.c b/drivers/media/pci/saa7146/mxb.c
index 504d788..930218c 100644
--- a/drivers/media/pci/saa7146/mxb.c
+++ b/drivers/media/pci/saa7146/mxb.c
@@ -819,7 +819,7 @@ static struct saa7146_pci_extension_data mxb = {
.ext = &extension,
};
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
{
.vendor = PCI_VENDOR_ID_PHILIPS,
.device = PCI_DEVICE_ID_PHILIPS_SAA7146,
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 8/9] [media] saa7146: hexium_orion: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
` (6 preceding siblings ...)
2017-08-02 17:14 ` [PATCH 7/9] [media] saa7146: mxb: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
2017-08-02 17:14 ` [PATCH 9/9] [media] saa7146: hexium_gemini: " Arvind Yadav
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/saa7146/hexium_orion.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/saa7146/hexium_orion.c b/drivers/media/pci/saa7146/hexium_orion.c
index c306a92..01f0158 100644
--- a/drivers/media/pci/saa7146/hexium_orion.c
+++ b/drivers/media/pci/saa7146/hexium_orion.c
@@ -427,7 +427,7 @@ static struct saa7146_pci_extension_data hexium_orion_4bnc = {
.ext = &extension,
};
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
{
.vendor = PCI_VENDOR_ID_PHILIPS,
.device = PCI_DEVICE_ID_PHILIPS_SAA7146,
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 9/9] [media] saa7146: hexium_gemini: constify pci_device_id.
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
` (7 preceding siblings ...)
2017-08-02 17:14 ` [PATCH 8/9] [media] saa7146: hexium_orion: " Arvind Yadav
@ 2017-08-02 17:14 ` Arvind Yadav
8 siblings, 0 replies; 10+ messages in thread
From: Arvind Yadav @ 2017-08-02 17:14 UTC (permalink / raw)
To: mchehab, sakari.ailus, hverkuil; +Cc: linux-media, linux-kernel
pci_device_id are not supposed to change at runtime. All functions
working with pci_device_id provided by <media/drv-intf/saa7146.h>
and <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/saa7146/hexium_gemini.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/saa7146/hexium_gemini.c b/drivers/media/pci/saa7146/hexium_gemini.c
index c889ec9..f708cab 100644
--- a/drivers/media/pci/saa7146/hexium_gemini.c
+++ b/drivers/media/pci/saa7146/hexium_gemini.c
@@ -363,7 +363,7 @@ static struct saa7146_pci_extension_data hexium_gemini_dual_4bnc = {
.ext = &hexium_extension,
};
-static struct pci_device_id pci_tbl[] = {
+static const struct pci_device_id pci_tbl[] = {
{
.vendor = PCI_VENDOR_ID_PHILIPS,
.device = PCI_DEVICE_ID_PHILIPS_SAA7146,
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-08-02 17:17 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-02 17:14 [PATCH 0/9] constify media pci_device_id/pci_tbl Arvind Yadav
2017-08-02 17:14 ` [PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id Arvind Yadav
2017-08-02 17:14 ` [PATCH 2/9] [media] ttpci: budget: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 3/9] [media] ttpci: budget-patch: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 4/9] [media] ttpci: budget-ci: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 5/9] [media] ttpci: budget-av: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 6/9] [media] ttpci: av7110: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 7/9] [media] saa7146: mxb: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 8/9] [media] saa7146: hexium_orion: " Arvind Yadav
2017-08-02 17:14 ` [PATCH 9/9] [media] saa7146: hexium_gemini: " Arvind Yadav
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).