Commits · 5d30566f7383f7f33a1c8ece38f5306bcd12890e · Kirill Smelkov / linux

21 Jul, 2014 40 commits

staging/nvec: Remove double const qualifier · 5d30566f

Thierry Reding authored Jul 21, 2014

The SIMPLE_DEV_PM_OPS macro already uses the const qualifier, so there's
no need to repeat it.
Signed-off-by: Thierry Reding <treding@nvidia.com>
Acked-by: Marc Dietrich <marvin24@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

5d30566f

staging: vt6655: Fixed checkpatch errors · 7354a175

Petar Dimitrijevic authored Jul 20, 2014

This patch fixes the errors report by the checkpatch script.
Most of them are use of C99 comments.

The checkpatch doesn't report any errors after the clean up.
However, please not that there are still ton of warnings left.
Signed-off-by: Petar Dimitrijevic <petar.dimitrijevic@vorteksed.com.mk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

7354a175

Staging: vt6655: Remove redundant comments in wcmd.h · 4025ef18

Igor Bezukh authored Jul 20, 2014

Removed redundant comments in the header file wcmd.h
Signed-off-by: Igor Bezukh <igbzkh@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

4025ef18

Staging: bcm: nvm.c: replaced member accessing with variable · ae5ad419

Matthias Beyer authored Jul 20, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

ae5ad419

Staging: bcm: nvm.c: Removed indentation level by concatenating nested if statements · aabaf67f
Matthias Beyer authored Jul 20, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
aabaf67f
Staging: bcm: nvm.c: Removed indentation level by using continue statement · f3ebe237
Matthias Beyer authored Jul 20, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
f3ebe237

Staging: bcm: nvm.c: Outsourced chunk of code into function · 093300f4

Matthias Beyer authored Jul 20, 2014

This patch outsources a chunk of code into an own function. It also
refactors the variable names which are used within this function.

The function name may be not appropriate.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

093300f4

Staging: bcm: nvm.c: Shortened lines · 3ccc0fd5

Matthias Beyer authored Jul 20, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

3ccc0fd5

staging: lustre: lov: Add a space before open braces '{' in lov_request.c · e010f2af

Riccardo Lucchese authored Jul 21, 2014

Fix the following checkpatch.pl issue in lov_request.c:
ERROR: space required before the open brace '{'
Signed-off-by: Riccardo Lucchese <riccardo.lucchese@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e010f2af

staging: lustre: lov: Add a blank line after declarations in lov_request.c · 5c816ad7

Riccardo Lucchese authored Jul 21, 2014

Fix the following checkpatch.pl issue in lov_request.c:
WARNING: Missing a blank line after declarations
Signed-off-by: Riccardo Lucchese <riccardo.lucchese@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

5c816ad7

staging: lustre: lov: Cleanup lov_check_set() in lov_request.c · bd9cb12f

Riccardo Lucchese authored Jul 21, 2014

Make the code clearer by introducing a local variable and removing the
unnecessary 'if' statement.
Signed-off-by: Riccardo Lucchese <riccardo.lucchese@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

bd9cb12f

staging: android: ion: ion_cma_heap.c: Fix checkpatch warning · f0ca3e87

Phong Tran authored Jul 20, 2014

This patch fix coding style:

- Remove "fail memory allocation" waring
- Remove return of void function

Tested by compilation only
Signed-off-by: Phong Tran <tranmanphong@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

f0ca3e87

staging: sep: Fix blank lines issue reported by checkpatch · bc0c9da5

LABBE Corentin authored Jul 19, 2014

Signed-off-by: LABBE Corentin <clabbe.montjoie@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

bc0c9da5

staging: sep: Fix misceanellous problems reported by checkpatch · f0949457

LABBE Corentin authored Jul 19, 2014

Signed-off-by: LABBE Corentin <clabbe.montjoie@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

f0949457

