Commits · cb3aadaec89109edb60d4829ce82e50b27cb4ce6 · Kirill Smelkov / linux

16 Jul, 2014 1 commit

staging: comedi: removing not useful `else` after return · cb3aadae

Kinka Huang authored Jul 15, 2014

Signed-off-by: Kinka Huang <kinkabrain@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

cb3aadae

15 Jul, 2014 39 commits

staging: vt6656: usbpipe irrelevant function descriptions · f24937db

Malcolm Priestley authored Jul 15, 2014

All these have no or vague meaning.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

f24937db

staging: vt6656: rename s_nsBulkInUsbIoCompleteRead to vnt_submit_rx_urb_complete · 08823af4
Malcolm Priestley authored Jul 15, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
08823af4

staging: vt6656: rename PIPEnsBulkInUsbRead to vnt_submit_rx_urb · 2dc37af0

Malcolm Priestley authored Jul 15, 2014

The function just submits to rx urbs.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

2dc37af0

staging: vt6656: rename s_nsBulkOutIoCompleteWrite to vnt_tx_context_complete · ceebd903
Malcolm Priestley authored Jul 15, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
ceebd903

staging: vt6656: rename PIPEnsSendBulkOut to vnt_tx_context. · 476e7d97

Malcolm Priestley authored Jul 15, 2014

We are just tx the context
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

476e7d97

staging: vt6656: rename s_nsInterruptUsbIoCompleteRead to vnt_start_interrupt_urb_complete · 34f98e3f
Malcolm Priestley authored Jul 15, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
34f98e3f

staging: vt6656: usbpipe rename PIPEnsInterruptRead to vnt_start_interrupt_urb · b0bc5724

Malcolm Priestley authored Jul 15, 2014

Drop Read and PIPE. The function starts the interrupt urb.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

b0bc5724

staging: vt6656: int.c remove irrelevant function description · 0d901321

Malcolm Priestley authored Jul 15, 2014

The function does not directly poll from here
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

0d901321

staging: vt6656: rename INTvWorkItem to vnt_int_start_interrupt · 62001939

Malcolm Priestley authored Jul 15, 2014

This function does run as work item, it starts interrupts from the
device.

Rename to vnt_int_start_interrupt
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

62001939

staging: vt6656: int rename INTnsProcessData to vnt_int_process_data · e360d2b8
Malcolm Priestley authored Jul 15, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
e360d2b8

staging: vt6656: int.c Replace DBG_PRT debug messages · 50730604

Malcolm Priestley authored Jul 15, 2014

Replace with dev_dbg
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

50730604

staging: vt6656: INTvWorkItem remove camel case · 526fed3e

Malcolm Priestley authored Jul 15, 2014

Camel case changes
pDevice -> priv
ntStatus -> status
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

526fed3e

staging: android: Clean up else statement from binder_send_failed_reply · d4ec15e1

Lucas Tanure authored Jul 13, 2014

Kernel coding style. Remove useless else statement after return.

Changes from v1 and v2: Fix warning for mixed declarations and code.
Declaration of "struct binder_transaction *next" made outside of while.

Changes from v3: Removed initialization to NULL for next variable.
Signed-off-by: Lucas Tanure <tanure@linux.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

d4ec15e1

staging: android: Clean up else statement from sync_fence_poll() · 6a44b50f

Lucas Tanure authored Jul 15, 2014

Kernel coding style. Remove useless else statement after return.
Signed-off-by: Lucas Tanure <tanure@linux.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

6a44b50f

staging: octeon: fix coding style · 38064eb1

Rahul Bedarkar authored Jul 14, 2014

This patch fixes checkpatch.pl warning. Add blank line after
declaration.
Signed-off-by: Rahul Bedarkar <rahulbedarkar89@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

38064eb1

Staging: rtl8821ae: delete unneeded lines in cam.c · 05c9c195

Joerg C. Meyer authored Jul 14, 2014

This is a patch to the cam.c file that removes some unneeded lines of commented-out code
Signed-off-by: Joerg C. Meyer <joerg@meyer.homedns.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

05c9c195

