Adithya Krishnamurthy
c994a4d519
Staging:tidspbridge Fix checkpatch.pl warning char * array declaration might be better as static const
...
Fixed checkpatch "WARNING: char * array declaration might be better as static const"
Signed-off-by: Adithya Krishnamurthy <linux.challenge1@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 21:00:42 -07:00
Kinka Huang
cb3aadaec8
staging: comedi: removing not useful else after return
...
Signed-off-by: Kinka Huang <kinkabrain@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 19:59:16 -07:00
Malcolm Priestley
f24937db60
staging: vt6656: usbpipe irrelevant function descriptions
...
All these have no or vague meaning.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:40 -07:00
Malcolm Priestley
08823af422
staging: vt6656: rename s_nsBulkInUsbIoCompleteRead to vnt_submit_rx_urb_complete
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:40 -07:00
Malcolm Priestley
2dc37af09b
staging: vt6656: rename PIPEnsBulkInUsbRead to vnt_submit_rx_urb
...
The function just submits to rx urbs.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:40 -07:00
Malcolm Priestley
ceebd9036a
staging: vt6656: rename s_nsBulkOutIoCompleteWrite to vnt_tx_context_complete
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:40 -07:00
Malcolm Priestley
476e7d97aa
staging: vt6656: rename PIPEnsSendBulkOut to vnt_tx_context.
...
We are just tx the context
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
34f98e3f27
staging: vt6656: rename s_nsInterruptUsbIoCompleteRead to vnt_start_interrupt_urb_complete
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
b0bc5724f6
staging: vt6656: usbpipe rename PIPEnsInterruptRead to vnt_start_interrupt_urb
...
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 >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
0d901321fa
staging: vt6656: int.c remove irrelevant function description
...
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 >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
6200193979
staging: vt6656: rename INTvWorkItem to vnt_int_start_interrupt
...
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 >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
e360d2b8fa
staging: vt6656: int rename INTnsProcessData to vnt_int_process_data
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
50730604c9
staging: vt6656: int.c Replace DBG_PRT debug messages
...
Replace with dev_dbg
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:39 -07:00
Malcolm Priestley
526fed3e16
staging: vt6656: INTvWorkItem remove camel case
...
Camel case changes
pDevice -> priv
ntStatus -> status
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 12:11:39 -07:00
Lucas Tanure
d4ec15e16f
staging: android: Clean up else statement from binder_send_failed_reply
...
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 >
2014-07-15 12:08:52 -07:00
Lucas Tanure
6a44b50f98
staging: android: Clean up else statement from sync_fence_poll()
...
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 >
2014-07-15 12:08:13 -07:00
Rahul Bedarkar
38064eb1a4
staging: octeon: fix coding style
...
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 >
2014-07-15 08:32:55 -07:00
Joerg C. Meyer
05c9c195a2
Staging: rtl8821ae: delete unneeded lines in cam.c
...
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 >
2014-07-15 08:32:55 -07:00
Daeseok Youn
825122353a
staging: dgap: remove dgap_newnode()
...
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 >
2014-07-15 08:26:29 -07:00
Daeseok Youn
7bab00ff65
staging: dgap: remove unused a parameter in dgap_gettok()
...
The "p" as parameter is unused.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 08:26:29 -07:00
Daeseok Youn
e9cc5b2bb1
staging: dgap: fix a typo in dgap_gettok()
...
The "boar" should be "board".
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 08:26:29 -07:00
Daeseok Youn
8e82ce0223
staging: dgap: remove unused case value in dgap_parsefile()
...
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 >
2014-07-15 08:26:29 -07:00
Daeseok Youn
dfa30ac139
staging: dgap: remove redundant error value check
...
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 >
2014-07-15 08:26:29 -07:00
Matthias Beyer
e86bd61ee6
Staging: bcm: PHSModule.c: Shortened lines
...
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
3be738ad55
Staging: bcm: PHSModule.c: Replaced if-else return code with simple return assertion
...
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
91283491c6
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
18002f567f
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
1c53f0877e
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
c842dd2ddb
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
cf423564fa
Staging: bcm: PHSModule.c: Replaced member accessing with variable 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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
14d8e91596
Staging: bcm: PHSModule.c: Added const keyword to ValidatePHSRUleComplete() argument
...
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 >
2014-07-15 08:09:49 -07:00
Matthias Beyer
cc872f8070
Staging: bcm: PHSModule.c: Rewrote ValidatePHSRuleComplete()
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
c8020cc93e
Staging: bcm: PHSModule.c: Replaced indentation level with goto jump on bad condition
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
3ed150a178
Staging: bcm: PHSModule.c: Replaced member accessing with variable in free_phs_service()
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
b43b245fb3
Staging: bcm: PHSModule.c: Added missing braces around else block
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
babb7f9479
Staging: bcm: PHSModule.c: Whitespace cleanup in PhsCompress()
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
a984b124aa
Staging: bcm: PHSModule.c: Replaced member accessing with variables
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
fc38bc1d68
Staging: bcm: PHSModule.c: Reduced indentation by using jump label in PhsDeleteSFRules()
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
93cc6ae8b3
Staging: bcm: PHSModule.c: Shortened lines
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
525afdd170
Staging: bcm: PHSModule.c: Reduced indentation level by using jump label
...
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
31f4f3fb15
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 >
2014-07-15 08:09:48 -07:00
Matthias Beyer
413852b6d4
Staging: bcm: PHSModule.c: Replaced member accessing with variable in PhsDeletePHSRule()
...
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 >
2014-07-15 08:09:47 -07:00
Matthias Beyer
c50e0b5c77
Staging: bcm: PHSModule.c: Outsourced debug printing for phs classifier entry
...
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 >
2014-07-15 08:09:47 -07:00
Matthias Beyer
20d174e2b4
Staging: bcm: PHSModule.c: Outsourced debug printing in own function
...
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 >
2014-07-15 08:09:47 -07:00
Matthias Beyer
ca89a2950c
Staging: bcm: PHSModule.c: Whitespace & Indentation cleanup
...
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 >
2014-07-15 08:09:47 -07:00
Peter Senna Tschudin
495e2be7e5
staging: vt6556: Remove double parentheses
...
This patch cleanup a checkpatch warning by removing double parentheses
from if condition. Additionally it fixes a line over 80 chars.
Tested by compilation only.
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 08:05:16 -07:00
Malcolm Priestley
ed0db5135f
staging: vt6656: set BBvSetAntennaMode in device_init_registers
...
BBvSetAntennaMode is not set at all, set to byRxAntennaMode
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 07:50:52 -07:00
Malcolm Priestley
6fdd946afe
staging: vt6656: device.h remove old wireless stats
...
These are dead nolonger of use.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-15 07:50:51 -07:00
Chen Gang
d0f17b64ec
drivers: staging: lustre: Use 'force_die' instead of 'die' to avoid compiling issue
...
Some of architectures have already defined 'die' as macro, so can not
use it as declaration or definition in other modules, or it will cause
compiling issue.
So use more precise name 'force_die' (like 'wrap_bulk') instead of. And
the related error (with allmodconfig under score):
CC [M] drivers/staging/lustre/lustre/ptlrpc/sec.o
drivers/staging/lustre/lustre/ptlrpc/sec.c: In function 'sptlrpc_cli_ctx_expire':
drivers/staging/lustre/lustre/ptlrpc/sec.c:309:13: error: 'struct ptlrpc_ctx_ops' has no member named '__die'
ctx->cc_ops->die(ctx, 0);
^
drivers/staging/lustre/lustre/ptlrpc/sec.c: In function 'ctx_refresh_timeout':
drivers/staging/lustre/lustre/ptlrpc/sec.c:594:26: error: 'struct ptlrpc_ctx_ops' has no member named '__die'
req->rq_cli_ctx->cc_ops->die(req->rq_cli_ctx, 0);
^
make[5]: *** [drivers/staging/lustre/lustre/ptlrpc/sec.o] Error 1
make[4]: *** [drivers/staging/lustre/lustre/ptlrpc] Error 2
make[3]: *** [drivers/staging/lustre/lustre] Error 2
make[2]: *** [drivers/staging/lustre] Error 2
make[1]: *** [drivers/staging] Error 2
make: *** [drivers] Error 2
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-14 18:40:12 -07:00
Greg Kroah-Hartman
589e1d10ff
Merge 3.16-rc5 into staging-next
...
We want the fixes in -rc5 in here as well.
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 15:35:56 -07:00