Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

USB: serial: generic: clean up urb->status usage

This done in anticipation of removal of urb->status, which will make
that patch easier to review and apply in the future.


Cc: <linux-usb-devel@lists.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

+8 -4
+8 -4
drivers/usb/serial/generic.c
··· 325 325 { 326 326 struct usb_serial_port *port = (struct usb_serial_port *)urb->context; 327 327 unsigned char *data = urb->transfer_buffer; 328 + int status = urb->status; 328 329 329 330 dbg("%s - port %d", __FUNCTION__, port->number); 330 331 331 - if (unlikely(urb->status != 0)) { 332 - dbg("%s - nonzero read bulk status received: %d", __FUNCTION__, urb->status); 332 + if (unlikely(status != 0)) { 333 + dbg("%s - nonzero read bulk status received: %d", 334 + __FUNCTION__, status); 333 335 return; 334 336 } 335 337 ··· 349 347 void usb_serial_generic_write_bulk_callback (struct urb *urb) 350 348 { 351 349 struct usb_serial_port *port = (struct usb_serial_port *)urb->context; 350 + int status = urb->status; 352 351 353 352 dbg("%s - port %d", __FUNCTION__, port->number); 354 353 355 354 port->write_urb_busy = 0; 356 - if (urb->status) { 357 - dbg("%s - nonzero write bulk status received: %d", __FUNCTION__, urb->status); 355 + if (status) { 356 + dbg("%s - nonzero write bulk status received: %d", 357 + __FUNCTION__, status); 358 358 return; 359 359 } 360 360