Commits · ac4ddad67acb9f67b42902939df64980fcbdbae5 · Kirill Smelkov / linux

28 Jun, 2014 1 commit

staging: winbond: remove driver · ac4ddad6

Kristina Martšenko authored Jun 29, 2014

The driver hasn't been cleaned up and nobody is working to do so, so
remove it.
Signed-off-by: Kristina Martšenko <kristina.martsenko@gmail.com>
Cc: Pavel Machek <pavel@ucw.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

ac4ddad6

27 Jun, 2014 39 commits

staging: bcm: nvm.c: Cleaning up check unsigned is less than zero · c0cd6e5b

Rickard Strandqvist authored Jun 25, 2014

Remove checking if a unsigned int is less than zero

This was found using a static code analysis program called cppcheck.
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

c0cd6e5b

Staging: bcm: hostmibs.c: Shortened lines · fa1e4a7f

Matthias Beyer authored Jun 23, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Jake Edge <jake@lwn.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

fa1e4a7f

Staging: bcm: replaced member accessing with variable for readability · ecf97445

Matthias Beyer authored Jun 23, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Reviewed-by: Jake Edge <jake@lwn.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

ecf97445

Staging: bcm: Added missing blank line after declarations · f687299f

Matthias Beyer authored Jun 23, 2014

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

f687299f

Staging: bcm: Replaced member accssing with variable · ab78899a

Matthias Beyer authored Jun 23, 2014

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

ab78899a

Staging: bcm: Whitespace for indentation fixed · 55eb46e9

Matthias Beyer authored Jun 23, 2014

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

55eb46e9

Staging: bcm: line length fixed · 3a7f7cae

Matthias Beyer authored Jun 23, 2014

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

3a7f7cae

Staging: bcm: Line length cleanup · cd77fe77

Matthias Beyer authored Jun 23, 2014

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

cd77fe77

Staging: bcm: Shortened some lines in select_alternate_setting_for_highspeed_modem() · 269027e9
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
269027e9

Staging: bcm: Outsourced selecting of alternate setting · b3110cd8

Matthias Beyer authored Jun 23, 2014

This patch outsources a chunk of code which is for selecting an
alternate setting for highspeed modem into a new function.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

b3110cd8

Staging: bcm: Merged some lines which got shorter by the latest patch · b4ad0f04
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
b4ad0f04
Staging: bcm: Replaced member accessing with variable in InterfaceAdapterInit() · a02b67b6
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
a02b67b6
Staging: bcm: Replaced member accessing with variable in device_run() · 9e58842f
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
9e58842f

Staging: bcm: replaced member accessing by variable · 3b068e54

Matthias Beyer authored Jun 23, 2014

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

3b068e54

Staging: bcm: Shortened some lines · d3842d56

Matthias Beyer authored Jun 23, 2014

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

d3842d56

Staging: bcm: led_control.c: Replaced nested ifs with logical AND concatenation · f4367dfe
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
f4367dfe

Staging: bcm: led_control.c: Replaced member accessing by variable · 2d2ea216

Matthias Beyer authored Jun 23, 2014

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

2d2ea216

Staging: bcm: led_control.c: Replaced member accessing by variable · 76a60d57

Matthias Beyer authored Jun 23, 2014

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

76a60d57

Staging: bcm: led_control.c: Replaced doubled if-check by logical OR in one check · a143aa2a
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
a143aa2a
Staging: bcm: led_control.c: Replaced time selecting ifelse with ternary operator · 89550511
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
89550511

Staging: bcm: led_control.c: More variable names refactored · 9c805bd9

Matthias Beyer authored Jun 23, 2014

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

9c805bd9

Staging: bcm: led_control.c: Refactored line lengths · 60e66d87

Matthias Beyer authored Jun 23, 2014

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

60e66d87

Staging: bcm: led_control.c: Removed dead code · ea20548a

Matthias Beyer authored Jun 23, 2014

This patch removes the never-entered else-block inside
blink_in_normal_bandwidth(). The condition of the preceding if statement
is always true.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

ea20548a

Staging: bcm: led_control.c: Refactored variable name · 60654a84

Matthias Beyer authored Jun 23, 2014

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

60654a84

