2015-05-01 15:37:34

by Krzysztof Kozłowski

Subject: [PATCH] watchdog: max63xx_wdt: Constify platform_device_id

The platform_device_id is not modified by the driver and core uses it as
const.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/watchdog/max63xx_wdt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/watchdog/max63xx_wdt.c b/drivers/watchdog/max63xx_wdt.c
index 08da3114accb..b3a1130b8eeb 100644
--- a/drivers/watchdog/max63xx_wdt.c
+++ b/drivers/watchdog/max63xx_wdt.c
@@ -214,7 +214,7 @@ static int max63xx_wdt_remove(struct platform_device *pdev)
return 0;
}

-static struct platform_device_id max63xx_id_table[] = {
+static const struct platform_device_id max63xx_id_table[] = {
{ "max6369_wdt", (kernel_ulong_t)max6369_table, },
{ "max6370_wdt", (kernel_ulong_t)max6369_table, },
{ "max6371_wdt", (kernel_ulong_t)max6371_table, },
--
2.1.4

2015-05-01 18:05:22

by Guenter Roeck

Subject: Re: [PATCH] watchdog: max63xx_wdt: Constify platform_device_id

On Sat, May 02, 2015 at 12:36:52AM +0900, Krzysztof Kozlowski wrote:
> The platform_device_id is not modified by the driver and core uses it as
> const.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Guenter Roeck <[email protected]>