diff mbox

[2/2] power: at91-reset: Constify platform_device_id

Message ID 1430495204-31363-2-git-send-email-k.kozlowski.k@gmail.com (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Krzysztof Kozlowski May 1, 2015, 3:46 p.m. UTC 
The platform_device_id is not modified by the driver and core uses it as
const.

Signed-off-by: Krzysztof Kozlowski <k.kozlowski.k@gmail.com>
---
 drivers/power/reset/at91-reset.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alexandre Belloni May 3, 2015, 10:03 a.m. UTC | #1 
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 <k.kozlowski.k@gmail.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

> ---
>  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
>
Sebastian Reichel May 24, 2015, 7:41 p.m. UTC | #2 
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.
diff mbox

Patch

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 */ }