2015-05-01 15:47:07

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 1/2] power: axp288_fuel_gauge: 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/power/axp288_fuel_gauge.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/axp288_fuel_gauge.c b/drivers/power/axp288_fuel_gauge.c
index ca1cc5a47eb1..35932c768aa4 100644
--- a/drivers/power/axp288_fuel_gauge.c
+++ b/drivers/power/axp288_fuel_gauge.c
@@ -1117,7 +1117,7 @@ static int axp288_fuel_gauge_probe(struct platform_device *pdev)
return ret;
}

-static struct platform_device_id axp288_fg_id_table[] = {
+static const struct platform_device_id axp288_fg_id_table[] = {
{ .name = DEV_NAME },
{},
};
--
2.1.4


2015-05-01 15:47:10

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 2/2] power: at91-reset: 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/power/reset/at91-reset.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/reset/at91-reset.c b/drivers/power/reset/at91-reset.c
index 01c7055c4200..4bfd9056eef5 100644
--- a/drivers/power/reset/at91-reset.c
+++ b/drivers/power/reset/at91-reset.c
@@ -243,7 +243,7 @@ static int at91_reset_probe(struct platform_device *pdev)
return 0;
}

-static struct platform_device_id at91_reset_plat_match[] = {
+static const struct platform_device_id at91_reset_plat_match[] = {
{ "at91-sam9260-reset", (unsigned long)at91sam9260_restart },
{ "at91-sam9g45-reset", (unsigned long)at91sam9g45_restart },
{ /* sentinel */ }
--
2.1.4

2015-05-03 10:04:05

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH 2/2] power: at91-reset: Constify platform_device_id

On 02/05/2015 at 00:46:44 +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]>
Acked-by: Alexandre Belloni <[email protected]>

> ---
> drivers/power/reset/at91-reset.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/power/reset/at91-reset.c b/drivers/power/reset/at91-reset.c
> index 01c7055c4200..4bfd9056eef5 100644
> --- a/drivers/power/reset/at91-reset.c
> +++ b/drivers/power/reset/at91-reset.c
> @@ -243,7 +243,7 @@ static int at91_reset_probe(struct platform_device *pdev)
> return 0;
> }
>
> -static struct platform_device_id at91_reset_plat_match[] = {
> +static const struct platform_device_id at91_reset_plat_match[] = {
> { "at91-sam9260-reset", (unsigned long)at91sam9260_restart },
> { "at91-sam9g45-reset", (unsigned long)at91sam9g45_restart },
> { /* sentinel */ }
> --
> 2.1.4
>

--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

2015-05-24 19:41:15

by Sebastian Reichel

[permalink] [raw]
Subject: Re: [PATCH 1/2] power: axp288_fuel_gauge: Constify platform_device_id

Hi,

On Sat, May 02, 2015 at 12:46:43AM +0900, Krzysztof Kozlowski wrote:
> The platform_device_id is not modified by the driver and core uses it as
> const.

Thanks, queued.

-- Sebastian

Attachments:
(No filename) (189.00 B)
 signature.asc (819.00 B)
Digital signature Download all attachments

2015-05-24 19:41:37

by Sebastian Reichel

[permalink] [raw]
Subject: Re: [PATCH 2/2] power: at91-reset: Constify platform_device_id

Hi,

On Sat, May 02, 2015 at 12:46:44AM +0900, Krzysztof Kozlowski wrote:
> The platform_device_id is not modified by the driver and core uses
> it as const.

Thanks, queued.

Attachments:
(No filename) (175.00 B)
 signature.asc (819.00 B)
Digital signature Download all attachments