On Thu, Apr 07, 2005 at 02:18:39PM -0700, Greg KH wrote:
> Jean's point is that you should send an individual patch for each type
> of individual change.  It's ok to say "patch 3 requires you to have
> applied patches 1 and 2" and so on.  Please split this up better.

Here it is...

Use i2c_transfer to send message, so we get proper bus locking.

===== drivers/i2c/chips/ds1337.c 1.1 vs edited =====
--- 1.1/drivers/i2c/chips/ds1337.c      2005-03-31 22:58:08 +02:00
+++ edited/drivers/i2c/chips/ds1337.c   2005-04-08 00:18:45 +02:00
@@ -3,7 +3,7 @@
  *
  *  Copyright (C) 2005 James Chapman <[EMAIL PROTECTED]>
  *
- *     based on linux/drivers/acron/char/pcf8583.c
+ *     based on linux/drivers/acorn/char/pcf8583.c
  *  Copyright (C) 2000 Russell King
  *
  * This program is free software; you can redistribute it and/or modify
@@ -119,8 +119,7 @@
        msg[1].len = sizeof(buf);
        msg[1].buf = &buf[0];
 
-       result = client->adapter->algo->master_xfer(client->adapter,
-                                                   &msg[0], 2);
+       result = i2c_transfer(client->adapter, msg, 2);
 
        dev_dbg(&client->adapter->dev,
                "%s: [%d] %02x %02x %02x %02x %02x %02x %02x\n",
@@ -194,8 +193,7 @@
        msg[0].len = sizeof(buf);
        msg[0].buf = &buf[0];
 
-       result = client->adapter->algo->master_xfer(client->adapter,
-                                                   &msg[0], 1);
+       result = i2c_transfer(client->adapter, msg, 1);
        if (result < 0) {
                dev_err(&client->adapter->dev, "ds1337[%d]: error "
                        "writing data! %d\n", data->id, result);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to