Re: [PATCH] therm_adt746x: Don't access non-existing register

From: Michel Dänzer
Date: Thu Dec 03 2009 - 05:24:33 EST


On Wed, 2009-12-02 at 10:10 +0100, Jean Delvare wrote:
> On Wed, 02 Dec 2009 09:54:02 +0100, Michel DÃnzer wrote:
> > On Wed, 2009-12-02 at 09:19 +0100, Jean Delvare wrote:
> > > Michel, Darrick,
> > >
> > > Apparently you are using the therm_adt746x driver, so maybe you would
> > > be interested in testing and carrying the following patch. I've sent it
> > > to Colin Leroy, who is listed as the maintainer for this driver, 1.5
> > > month ago, but did not hear back.
> >
> > The driver seems to load and work fine here (on a PowerBook5,8) with the
> > patch, so feel free to add a Tested-By: line for me.
>
> Will do, thanks.
>
> BTW, I also have i2c-powermac patches queued for 2.6.33 if you want to
> give them a try:
>
> ftp://ftp.kernel.org/pub/linux/kernel/people/jdelvare/linux-2.6/jdelvare-i2c/i2c-powermac-*.patch
>
> (Don't look for patch 5/6, it was broken so I discarded it.)

My PowerBook5,8 boots and works fine with all of these applied and
CONFIG_I2C_POWERMAC=y, so for all of them:

Tested-by: Michel DÃnzer <michel@xxxxxxxxxxx>


--
Earthling Michel DÃnzer | http://www.vmware.com
Libre software enthusiast | Debian, X and DRI developer

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/