lkml.org 
[lkml]   [2003]   [Mar]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: w83781d i2c driver updated for 2.5.66 (without sysfs support)
On Wed, Mar 26, 2003 at 09:04:33PM +0200, Martin Schlemmer wrote:
> Hi
>
> Ok, this is the w83781d driver updated for 2.5.66bk2. It works
> over here.

Looks nice, thanks.

Some of the nasty casts should be fixed up though. Stuff like:

> + ERROR7:
> + if (!is_isa)
> + i2c_detach_client(&
> + (((struct w83781d_data
> + *) (i2c_get_clientdata(new_client)))->
> + lm75[1]));
> + ERROR6:
> + if (!is_isa)
> + i2c_detach_client(&
> + (((struct w83781d_data
> + *) (i2c_get_clientdata(new_client)))->
> + lm75[0]));
> + ERROR5:
> + if (!is_isa)
> + kfree(((struct w83781d_data *) (i2c_get_clientdata(new_client)))->
> + lm75);

Is just obnoxious :)

I'll hold off sending this driver to Linus until it gets cleaned up with
sysfs entries, as I'd rather not pollute /proc and sysctls anymore.

thanks,

greg k-h
-
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/

\
 
 \ /
  Last update: 2005-03-22 13:34    [W:0.387 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site