lkml.org 
[lkml]   [2015]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v4] USB: serial: add Moxa UPORT 11x0 driver
Hi Johan

Ok, I'll include a changelog in future patches. Here is the current changlog :

Changes in v2:
* Use generic implementation for read/write code
* Use TIOCSRS485/TIOCGRS485 instead of moxa specific ioctl
* Other problems pointed out by Johan review

Changes in v3:
* Drop unused defines
* Fix style problems
* Remove port identification code since this driver is for one-port devices only
* Fix debug messages
* Use a mutex to protect shadow_mc
* Use usb_serial_generic_open and close
* Other problems pointed out by Johan review

Changes in v4:
* mxu1_startup return -ENODEV on purpose if the firmware was downloaded.
It avoids port_probe to fail since the device is reset.

Thanks,
Mathieu

On Wed, Nov 11, 2015 at 10:40:04AM +0100, Johan Hovold wrote:
> On Wed, Nov 11, 2015 at 10:35:47AM +0100, Mathieu OTHACEHE wrote:
> > Add a driver which supports :
> >
> > - UPort 1110 : 1 port RS-232 USB to Serial Hub.
> > - UPort 1130 : 1 port RS-422/485 USB to Serial Hub.
> > - UPort 1130I : 1 port RS-422/485 USB to Serial Hub with Isolation.
> > - UPort 1150 : 1 port RS-232/422/485 USB to Serial Hub.
> > - UPort 1150I : 1 port RS-232/422/485 USB to Serial Hub with Isolation.
> >
> > This driver is based on GPL MOXA driver written by Hen Huang and available
> > on MOXA website. The original driver was based on io_ti serial driver.
> >
> > Signed-off-by: Mathieu OTHACEHE <m.othacehe@gmail.com>
> > ---
>
> Thanks for the v4. It'll be a few days still before I have a chance to
> review it.
>
> What changed since v3 (and v2)?
>
> Please include a summary of the changes below the cutoff line (i.e.
> here) when you send a new revision of a patch.
>
> Thanks,
> Johan


\
 
 \ /
  Last update: 2015-11-11 16:01    [W:0.139 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site