[lkml]   [2006]   [Aug]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] Moschip 7840 USB-Serial Driver
On Tue, Aug 29, 2006 at 02:31:17AM -0500, Paul B Schroeder wrote:
> Worked with the tech support folks at Moschip Semiconductor to get the
> driver working on the latest kernel. We've been using it for a bit now and
> it appears to be working well. They're okay with kernel inclusion of the
> driver. I've cleaned it up a bit and put this patch together.
> The patch is against 2.6.18-rc5. Please apply, it can be found here:

Great, but can you submit it as per the Documentation/SubmittingPatches

Also, I've included the following text at the top of the file in my port
of the mos7720 driver from the same developers. It also applies to this
driver and you should add it too:

* Originally based on drivers/usb/serial/io_edgeport.c which is:
* Copyright (C) 2000 Inside Out Networks, All rights reserved.
* Copyright (C) 2001-2002 Greg Kroah-Hartman <>

You can see the same function names and comments, as well as variable
usages where they don't really need to be used.

Can you also please clean up those functions to be more kernel-like, and
fix the variable names to follow the coding style rules of the kernel?

The #defines at the start of the file also can be removed, as they are
not needed (TRUE and FALSE don't belong in a driver, use 0 and -ERR

Also, look at the variables in struct moschip_port. A lot of them can
be simply removed as they are never used. The wait queues are one
example of these. And the use of the "__" type of variables should be
examined, as I don't think it's necessary as these variables never cross
the kernel/user boundry.

There are also a lot of global variables that should be static.

I'll go over the rest when you submit it in an email message so it can
be quoted.

It's a great start though, I remember what the original version looked
like :)


greg k-h
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2006-08-29 09:55    [W:0.033 / U:1.512 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site