Re: Subject: [PATCH v3 1/2] USB: serial: cp210x: Enable usb generic throttle/unthrottle

2020-06-23 Thread gre...@linuxfoundation.org
On Wed, Jun 24, 2020 at 04:11:21AM +, Phu Luu wrote:
> Assign the .throttle and .unthrottle functions to be generic function
> in the driver structure to prevent data loss that can otherwise occur
> if the host does not enable USB throttling.
> 
> Signed-off-by: Phu Luu An 
> Signed-off-by: Brant Merryman 

Why does your Subject: line have "Subject:" in it again?

Can you please fix up and resend these patches?

thanks,

greg k-h


Subject: [PATCH v3 1/2] USB: serial: cp210x: Enable usb generic throttle/unthrottle

2020-06-23 Thread Phu Luu
Assign the .throttle and .unthrottle functions to be generic function
in the driver structure to prevent data loss that can otherwise occur
if the host does not enable USB throttling.

Signed-off-by: Phu Luu An 
Signed-off-by: Brant Merryman 
---
06/09/2020: Patch v3 1/2 Modified based on feedback from Johan Hovold 

12/18/2019: Patch v2 Broken into two patches and modified based on feedback 
from Johan Hovold 
12/09/2019: Initial submission of patch "Proper RTS control when buffers fill"

 drivers/usb/serial/cp210x.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/usb/serial/cp210x.c b/drivers/usb/serial/cp210x.c
index f5143eedbc48..bcceb4ad8be0 100644
--- a/drivers/usb/serial/cp210x.c
+++ b/drivers/usb/serial/cp210x.c
@@ -272,6 +272,8 @@ static struct usb_serial_driver cp210x_device = {
.break_ctl  = cp210x_break_ctl,
.set_termios= cp210x_set_termios,
.tx_empty   = cp210x_tx_empty,
+   .throttle   = usb_serial_generic_throttle,
+   .unthrottle = usb_serial_generic_unthrottle,
.tiocmget   = cp210x_tiocmget,
.tiocmset   = cp210x_tiocmset,
.attach = cp210x_attach,
-- 
2.17.0