Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1701191 > unrolled thread

[PATCH 00/10] constify char pci_device_id

Started byArvind Yadav <arvind.yadav.cs@gmail.com>
First post2017-08-01 18:10 +0200
Last post2017-08-01 18:30 +0200
Articles 10 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 00/10] constify char pci_device_id Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:10 +0200
    [PATCH 02/10] agp: amd-k7: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:10 +0200
    [PATCH 06/10] agp: efficeon: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:10 +0200
    [PATCH 01/10] agp: uninorth: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:10 +0200
      Re: [PATCH 01/10] agp: uninorth: constify pci_device_id. Dave Airlie <airlied@linux.ie> - 2017-08-04 09:20 +0200
    [PATCH 05/10] agp: ati: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:10 +0200
    [PATCH 07/10] agp: sis: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:30 +0200
    [PATCH 10/10] applicom: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:30 +0200
    [PATCH 08/10] agp: amd64: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:30 +0200
    [PATCH 09/10] agp: nvidia: constify pci_device_id. Arvind Yadav <arvind.yadav.cs@gmail.com> - 2017-08-01 18:30 +0200

#1701191 — [PATCH 00/10] constify char pci_device_id

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:10 +0200
Subject[PATCH 00/10] constify char pci_device_id
Message-ID<u9HU5-68E-3@gated-at.bofh.it>
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 (10):
  [PATCH 01/10] agp: uninorth: constify pci_device_id.
  [PATCH 02/10] agp: amd-k7: constify pci_device_id.
  [PATCH 03/10] agp: intel: constify pci_device_id.
  [PATCH 04/10] agp: ali: constify pci_device_id.
  [PATCH 05/10] agp: ati: constify pci_device_id.
  [PATCH 06/10] agp: efficeon: constify pci_device_id.
  [PATCH 07/10] agp: sis: constify pci_device_id.
  [PATCH 08/10] agp: amd64: constify pci_device_id.
  [PATCH 09/10] agp: nvidia: constify pci_device_id.
  [PATCH 10/10] applicom: constify pci_device_id.

 drivers/char/agp/ali-agp.c      | 2 +-
 drivers/char/agp/amd-k7-agp.c   | 4 ++--
 drivers/char/agp/amd64-agp.c    | 2 +-
 drivers/char/agp/ati-agp.c      | 2 +-
 drivers/char/agp/efficeon-agp.c | 2 +-
 drivers/char/agp/intel-agp.c    | 2 +-
 drivers/char/agp/nvidia-agp.c   | 2 +-
 drivers/char/agp/sis-agp.c      | 2 +-
 drivers/char/agp/uninorth-agp.c | 2 +-
 drivers/char/applicom.c         | 2 +-
 10 files changed, 11 insertions(+), 11 deletions(-)

-- 
2.7.4

[toc] | [next] | [standalone]

#1701193 — [PATCH 02/10] agp: amd-k7: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:10 +0200
Subject[PATCH 02/10] agp: amd-k7: constify pci_device_id.
Message-ID<u9HU6-68E-21@gated-at.bofh.it>
In reply to#1701191
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/char/agp/amd-k7-agp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/char/agp/amd-k7-agp.c b/drivers/char/agp/amd-k7-agp.c
index 5fbd333..b450544 100644
--- a/drivers/char/agp/amd-k7-agp.c
+++ b/drivers/char/agp/amd-k7-agp.c
@@ -21,7 +21,7 @@
 #define AMD_TLBFLUSH	0x0c	/* In mmio region (32-bit register) */
 #define AMD_CACHEENTRY	0x10	/* In mmio region (32-bit register) */
 
