[PATCH 2.6] Prevent i2c-dev oops with debug
Greg KH
greg at kroah.com
Wed Mar 10 00:02:17 CET 2004
On Tue, Mar 02, 2004 at 10:49:09PM +0100, Jean Delvare wrote:
> Hi Greg,
>
> Looks like i2c-dev suffers the same problem with dev_dbg as i2c-core did
> twice. Here is a patch that fixed a oops I and another user were
> experiencing when running sensors-detect.
>
> Please note that I have no idea if this is the correct way to fix the
> problem. I mostly copied the fix from i2c-core. I don't quite understand
> what we are supposed to pass to dev_dbg.
Hm, I'll apply this for now. I'm working on fixing the i2c-dev driver
up a bunch more, but this is good for now.
> BTW, I read that you don't plan to send any i2c patch to Linus before
> 2.6.4. I agree that we need to calm down a bit ;) but there are a
> several oops preventing patches in my list (i2c-core, i2c-i801,
> i2c-sis5595, i2c-ali1535 and now i2c-dev). Shouldn't these simple but
> important fixes be sent to Linus?
They only happen when people enable debugging, right? That's a pretty
simple fix :)
> Also, I sent a mail to you some times ago about classes. The fact that 3
> bus drivers are unusable at the moment sounds like an important issue to
> me, and we would need to decide about what should be done.
Sorry, getting behind these days... I saw your recent patch and that
looks good for now. I'll put it in the queue to apply.
thanks,
greg k-h
More information about the lm-sensors
mailing list