staging: dgap: remove dgap_newnode() · 82512235

Daeseok Youn authored Jul 15, 2014

The dgap_newnode() is useless for creating new node.
So just use kzalloc and set a type in case statement.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

82512235

staging: dgap: remove unused a parameter in dgap_gettok() · 7bab00ff

Daeseok Youn authored Jul 15, 2014

The "p" as parameter is unused.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

7bab00ff

staging: dgap: fix a typo in dgap_gettok() · e9cc5b2b

Daeseok Youn authored Jul 15, 2014

The "boar" should be "board".
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e9cc5b2b

staging: dgap: remove unused case value in dgap_parsefile() · 8e82ce02

Daeseok Youn authored Jul 15, 2014

If rc is zero, this function will returns with an error and
cannot reach switch-case statement.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

8e82ce02

staging: dgap: remove redundant error value check · dfa30ac1

Daeseok Youn authored Jul 15, 2014

The retval in dgap_block_til_ready() is initialized to zero,
and if no error has occurred in this function, the retval has a zero.
So it doesn't need to check "retval" itself.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

dfa30ac1

Staging: bcm: PHSModule.c: Shortened lines · e86bd61e

Matthias Beyer authored Jul 15, 2014

This patch shortenes all lines where possible without code refactoring,
as well as comment lines. It does not change the wording of comments.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e86bd61e

Staging: bcm: PHSModule.c: Replaced if-else return code with simple return assertion · 3be738ad

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

3be738ad

Staging: bcm: PHSModule.c: Replaced member accessing with variable in... · 91283491

Matthias Beyer authored Jul 15, 2014

Staging: bcm: PHSModule.c: Replaced member accessing with variable in CreateSFToClassifierRuleMapping()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

91283491

Staging: bcm: PHSModule.c: Simplified nested if statements by linking them... · 18002f56

Matthias Beyer authored Jul 15, 2014

Staging: bcm: PHSModule.c: Simplified nested if statements by linking them with logical AND in GetPhsRuleEntry()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

18002f56

Staging: bcm: PHSModule.c: Simplified nested if statements by linking them... · 1c53f087

Matthias Beyer authored Jul 15, 2014

Staging: bcm: PHSModule.c: Simplified nested if statements by linking them with logical AND in GetServiceFlowEntry()
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1c53f087

Staging: bcm: PHSModule.c: Simplified nested if statements by linking them... · c842dd2d

Matthias Beyer authored Jul 15, 2014

c842dd2d

Staging: bcm: PHSModule.c: Replaced member accessing with variable in GetServiceFlowEntry() · cf423564

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

cf423564

Staging: bcm: PHSModule.c: Added const keyword to ValidatePHSRUleComplete() argument · 14d8e915

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

14d8e915

Staging: bcm: PHSModule.c: Rewrote ValidatePHSRuleComplete() · cc872f80

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

cc872f80

Staging: bcm: PHSModule.c: Replaced indentation level with goto jump on bad condition · c8020cc9

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

c8020cc9

Staging: bcm: PHSModule.c: Replaced member accessing with variable in free_phs_service() · 3ed150a1

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

3ed150a1

Staging: bcm: PHSModule.c: Added missing braces around else block · b43b245f

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

b43b245f

Staging: bcm: PHSModule.c: Whitespace cleanup in PhsCompress() · babb7f94

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

babb7f94

Staging: bcm: PHSModule.c: Replaced member accessing with variables · a984b124

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a984b124

Staging: bcm: PHSModule.c: Reduced indentation by using jump label in PhsDeleteSFRules() · fc38bc1d

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

fc38bc1d

Staging: bcm: PHSModule.c: Shortened lines · 93cc6ae8

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

93cc6ae8

Staging: bcm: PHSModule.c: Reduced indentation level by using jump label · 525afdd1

Matthias Beyer authored Jul 15, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

525afdd1

Staging: bcm: PHSModule.c: Replaced nested if statements with logical AND... · 31f4f3fb

Matthias Beyer authored Jul 15, 2014

Staging: bcm: PHSModule.c: Replaced nested if statements with logical AND concatenation of the conditions
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

31f4f3fb