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 }; |
