diff mbox

hwmon: (amc6821) basic device tree support

Message ID 1471579258-16148-1-git-send-email-matthew.weber@rockwellcollins.com (mailing list archive)
State Rejected
Headers show

Commit Message

Matt Weber Aug. 19, 2016, 4 a.m. UTC
Enables basic enumeration of a amc6821 device via device tree.

Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>
Signed-off-by: Ronak Desai <ronak.desai@rockwellcollins.com>
---
 drivers/hwmon/amc6821.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Guenter Roeck Aug. 19, 2016, 4:37 a.m. UTC | #1
On 08/18/2016 09:00 PM, Matt Weber wrote:
> Enables basic enumeration of a amc6821 device via device tree.
>

Same question as with ucd9000. i2c devices normally instantiate on devicetree
based on the i2c id table. Are you sure this is needed ?

Guenter

> Signed-off-by: Matthew Weber <matthew.weber@rockwellcollins.com>
> Signed-off-by: Ronak Desai <ronak.desai@rockwellcollins.com>
> ---
>  drivers/hwmon/amc6821.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/drivers/hwmon/amc6821.c b/drivers/hwmon/amc6821.c
> index 12e851a..77b5cf4 100644
> --- a/drivers/hwmon/amc6821.c
> +++ b/drivers/hwmon/amc6821.c
> @@ -32,6 +32,9 @@
>  #include <linux/err.h>
>  #include <linux/mutex.h>
>
> +#ifdef CONFIG_OF
> +#include <linux/of.h>
> +#endif
>  /*
>   * Addresses to scan.
>   */
> @@ -996,6 +999,12 @@ static int amc6821_probe(struct i2c_client *client,
>  	return PTR_ERR_OR_ZERO(hwmon_dev);
>  }
>
> +#ifdef CONFIG_OF
> +static const struct of_device_id amc6821_dt_match[] = {
> +	{ .compatible = "ti,amc6821", .data = (void *)amc6821 },
> +	{ }
> +};
> +#endif
>  static const struct i2c_device_id amc6821_id[] = {
>  	{ "amc6821", amc6821 },
>  	{ }
> @@ -1007,6 +1016,9 @@ static struct i2c_driver amc6821_driver = {
>  	.class = I2C_CLASS_HWMON,
>  	.driver = {
>  		.name	= "amc6821",
> +#ifdef CONFIG_OF
> +		.of_match_table = of_match_ptr(amc6821_dt_match),
> +#endif
>  	},
>  	.probe = amc6821_probe,
>  	.id_table = amc6821_id,
>

--
To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/hwmon/amc6821.c b/drivers/hwmon/amc6821.c
index 12e851a..77b5cf4 100644
--- a/drivers/hwmon/amc6821.c
+++ b/drivers/hwmon/amc6821.c
@@ -32,6 +32,9 @@ 
 #include <linux/err.h>
 #include <linux/mutex.h>
 
+#ifdef CONFIG_OF
+#include <linux/of.h>
+#endif
 /*
  * Addresses to scan.
  */
@@ -996,6 +999,12 @@  static int amc6821_probe(struct i2c_client *client,
 	return PTR_ERR_OR_ZERO(hwmon_dev);
 }
 
+#ifdef CONFIG_OF
+static const struct of_device_id amc6821_dt_match[] = {
+	{ .compatible = "ti,amc6821", .data = (void *)amc6821 },
+	{ }
+};
+#endif
 static const struct i2c_device_id amc6821_id[] = {
 	{ "amc6821", amc6821 },
 	{ }
@@ -1007,6 +1016,9 @@  static struct i2c_driver amc6821_driver = {
 	.class = I2C_CLASS_HWMON,
 	.driver = {
 		.name	= "amc6821",
+#ifdef CONFIG_OF
+		.of_match_table = of_match_ptr(amc6821_dt_match),
+#endif
 	},
 	.probe = amc6821_probe,
 	.id_table = amc6821_id,