* [PATCH 1/5] rapidio: idt_gen2: constify rio_device_id
2017-08-26 8:03 [PATCH 0/5] constify rio_device_id Arvind Yadav
@ 2017-08-26 8:03 ` Arvind Yadav
2017-08-26 8:03 ` [PATCH 2/5] rapidio: idt_gen3: " Arvind Yadav
` (4 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: Arvind Yadav @ 2017-08-26 8:03 UTC (permalink / raw)
To: mporter, alexandre.bounine; +Cc: linux-kernel
rio_device_id are not supposed to change at runtime. rio driver is
working with const 'id_table'. So mark the non-const rio_device_id
structs as const.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
drivers/rapidio/switches/idt_gen2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rapidio/switches/idt_gen2.c b/drivers/rapidio/switches/idt_gen2.c
index e67b923..4931ed7 100644
--- a/drivers/rapidio/switches/idt_gen2.c
+++ b/drivers/rapidio/switches/idt_gen2.c
@@ -458,7 +458,7 @@ static void idtg2_remove(struct rio_dev *rdev)
idtg2_sysfs(rdev, false);
}
-static struct rio_device_id idtg2_id_table[] = {
+static const struct rio_device_id idtg2_id_table[] = {
{RIO_DEVICE(RIO_DID_IDTCPS1848, RIO_VID_IDT)},
{RIO_DEVICE(RIO_DID_IDTCPS1616, RIO_VID_IDT)},
{RIO_DEVICE(RIO_DID_IDTVPS1616, RIO_VID_IDT)},
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 2/5] rapidio: idt_gen3: constify rio_device_id
2017-08-26 8:03 [PATCH 0/5] constify rio_device_id Arvind Yadav
2017-08-26 8:03 ` [PATCH 1/5] rapidio: idt_gen2: " Arvind Yadav
@ 2017-08-26 8:03 ` Arvind Yadav
2017-08-26 8:03 ` [PATCH 3/5] rapidio: idtcps: " Arvind Yadav
` (3 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: Arvind Yadav @ 2017-08-26 8:03 UTC (permalink / raw)
To: mporter, alexandre.bounine; +Cc: linux-kernel
rio_device_id are not supposed to change at runtime. rio driver is
working with const 'id_table'. So mark the non-const rio_device_id
structs as const.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
drivers/rapidio/switches/idt_gen3.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rapidio/switches/idt_gen3.c b/drivers/rapidio/switches/idt_gen3.c
index c5923a5..85a3908 100644
--- a/drivers/rapidio/switches/idt_gen3.c
+++ b/drivers/rapidio/switches/idt_gen3.c
@@ -348,7 +348,7 @@ static void idtg3_shutdown(struct rio_dev *rdev)
}
}
-static struct rio_device_id idtg3_id_table[] = {
+static const struct rio_device_id idtg3_id_table[] = {
{RIO_DEVICE(RIO_DID_IDTRXS1632, RIO_VID_IDT)},
{RIO_DEVICE(RIO_DID_IDTRXS2448, RIO_VID_IDT)},
{ 0, } /* terminate list */
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 3/5] rapidio: idtcps: constify rio_device_id
2017-08-26 8:03 [PATCH 0/5] constify rio_device_id Arvind Yadav
2017-08-26 8:03 ` [PATCH 1/5] rapidio: idt_gen2: " Arvind Yadav
2017-08-26 8:03 ` [PATCH 2/5] rapidio: idt_gen3: " Arvind Yadav
@ 2017-08-26 8:03 ` Arvind Yadav
2017-08-26 8:03 ` [PATCH 4/5] rapidio: tsi568: " Arvind Yadav
` (2 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: Arvind Yadav @ 2017-08-26 8:03 UTC (permalink / raw)
To: mporter, alexandre.bounine; +Cc: linux-kernel
rio_device_id are not supposed to change at runtime. rio driver is
working with const 'id_table'. So mark the non-const rio_device_id
structs as const.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
drivers/rapidio/switches/idtcps.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rapidio/switches/idtcps.c b/drivers/rapidio/switches/idtcps.c
index 7fbb60d..4058ce2 100644
--- a/drivers/rapidio/switches/idtcps.c
+++ b/drivers/rapidio/switches/idtcps.c
@@ -168,7 +168,7 @@ static void idtcps_remove(struct rio_dev *rdev)
spin_unlock(&rdev->rswitch->lock);
}
-static struct rio_device_id idtcps_id_table[] = {
+static const struct rio_device_id idtcps_id_table[] = {
{RIO_DEVICE(RIO_DID_IDTCPS6Q, RIO_VID_IDT)},
{RIO_DEVICE(RIO_DID_IDTCPS8, RIO_VID_IDT)},
{RIO_DEVICE(RIO_DID_IDTCPS10Q, RIO_VID_IDT)},
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 4/5] rapidio: tsi568: constify rio_device_id
2017-08-26 8:03 [PATCH 0/5] constify rio_device_id Arvind Yadav
` (2 preceding siblings ...)
2017-08-26 8:03 ` [PATCH 3/5] rapidio: idtcps: " Arvind Yadav
@ 2017-08-26 8:03 ` Arvind Yadav
2017-08-26 8:03 ` [PATCH 5/5] rapidio: tsi57x: " Arvind Yadav
2017-09-25 16:45 ` [PATCH 0/5] " Bounine, Alexandre
5 siblings, 0 replies; 9+ messages in thread
From: Arvind Yadav @ 2017-08-26 8:03 UTC (permalink / raw)
To: mporter, alexandre.bounine; +Cc: linux-kernel
rio_device_id are not supposed to change at runtime. rio driver is
working with const 'id_table'. So mark the non-const rio_device_id
structs as const.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
drivers/rapidio/switches/tsi568.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rapidio/switches/tsi568.c b/drivers/rapidio/switches/tsi568.c
index 8a43561..1214628b 100644
--- a/drivers/rapidio/switches/tsi568.c
+++ b/drivers/rapidio/switches/tsi568.c
@@ -169,7 +169,7 @@ static void tsi568_remove(struct rio_dev *rdev)
spin_unlock(&rdev->rswitch->lock);
}
-static struct rio_device_id tsi568_id_table[] = {
+static const struct rio_device_id tsi568_id_table[] = {
{RIO_DEVICE(RIO_DID_TSI568, RIO_VID_TUNDRA)},
{ 0, } /* terminate list */
};
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 5/5] rapidio: tsi57x: constify rio_device_id
2017-08-26 8:03 [PATCH 0/5] constify rio_device_id Arvind Yadav
` (3 preceding siblings ...)
2017-08-26 8:03 ` [PATCH 4/5] rapidio: tsi568: " Arvind Yadav
@ 2017-08-26 8:03 ` Arvind Yadav
2017-09-25 16:45 ` [PATCH 0/5] " Bounine, Alexandre
5 siblings, 0 replies; 9+ messages in thread
From: Arvind Yadav @ 2017-08-26 8:03 UTC (permalink / raw)
To: mporter, alexandre.bounine; +Cc: linux-kernel
rio_device_id are not supposed to change at runtime. rio driver is
working with const 'id_table'. So mark the non-const rio_device_id
structs as const.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
drivers/rapidio/switches/tsi57x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/rapidio/switches/tsi57x.c b/drivers/rapidio/switches/tsi57x.c
index 2700d15..9f063e2 100644
--- a/drivers/rapidio/switches/tsi57x.c
+++ b/drivers/rapidio/switches/tsi57x.c
@@ -336,7 +336,7 @@ static void tsi57x_remove(struct rio_dev *rdev)
spin_unlock(&rdev->rswitch->lock);
}
-static struct rio_device_id tsi57x_id_table[] = {
+static const struct rio_device_id tsi57x_id_table[] = {
{RIO_DEVICE(RIO_DID_TSI572, RIO_VID_TUNDRA)},
{RIO_DEVICE(RIO_DID_TSI574, RIO_VID_TUNDRA)},
{RIO_DEVICE(RIO_DID_TSI577, RIO_VID_TUNDRA)},
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* RE: [PATCH 0/5] constify rio_device_id
2017-08-26 8:03 [PATCH 0/5] constify rio_device_id Arvind Yadav
` (4 preceding siblings ...)
2017-08-26 8:03 ` [PATCH 5/5] rapidio: tsi57x: " Arvind Yadav
@ 2017-09-25 16:45 ` Bounine, Alexandre
2017-09-25 23:14 ` Andrew Morton
5 siblings, 1 reply; 9+ messages in thread
From: Bounine, Alexandre @ 2017-09-25 16:45 UTC (permalink / raw)
To: Andrew Morton; +Cc: linux-kernel, mporter, Arvind Yadav
Acked-by: Alexandre Bounine <alexandre.bounine@idt.com>
Hi Andrew,
Please apply the patches listed below. I will forward them to you.
Thank you,
Alex.
-----Original Message-----
From: Arvind Yadav [mailto:arvind.yadav.cs@gmail.com]
Sent: Saturday, August 26, 2017 4:04 AM
To: mporter@kernel.crashing.org; Bounine, Alexandre <Alexandre.Bounine@idt.com>
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH 0/5] constify rio_device_id
rio_device_id are not supposed to change at runtime. rio driver is
working with const 'id_table'. So mark the non-const rio_device_id
structs as const.
Arvind Yadav (5):
[PATCH 1/5] rapidio: idt_gen2: constify rio_device_id
[PATCH 2/5] rapidio: idt_gen3: constify rio_device_id
[PATCH 3/5] rapidio: idtcps: constify rio_device_id
[PATCH 4/5] rapidio: tsi568: constify rio_device_id
[PATCH 5/5] rapidio: tsi57x: constify rio_device_id
drivers/rapidio/switches/idt_gen2.c | 2 +-
drivers/rapidio/switches/idt_gen3.c | 2 +-
drivers/rapidio/switches/idtcps.c | 2 +-
drivers/rapidio/switches/tsi568.c | 2 +-
drivers/rapidio/switches/tsi57x.c | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH 0/5] constify rio_device_id
2017-09-25 16:45 ` [PATCH 0/5] " Bounine, Alexandre
@ 2017-09-25 23:14 ` Andrew Morton
2017-09-26 11:27 ` Bounine, Alexandre
0 siblings, 1 reply; 9+ messages in thread
From: Andrew Morton @ 2017-09-25 23:14 UTC (permalink / raw)
To: Bounine, Alexandre; +Cc: linux-kernel, mporter, Arvind Yadav
On Mon, 25 Sep 2017 16:45:09 +0000 "Bounine, Alexandre" <Alexandre.Bounine@idt.com> wrote:
> Acked-by: Alexandre Bounine <alexandre.bounine@idt.com>
>
> Hi Andrew,
>
> Please apply the patches listed below. I will forward them to you.
I'm subscribed to linux-kernel so the extra forwardings weren't needed,
thanks.
I'm thinking we might as well fold all these into a single patch, save
a few electrons?
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 0/5] constify rio_device_id
2017-09-25 23:14 ` Andrew Morton
@ 2017-09-26 11:27 ` Bounine, Alexandre
0 siblings, 0 replies; 9+ messages in thread
From: Bounine, Alexandre @ 2017-09-26 11:27 UTC (permalink / raw)
To: Andrew Morton; +Cc: linux-kernel, mporter, Arvind Yadav
Yes, single patch makes sense.
-----Original Message-----
From: Andrew Morton [mailto:akpm@linux-foundation.org]
Sent: Monday, September 25, 2017 7:14 PM
To: Bounine, Alexandre <Alexandre.Bounine@idt.com>
Cc: linux-kernel@vger.kernel.org; mporter@kernel.crashing.org; Arvind Yadav <arvind.yadav.cs@gmail.com>
Subject: Re: [PATCH 0/5] constify rio_device_id
On Mon, 25 Sep 2017 16:45:09 +0000 "Bounine, Alexandre" <Alexandre.Bounine@idt.com> wrote:
> Acked-by: Alexandre Bounine <alexandre.bounine@idt.com>
>
> Hi Andrew,
>
> Please apply the patches listed below. I will forward them to you.
I'm subscribed to linux-kernel so the extra forwardings weren't needed, thanks.
I'm thinking we might as well fold all these into a single patch, save a few electrons?
^ permalink raw reply [flat|nested] 9+ messages in thread