Skip to contentSkip to conversation

ACPI / PMIC: constify platform_device_id

PATCH In review Arvind Yadav 3 participants 3 messages Nov 29, 2017 – Dec 16, 2017

Subsystems (2)
ACPI ACPI PMIC drivers
lore
Share & tools
lore

Arvind Yadav

author
#0 lore raw

PATCH v2 1/1 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]>

---
changes in v2:
Resend all changes as single patch.

 drivers/acpi/pmic/intel_pmic_bxtwc.c | 2 +-
 drivers/acpi/pmic/intel_pmic_chtwc.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/acpi/pmic/intel_pmic_bxtwc.c +1 -1
index 90011aa..2012d1d 100644
@@ -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" },
 	{},
 };
drivers/acpi/pmic/intel_pmic_chtwc.c +1 -1
index 85636d7..813b829 100644
@@ -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" },
 	{},
 };
Signature · 1 line
1.9.1