mbox series

[0/2] USB: serial: allow hung up ports to be suspended

Message ID 20210910121128.18664-1-johan@kernel.org (mailing list archive)
Headers show
Series USB: serial: allow hung up ports to be suspended | expand

Message

Johan Hovold Sept. 10, 2021, 12:11 p.m. UTC 
User space can keep a tty open indefinitely and that should not prevent
a hung up port and its USB device from being runtime suspended.

Also clean up a few related error labels in a preparatory patch.

Johan


Johan Hovold (2):
  USB: serial: clean up core error labels
  USB: serial: allow hung up ports to be suspended

 drivers/usb/serial/usb-serial.c | 59 +++++++++++++++++----------------
 1 file changed, 30 insertions(+), 29 deletions(-)

Comments

Greg KH Sept. 10, 2021, 12:41 p.m. UTC | #1 
On Fri, Sep 10, 2021 at 02:11:26PM +0200, Johan Hovold wrote:
> User space can keep a tty open indefinitely and that should not prevent
> a hung up port and its USB device from being runtime suspended.
> 
> Also clean up a few related error labels in a preparatory patch.
> 
> Johan
> 
> 
> Johan Hovold (2):
>   USB: serial: clean up core error labels
>   USB: serial: allow hung up ports to be suspended
> 
>  drivers/usb/serial/usb-serial.c | 59 +++++++++++++++++----------------
>  1 file changed, 30 insertions(+), 29 deletions(-)
> 
> -- 
> 2.32.0
> 

Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Johan Hovold Sept. 20, 2021, 9:50 a.m. UTC | #2 
On Fri, Sep 10, 2021 at 02:41:24PM +0200, Greg Kroah-Hartman wrote:
> On Fri, Sep 10, 2021 at 02:11:26PM +0200, Johan Hovold wrote:
> > User space can keep a tty open indefinitely and that should not prevent
> > a hung up port and its USB device from being runtime suspended.
> > 
> > Also clean up a few related error labels in a preparatory patch.
> > 
> > Johan
> > 
> > 
> > Johan Hovold (2):
> >   USB: serial: clean up core error labels
> >   USB: serial: allow hung up ports to be suspended
> > 
> >  drivers/usb/serial/usb-serial.c | 59 +++++++++++++++++----------------
> >  1 file changed, 30 insertions(+), 29 deletions(-)
> > 
> > -- 
> > 2.32.0
> > 
> 
> Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Thanks for reviewing. Now applied.

Johan