Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932528Ab1CWKKO (ORCPT ); Wed, 23 Mar 2011 06:10:14 -0400 Received: from zone0.gcu-squad.org ([212.85.147.21]:29356 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756158Ab1CWKKL (ORCPT ); Wed, 23 Mar 2011 06:10:11 -0400 Date: Wed, 23 Mar 2011 11:09:56 +0100 From: Jean Delvare To: Jiri Slaby Cc: jirislaby@gmail.com, linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org, Tomoya MORINAGA , Ben Dooks Subject: Re: [PATCH 5/7] I2C: eg20t, remove unused variable Message-ID: <20110323110956.2edf6f33@endymion.delvare> In-Reply-To: <1300870200-24405-5-git-send-email-jslaby@suse.cz> References: <1300870200-24405-1-git-send-email-jslaby@suse.cz> <1300870200-24405-5-git-send-email-jslaby@suse.cz> X-Mailer: Claws Mail 3.7.5 (GTK+ 2.20.1; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2047 Lines: 56 Hi Jiri, On Wed, 23 Mar 2011 09:49:58 +0100, Jiri Slaby wrote: > drivers/i2c/busses/i2c-eg20t.c:684:2: warning: Value stored to 'msglen' is never read > msglen = (pmsg->len) - (subaddrlen + 1); > ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > Signed-off-by: Jiri Slaby > Cc: Jean Delvare > Cc: linux-i2c@vger.kernel.org > --- > drivers/i2c/busses/i2c-eg20t.c | 8 -------- > 1 files changed, 0 insertions(+), 8 deletions(-) This driver is under Ben Dooks's jurisdiction, not mine. You should get this patch reviewed by him and Tomoya MORINAGA (both Cc'd.) It may be that your simple fix is correct, but it may also be that the unused variables should be used somewhere in the code for correctness. I can't tell. > > diff --git a/drivers/i2c/busses/i2c-eg20t.c b/drivers/i2c/busses/i2c-eg20t.c > index 878a120..ea73252 100644 > --- a/drivers/i2c/busses/i2c-eg20t.c > +++ b/drivers/i2c/busses/i2c-eg20t.c > @@ -651,8 +651,6 @@ static s32 pch_i2c_xfer(struct i2c_adapter *i2c_adap, > struct i2c_msg *pmsg; > u32 i = 0; > u32 status; > - u32 msglen; > - u32 subaddrlen; > s32 ret; > > struct i2c_algo_pch_data *adap = i2c_adap->algo_data; > @@ -676,12 +674,6 @@ static s32 pch_i2c_xfer(struct i2c_adapter *i2c_adap, > status = pmsg->flags; > pch_dbg(adap, > "After invoking I2C_MODE_SEL :flag= 0x%x\n", status); > - /* calculate sub address length and message length */ > - /* these are applicable only for buffer mode */ > - subaddrlen = pmsg->buf[0]; > - /* calculate actual message length excluding > - * the sub address fields */ > - msglen = (pmsg->len) - (subaddrlen + 1); > if (status & (I2C_M_RD)) { > pch_dbg(adap, "invoking pch_i2c_readbytes\n"); > ret = pch_i2c_readbytes(i2c_adap, pmsg, (i + 1 == num), -- Jean Delvare -- 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/