2017-11-28 10:05:17

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 0/2] ACPI / PMIC: 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.

Arvind Yadav (2):
[PATCH 1/2] ACPI / PMIC: constify platform_device_id
[PATCH 2/2] ACPI / PMIC: Constify platform_device_id

drivers/acpi/pmic/intel_pmic_bxtwc.c | 2 +-
drivers/acpi/pmic/intel_pmic_chtwc.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

--
1.9.1


From 1585303686724933226@xxx Tue Nov 28 10:03:29 +0000 2017
X-GM-THRID: 1585303686724933226
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread


2017-11-28 10:03:29

by Arvind Yadav

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

diff --git a/drivers/acpi/pmic/intel_pmic_chtwc.c b/drivers/acpi/pmic/intel_pmic_chtwc.c
index 85636d7..813b829 100644
--- a/drivers/acpi/pmic/intel_pmic_chtwc.c
+++ b/drivers/acpi/pmic/intel_pmic_chtwc.c
@@ -260,7 +260,7 @@ static int intel_cht_wc_pmic_opregion_probe(struct platform_device *pdev)
&intel_cht_wc_pmic_opregion_data);
}

-static struct platform_device_id cht_wc_opregion_id_table[] = {
+static const struct platform_device_id cht_wc_opregion_id_table[] = {
{ .name = "cht_wcove_region" },
{},
};
--
1.9.1


From 1585375848857878399@xxx Wed Nov 29 05:10:28 +0000 2017
X-GM-THRID: 1585316485511785462
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread

2017-11-28 10:04:08

by Arvind Yadav

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

diff --git a/drivers/acpi/pmic/intel_pmic_bxtwc.c b/drivers/acpi/pmic/intel_pmic_bxtwc.c
index 90011aa..2012d1d 100644
--- a/drivers/acpi/pmic/intel_pmic_bxtwc.c
+++ b/drivers/acpi/pmic/intel_pmic_bxtwc.c
@@ -400,7 +400,7 @@ static int intel_bxtwc_pmic_opregion_probe(struct platform_device *pdev)
&intel_bxtwc_pmic_opregion_data);
}

-static struct platform_device_id bxt_wc_opregion_id_table[] = {
+static const struct platform_device_id bxt_wc_opregion_id_table[] = {
{ .name = "bxt_wcove_region" },
{},
};
--
1.9.1


From 1585307846333638693@xxx Tue Nov 28 11:09:36 +0000 2017
X-GM-THRID: 1585307846333638693
X-Gmail-Labels: Inbox,Category Promotions,HistoricalUnread