-static struct pci_device_id agp_amdk7_pci_table[];
+static const struct pci_device_id agp_amdk7_pci_table[];
 
 struct amd_page_map {
 	unsigned long *real;
@@ -508,7 +508,7 @@ static int agp_amdk7_resume(struct pci_dev *pdev)
 #endif /* CONFIG_PM */
 
 /* must be the same order as name table above */
-static struct pci_device_id agp_amdk7_pci_table[] = {
+static const struct pci_device_id agp_amdk7_pci_table[] = {
 	{
 	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 	.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1701196 — [PATCH 06/10] agp: efficeon: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:10 +0200
Subject[PATCH 06/10] agp: efficeon: constify pci_device_id.
Message-ID<u9HU6-68E-25@gated-at.bofh.it>
In reply to#1701191
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/char/agp/efficeon-agp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/agp/efficeon-agp.c b/drivers/char/agp/efficeon-agp.c
index 533cb6d..7f88490 100644
--- a/drivers/char/agp/efficeon-agp.c
+++ b/drivers/char/agp/efficeon-agp.c
@@ -427,7 +427,7 @@ static int agp_efficeon_resume(struct pci_dev *pdev)
 }
 #endif
 
-static struct pci_device_id agp_efficeon_pci_table[] = {
+static const struct pci_device_id agp_efficeon_pci_table[] = {
 	{
 	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 	.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1701200 — [PATCH 01/10] agp: uninorth: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:10 +0200
Subject[PATCH 01/10] agp: uninorth: constify pci_device_id.
Message-ID<u9HU7-68E-33@gated-at.bofh.it>
In reply to#1701191
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/char/agp/uninorth-agp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/agp/uninorth-agp.c b/drivers/char/agp/uninorth-agp.c
index fdced54..c381c8e 100644
--- a/drivers/char/agp/uninorth-agp.c
+++ b/drivers/char/agp/uninorth-agp.c
@@ -679,7 +679,7 @@ static void agp_uninorth_remove(struct pci_dev *pdev)
 	agp_put_bridge(bridge);
 }
 
-static struct pci_device_id agp_uninorth_pci_table[] = {
+static const struct pci_device_id agp_uninorth_pci_table[] = {
 	{
 	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 	.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1703732 — Re: [PATCH 01/10] agp: uninorth: constify pci_device_id.

FromDave Airlie <airlied@linux.ie>
Date2017-08-04 09:20 +0200
SubjectRe: [PATCH 01/10] agp: uninorth: constify pci_device_id.
Message-ID<uaF3P-3UX-1@gated-at.bofh.it>
In reply to#1701200
On Tue, 1 Aug 2017, Arvind Yadav wrote:

> 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>

I've merged all the agp ones into drm-next.

Dave.

> ---
>  drivers/char/agp/uninorth-agp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/char/agp/uninorth-agp.c b/drivers/char/agp/uninorth-agp.c
> index fdced54..c381c8e 100644
> --- a/drivers/char/agp/uninorth-agp.c
> +++ b/drivers/char/agp/uninorth-agp.c
> @@ -679,7 +679,7 @@ static void agp_uninorth_remove(struct pci_dev *pdev)
>  	agp_put_bridge(bridge);
>  }
>  
> -static struct pci_device_id agp_uninorth_pci_table[] = {
> +static const struct pci_device_id agp_uninorth_pci_table[] = {
>  	{
>  	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
>  	.class_mask	= ~0,
>

[toc] | [prev] | [next] | [standalone]

#1701201 — [PATCH 05/10] agp: ati: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:10 +0200
Subject[PATCH 05/10] agp: ati: constify pci_device_id.
Message-ID<u9HU7-68E-37@gated-at.bofh.it>
In reply to#1701191
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/char/agp/ati-agp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/agp/ati-agp.c b/drivers/char/agp/ati-agp.c
index 0b5ec7a..88b4cbe 100644
--- a/drivers/char/agp/ati-agp.c
+++ b/drivers/char/agp/ati-agp.c
@@ -540,7 +540,7 @@ static void agp_ati_remove(struct pci_dev *pdev)
 	agp_put_bridge(bridge);
 }
 
-static struct pci_device_id agp_ati_pci_table[] = {
+static const struct pci_device_id agp_ati_pci_table[] = {
 	{
 	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 	.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1701211 — [PATCH 07/10] agp: sis: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:30 +0200
Subject[PATCH 07/10] agp: sis: constify pci_device_id.
Message-ID<u9Ids-6fm-15@gated-at.bofh.it>
In reply to#1701191
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/char/agp/sis-agp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/agp/sis-agp.c b/drivers/char/agp/sis-agp.c
index 2c74038..14909fc 100644
--- a/drivers/char/agp/sis-agp.c
+++ b/drivers/char/agp/sis-agp.c
@@ -237,7 +237,7 @@ static int agp_sis_resume(struct pci_dev *pdev)
 
 #endif /* CONFIG_PM */
 
-static struct pci_device_id agp_sis_pci_table[] = {
+static const struct pci_device_id agp_sis_pci_table[] = {
 	{
 		.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 		.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1701213 — [PATCH 10/10] applicom: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:30 +0200
Subject[PATCH 10/10] applicom: constify pci_device_id.
Message-ID<u9Ids-6fm-17@gated-at.bofh.it>
In reply to#1701191
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/char/applicom.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/applicom.c b/drivers/char/applicom.c
index b67263d..c0a5b1f 100644
--- a/drivers/char/applicom.c
+++ b/drivers/char/applicom.c
@@ -67,7 +67,7 @@ static char *applicom_pci_devnames[] = {
 	"PCI2000PFB"
 };
 
-static struct pci_device_id applicom_pci_tbl[] = {
+static const struct pci_device_id applicom_pci_tbl[] = {
 	{ PCI_VDEVICE(APPLICOM, PCI_DEVICE_ID_APPLICOM_PCIGENERIC) },
 	{ PCI_VDEVICE(APPLICOM, PCI_DEVICE_ID_APPLICOM_PCI2000IBS_CAN) },
 	{ PCI_VDEVICE(APPLICOM, PCI_DEVICE_ID_APPLICOM_PCI2000PFB) },
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1701218 — [PATCH 08/10] agp: amd64: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:30 +0200
Subject[PATCH 08/10] agp: amd64: constify pci_device_id.
Message-ID<u9Ids-6fm-29@gated-at.bofh.it>
In reply to#1701191
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/char/agp/amd64-agp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/agp/amd64-agp.c b/drivers/char/agp/amd64-agp.c
index c99cd19..e50c29c 100644
--- a/drivers/char/agp/amd64-agp.c
+++ b/drivers/char/agp/amd64-agp.c
@@ -610,7 +610,7 @@ static int agp_amd64_resume(struct pci_dev *pdev)
 
 #endif /* CONFIG_PM */
 
-static struct pci_device_id agp_amd64_pci_table[] = {
+static const struct pci_device_id agp_amd64_pci_table[] = {
 	{
 	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 	.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1701219 — [PATCH 09/10] agp: nvidia: constify pci_device_id.

FromArvind Yadav <arvind.yadav.cs@gmail.com>
Date2017-08-01 18:30 +0200
Subject[PATCH 09/10] agp: nvidia: constify pci_device_id.
Message-ID<u9Ids-6fm-31@gated-at.bofh.it>
In reply to#1701191
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/char/agp/nvidia-agp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/agp/nvidia-agp.c b/drivers/char/agp/nvidia-agp.c
index 6c8d39c..828b344 100644
--- a/drivers/char/agp/nvidia-agp.c
+++ b/drivers/char/agp/nvidia-agp.c
@@ -420,7 +420,7 @@ static int agp_nvidia_resume(struct pci_dev *pdev)
 #endif
 
 
-static struct pci_device_id agp_nvidia_pci_table[] = {
+static const struct pci_device_id agp_nvidia_pci_table[] = {
 	{
 	.class		= (PCI_CLASS_BRIDGE_HOST << 8),
 	.class_mask	= ~0,
-- 
2.7.4

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web