All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keerthy <j-keerthy@ti.com>
To: <tony@atomide.com>, <milo.kim@ti.com>, <lee.jones@linaro.org>
Cc: <j-keerthy@ti.com>, <linux-omap@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: [PATCH 1/9] mfd: tps65217: Remove redundant i2c_device_id
Date: Sun, 4 Jun 2017 14:48:36 +0530	[thread overview]
Message-ID: <1496567924-4278-2-git-send-email-j-keerthy@ti.com> (raw)
In-Reply-To: <1496567924-4278-1-git-send-email-j-keerthy@ti.com>

With the introduction of probe_new function the i2c_device_id
is no longer needed. Hence the remove the same and use probe_new
instead of probe.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---

Boot tested on am335x-bone and checked for regulator registrations.

 drivers/mfd/tps65217.c | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
index f769c7d..c2e8385 100644
--- a/drivers/mfd/tps65217.c
+++ b/drivers/mfd/tps65217.c
@@ -316,12 +316,11 @@ static bool tps65217_volatile_reg(struct device *dev, unsigned int reg)
 };
 MODULE_DEVICE_TABLE(of, tps65217_of_match);
 
-static int tps65217_probe(struct i2c_client *client,
-				const struct i2c_device_id *ids)
+static int tps65217_probe(struct i2c_client *client)
 {
 	struct tps65217 *tps;
 	unsigned int version;
-	unsigned long chip_id = ids->driver_data;
+	unsigned long chip_id = 0;
 	const struct of_device_id *match;
 	bool status_off = false;
 	int ret;
@@ -418,19 +417,12 @@ static int tps65217_remove(struct i2c_client *client)
 	return 0;
 }
 
-static const struct i2c_device_id tps65217_id_table[] = {
-	{"tps65217", TPS65217},
-	{ /* sentinel */ }
-};
-MODULE_DEVICE_TABLE(i2c, tps65217_id_table);
-
 static struct i2c_driver tps65217_driver = {
 	.driver		= {
 		.name	= "tps65217",
 		.of_match_table = tps65217_of_match,
 	},
-	.id_table	= tps65217_id_table,
-	.probe		= tps65217_probe,
+	.probe_new	= tps65217_probe,
 	.remove		= tps65217_remove,
 };
 
-- 
1.9.1

WARNING: multiple messages have this Message-ID (diff)
From: Keerthy <j-keerthy@ti.com>
To: tony@atomide.com, milo.kim@ti.com, lee.jones@linaro.org
Cc: j-keerthy@ti.com, linux-omap@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH 1/9] mfd: tps65217: Remove redundant i2c_device_id
Date: Sun, 4 Jun 2017 14:48:36 +0530	[thread overview]
Message-ID: <1496567924-4278-2-git-send-email-j-keerthy@ti.com> (raw)
In-Reply-To: <1496567924-4278-1-git-send-email-j-keerthy@ti.com>

With the introduction of probe_new function the i2c_device_id
is no longer needed. Hence the remove the same and use probe_new
instead of probe.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---

Boot tested on am335x-bone and checked for regulator registrations.

 drivers/mfd/tps65217.c | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
index f769c7d..c2e8385 100644
--- a/drivers/mfd/tps65217.c
+++ b/drivers/mfd/tps65217.c
@@ -316,12 +316,11 @@ static bool tps65217_volatile_reg(struct device *dev, unsigned int reg)
 };
 MODULE_DEVICE_TABLE(of, tps65217_of_match);
 
-static int tps65217_probe(struct i2c_client *client,
-				const struct i2c_device_id *ids)
+static int tps65217_probe(struct i2c_client *client)
 {
 	struct tps65217 *tps;
 	unsigned int version;
-	unsigned long chip_id = ids->driver_data;
+	unsigned long chip_id = 0;
 	const struct of_device_id *match;
 	bool status_off = false;
 	int ret;
@@ -418,19 +417,12 @@ static int tps65217_remove(struct i2c_client *client)
 	return 0;
 }
 
-static const struct i2c_device_id tps65217_id_table[] = {
-	{"tps65217", TPS65217},
-	{ /* sentinel */ }
-};
-MODULE_DEVICE_TABLE(i2c, tps65217_id_table);
-
 static struct i2c_driver tps65217_driver = {
 	.driver		= {
 		.name	= "tps65217",
 		.of_match_table = tps65217_of_match,
 	},
-	.id_table	= tps65217_id_table,
-	.probe		= tps65217_probe,
+	.probe_new	= tps65217_probe,
 	.remove		= tps65217_remove,
 };
 
-- 
1.9.1

  reply	other threads:[~2017-06-04  9:19 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-04  9:18 [PATCH 0/9] mfd: Remove redundant i2c_device_id Keerthy
2017-06-04  9:18 ` Keerthy
2017-06-04  9:18 ` Keerthy [this message]
2017-06-04  9:18   ` [PATCH 1/9] mfd: tps65217: " Keerthy
2017-06-04  9:18 ` [PATCH 2/9] mfd: tps65218: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 3/9] mfd: palmas: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 4/9] mfd: lp873x: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 5/9] mfd: lp8788: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 6/9] mfd: lp3943: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 7/9] mfd: tps65086: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 8/9] mfd: tps65090: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-04  9:18 ` [PATCH 9/9] mfd: tps6586x: " Keerthy
2017-06-04  9:18   ` Keerthy
2017-06-05 10:37 ` [PATCH 0/9] mfd: " Lee Jones
2017-06-05 14:37   ` Keerthy
2017-06-05 14:37     ` Keerthy
2017-06-06  7:17     ` Lee Jones
2017-06-06  7:30       ` Keerthy
2017-06-06  7:30         ` Keerthy
2017-06-06  7:48         ` Keerthy
2017-06-06  7:48           ` Keerthy
2017-06-06 10:44           ` Tony Lindgren
2017-06-06 11:01             ` Keerthy
2017-06-06 11:01               ` Keerthy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1496567924-4278-2-git-send-email-j-keerthy@ti.com \
    --to=j-keerthy@ti.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=milo.kim@ti.com \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.