2017-08-13 11:12:00

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 0/4] constify net platform_device_id

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

Arvind Yadav (4):
[PATCH 1/4] can: constify platform_device_id
[PATCH 2/4] net: dpaa_eth: constify platform_device_id
[PATCH 3/4] net: sh_eth: constify platform_device_id
[PATCH 4/4] net: ti: cpsw:: constify platform_device_id

drivers/net/can/c_can/c_can_platform.c | 2 +-
drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 2 +-
drivers/net/ethernet/renesas/sh_eth.c | 2 +-
drivers/net/ethernet/ti/cpsw.c | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)

--
2.7.4


2017-08-13 11:12:07

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 1/4] can: constify platform_device_id

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

Signed-off-by: Arvind Yadav <[email protected]>
---
drivers/net/can/c_can/c_can_platform.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/can/c_can/c_can_platform.c b/drivers/net/can/c_can/c_can_platform.c
index e36d105..46a746e 100644
--- a/drivers/net/can/c_can/c_can_platform.c
+++ b/drivers/net/can/c_can/c_can_platform.c
@@ -220,7 +220,7 @@ static const struct c_can_driver_data am3352_dcan_drvdata = {
.raminit_bits = am3352_raminit_bits,
};

-static struct platform_device_id c_can_id_table[] = {
+static const struct platform_device_id c_can_id_table[] = {
{
.name = KBUILD_MODNAME,
.driver_data = (kernel_ulong_t)&c_can_drvdata,
--
2.7.4

2017-08-14 03:13:01

by David Miller

[permalink] [raw]
Subject: Re: [PATCH 0/4] constify net platform_device_id

From: Arvind Yadav <[email protected]>
Date: Sun, 13 Aug 2017 16:41:44 +0530

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

Series applied, thanks.