Thread (10 messages) 10 messages, 4 authors, 2009-12-25

Re: [PATCH] Add Driver for Logitech Flight System G940

From: Jiri Kosina <hidden>
Date: 2009-12-15 13:53:27

On Thu, 10 Dec 2009, Gary Stein wrote:
This is a cleaned up and updated driver for the G940.  I followed most 
of the checkpatch notes except for some of the 80 column stuff, the 
sections of where most of that code is hid-core.c also doesn't seem to 
care about it.
Yup, that's OK, thanks.

Please also fix up a few comments on various places which are wrongly 
intended, such as
+       /* maxusage should be 63, maximum fields
+               likely a better way to ensure this data is clean */
which should rather be in a format more similar to

	/* 
	 * first comment line
	 * second comment line
	 */
However, also in this code is the (int) cast fix that I send before to 
linux-kernel in ff-memless.  Without this, my driver or the normal 
logitech driver for other sticks seems to work for me.
Dmitry, will you pick that one up for your tree? 

Thanks,

-- 
Jiri Kosina
SUSE Labs, Novell Inc.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help