Staging: bcm: led_control.c: Outsourced code chunk · 4fb9ab4e

Matthias Beyer authored Jun 23, 2014

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

4fb9ab4e

Staging: bcm: led_control.c: Replaced member accessing · e4e9d83e

Matthias Beyer authored Jun 23, 2014

This patch replaces member accessing with a variable in
BcmGetGPIOPinInfo().
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e4e9d83e

Staging: bcm: led_control.c: Simplified nested if conditions · 7a0304c0

Matthias Beyer authored Jun 23, 2014

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

7a0304c0

Staging: bcm: led_control.{c,h}: Fixed macro · a1082a4d

Matthias Beyer authored Jun 23, 2014

This patch fixes the TURN_{ON,OFF}_LED() macros by adding the adapter as
parameter.
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a1082a4d

Staging: bcm: led_control.c: Outsourced adapter driver state handling · ce2705c3
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
ce2705c3
Staging: bcm: led_control.c: Indentation/Whitespace/Line length cleanup · 9bcfab20
Matthias Beyer authored Jun 23, 2014
```
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
9bcfab20

Staging: slicoss: fixed frivolous else statement warnings in slicoss.c · 351e836f

Vincent Heuken authored Jun 23, 2014

This is a patch for the slicoss.c file that fixes four instances
of the following warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Vincent Heuken <me@vincentheuken.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

351e836f

staging: emxx_udc: Use module_platform_driver · 464cad2c

Sachin Kamat authored Jun 23, 2014

module_platform_driver removes some boiler plate and makes
code simpler.
Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

464cad2c

staging: emxx_udc: Fix build error · a790ebc1

Sachin Kamat authored Jun 23, 2014

devm_request_and_ioremap has been removed since commit c9d53c0f
("devres: remove devm_request_and_ioremap()") Use devm_ioremap_resource
instead. While at it, also remove redundant error message as it is now
handled by devm_ioremap_resource. Without this patch we get the following
build error:
drivers/staging/emxx_udc/emxx_udc.c:3370:2: error: implicit declaration of function ‘devm_request_and_ioremap’ [-Werror=implicit-function-declaration]
Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a790ebc1

staging: lustre: llite: dir.c - fix for coding style issues · b0126abf

Anil Belur authored Jun 25, 2014

- this fixes some coding style issues "ERROR: "(foo*)" should be "(foo *)"
Signed-off-by: Anil Belur <askb23@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

b0126abf

staging: lustre: llite: dir.c - fixes indented labels in code · eb73f514

Anil Belur authored Jun 25, 2014

- this fixes "WARNING: labels should not be indented"
Signed-off-by: Anil Belur <askb23@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

eb73f514

staging: lustre: llite: llite_capa.c - fixed warning to use recomended headers <linux/uaccess.h> · 576eb39a

Anil Belur authored Jun 25, 2014

- this fixes "WARNING: Use #include <linux/uaccess.h> instead of
  <asm/uaccess.h>"
Signed-off-by: Anil Belur <askb23@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

576eb39a

staging: lustre: llite: llite_capa.c - fixes checkpatch errors - unecessary... · 8bcf3c23

Anil Belur authored Jun 25, 2014

staging: lustre: llite: llite_capa.c - fixes checkpatch errors - unecessary init of static variables to 0

- this commit fixes some "ERROR: do not initialise statics to 0 or NULL"
Signed-off-by: Anil Belur <askb23@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

8bcf3c23

staging: lustre: lustre: libcfs: fail.c - fix sparse warnings EXPORT_SYMBOL · 0d2f6bb4

Anil Belur authored Jun 23, 2014

- this fixes "WARNING: EXPORT_SYMBOL(foo); should immediately follow its
  function/variable"
Signed-off-by: Anil Belur <askb23@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

0d2f6bb4

staging/lustre/lnet: abort messages whose MD has been unlinked · dee2857e

Isaac Huang authored Jun 22, 2014

If LNetMDUnlink has been called, all outgoing messages
on that MD should be aborted before lnet_ni_send() is
called.
Signed-off-by: Isaac Huang <he.huang@intel.com>
Reviewed-on: http://review.whamcloud.com/8041
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4006Reviewed-by: Liang Zhen <liang.zhen@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Signed-off-by: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

dee2857e