regulator: stw481x-vmmc: Constify static structs
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Sun, 13 Sep 2020 08:41:12 +0000 (10:41 +0200)
committerMark Brown <broonie@kernel.org>
Mon, 14 Sep 2020 13:24:38 +0000 (14:24 +0100)
The only usage of stw481x_vmmc_ops is to assign its address to the ops
field in the regulator_desc struct which is a const pointer.

The only usage of vmmc_regulator is to pass its address to
of_get_regulator_init_data() and devm_regulator_register(), both which
take const pointers.

Make both of them const to allow the compiler to put them in read-only
memory.

Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Link: https://lore.kernel.org/r/20200913084114.8851-4-rikard.falkeborn@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/stw481x-vmmc.c

index 6dc2316..127ab43 100644 (file)
@@ -27,7 +27,7 @@ static const unsigned int stw481x_vmmc_voltages[] = {
        3300000,
 };
 
-static struct regulator_ops stw481x_vmmc_ops = {
+static const struct regulator_ops stw481x_vmmc_ops = {
        .list_voltage = regulator_list_voltage_table,
        .enable      = regulator_enable_regmap,
        .disable     = regulator_disable_regmap,
@@ -36,7 +36,7 @@ static struct regulator_ops stw481x_vmmc_ops = {
        .set_voltage_sel = regulator_set_voltage_sel_regmap,
 };
 
-static struct regulator_desc vmmc_regulator = {
+static const struct regulator_desc vmmc_regulator = {
        .name = "VMMC",
        .id   = 0,
        .ops  = &stw481x_vmmc_ops,