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.mallet@xxxxxxx> > --- > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html