Commit b3fce99a authored by Jonathan Cameron's avatar Jonathan Cameron

iio:chemical:vz89x: Drop of_match_ptr protection and use generic fw accessors

This change allow the driver to be used with ACPI PRP0001 and removes
an antipattern that I want to avoid being copied into new IIO drivers.

The handling of match_data uses a different approach as
device_get_match_data() doesn't distinguish between no match, and
a match but with NULL data.
Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Acked-by: default avatarMatt Ranostay <matt.ranostay@konsulko.com>
Link: https://lore.kernel.org/r/20200910173242.621168-30-jic23@kernel.org
parent e12b3a61
...@@ -10,8 +10,7 @@ ...@@ -10,8 +10,7 @@
#include <linux/mutex.h> #include <linux/mutex.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/i2c.h> #include <linux/i2c.h>
#include <linux/of.h> #include <linux/mod_devicetable.h>
#include <linux/of_device.h>
#include <linux/iio/iio.h> #include <linux/iio/iio.h>
#include <linux/iio/sysfs.h> #include <linux/iio/sysfs.h>
...@@ -355,7 +354,6 @@ static int vz89x_probe(struct i2c_client *client, ...@@ -355,7 +354,6 @@ static int vz89x_probe(struct i2c_client *client,
struct device *dev = &client->dev; struct device *dev = &client->dev;
struct iio_dev *indio_dev; struct iio_dev *indio_dev;
struct vz89x_data *data; struct vz89x_data *data;
const struct of_device_id *of_id;
int chip_id; int chip_id;
indio_dev = devm_iio_device_alloc(dev, sizeof(*data)); indio_dev = devm_iio_device_alloc(dev, sizeof(*data));
...@@ -371,11 +369,10 @@ static int vz89x_probe(struct i2c_client *client, ...@@ -371,11 +369,10 @@ static int vz89x_probe(struct i2c_client *client,
else else
return -EOPNOTSUPP; return -EOPNOTSUPP;
of_id = of_match_device(vz89x_dt_ids, dev); if (!dev_fwnode(dev))
if (!of_id)
chip_id = id->driver_data; chip_id = id->driver_data;
else else
chip_id = (unsigned long)of_id->data; chip_id = (unsigned long)device_get_match_data(dev);
i2c_set_clientdata(client, indio_dev); i2c_set_clientdata(client, indio_dev);
data->client = client; data->client = client;
...@@ -403,7 +400,7 @@ MODULE_DEVICE_TABLE(i2c, vz89x_id); ...@@ -403,7 +400,7 @@ MODULE_DEVICE_TABLE(i2c, vz89x_id);
static struct i2c_driver vz89x_driver = { static struct i2c_driver vz89x_driver = {
.driver = { .driver = {
.name = "vz89x", .name = "vz89x",
.of_match_table = of_match_ptr(vz89x_dt_ids), .of_match_table = vz89x_dt_ids,
}, },
.probe = vz89x_probe, .probe = vz89x_probe,
.id_table = vz89x_id, .id_table = vz89x_id,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment