| Message ID | 30f8e66e4397f59747e9843e7c09b0c42b2d5804.1502623863.git.arvind.yadav.cs@gmail.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Zhang Rui |
| Headers | show |
On Sun, 2017-08-13 at 17:03 +0530, Arvind Yadav wrote: > 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 <arvind.yadav.cs@gmail.com> applied. thanks, rui > --- > drivers/thermal/max77620_thermal.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/max77620_thermal.c > b/drivers/thermal/max77620_thermal.c > index 159bbce..3c74b91 100644 > --- a/drivers/thermal/max77620_thermal.c > +++ b/drivers/thermal/max77620_thermal.c > @@ -149,7 +149,7 @@ static int max77620_thermal_probe(struct > platform_device *pdev) > return 0; > } > > -static struct platform_device_id max77620_thermal_devtype[] = { > +static const struct platform_device_id max77620_thermal_devtype[] = > { > { .name = "max77620-thermal", }, > {}, > };
diff --git a/drivers/thermal/max77620_thermal.c b/drivers/thermal/max77620_thermal.c index 159bbce..3c74b91 100644 --- a/drivers/thermal/max77620_thermal.c +++ b/drivers/thermal/max77620_thermal.c @@ -149,7 +149,7 @@ static int max77620_thermal_probe(struct platform_device *pdev) return 0; } -static struct platform_device_id max77620_thermal_devtype[] = { +static const struct platform_device_id max77620_thermal_devtype[] = { { .name = "max77620-thermal", }, {}, };
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 <arvind.yadav.cs@gmail.com> --- drivers/thermal/max77620_thermal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)