On Wed, May 03, 2017 at 09:23:59PM +0200, Anthony Mallet wrote:
> Commit 557aaa7ffab6 ("ft232: support the ASYNC_LOW_LATENCY
> flag") enables unprivileged users to set the FTDI latency timer,
> but there was a logic flaw that skipped sending the corresponding
> USB control message to the device.
>
> Signed-off-by: Anthony Mallet <anthony.mal...@laas.fr>
> ---
> drivers/usb/serial/ftdi_sio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/serial/ftdi_sio.c b/drivers/usb/serial/ftdi_sio.c
> index c540de1..bf04eea 100644
> --- a/drivers/usb/serial/ftdi_sio.c
> +++ b/drivers/usb/serial/ftdi_sio.c
> @@ -1505,9 +1505,9 @@ static int set_serial_info(struct tty_struct *tty,
> (new_serial.flags & ASYNC_FLAGS));
> priv->custom_divisor = new_serial.custom_divisor;
>
> +check_and_exit:
> write_latency_timer(port);
>
> -check_and_exit:
> if ((old_priv.flags & ASYNC_SPD_MASK) !=
> (priv->flags & ASYNC_SPD_MASK)) {
> if ((priv->flags & ASYNC_SPD_MASK) == ASYNC_SPD_HI)
Patch still looks good, except that now all leading tabs have been
replaced with spaces (possibly a mailer issue) so that it no longer
applies.
One more try? :)
Thanks,
Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
