misc: nct1008: debugfs support for reg dump

Debugfs support added in nct1008 driver to enable register dump.

Reviewed-on: http://git-master/r/51856
(cherry picked from commit 4e7b8c8e1a7a43e4d745289fc9f20a11556eca96)

Reviewed-on: http://git-master/r/54992
(cherry picked from commit 5b512a6c63bd65375c9362057105f76a7ef97d4a)

Change-Id: Icf9607b12462146e3e046cfc10c3e89424142e5f
Reviewed-on: http://git-master/r/58598
Tested-by: Bitan Biswas <bbiswas@nvidia.com>
Reviewed-by: Diwakar Tundlam <dtundlam@nvidia.com>
Reviewed-by: Varun Wadekar <vwadekar@nvidia.com>

Rebase-Id: R95221409d54694937ef226f01572b58c009a8361

1 files changed, 81 insertions, 2 deletions

diff --git a/drivers/misc/nct1008.c b/drivers/misc/nct1008.c
index f3bd0fe12960..3a830b775dc8 100644
--- a/drivers/misc/nct1008.c
+++ b/drivers/misc/nct1008.c
@@ -374,6 +374,81 @@ static const struct attribute_group nct1008_attr_group = {
 	.attrs = nct1008_attributes,
 };
 
+#ifdef CONFIG_DEBUG_FS
+#include <linux/debugfs.h>
+#include <linux/seq_file.h>
+static void print_reg(const char *reg_name, struct seq_file *s,
+		int offset)
+{
+	struct nct1008_data *nct_data = s->private;
+	int ret;
+
+	ret = i2c_smbus_read_byte_data(nct_data->client,
+		offset);
+	if (ret >= 0)
+		seq_printf(s, "Reg %s Addr = 0x%02x Reg 0x%02x "
+		"Value 0x%02x\n", reg_name,
+		nct_data->client->addr,
+			offset, ret);
+	else
+		seq_printf(s, "%s: line=%d, i2c read error=%d\n",
+		__func__, __LINE__, ret);
+}
+
+static int dbg_nct1008_show(struct seq_file *s, void *unused)
+{
+	seq_printf(s, "nct1008 Registers\n");
+	seq_printf(s, "------------------\n");
+	print_reg("Local Temp Value    ",     s, 0x00);
+	print_reg("Ext Temp Value Hi   ",     s, 0x01);
+	print_reg("Status              ",     s, 0x02);
+	print_reg("Configuration       ",     s, 0x03);
+	print_reg("Conversion Rate     ",     s, 0x04);
+	print_reg("Local Temp Hi Limit ",     s, 0x05);
+	print_reg("Local Temp Lo Limit ",     s, 0x06);
+	print_reg("Ext Temp Hi Limit Hi",     s, 0x07);
+	print_reg("Ext Temp Hi Limit Lo",     s, 0x13);
+	print_reg("Ext Temp Lo Limit Hi",     s, 0x08);
+	print_reg("Ext Temp Lo Limit Lo",     s, 0x14);
+	print_reg("Ext Temp Value Lo   ",     s, 0x10);
+	print_reg("Ext Temp Offset Hi  ",     s, 0x11);
+	print_reg("Ext Temp Offset Lo  ",     s, 0x12);
+	print_reg("Ext THERM Limit     ",     s, 0x19);
+	print_reg("Local THERM Limit   ",     s, 0x20);
+	print_reg("THERM Hysteresis    ",     s, 0x21);
+	print_reg("Consecutive ALERT   ",     s, 0x22);
+	return 0;
+}
+
+static int dbg_nct1008_open(struct inode *inode, struct file *file)
+{
+	return single_open(file, dbg_nct1008_show, inode->i_private);
+}
+
+static const struct file_operations debug_fops = {
+	.open		= dbg_nct1008_open,
+	.read		= seq_read,
+	.llseek		= seq_lseek,
+	.release	= single_release,
+};
+
+static int __init nct1008_debuginit(struct nct1008_data *nct)
+{
+	int err;
+	err = debugfs_create_file("nct1008", S_IRUGO, NULL,
+			nct, &debug_fops);
+	if (err < 0)
+		pr_err("Error: %s debugfs not supported, error=%d\n",
+			__func__, err);
+	return err;
+}
+#else
+static int __init nct1008_debuginit(struct nct1008_data *nct)
+{
+	return 0;
+}
+#endif
+
 static int nct1008_enable(struct i2c_client *client)
 {
 	struct nct1008_data *data = i2c_get_clientdata(client);
@@ -595,12 +670,12 @@ static void nct1008_power_control(struct nct1008_data *data, bool is_enable)
 {
 	int ret;
 	if (!data->nct_reg) {
-		data->nct_reg = regulator_get(data->client->dev, "vdd");
+		data->nct_reg = regulator_get(&data->client->dev, "vdd");
 		if (IS_ERR_OR_NULL(data->nct_reg)) {
 			dev_warn(&data->client->dev, "Error [%d] in"
 				"getting the regulator handle for vdd "
 				"of %s\n", (int)data->nct_reg,
-				dev_name(data->client->dev));
+				dev_name(&data->client->dev));
 			data->nct_reg = NULL;
 			return;
 		}
@@ -845,6 +920,10 @@ static int __devinit nct1008_probe(struct i2c_client *client,
 		goto error;
 	}
 
+	err = nct1008_debuginit(data);
+	if (err < 0)
+		err = 0; /* without debugfs we may continue */
+
 	/* switch to extended mode reports correct temperature
 	 * from next measurement cycle */
 	if (data->plat_data.ext_range) {