hwmon: (tmp421) Add OF device ID table
The driver doesn't have a struct of_device_id table but supported devices are registered via Device Trees. This is working on the assumption that a I2C device registered via OF will always match a legacy I2C device ID and that the MODALIAS reported will always be of the form i2c:<device>. But this could change in the future so the correct approach is to have an OF device ID table if the devices are registered via OF. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> Signed-off-by: Guenter Roeck <linux@roeck-us.net>
This commit is contained in:
parent
72fc64c68d
commit
a1253027b7
|
@ -29,6 +29,7 @@
|
||||||
#include <linux/hwmon-sysfs.h>
|
#include <linux/hwmon-sysfs.h>
|
||||||
#include <linux/err.h>
|
#include <linux/err.h>
|
||||||
#include <linux/mutex.h>
|
#include <linux/mutex.h>
|
||||||
|
#include <linux/of_device.h>
|
||||||
#include <linux/sysfs.h>
|
#include <linux/sysfs.h>
|
||||||
|
|
||||||
/* Addresses to scan */
|
/* Addresses to scan */
|
||||||
|
@ -69,6 +70,31 @@ static const struct i2c_device_id tmp421_id[] = {
|
||||||
};
|
};
|
||||||
MODULE_DEVICE_TABLE(i2c, tmp421_id);
|
MODULE_DEVICE_TABLE(i2c, tmp421_id);
|
||||||
|
|
||||||
|
static const struct of_device_id tmp421_of_match[] = {
|
||||||
|
{
|
||||||
|
.compatible = "ti,tmp421",
|
||||||
|
.data = (void *)2
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.compatible = "ti,tmp422",
|
||||||
|
.data = (void *)3
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.compatible = "ti,tmp423",
|
||||||
|
.data = (void *)4
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.compatible = "ti,tmp441",
|
||||||
|
.data = (void *)2
|
||||||
|
},
|
||||||
|
{
|
||||||
|
.compatible = "ti,tmp422",
|
||||||
|
.data = (void *)3
|
||||||
|
},
|
||||||
|
{ },
|
||||||
|
};
|
||||||
|
MODULE_DEVICE_TABLE(of, tmp421_of_match);
|
||||||
|
|
||||||
struct tmp421_data {
|
struct tmp421_data {
|
||||||
struct i2c_client *client;
|
struct i2c_client *client;
|
||||||
struct mutex update_lock;
|
struct mutex update_lock;
|
||||||
|
@ -78,7 +104,7 @@ struct tmp421_data {
|
||||||
struct hwmon_chip_info chip;
|
struct hwmon_chip_info chip;
|
||||||
char valid;
|
char valid;
|
||||||
unsigned long last_updated;
|
unsigned long last_updated;
|
||||||
int channels;
|
unsigned long channels;
|
||||||
u8 config;
|
u8 config;
|
||||||
s16 temp[4];
|
s16 temp[4];
|
||||||
};
|
};
|
||||||
|
@ -272,7 +298,11 @@ static int tmp421_probe(struct i2c_client *client,
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
mutex_init(&data->update_lock);
|
mutex_init(&data->update_lock);
|
||||||
data->channels = id->driver_data;
|
if (client->dev.of_node)
|
||||||
|
data->channels = (unsigned long)
|
||||||
|
of_device_get_match_data(&client->dev);
|
||||||
|
else
|
||||||
|
data->channels = id->driver_data;
|
||||||
data->client = client;
|
data->client = client;
|
||||||
|
|
||||||
err = tmp421_init_client(client);
|
err = tmp421_init_client(client);
|
||||||
|
@ -301,6 +331,7 @@ static struct i2c_driver tmp421_driver = {
|
||||||
.class = I2C_CLASS_HWMON,
|
.class = I2C_CLASS_HWMON,
|
||||||
.driver = {
|
.driver = {
|
||||||
.name = "tmp421",
|
.name = "tmp421",
|
||||||
|
.of_match_table = of_match_ptr(tmp421_of_match),
|
||||||
},
|
},
|
||||||
.probe = tmp421_probe,
|
.probe = tmp421_probe,
|
||||||
.id_table = tmp421_id,
|
.id_table = tmp421_id,
|
||||||
|
|
Loading…
Reference in New Issue