diff options
| author |   Hans de Goede <hdegoede@redhat.com> | 2021-01-20 22:49:54 +0100 |
|---|---|---|
| committer |   Lee Jones <lee.jones@linaro.org> | 2021-02-04 13:54:56 +0000 |
| commit | 039da225d644e523a3bbec78ee258b25fe0676f9 (patch) | |
| tree | ae19020336350a4a25671766a1c92c2082f24c26 /drivers/mfd/arizona-i2c.c | |
| parent | mfd: arizona: Add MODULE_SOFTDEP("pre: arizona_ldo1") (diff) | |
| download | linux-dev-039da225d644e523a3bbec78ee258b25fe0676f9.tar.xz linux-dev-039da225d644e523a3bbec78ee258b25fe0676f9.zip | |
mfd: arizona: Replace arizona_of_get_type() with device_get_match_data()
Replace the custom arizona_of_get_type() function with the generic
device_get_match_data() helper. Besides being a nice cleanup this
also makes it easier to add support for binding to ACPI enumerated
devices.
While at it also fix a possible NULL pointer deref of the id
argument to the probe functions (this could happen on e.g. manual
driver binding through sysfs).
Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd/arizona-i2c.c')
| -rw-r--r-- | drivers/mfd/arizona-i2c.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/mfd/arizona-i2c.c b/drivers/mfd/arizona-i2c.c index 2a4a3a164d0a..5e83b730c4ce 100644 --- a/drivers/mfd/arizona-i2c.c +++ b/drivers/mfd/arizona-i2c.c @@ -23,14 +23,16 @@ static int arizona_i2c_probe(struct i2c_client *i2c, const struct i2c_device_id *id) { + const void *match_data; struct arizona *arizona; const struct regmap_config *regmap_config = NULL; - unsigned long type; + unsigned long type = 0; int ret; - if (i2c->dev.of_node) - type = arizona_of_get_type(&i2c->dev); - else + match_data = device_get_match_data(&i2c->dev); + if (match_data) + type = (unsigned long)match_data; + else if (id) type = id->driver_data; switch (type) { |