[PATCH] I2C: Kill address ranges in non-sensors i2c chip drivers

Some months ago, you killed the address ranges mechanism from all
sensors i2c chip drivers (both the module parameters and the in-code
address lists). I think it was a very good move, as the ranges can
easily be replaced by individual addresses, and this allowed for
significant cleanups in the i2c core (let alone the impressive size
shrink for all these drivers).

Unfortunately you did not do the same for non-sensors i2c chip drivers.
These need the address ranges even less, so we could get rid of the
ranges here as well for another significant i2c core cleanup. Here comes
a patch which does just that. Since the process is exactly the same as
what you did for the other drivers set already, I did not split this one
in parts.

A documentation update is included.

The change saves 308 bytes in the i2c core, and an average 1382 bytes
for chip drivers which use I2C_CLIENT_INSMOD, 126 bytes for those which
do not.

This change is required if we want to merge the sensors and non-sensors
i2c code (and we want to do this).

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Index: gregkh-2.6/Documentation/i2c/writing-clients
===================================================================

4 files changed, 0 insertions, 42 deletions

diff --git a/drivers/i2c/chips/isp1301_omap.c b/drivers/i2c/chips/isp1301_omap.c
index 7f29a8aff165..354a26295672 100644
--- a/drivers/i2c/chips/isp1301_omap.c
+++ b/drivers/i2c/chips/isp1301_omap.c
@@ -145,7 +145,6 @@ static inline void notresponding(struct isp1301 *isp)
 static unsigned short normal_i2c[] = {
         ISP_BASE, ISP_BASE + 1,
         I2C_CLIENT_END };
-static unsigned short normal_i2c_range[] = { I2C_CLIENT_END };
 
 I2C_CLIENT_INSMOD;
 
diff --git a/drivers/i2c/chips/m41t00.c b/drivers/i2c/chips/m41t00.c
index e771566dffa8..5e463c47bfbc 100644
--- a/drivers/i2c/chips/m41t00.c
+++ b/drivers/i2c/chips/m41t00.c
@@ -40,11 +40,8 @@ static unsigned short normal_addr[] = { 0x68, I2C_CLIENT_END };
 
 static struct i2c_client_address_data addr_data = {
         .normal_i2c             = normal_addr,
-        .normal_i2c_range       = ignore,
         .probe                  = ignore,
-        .probe_range            = ignore,
         .ignore                 = ignore,
-        .ignore_range           = ignore,
         .force                  = ignore,
 };
 
diff --git a/drivers/i2c/chips/rtc8564.c b/drivers/i2c/chips/rtc8564.c
index 5a9deddb626b..30f553e73700 100644
--- a/drivers/i2c/chips/rtc8564.c
+++ b/drivers/i2c/chips/rtc8564.c
@@ -66,11 +66,8 @@ static unsigned short normal_addr[] = { 0x51, I2C_CLIENT_END };
 
 static struct i2c_client_address_data addr_data = {
         .normal_i2c             = normal_addr,
-        .normal_i2c_range       = ignore,
         .probe                  = ignore,
-        .probe_range            = ignore,
         .ignore                 = ignore,
-        .ignore_range           = ignore,
         .force                  = ignore,
 };
 
diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
index a22e53badacb..4cc8c9f7211c 100644
--- a/drivers/i2c/i2c-core.c
+++ b/drivers/i2c/i2c-core.c
@@ -742,18 +742,6 @@ int i2c_probe(struct i2c_adapter *adapter,
                                 found = 1;
                         }
                 }
-                for (i = 0;
-                     !found && (address_data->ignore_range[i] != I2C_CLIENT_END);
-                     i += 3) {
-                        if (((adap_id == address_data->ignore_range[i]) ||
-                            ((address_data->ignore_range[i]==ANY_I2C_BUS))) &&
-                            (addr >= address_data->ignore_range[i+1]) &&
-                            (addr <= address_data->ignore_range[i+2])) {
-                                dev_dbg(&adapter->dev, "found ignore_range parameter for adapter %d, "
-                                        "addr %04x\n", adap_id,addr);
-                                found = 1;
-                        }
-                }
                 if (found) 
                         continue;
 
@@ -770,17 +758,6 @@ int i2c_probe(struct i2c_adapter *adapter,
                 }
 
                 for (i = 0;
-                     !found && (address_data->normal_i2c_range[i] != I2C_CLIENT_END);
-                     i += 2) {
-                        if ((addr >= address_data->normal_i2c_range[i]) &&
-                            (addr <= address_data->normal_i2c_range[i+1])) {
-                                found = 1;
-                                dev_dbg(&adapter->dev, "found normal i2c_range entry for adapter %d, "
-                                        "addr %04x\n", adap_id,addr);
-                        }
-                }
-
-                for (i = 0;
                      !found && (address_data->probe[i] != I2C_CLIENT_END);
                      i += 2) {
                         if (((adap_id == address_data->probe[i]) ||
@@ -791,18 +768,6 @@ int i2c_probe(struct i2c_adapter *adapter,
                                         "addr %04x\n", adap_id,addr);
                         }
                 }
-                for (i = 0;
-                     !found && (address_data->probe_range[i] != I2C_CLIENT_END);
-                     i += 3) {
-                        if (((adap_id == address_data->probe_range[i]) ||
-                           (address_data->probe_range[i] == ANY_I2C_BUS)) &&
-                           (addr >= address_data->probe_range[i+1]) &&
-                           (addr <= address_data->probe_range[i+2])) {
-                                found = 1;
-                                dev_dbg(&adapter->dev, "found probe_range parameter for adapter %d, "
-                                        "addr %04x\n", adap_id,addr);
-                        }
-                }
                 if (!found) 
                         continue;