[PATCH v1 1/2] hwmon: (pmbus/mp2869) Remove unused driver data

From: Uwe Kleine-König (The Capable Hub)

Date: Tue May 12 2026 - 12:32:40 EST


The driver doesn't make use of the i2c .driver_data and the of .data. So
drop the useless and irritating assignments.

While touching all these lines, use named initializers for the i2c
device ids and convert to the most common usage of spacing in such
arrays.

Signed-off-by: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@xxxxxxxxxxxx>
---
drivers/hwmon/pmbus/mp2869.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/hwmon/pmbus/mp2869.c b/drivers/hwmon/pmbus/mp2869.c
index 4647892e5112..d345be920429 100644
--- a/drivers/hwmon/pmbus/mp2869.c
+++ b/drivers/hwmon/pmbus/mp2869.c
@@ -632,20 +632,20 @@ static int mp2869_probe(struct i2c_client *client)
}

static const struct i2c_device_id mp2869_id[] = {
- {"mp2869", 0},
- {"mp29608", 1},
- {"mp29612", 2},
- {"mp29816", 3},
- {}
+ { .name = "mp2869" },
+ { .name = "mp29608" },
+ { .name = "mp29612" },
+ { .name = "mp29816" },
+ { }
};
MODULE_DEVICE_TABLE(i2c, mp2869_id);

static const struct of_device_id __maybe_unused mp2869_of_match[] = {
- {.compatible = "mps,mp2869", .data = (void *)0},
- {.compatible = "mps,mp29608", .data = (void *)1},
- {.compatible = "mps,mp29612", .data = (void *)2},
- {.compatible = "mps,mp29816", .data = (void *)3},
- {}
+ { .compatible = "mps,mp2869" },
+ { .compatible = "mps,mp29608" },
+ { .compatible = "mps,mp29612" },
+ { .compatible = "mps,mp29816" },
+ { }
};
MODULE_DEVICE_TABLE(of, mp2869_of_match);

--
2.47.3