[PATCH v2] mfd: madera-i2c: Simplify probe()
Simplify probe() by replacing of_device_get_match_data() and ID lookup for retrieving match data by i2c_get_match_data().
Signed-off-by: Biju Das biju.das.jz@bp.renesas.com Acked-by: Charles Keepax ckeepax@opensource.cirrus.com --- Note: This patch is only compile tested.
v1->v2: * Added Ack tag from Charles Keepax. * Dropped blank line before switch statement. --- drivers/mfd/madera-i2c.c | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/mfd/madera-i2c.c b/drivers/mfd/madera-i2c.c index a404ea26bc79..33781c631345 100644 --- a/drivers/mfd/madera-i2c.c +++ b/drivers/mfd/madera-i2c.c @@ -18,21 +18,14 @@
static int madera_i2c_probe(struct i2c_client *i2c) { - const struct i2c_device_id *id = i2c_client_get_device_id(i2c); struct madera *madera; const struct regmap_config *regmap_16bit_config = NULL; const struct regmap_config *regmap_32bit_config = NULL; - const void *of_data; unsigned long type; const char *name; int ret;
- of_data = of_device_get_match_data(&i2c->dev); - if (of_data) - type = (unsigned long)of_data; - else - type = id->driver_data; - + type = (unsigned long)i2c_get_match_data(i2c); switch (type) { case CS47L15: if (IS_ENABLED(CONFIG_MFD_CS47L15)) {
On Mon, Aug 28, 2023 at 04:49:41PM +0100, Biju Das wrote:
Simplify probe() by replacing of_device_get_match_data() and ID lookup for retrieving match data by i2c_get_match_data().
...
- type = (unsigned long)i2c_get_match_data(i2c);
uintptr_t ?
Hi Andy Shevchenko,
Subject: Re: [PATCH v2] mfd: madera-i2c: Simplify probe()
On Mon, Aug 28, 2023 at 04:49:41PM +0100, Biju Das wrote:
Simplify probe() by replacing of_device_get_match_data() and ID lookup for retrieving match data by i2c_get_match_data().
...
- type = (unsigned long)i2c_get_match_data(i2c);
uintptr_t ?
Oops, missed it.
Cheers, Biju
participants (2)
-
Andy Shevchenko
-
Biju Das