Re: [PATCH v2] staging: gdm7240: a TTY rewrite according to the latest TTY APIs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Aug 13, 2013 at 07:46:01PM +0900, Won Kang wrote:
> Removed the old style reference countings and termios.
> Renamed variables to meaninful ones.
> 
> Fixed mis-use of mutex for gdm_table. gdm_table is refered to only
> inside tty_install and port destrcut, and usb callbacks use internal
> reference which was saved during urb submission
> 
> Signed-off-by: Won Kang <wonkang@xxxxxxxxxxx>
> ---
> v2: fixed mutex bug, added ChangeLog, added missing kfree
>  drivers/staging/gdm724x/ChangeLog |   62 ++++++++

Why a changelog?  The kernel has a changelog, in the git commit log, if
every individual driver had a separate changelog, the tree would be a
mess, right?  It also looks like you included things that are already in
the kernel changelog (patches from me and others), so this is duplicated
effort.

>  drivers/staging/gdm724x/gdm_mux.c |    9 +-
>  drivers/staging/gdm724x/gdm_mux.h |   14 +-
>  drivers/staging/gdm724x/gdm_tty.c |  295 +++++++++++++++++--------------------
>  drivers/staging/gdm724x/gdm_tty.h |   36 +++--
>  5 files changed, 237 insertions(+), 179 deletions(-)
>  create mode 100644 drivers/staging/gdm724x/ChangeLog
> 
> diff --git a/drivers/staging/gdm724x/ChangeLog b/drivers/staging/gdm724x/ChangeLog
> new file mode 100644
> index 0000000..506e60e
> --- /dev/null
> +++ b/drivers/staging/gdm724x/ChangeLog
> @@ -0,0 +1,62 @@
> +2013-08-13  Won Kang <wonkang@xxxxxxxxxxx>
> +	* Fixed mis-use of mutex for gdm_table. gdm_table is refered to only
> +	* inside tty_install and port destrcut, and usb callbacks use internal
> +	* reference which was saved during urb submission
> +
> +2013-08-09  Won Kang <wonkang@xxxxxxxxxxx>
> +	* A rewrite was needed to properly use latest TTY AAPI
> +	*
> +	* Original TTY codes were written from kernel 2.6.x, doing tty reference
> +	* counting within the this driver. This is fixed to use tty core APIs.
> +	*
> +	* The usage count of tty port is no longer handled by the driver.
> +	*
> +	* termios members of tty_operations as the ports are virtual and
> +	* have no real effect.
> +	*
> +	* Renamed variables to be more clear as they were confusing due to the
> +	* similarities with the tty core structures.

Please send these two patches, not mushed together as one, so I can
apply them that way.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux