[PATCH linux dev-4.10 1/4] i2c: fsi: Remove ida counter

Joel Stanley joel at jms.id.au
Fri Jul 7 00:35:00 AEST 2017


It's not clear what the intention is. Regardless, it is not necessary so
remove it.

Signed-off-by: Joel Stanley <joel at jms.id.au>
---
 drivers/i2c/busses/i2c-fsi.c | 17 +++--------------
 1 file changed, 3 insertions(+), 14 deletions(-)

diff --git a/drivers/i2c/busses/i2c-fsi.c b/drivers/i2c/busses/i2c-fsi.c
index 01816466ec86..da045777624e 100644
--- a/drivers/i2c/busses/i2c-fsi.c
+++ b/drivers/i2c/busses/i2c-fsi.c
@@ -135,7 +135,6 @@ struct fsi_i2c_master {
 	int			idx;
 	u8			fifo_size;
 	struct list_head	ports;
-	struct ida		ida;
 	wait_queue_head_t	wait;
 	struct semaphore	lock;
 };
@@ -554,7 +553,7 @@ static int fsi_i2c_probe(struct device *dev)
 	struct fsi_i2c_master *i2c;
 	struct fsi_i2c_port *port;
 	struct device_node *np;
-	int rc, idx;
+	int rc;
 	u32 port_no;
 
 	i2c = devm_kzalloc(dev, sizeof(*i2c), GFP_KERNEL);
@@ -565,7 +564,6 @@ static int fsi_i2c_probe(struct device *dev)
 	sema_init(&i2c->lock, 1);
 	i2c->fsi = to_fsi_dev(dev);
 	i2c->idx = ida_simple_get(&fsi_i2c_ida, 1, INT_MAX, GFP_KERNEL);
-	ida_init(&i2c->ida);
 	INIT_LIST_HEAD(&i2c->ports);
 
 	/* add adapter for each i2c port of the master */
@@ -574,12 +572,6 @@ static int fsi_i2c_probe(struct device *dev)
 		if (rc || port_no > 0xFFFF)
 			continue;
 
-		/* make sure we don't overlap index with a buggy dts */
-		idx = ida_simple_get(&i2c->ida, port_no,
-				port_no + 1, GFP_KERNEL);
-		if (idx < 0)
-			continue;
-
 		port = devm_kzalloc(dev, sizeof(*port), GFP_KERNEL);
 		if (!port)
 			return -ENOMEM;
@@ -596,9 +588,8 @@ static int fsi_i2c_probe(struct device *dev)
 		port->adapter.nr = (i2c->idx * I2C_MASTER_NR_OFFSET) +
 			port_no;
 
-		snprintf(port->adapter.name,
-				sizeof(port->adapter.name),
-				"fsi%d_i2c%u", i2c->idx, port_no);
+		snprintf(port->adapter.name, sizeof(port->adapter.name),
+				"fsi_i2c%u", port_no);
 
 		rc = i2c_add_numbered_adapter(&port->adapter);
 		if (rc < 0)
@@ -625,8 +616,6 @@ static int fsi_i2c_remove(struct device *dev)
 		i2c_del_adapter(&port->adapter);
 	}
 
-	ida_destroy(&i2c->ida);
-
 	ida_simple_remove(&fsi_i2c_ida, i2c->idx);
 
 	return 0;
-- 
2.13.2



More information about the openbmc mailing list