From mboxrd@z Thu Jan 1 00:00:00 1970 From: greg@kroah.com (Greg KH) Date: Tue, 25 Jul 2006 21:40:57 +0000 Subject: [lm-sensors] [PATCH 1/13] [PATCH] i2c: Fix 'ignore' module Message-Id: <20060725214057.GA10155@kroah.com> List-Id: References: <11527468454006-git-send-email-greg@kroah.com> In-Reply-To: <11527468454006-git-send-email-greg@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Tue, Jul 25, 2006 at 09:05:18PM +0200, Jean Delvare wrote: > Hi all, > > > From: Mark M. Hoffman > > > > This patch fixes a bug in the handling of 'ignore' module parameters of I2C > > client drivers. > > > > Signed-off-by: Mark M. Hoffman > > Signed-off-by: Jean Delvare > > Signed-off-by: Greg Kroah-Hartman > > --- > > drivers/i2c/i2c-core.c | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c > > index a45155f..9cb277d 100644 > > --- a/drivers/i2c/i2c-core.c > > +++ b/drivers/i2c/i2c-core.c > > @@ -756,9 +756,9 @@ int i2c_probe(struct i2c_adapter *adapte > > "parameter for adapter %d, " > > "addr 0x%02x\n", adap_id, > > address_data->ignore[j + 1]); > > + ignore = 1; > > + break; > > } > > - ignore = 1; > > - break; > > } > > if (ignore) > > continue; > > I am wondering, do we want to put this fix in 2.6.17.y (and 2.6.16.y)? > It is a regression, it affects virtually every i2c chip driver and the > fix is straightforward, but OTOH it has been broken since 2.6.14-rc1 (10 > months ago) and nobody ever complained. Opinion anyone? Sounds good to me, send it to stable at kernel.org and I'll add it to the queue. thanks, greg k-h