staging: sep: No else is necessary after a break (reported by checkpatch) · ba5a40d6
LABBE Corentin authored Jul 19, 2014
```
Signed-off-by: LABBE Corentin <clabbe.montjoie@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
ba5a40d6

staging: vt6656: struct vnt_usb_send_context context into vnt_get_duration_le · cca48e3c

Malcolm Priestley authored Jul 20, 2014

Removing priv, frame_length and need_ack
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

cca48e3c

staging; vt6656: rxtx add current_rate into vnt_usb_send_context as tx_rate · 798f0601

Malcolm Priestley authored Jul 20, 2014

Create u8 tx_rate and remove current_rate from function declarations
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

798f0601

staging: vt6656: rxtx move frame_len/size into vnt_usb_send_context as frame_len · 0f5567cb

Malcolm Priestley authored Jul 20, 2014

Removing frame_len and frame_size from function declarations
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

0f5567cb

staging: vt6656: move need_ack into into vnt_usb_send_context · 2eca8dbb

Malcolm Priestley authored Jul 20, 2014

Fixing need_ack to u8 and removing from function declarations.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

2eca8dbb

staging: vt6656: rxtx move pkt_type into vnt_usb_send_context · e8c9875c

Malcolm Priestley authored Jul 20, 2014

Removing pkt_type from functions
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e8c9875c

staging: vt6656: vnt_get_duration_le fix typo piv to priv · 0871dc69

Malcolm Priestley authored Jul 20, 2014

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

0871dc69

staging: vt6656: dead code remove channel switch. · bab97c05

Malcolm Priestley authored Jul 20, 2014

The driver needs to do anything special for channel switch.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

bab97c05

staging: vt6656: struct vnt_private rename byBBPreEDIndex with bb_pre_ed_index · 43fd1924

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

43fd1924

staging: vt6656: struct vnt_private replace byBBPreEDRSSI with bb_pre_ed_rssi · 12687222

Malcolm Priestley authored Jul 20, 2014

Remove type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

12687222

staging: vt6656: struct vnt_private remove dead code ldBmThreshold · 975c89f3

Malcolm Priestley authored Jul 20, 2014

ldBmThreshold was used in old function s_vCheckSensitivity which
had been disabled in orginal vendors code.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

975c89f3

staging: vt6656: struct vnt_private replace abyBBVGA with bb_vga · c37cbd37

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

c37cbd37

staging: vt6656: struct vnt_private replace byAutoFBCtrl with auto_fb_ctrl · a6177aef

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a6177aef

staging: vt6656: struct vnt_private replace wSeqCounter with seq_counter · 1b2bc0aa

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1b2bc0aa

staging: vt6656: struct vnt_private remove dead variable bBarkerPreambleMd · ba06e38c
Malcolm Priestley authored Jul 20, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
ba06e38c

staging: vt6656: struct vnt_private replace bShortSlotTime with short_slot_time · a641c9ec

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a641c9ec

staging: vt6656: struct vnt_private remove camel case retry limit · 388e5cb8

Malcolm Priestley authored Jul 20, 2014

camel case changes
byShortRetryLimit -> short_retry_limit
byLongRetryLimit -> long_retry_limit
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

388e5cb8

staging: vt6656: struct vnt_private replace wCurrentRate with current_rate · 8b84c1da

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

8b84c1da

staging: vt6656: struct vnt_private rf power table remove camel case · 5a97491c

Malcolm Priestley authored Jul 20, 2014

Camel case changes
byCCKPwr -> cck_pwr
byOFDMPwrG -> ofdm_pwr_g
byOFDMPwrA -> ofdm_pwr_a
byCurPwr -> power
abyCCKPwrTbl -> cck_pwr_tbl
abyOFDMPwrTbl -> ofdm_pwr_tbl
abyOFDMAPwrTbl -> ofdm_a_pwr_tbl
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

5a97491c

staging: vt6656: struct vnt_private replace byPreambleType with preamble_type · 98e93fe5

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

98e93fe5

staging: vt6656: struct vnt_private replace abyEEPROM with eeprom · bbb11263

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

bbb11263

staging: vt6656: struct vnt_private replace byTopCCKBasicRate with top_cck_basic_rate · d80bf43c

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

d80bf43c

staging: vt6656: struct vnt_private replace byTopOFDMBasicRate with top_ofdm_basic_rate · 3c8a5b25

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

3c8a5b25

staging: vt6656: struct vnt_private replace wBasicRate with basic_rates · 93a73558

Malcolm Priestley authored Jul 20, 2014

The same name as in struct ieee80211_bss_conf
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

93a73558

staging: vt6656: struct vnt_private replace byPacketType with packet_type · e12471db

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e12471db

staging: vt6656: struct vnt_private replace byBBType with bb_type · 65df77e2

Malcolm Priestley authored Jul 20, 2014

Removing type prefix and camel case.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

65df77e2