* [PATCH 1/6] mtd: ichxrom: constify pci_device_id.
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
@ 2017-08-03 16:22 ` Arvind Yadav
2017-08-03 16:22 ` [PATCH 2/6] mtd: amd76xrom: " Arvind Yadav
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Arvind Yadav @ 2017-08-03 16:22 UTC (permalink / raw)
To: dwmw2, computersforpeace, boris.brezillon, marek.vasut, richard,
cyrille.pitchen
Cc: linux-mtd, 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/mtd/maps/ichxrom.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/maps/ichxrom.c b/drivers/mtd/maps/ichxrom.c
index 976d42f..1888c5b 100644
--- a/drivers/mtd/maps/ichxrom.c
+++ b/drivers/mtd/maps/ichxrom.c
@@ -323,7 +323,7 @@ static void ichxrom_remove_one(struct pci_dev *pdev)
ichxrom_cleanup(window);
}
-static struct pci_device_id ichxrom_pci_tbl[] = {
+static const struct pci_device_id ichxrom_pci_tbl[] = {
{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_82801BA_0,
PCI_ANY_ID, PCI_ANY_ID, },
{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_82801CA_0,
@@ -351,7 +351,7 @@ static struct pci_driver ichxrom_driver = {
static int __init init_ichxrom(void)
{
struct pci_dev *pdev;
- struct pci_device_id *id;
+ const struct pci_device_id *id;
pdev = NULL;
for (id = ichxrom_pci_tbl; id->vendor; id++) {
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 2/6] mtd: amd76xrom: constify pci_device_id.
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
2017-08-03 16:22 ` [PATCH 1/6] mtd: ichxrom: constify pci_device_id Arvind Yadav
@ 2017-08-03 16:22 ` Arvind Yadav
2017-08-03 16:22 ` [PATCH 3/6] mtd: esb2rom: " Arvind Yadav
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Arvind Yadav @ 2017-08-03 16:22 UTC (permalink / raw)
To: dwmw2, computersforpeace, boris.brezillon, marek.vasut, richard,
cyrille.pitchen
Cc: linux-mtd, 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/mtd/maps/amd76xrom.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/maps/amd76xrom.c b/drivers/mtd/maps/amd76xrom.c
index f2b6866..26de0a1 100644
--- a/drivers/mtd/maps/amd76xrom.c
+++ b/drivers/mtd/maps/amd76xrom.c
@@ -296,7 +296,7 @@ static void amd76xrom_remove_one(struct pci_dev *pdev)
amd76xrom_cleanup(window);
}
-static struct pci_device_id amd76xrom_pci_tbl[] = {
+static const struct pci_device_id amd76xrom_pci_tbl[] = {
{ PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_VIPER_7410,
PCI_ANY_ID, PCI_ANY_ID, },
{ PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_VIPER_7440,
@@ -319,7 +319,7 @@ static struct pci_driver amd76xrom_driver = {
static int __init init_amd76xrom(void)
{
struct pci_dev *pdev;
- struct pci_device_id *id;
+ const struct pci_device_id *id;
pdev = NULL;
for(id = amd76xrom_pci_tbl; id->vendor; id++) {
pdev = pci_get_device(id->vendor, id->device, NULL);
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 3/6] mtd: esb2rom: constify pci_device_id.
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
2017-08-03 16:22 ` [PATCH 1/6] mtd: ichxrom: constify pci_device_id Arvind Yadav
2017-08-03 16:22 ` [PATCH 2/6] mtd: amd76xrom: " Arvind Yadav
@ 2017-08-03 16:22 ` Arvind Yadav
2017-08-03 16:22 ` [PATCH 4/6] mtd: ck804xrom: " Arvind Yadav
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Arvind Yadav @ 2017-08-03 16:22 UTC (permalink / raw)
To: dwmw2, computersforpeace, boris.brezillon, marek.vasut, richard,
cyrille.pitchen
Cc: linux-mtd, 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/mtd/maps/esb2rom.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/maps/esb2rom.c b/drivers/mtd/maps/esb2rom.c
index 9646b07..da9f6d7 100644
--- a/drivers/mtd/maps/esb2rom.c
+++ b/drivers/mtd/maps/esb2rom.c
@@ -384,7 +384,7 @@ static void esb2rom_remove_one(struct pci_dev *pdev)
esb2rom_cleanup(window);
}
-static struct pci_device_id esb2rom_pci_tbl[] = {
+static const struct pci_device_id esb2rom_pci_tbl[] = {
{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_82801BA_0,
PCI_ANY_ID, PCI_ANY_ID, },
{ PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_82801CA_0,
@@ -414,7 +414,7 @@ static struct pci_driver esb2rom_driver = {
static int __init init_esb2rom(void)
{
struct pci_dev *pdev;
- struct pci_device_id *id;
+ const struct pci_device_id *id;
int retVal;
pdev = NULL;
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 4/6] mtd: ck804xrom: constify pci_device_id.
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
` (2 preceding siblings ...)
2017-08-03 16:22 ` [PATCH 3/6] mtd: esb2rom: " Arvind Yadav
@ 2017-08-03 16:22 ` Arvind Yadav
2017-08-03 16:22 ` [PATCH 5/6] mtd: intel_vr_nor: " Arvind Yadav
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Arvind Yadav @ 2017-08-03 16:22 UTC (permalink / raw)
To: dwmw2, computersforpeace, boris.brezillon, marek.vasut, richard,
cyrille.pitchen
Cc: linux-mtd, 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/mtd/maps/ck804xrom.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/maps/ck804xrom.c b/drivers/mtd/maps/ck804xrom.c
index 4f206a9..584962e 100644
--- a/drivers/mtd/maps/ck804xrom.c
+++ b/drivers/mtd/maps/ck804xrom.c
@@ -326,7 +326,7 @@ static void ck804xrom_remove_one(struct pci_dev *pdev)
ck804xrom_cleanup(window);
}
-static struct pci_device_id ck804xrom_pci_tbl[] = {
+static const struct pci_device_id ck804xrom_pci_tbl[] = {
{ PCI_DEVICE(PCI_VENDOR_ID_NVIDIA, 0x0051), .driver_data = DEV_CK804 },
{ PCI_DEVICE(PCI_VENDOR_ID_NVIDIA, 0x0360), .driver_data = DEV_MCP55 },
{ PCI_DEVICE(PCI_VENDOR_ID_NVIDIA, 0x0361), .driver_data = DEV_MCP55 },
@@ -353,7 +353,7 @@ static struct pci_driver ck804xrom_driver = {
static int __init init_ck804xrom(void)
{
struct pci_dev *pdev;
- struct pci_device_id *id;
+ const struct pci_device_id *id;
int retVal;
pdev = NULL;
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 5/6] mtd: intel_vr_nor: constify pci_device_id.
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
` (3 preceding siblings ...)
2017-08-03 16:22 ` [PATCH 4/6] mtd: ck804xrom: " Arvind Yadav
@ 2017-08-03 16:22 ` Arvind Yadav
2017-08-03 16:22 ` [PATCH 6/6] mtd: pci: " Arvind Yadav
2017-08-17 13:02 ` [PATCH 0/6] constify mtd pci_device_id Boris Brezillon
6 siblings, 0 replies; 8+ messages in thread
From: Arvind Yadav @ 2017-08-03 16:22 UTC (permalink / raw)
To: dwmw2, computersforpeace, boris.brezillon, marek.vasut, richard,
cyrille.pitchen
Cc: linux-mtd, 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/mtd/maps/intel_vr_nor.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/maps/intel_vr_nor.c b/drivers/mtd/maps/intel_vr_nor.c
index 8bf7977..dd5d685 100644
--- a/drivers/mtd/maps/intel_vr_nor.c
+++ b/drivers/mtd/maps/intel_vr_nor.c
@@ -170,7 +170,7 @@ static int vr_nor_init_maps(struct vr_nor_mtd *p)
return err;
}
-static struct pci_device_id vr_nor_pci_ids[] = {
+static const struct pci_device_id vr_nor_pci_ids[] = {
{PCI_DEVICE(PCI_VENDOR_ID_INTEL, 0x500D)},
{0,}
};
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 6/6] mtd: pci: constify pci_device_id.
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
` (4 preceding siblings ...)
2017-08-03 16:22 ` [PATCH 5/6] mtd: intel_vr_nor: " Arvind Yadav
@ 2017-08-03 16:22 ` Arvind Yadav
2017-08-17 13:02 ` [PATCH 0/6] constify mtd pci_device_id Boris Brezillon
6 siblings, 0 replies; 8+ messages in thread
From: Arvind Yadav @ 2017-08-03 16:22 UTC (permalink / raw)
To: dwmw2, computersforpeace, boris.brezillon, marek.vasut, richard,
cyrille.pitchen
Cc: linux-mtd, 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/mtd/maps/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/maps/pci.c b/drivers/mtd/maps/pci.c
index eb0242e..7b3bb40 100644
--- a/drivers/mtd/maps/pci.c
+++ b/drivers/mtd/maps/pci.c
@@ -228,7 +228,7 @@ static struct mtd_pci_info intel_dc21285_info = {
* PCI device ID table
*/
-static struct pci_device_id mtd_pci_ids[] = {
+static const struct pci_device_id mtd_pci_ids[] = {
{
.vendor = PCI_VENDOR_ID_INTEL,
.device = 0x530d,
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 0/6] constify mtd pci_device_id
2017-08-03 16:22 [PATCH 0/6] constify mtd pci_device_id Arvind Yadav
` (5 preceding siblings ...)
2017-08-03 16:22 ` [PATCH 6/6] mtd: pci: " Arvind Yadav
@ 2017-08-17 13:02 ` Boris Brezillon
6 siblings, 0 replies; 8+ messages in thread
From: Boris Brezillon @ 2017-08-17 13:02 UTC (permalink / raw)
To: Arvind Yadav
Cc: dwmw2, computersforpeace, marek.vasut, richard, cyrille.pitchen,
linux-mtd, linux-kernel
Le Thu, 3 Aug 2017 21:52:04 +0530,
Arvind Yadav <arvind.yadav.cs@gmail.com> a écrit :
> 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.
Applied the whole series to l2-mtd/master.
Thanks,
Boris
>
> Arvind Yadav (6):
> [PATCH 1/6] mtd: maps: ichxrom: constify pci_device_id.
> [PATCH 2/6] mtd: amd76xrom: constify pci_device_id.
> [PATCH 3/6] mtd: esb2rom: constify pci_device_id.
> [PATCH 4/6] mtd: ck804xrom: constify pci_device_id.
> [PATCH 5/6] mtd: intel_vr_nor: constify pci_device_id.
> [PATCH 6/6] mtd: pci: constify pci_device_id.
>
> drivers/mtd/maps/amd76xrom.c | 4 ++--
> drivers/mtd/maps/ck804xrom.c | 4 ++--
> drivers/mtd/maps/esb2rom.c | 4 ++--
> drivers/mtd/maps/ichxrom.c | 4 ++--
> drivers/mtd/maps/intel_vr_nor.c | 2 +-
> drivers/mtd/maps/pci.c | 2 +-
> 6 files changed, 10 insertions(+), 10 deletions(-)
>
^ permalink raw reply [flat|nested] 8+ messages in thread