Re: [PATCH 4/5] USB : serial : invoke dcd_change ldisc's handler.

[Greg KH <gregkh@linuxfoundation.org>]

On Mon, Sep 09, 2013 at 06:01:19PM +0200, Paul Chavent wrote:
> In order to have the PPS line discipline to work with usb devices.
> 
> Signed-off-by: Paul Chavent <paul.chavent@onera.fr>
> ---
>  drivers/usb/serial/generic.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c
> index 91f0592..a18a086 100644
> --- a/drivers/usb/serial/generic.c
> +++ b/drivers/usb/serial/generic.c
> @@ -567,6 +567,13 @@ void usb_serial_handle_dcd_change(struct usb_serial_port *usb_port,
>  {
>  	struct tty_port *port = &usb_port->port;
>  	struct tty_struct *tty = tty_port_tty_get(port);
> +	struct tty_ldisc *ld = tty ? tty_ldisc_ref(tty) : NULL;
> +
> +	if (ld) {

Minor nit, I hate the ? : mode of C, and as you are doing another if
statement right after it, it's pretty redundant here.  So can you just
merge them together into one if chain?

thanks,

greg k-h