diff options
author | Joe Perches <joe@perches.com> | 2010-08-04 13:40:08 -0400 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2010-08-10 08:25:42 -0400 |
commit | 7f26b3a7533bbc1ddd88b297c935ee4da8f74cea (patch) | |
tree | 28ba09da29cf391405a932ef243ea49d96a21204 /drivers/usb/image | |
parent | 429d646d9aeafdb57717dab75c807215f5cce9c2 (diff) |
drivers/usb: Remove unnecessary return's from void functions
Greg prefers this to go through the trivial tree.
http://lkml.org/lkml/2010/6/24/1
There are about 2500 void functions in drivers/usb
Only a few used return; at end of function.
Standardize them a bit.
Moved a statement down a line in drivers/usb/host/u132-hcd.c
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/usb/image')
-rw-r--r-- | drivers/usb/image/microtek.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/drivers/usb/image/microtek.c b/drivers/usb/image/microtek.c index 3a6bcd5fee09..5a47805d9580 100644 --- a/drivers/usb/image/microtek.c +++ b/drivers/usb/image/microtek.c | |||
@@ -398,7 +398,6 @@ void mts_int_submit_urb (struct urb* transfer, | |||
398 | context->srb->result = DID_ERROR << 16; | 398 | context->srb->result = DID_ERROR << 16; |
399 | mts_transfer_cleanup(transfer); | 399 | mts_transfer_cleanup(transfer); |
400 | } | 400 | } |
401 | return; | ||
402 | } | 401 | } |
403 | 402 | ||
404 | 403 | ||
@@ -409,7 +408,6 @@ static void mts_transfer_cleanup( struct urb *transfer ) | |||
409 | 408 | ||
410 | if ( likely(context->final_callback != NULL) ) | 409 | if ( likely(context->final_callback != NULL) ) |
411 | context->final_callback(context->srb); | 410 | context->final_callback(context->srb); |
412 | |||
413 | } | 411 | } |
414 | 412 | ||
415 | static void mts_transfer_done( struct urb *transfer ) | 413 | static void mts_transfer_done( struct urb *transfer ) |
@@ -420,8 +418,6 @@ static void mts_transfer_done( struct urb *transfer ) | |||
420 | context->srb->result |= (unsigned)(*context->scsi_status)<<1; | 418 | context->srb->result |= (unsigned)(*context->scsi_status)<<1; |
421 | 419 | ||
422 | mts_transfer_cleanup(transfer); | 420 | mts_transfer_cleanup(transfer); |
423 | |||
424 | return; | ||
425 | } | 421 | } |
426 | 422 | ||
427 | 423 | ||
@@ -452,8 +448,6 @@ static void mts_data_done( struct urb* transfer ) | |||
452 | } | 448 | } |
453 | 449 | ||
454 | mts_get_status(transfer); | 450 | mts_get_status(transfer); |
455 | |||
456 | return; | ||
457 | } | 451 | } |
458 | 452 | ||
459 | 453 | ||
@@ -496,8 +490,6 @@ static void mts_command_done( struct urb *transfer ) | |||
496 | mts_get_status(transfer); | 490 | mts_get_status(transfer); |
497 | } | 491 | } |
498 | } | 492 | } |
499 | |||
500 | return; | ||
501 | } | 493 | } |
502 | 494 | ||
503 | static void mts_do_sg (struct urb* transfer) | 495 | static void mts_do_sg (struct urb* transfer) |
@@ -522,7 +514,6 @@ static void mts_do_sg (struct urb* transfer) | |||
522 | sg[context->fragment].length, | 514 | sg[context->fragment].length, |
523 | context->fragment + 1 == scsi_sg_count(context->srb) ? | 515 | context->fragment + 1 == scsi_sg_count(context->srb) ? |
524 | mts_data_done : mts_do_sg); | 516 | mts_data_done : mts_do_sg); |
525 | return; | ||
526 | } | 517 | } |
527 | 518 | ||
528 | static const u8 mts_read_image_sig[] = { 0x28, 00, 00, 00 }; | 519 | static const u8 mts_read_image_sig[] = { 0x28, 00, 00, 00 }; |