On Wed, Dec 13, 2017 at 04:44:55PM +0300, Mikhail Zaytsev wrote: > The patch removes unused TIOCSSERIAL ioctl case and adds the default block > to the switch. > > Signed-off-by: Mikhail Zaytsev <flashed@xxxxxxx> > --- > drivers/usb/serial/ark3116.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/usb/serial/ark3116.c b/drivers/usb/serial/ark3116.c > index 23d46ef87..2e957c76f 100644 > --- a/drivers/usb/serial/ark3116.c > +++ b/drivers/usb/serial/ark3116.c > @@ -418,10 +418,8 @@ static int ark3116_ioctl(struct tty_struct *tty, > return -EFAULT; > > return 0; > - case TIOCSSERIAL: > - if (copy_from_user(&serstruct, user_arg, sizeof(serstruct))) > - return -EFAULT; > - return 0; > + default: > + break; > } > > return -ENOIOCTLCMD; This will make the ioctl return -ENOTTY to user space (e.g. setserial), which I guess should be fine as TIOCSSERIAL really isn't supported for these devices currently. But you should at least mention this changed behaviour in the commit message. Please also drop the ".c" part from the subject prefix while at it. 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
