On 06/16/2010 07:41 AM, Jean Delvare wrote:
Now that bus_for_each_drv() is no longer __must_check, we can drop the
dummy variable that was used to store the returned value.

Signed-off-by: Jean Delvare<[email protected]>
Cc: Justin P. Mattock<[email protected]>
---
Anticipating that my other patch will be accepted...

  drivers/i2c/i2c-core.c |    5 ++---
  1 file changed, 2 insertions(+), 3 deletions(-)

--- linux-2.6.35-rc3.orig/drivers/i2c/i2c-core.c        2010-06-16 
12:44:52.000000000 +0200
+++ linux-2.6.35-rc3/drivers/i2c/i2c-core.c     2010-06-16 12:44:54.000000000 
+0200
@@ -783,7 +783,7 @@ static int __process_new_adapter(struct

  static int i2c_register_adapter(struct i2c_adapter *adap)
  {
-       int res = 0, dummy;
+       int res = 0;

        /* Can't register until after driver model init */
        if (unlikely(WARN_ON(!i2c_bus_type.p))) {
@@ -821,8 +821,7 @@ static int i2c_register_adapter(struct i

        /* Notify drivers */
        mutex_lock(&core_lock);
-       dummy = bus_for_each_drv(&i2c_bus_type, NULL, adap,
-                                __process_new_adapter);
+       bus_for_each_drv(&i2c_bus_type, NULL, adap, __process_new_adapter);
        mutex_unlock(&core_lock);

        return 0;




cool thanks!! applied the above patch and now the warning is no longer there. Here is what I see:

  LD      kernel/built-in.o
  CC      drivers/i2c/i2c-core.o
  LD      drivers/i2c/built-in.o
  CC [M]  drivers/uio/uio.o

cheers,

Justin P. Mattock


--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to