Re: [PATCH 2/2] USB: serial: drop unused ASYNC flags

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

 



On Wed, Jan 25, 2017 at 06:22:54PM +0100, Johan Hovold wrote:
> Do not report ASYNC_SKIP_TEST or ASYNC_AUTO_IRQ as being set in
> TIOCGSERIAL handlers as these flags are not supported and do not really
> make any sense for USB serial devices in the first place.
> 
> Signed-off-by: Johan Hovold <johan@xxxxxxxxxx>
> ---
>  drivers/usb/serial/io_edgeport.c | 1 -
>  drivers/usb/serial/io_ti.c       | 1 -
>  drivers/usb/serial/mos7720.c     | 1 -
>  drivers/usb/serial/mos7840.c     | 1 -
>  drivers/usb/serial/opticon.c     | 1 -
>  drivers/usb/serial/quatech2.c    | 1 -
>  drivers/usb/serial/ssu100.c      | 1 -
>  drivers/usb/serial/whiteheat.c   | 1 -
>  8 files changed, 8 deletions(-)
> 
> diff --git a/drivers/usb/serial/io_edgeport.c b/drivers/usb/serial/io_edgeport.c
> index 8ab5f5b49ef3..92abf92e5669 100644
> --- a/drivers/usb/serial/io_edgeport.c
> +++ b/drivers/usb/serial/io_edgeport.c
> @@ -1572,7 +1572,6 @@ static int get_serial_info(struct edgeport_port *edge_port,
>  	tmp.line		= edge_port->port->minor;
>  	tmp.port		= edge_port->port->port_number;
>  	tmp.irq			= 0;
> -	tmp.flags		= ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size	= edge_port->maxTxCredits;
>  	tmp.baud_base		= 9600;
>  	tmp.close_delay		= 5*HZ;
> diff --git a/drivers/usb/serial/io_ti.c b/drivers/usb/serial/io_ti.c
> index 9a0db2965fbb..ceaeebaa6f90 100644
> --- a/drivers/usb/serial/io_ti.c
> +++ b/drivers/usb/serial/io_ti.c
> @@ -2468,7 +2468,6 @@ static int get_serial_info(struct edgeport_port *edge_port,
>  	tmp.line		= edge_port->port->minor;
>  	tmp.port		= edge_port->port->port_number;
>  	tmp.irq			= 0;
> -	tmp.flags		= ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size	= edge_port->port->bulk_out_size;
>  	tmp.baud_base		= 9600;
>  	tmp.close_delay		= 5*HZ;
> diff --git a/drivers/usb/serial/mos7720.c b/drivers/usb/serial/mos7720.c
> index acf431e02699..f075121c6e32 100644
> --- a/drivers/usb/serial/mos7720.c
> +++ b/drivers/usb/serial/mos7720.c
> @@ -1851,7 +1851,6 @@ static int get_serial_info(struct moschip_port *mos7720_port,
>  	tmp.line		= mos7720_port->port->minor;
>  	tmp.port		= mos7720_port->port->port_number;
>  	tmp.irq			= 0;
> -	tmp.flags		= ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size	= NUM_URBS * URB_TRANSFER_BUFFER_SIZE;
>  	tmp.baud_base		= 9600;
>  	tmp.close_delay		= 5*HZ;
> diff --git a/drivers/usb/serial/mos7840.c b/drivers/usb/serial/mos7840.c
> index 155006acf918..d1b92f582478 100644
> --- a/drivers/usb/serial/mos7840.c
> +++ b/drivers/usb/serial/mos7840.c
> @@ -1963,7 +1963,6 @@ static int mos7840_get_serial_info(struct moschip_port *mos7840_port,
>  	tmp.line = mos7840_port->port->minor;
>  	tmp.port = mos7840_port->port->port_number;
>  	tmp.irq = 0;
> -	tmp.flags = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size = NUM_URBS * URB_TRANSFER_BUFFER_SIZE;
>  	tmp.baud_base = 9600;
>  	tmp.close_delay = 5 * HZ;
> diff --git a/drivers/usb/serial/opticon.c b/drivers/usb/serial/opticon.c
> index b3c64f557d60..3937b9c3cc69 100644
> --- a/drivers/usb/serial/opticon.c
> +++ b/drivers/usb/serial/opticon.c
> @@ -343,7 +343,6 @@ static int get_serial_info(struct usb_serial_port *port,
>  	tmp.line		= port->minor;
>  	tmp.port		= 0;
>  	tmp.irq			= 0;
> -	tmp.flags		= ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size	= 1024;
>  	tmp.baud_base		= 9600;
>  	tmp.close_delay		= 5*HZ;
> diff --git a/drivers/usb/serial/quatech2.c b/drivers/usb/serial/quatech2.c
> index cf29128327d3..fdbb904d153f 100644
> --- a/drivers/usb/serial/quatech2.c
> +++ b/drivers/usb/serial/quatech2.c
> @@ -465,7 +465,6 @@ static int get_serial_info(struct usb_serial_port *port,
>  	tmp.line		= port->minor;
>  	tmp.port		= 0;
>  	tmp.irq			= 0;
> -	tmp.flags		= ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size	= port->bulk_out_size;
>  	tmp.baud_base		= 9600;
>  	tmp.close_delay		= 5*HZ;
> diff --git a/drivers/usb/serial/ssu100.c b/drivers/usb/serial/ssu100.c
> index 55814538ff1f..5aa7bbbeba3d 100644
> --- a/drivers/usb/serial/ssu100.c
> +++ b/drivers/usb/serial/ssu100.c
> @@ -339,7 +339,6 @@ static int get_serial_info(struct usb_serial_port *port,
>  	tmp.line		= port->minor;
>  	tmp.port		= 0;
>  	tmp.irq			= 0;
> -	tmp.flags		= ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  	tmp.xmit_fifo_size	= port->bulk_out_size;
>  	tmp.baud_base		= 9600;
>  	tmp.close_delay		= 5*HZ;
> diff --git a/drivers/usb/serial/whiteheat.c b/drivers/usb/serial/whiteheat.c
> index d3ea90bef84d..5ab65eb1dacc 100644
> --- a/drivers/usb/serial/whiteheat.c
> +++ b/drivers/usb/serial/whiteheat.c
> @@ -487,7 +487,6 @@ static int whiteheat_ioctl(struct tty_struct *tty,
>  		serstruct.type = PORT_16654;
>  		serstruct.line = port->minor;
>  		serstruct.port = port->port_number;
> -		serstruct.flags = ASYNC_SKIP_TEST | ASYNC_AUTO_IRQ;
>  		serstruct.xmit_fifo_size = kfifo_size(&port->write_fifo);
>  		serstruct.custom_divisor = 0;
>  		serstruct.baud_base = 460800;

This is probably my fault from back in 1999 or so, sorry about that...

Reviewed-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux