Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262806AbVDHNCg (ORCPT ); Fri, 8 Apr 2005 09:02:36 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262809AbVDHNCB (ORCPT ); Fri, 8 Apr 2005 09:02:01 -0400 Received: from smtp.seznam.cz ([212.80.76.43]:21912 "HELO smtp.seznam.cz") by vger.kernel.org with SMTP id S262806AbVDHNAU (ORCPT ); Fri, 8 Apr 2005 09:00:20 -0400 Date: Fri, 8 Apr 2005 15:00:21 +0200 To: Greg KH Cc: Jean Delvare , LKML , LM Sensors , James Chapman Subject: Re: [PATCH] ds1337 1/4 Message-ID: <20050408130021.GA7054@orphique> References: <20050407111631.GA21190@orphique> <20050407142804.GA11284@orphique> <20050407211839.GA5357@kroah.com> <20050407231758.GB27226@orphique> <20050407233628.GA6703@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050407233628.GA6703@kroah.com> User-Agent: Mutt/1.5.6+20040907i From: Ladislav Michl Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1688 Lines: 47 On Thu, Apr 07, 2005 at 04:36:29PM -0700, Greg KH wrote: > Oops, you forgot to add a Signed-off-by: line for every patch, as per > Documentation/SubmittingPatches. Care to redo them? Here it is (I'm sorry about that). Use i2c_transfer to send message, so we get proper bus locking. Signed-off-by: Ladislav Michl ===== 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 * - * 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 majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/