platform/kernel/linux-starfive.git
5 years agostaging: comedi: ni_mio_common: use insn->n in ni_eeprom_insn_read()
Ian Abbott [Tue, 19 Mar 2019 16:54:43 +0000 (16:54 +0000)]
staging: comedi: ni_mio_common: use insn->n in ni_eeprom_insn_read()

The `insn_read` handler for the EEPROM subdevice on E-series boards
(`ni_eeprom_insn_read()`) currently ignores `insn->n` (the number of
samples to read) and assumes a single sample is to be read into
`data[0]`.  Fortunately, the Comedi core ensures that `data[]` has a
length of at least 16 so there is no problem with array bounds.

The usual Comedi convention for `insn_read` handlers is to read the same
channel `insn->n` times into successive elements of `data[]` so let's do
that.  (Each channel number corresponds to a single EEPROM address.)
Since we do not expect the EEPROM data at a particular address to change
between readings, let's just read it once and copy the value `insn->n`
times.

Also, follow the usual Comedi convention and return `insn->n` from the
handler to indicate success (although any non-negative value will do).

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: comedi: ni_mio_common: use insn->n in ni_calib_insn_read()
Ian Abbott [Tue, 19 Mar 2019 16:54:42 +0000 (16:54 +0000)]
staging: comedi: ni_mio_common: use insn->n in ni_calib_insn_read()

The `insn_read` handler for the calibration subdevice
(`ni_calib_insn_read()`) currently ignores `insn->n` (the number of
samples to read) and assumes a single sample is to be read into
`data[0]`.  Fortunately, the Comedi core ensures that `data[]` has a
length of at least 16, so there is no problem with array bounds.

The usual Comedi convention for `insn_read` handlers is to read the same
channel `insn->n` times into successive elements of `data[]`, so let's
do that.

Also, follow the usual Comedi convention and return `insn->n` from the
handler to indicate success (although any non-negative value will do).

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: comedi: ni_mio_common: Use insn->n in ni_calib_insn_write()
Ian Abbott [Tue, 19 Mar 2019 16:54:41 +0000 (16:54 +0000)]
staging: comedi: ni_mio_common: Use insn->n in ni_calib_insn_write()

The `insn_write` handler for the calibration subdevice
(`ni_calib_insn_write()`) currently ignores `insn->n` (the number of
samples to write) and assumes a single sample is to be written, but
`insn->n` could be 0, meaning no samples should be written, in which
case `data[0]` is invalid.

Change `ni_calib_insn_write()` to only write to the calibration device
if `insn->n > 0`.  There isn't much point writing all the values when
`insn->n > 1`, so just write the last one (`data[insn->n - 1]`).

Also follow the usual Comedi convention and return `insn->n` from the
handler to indicate success (although any non-negative return value will
do as far as the Comedi core is concerned).

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Only unmap_sg if mapped
George Hilliard [Tue, 19 Mar 2019 02:20:12 +0000 (20:20 -0600)]
staging: mt7621-mmc: Only unmap_sg if mapped

A failure while processing the start command could cause dma_unmap_sg()
to be called without first calling dma_map_sg().

Since calling dma_unmap_sg() is only needed when data != NULL, move the
unmap call into the corresponding if {} block.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Fix BRUST -> BURST typo
George Hilliard [Tue, 19 Mar 2019 02:20:11 +0000 (20:20 -0600)]
staging: mt7621-mmc: Fix BRUST -> BURST typo

Obvious typo.  It is specified as BURST in the datasheet.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Immediately notify mmc layer of card change detection
George Hilliard [Tue, 19 Mar 2019 02:20:10 +0000 (20:20 -0600)]
staging: mt7621-mmc: Immediately notify mmc layer of card change detection

There is no need to delay notifying the mmc layer.  Schedule the delayed
work to run immediately.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Remove redundant host->mmc->f_max write
George Hilliard [Tue, 19 Mar 2019 02:20:09 +0000 (20:20 -0600)]
staging: mt7621-mmc: Remove redundant host->mmc->f_max write

This is set once during initialization and never changed.  Don't bother
setting it again in the interrupt handler.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Bill the caller for I/O time
George Hilliard [Tue, 19 Mar 2019 02:20:06 +0000 (20:20 -0600)]
staging: mt7621-mmc: Bill the caller for I/O time

When waiting on completions, use the _io variant so the caller is
charged as using I/O.

This should have no effect on the module's functionality, only improve
CPU accounting.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Use pinctrl subsystem to select SDXC pin mode
George Hilliard [Tue, 19 Mar 2019 02:20:05 +0000 (20:20 -0600)]
staging: mt7621-mmc: Use pinctrl subsystem to select SDXC pin mode

The driver previously grabbed the SD pins for itself, ignoring the pin
controller.  Remove this, and allow the pinctrl subsystem to set up the
pins using the device tree mappings.  This allows this driver to work on
related devices that have a different pin controller mapping, such as
the MT7688.  The hardcoded bit index was incorrect on that device.

The driver now needs a pin controller reference in its device tree node:

sdhci: /* ... */ {
compatible = "ralink,mt7620-sdhci";

pinctrl-names = "default";
pinctrl-0 = <&sdhci_pins>;

// ...
};

This change could break SD controller functionality on existing devices
whose device trees do not specify a pin controller and state for the SD
node.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Remove obsolete comments and variables
George Hilliard [Tue, 19 Mar 2019 02:20:03 +0000 (20:20 -0600)]
staging: mt7621-mmc: Remove obsolete comments and variables

These comments don't contain useful code or alternate implementation
ideas.  Remove them.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: fix unused variable compiler warning
George Hilliard [Tue, 19 Mar 2019 02:20:02 +0000 (20:20 -0600)]
staging: mt7621-mmc: fix unused variable compiler warning

The compiler complains:

    drivers/staging/mt7621-mmc/dbg.c: In function ‘msdc_debug_proc_write’:
    drivers/staging/mt7621-mmc/dbg.c:237:12: warning: unused variable ‘size’ [-Wunused-variable]
      int mode, size;
                ^~~~
    drivers/staging/mt7621-mmc/dbg.c:237:6: warning: unused variable ‘mode’ [-Wunused-variable]
      int mode, size;
          ^~~~

Remove these declarations.

Signed-off-by: George Hilliard <thirtythreeforty@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: fsl-dpaa2: ethsw: Remove return variable
Nishka Dasgupta [Tue, 19 Mar 2019 19:22:31 +0000 (00:52 +0530)]
staging: fsl-dpaa2: ethsw: Remove return variable

Remove return variable and return return value directly. Issue found by
Coccinelle using ret.cocci.

Signed-off-by: Nishka Dasgupta <nishka.dasgupta@yahoo.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: unisys: visorhba: Replace '--help--' with 'help' in Kconfig
Wentao Cai [Wed, 20 Mar 2019 04:44:59 +0000 (21:44 -0700)]
Staging: unisys: visorhba: Replace '--help--' with 'help' in Kconfig

Replace '--help--' in Kconfig and indent the subsequent text to silence
checkpatch.pl
warning:
WARNING: prefer 'help' over '---help---' for new help texts

Signed-off-by: Wentao Cai <etsai042@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: unisys: visorinput: Replace '--help--' with 'help' in Kconfig
Wentao Cai [Wed, 20 Mar 2019 04:45:00 +0000 (21:45 -0700)]
Staging: unisys: visorinput: Replace '--help--' with 'help' in Kconfig

Replace '--help--' in Kconfig and indent the subsequent text to silence
checkpatch.pl
warning:
WARNING: prefer 'help' over '---help---' for new help texts

Signed-off-by: Wentao Cai <etsai042@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: unisys: visornic: Replace '--help--' with 'help' in Kconfig
Wentao Cai [Wed, 20 Mar 2019 04:45:01 +0000 (21:45 -0700)]
Staging: unisys: visornic: Replace '--help--' with 'help' in Kconfig

Replace '--help--' in Kconfig and indent the subsequent text to silence
checkpatch.pl
warning:
WARNING: prefer 'help' over '---help---' for new help texts

Signed-off-by: Wentao Cai <etsai042@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: sm750fb: Eliminate camel case
Vatsala Narang [Tue, 19 Mar 2019 20:21:57 +0000 (01:51 +0530)]
staging: sm750fb: Eliminate camel case

Edit CamelCase function name
Issue found by checkpatch.pl

Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192u: ieee80211: ieee80211_tx.c: Adjust space around else.
Sanjana Sanikommu [Tue, 19 Mar 2019 17:46:28 +0000 (23:16 +0530)]
staging: rtl8192u: ieee80211: ieee80211_tx.c: Adjust space around else.

Fix the spacce around else.
Issue found by checkpatch.pl

ERROR: spaces required before that open brace '{'
ERROR: spaces required before that close brace '}'

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192u: ieee80211: ieee80211_rx.c: Fix NULL comparisions.
Sanjana Sanikommu [Tue, 19 Mar 2019 17:05:26 +0000 (22:35 +0530)]
staging: rtl8192u: ieee80211: ieee80211_rx.c: Fix NULL comparisions.

Replace NULL comparisons in the file.
Issue found by checkpatch.pl

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192u: r8192U_dm: Remove unnecessary blank lines.
Sanjana Sanikommu [Tue, 19 Mar 2019 15:26:48 +0000 (20:56 +0530)]
staging: rtl8192u: r8192U_dm: Remove unnecessary blank lines.

Remove unnecessary blank lines.
Issue found by checkpatch.pl

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192u: r8192U_dm: Fix space issue around operators.
Sanjana Sanikommu [Tue, 19 Mar 2019 15:25:17 +0000 (20:55 +0530)]
staging: rtl8192u: r8192U_dm: Fix space issue around operators.

Add spaces around the operators.
Issue found by checkpatch.pl

CHECK: spaces preferred around that '+'
CHECK: spaces preferred around that '*'

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192u: r8192U_dm: Fix alignment issue.
Sanjana Sanikommu [Tue, 19 Mar 2019 15:23:55 +0000 (20:53 +0530)]
staging: rtl8192u: r8192U_dm: Fix alignment issue.

Ajdust alignment to match open paranthesis.
Issue found by checkpatch.pl

CHECK: Alignment should match open paranthesis.

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: gasket: Remove return variable
Nishka Dasgupta [Tue, 19 Mar 2019 19:29:11 +0000 (00:59 +0530)]
staging: gasket: Remove return variable

Remove return variable and return the return value directly. Issue
suggested by ret.cocci.

Signed-off-by: Nishka Dasgupta <nishka.dasgupta@yahoo.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: greybus: Remove parentheses around variable
Nishka Dasgupta [Tue, 19 Mar 2019 20:03:08 +0000 (01:33 +0530)]
staging: greybus: Remove parentheses around variable

Remove parentheses around variable.

Signed-off-by: Nishka Dasgupta <nishka.dasgupta@yahoo.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: erofs: fix parenthesis alignment
Julian Merida [Mon, 18 Mar 2019 23:58:41 +0000 (20:58 -0300)]
staging: erofs: fix parenthesis alignment

Fix all checkpatch issues: "CHECK: Alignment should match open parenthesis"

Signed-off-by: Julian Merida <julianmr97@gmail.com>
Reviewed-by: Gao Xiang <gaoxiang25@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: fbtft: ternary statement to if statement.
Bhagyashri Dighole [Tue, 19 Mar 2019 11:00:37 +0000 (16:30 +0530)]
staging: fbtft: ternary statement to if statement.

Convert a ternary statement into a if statement which is detected while
resolving "WARNING: line over 80 characters". Use BIT() macro instead
manually left shifting.

Signed-off-by: Bhagyashri Dighole <digholebhagyashri@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-pci-phy: Add Spaces to Macro Definition
Emanuel Bennici [Mon, 18 Mar 2019 18:58:43 +0000 (19:58 +0100)]
staging: mt7621-pci-phy: Add Spaces to Macro Definition

Improve Code readability by adding Tabs and Spaces after #define

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8192u: r8180_93cx6.c: Fix space around '<<'.
Sanjana Sanikommu [Tue, 19 Mar 2019 12:34:18 +0000 (18:04 +0530)]
Staging: rtl8192u: r8180_93cx6.c: Fix space around '<<'.

Fix the check as per Linux kernel style for use of spaces.
Issue found by checkpatch.pl

CHECK:spaces preffered around that '<<'

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8723bs: Add missing NULL check for kmalloc
Madhumitha Prabakaran [Mon, 18 Mar 2019 23:12:36 +0000 (18:12 -0500)]
Staging: rtl8723bs: Add missing NULL check for kmalloc

Include missing NULL check for kmalloc in function rtw_init_evt_priv.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8723bs: Replace NULL comparison with !
Madhumitha Prabakaran [Mon, 18 Mar 2019 23:12:35 +0000 (18:12 -0500)]
Staging: rtl8723bs: Replace NULL comparison with !

Replace NULL comparison with ! in function rtw_init_cmd_priv, to
maintain Linux kernel coding style.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8723bs: Change values to standard error codes in functions
Madhumitha Prabakaran [Mon, 18 Mar 2019 23:12:34 +0000 (18:12 -0500)]
Staging: rtl8723bs: Change values to standard error codes in functions

Change values for standard error codes in functions rtw_init_cmd_priv and
rtw_init_evt_priv, as _SUCCESS should be 0 and _FAIL should be -ENOMEM.
Also, change the values in corresponding call sites of the functions.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8723bs: Remove unnecessary local variable in function
Madhumitha Prabakaran [Mon, 18 Mar 2019 23:12:33 +0000 (18:12 -0500)]
Staging: rtl8723bs: Remove unnecessary local variable in function

Remove unnecessary local variable 'res' in function and
replace the value directly in the return of the function.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: do not use __constant_cpu_to_le16
Sergey Senozhatsky [Mon, 18 Mar 2019 11:22:21 +0000 (20:22 +0900)]
staging: rtl8723bs: do not use __constant_cpu_to_le16

cpu_to_le16() is capable enough to detect __builtin_constant_p()
and to use an appropriate compile time ___constant_swahbXX()
function.

So we can use cpu_to_le16() instead of __constant_cpu_to_le16().

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: ks7010: remove redundant auth_type check
Colin Ian King [Mon, 18 Mar 2019 10:57:49 +0000 (10:57 +0000)]
staging: ks7010: remove redundant auth_type check

The range check on auth_type is redundant as there is a prior
check on the auth_type values and the only way the block is entered
is if auth_type is one of TYPE_PMK1, TYPE_GMK1 and TYPE_GMK2. Hence
the auth_type check can be removed.

Detected by static analysis with cppcheck.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8188eu: remove unused defines from wifi.h
Michael Straube [Mon, 18 Mar 2019 10:23:59 +0000 (11:23 +0100)]
staging: rtl8188eu: remove unused defines from wifi.h

All defined P2P* in wifi.h are unused in the driver code,
so remove them.

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8188eu: rtw_ieee80211: Fix indent for 'if' statement.
Sanjana Sanikommu [Mon, 18 Mar 2019 17:56:31 +0000 (23:26 +0530)]
Staging: rtl8188eu: rtw_ieee80211: Fix indent for 'if' statement.

Adjust indentation for the condition statement.
Issue found by checkpatch.pl

WARNING: suspect code indent for condition statements(8,24)

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8188eu: core: rtw_mlme.c: Replace NULL comparision.
Sanjana Sanikommu [Mon, 18 Mar 2019 14:29:44 +0000 (19:59 +0530)]
Staging: rtl8188eu: core: rtw_mlme.c: Replace NULL comparision.

Replace NULL comparison with '!' in the file rtw_mlme.c
Issue found by checkpatch.pl semantic patch results for rtw_mlme.c

CHECK:Comparision to NULL could be written "!candidate"

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: gasket: gasket_sysfs.c: Fix string split issue.
Sanjana Sanikommu [Mon, 18 Mar 2019 16:18:40 +0000 (21:48 +0530)]
Staging: gasket: gasket_sysfs.c: Fix string split issue.

Concatenate a multi-line string constant into a single
line.Issue found by checkpatch.pl.

WARNING:quoted string split across line.

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: wlan-ng: line over 80 character limit
Branden Bonaby [Mon, 18 Mar 2019 17:25:06 +0000 (13:25 -0400)]
staging: wlan-ng: line over 80 character limit

Align function to prevent it from going over
the 80 character a line limit for readability.
WARNING: line over 80 characters

Signed-off-by: Branden Bonaby <brandonbonaby94@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: speakup: fix printk KERN_LEVEL facility level warning
Jules Irenge [Mon, 18 Mar 2019 13:21:10 +0000 (13:21 +0000)]
staging: speakup: fix printk KERN_LEVEL facility level warning

Replace printk with pr_info to solve checkpatch.pl warning:
 " WARNING: printk() should include  KERN_<LEVEL> facility level"

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8188eu: core: rtw_pwrctrl.c: Fix a comparision warning.
Sanjana Sanikommu [Mon, 18 Mar 2019 12:56:45 +0000 (18:26 +0530)]
Staging: rtl8188eu: core: rtw_pwrctrl.c: Fix a comparision warning.

Move the constant to the right side of comparision.
Issue found by checkpatch.pl semantic patch results for rtw_ap.c

WARNING:Comparision should place the constant on the right side of the
test.

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agoStaging: rtl8192e: Remove parentheses around the right hand side of assignments
Bhanusree Pola [Mon, 18 Mar 2019 07:05:53 +0000 (12:35 +0530)]
Staging: rtl8192e: Remove parentheses around the right hand side of assignments

Avoid useless parentheses to the right hand side of an assignment.
Issue found using coccinelle.
The semantic patch that fixes the problem is as follows

// <smpl>
@r1 disable paren@
expression value,e;
@@

(
- value = (e)
+ value = e
)

@r2 depends on r1@
expression value,e;
constant c;
@@

(
- value = (e == c)
+ value = (e == c)
|
- value = (e <= c)
+ value = (e <= c)
|
- value = (e >= c)
+ value = (e >= c)
|
- value = (e != c)
+ value = (e != c)
)
// </smpl>

Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Code in core/rtw_mlme.c
Emanuel Bennici [Sun, 17 Mar 2019 18:31:13 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Code in core/rtw_mlme.c

The Functions `void rtw_surveydone_event_callback`, `void rtw_indicate_connect`,
`void rtw_stadel_event_callback` and `void _rtw_join_timeout_handler`
are using the Macro CONFIG_INTEL_WIDI which doesn't exists anymore.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Functions in os_dep/ioctl_linux.c
Emanuel Bennici [Sun, 17 Mar 2019 18:31:12 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Functions in os_dep/ioctl_linux.c

The Functions `static int rtw_widi_set()` and
`static int rtw_widi_set_probe_request()` part of dead code due to
CONFIG_INTEL_WIDI.
The fix removes the deadcode.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Code in os_dep/ioctl_linux.c
Emanuel Bennici [Sun, 17 Mar 2019 18:31:11 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Code in os_dep/ioctl_linux.c

Remove Dead Code in `static int rtw_wps_start`,
`struct iw_handler rtw_private_handler[]` and
`struct iw_priv_args rtw_private_args[]` since they use the Macro
CONFIG_INTEL_WIDI witch doesn't exists any more.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Code in os_dep/os_intfs.c
Emanuel Bennici [Sun, 17 Mar 2019 18:31:10 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Code in os_dep/os_intfs.c

The Functions `u8 rtw_init_drv_sw` and `u8 rtw_free_drv_sw` uses the
Macro CONFIG_INTEL_WIDI witch doesn't exists anymore.
This Patch removes those redundant Code Lines.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Code in os_dep/os_intfs.c
Emanuel Bennici [Sun, 17 Mar 2019 18:31:09 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Code in os_dep/os_intfs.c

`uint loadparam` uses the Macro CONFIG_INTEL_WIDI witch doesn't exists
anymore.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove unreachable Code in core/rtw_cmd.c
Emanuel Bennici [Sun, 17 Mar 2019 18:31:08 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove unreachable Code in core/rtw_cmd.c

The `case INTEl_WIDI_WK_CID`-Statement in Function `u8 rtw_drvextra_cmd_hdl`
wouldn't be used because of CONFIG_INTEL_WIDI.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove old unreachable Code
Emanuel Bennici [Sun, 17 Mar 2019 18:31:07 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove old unreachable Code

The commented code in core/rtw_mlme_ext.c obsolete because the macro
CONFIG_INTEL_WIDI no longer exists.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Code in collect_bss_info Function
Emanuel Bennici [Sun, 17 Mar 2019 18:31:06 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Code in collect_bss_info Function

In `u8 collect_bss_info` is a ifdef-Preprocessor who checks
CONFIG_INTEL_WIDI, since this Macro doesn't exists anymore this Patch
removes the Dead Code in core/rtw_mlme_ext.c

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Include in include/drv_types.h
Emanuel Bennici [Sun, 17 Mar 2019 18:31:05 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Include in include/drv_types.h

`rtw_intel_widi.h` is never included due to CONFIG_INTEL_WIDI.
This Patch removes the dead code.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove Dead Code in `struct mlme_priv`
Emanuel Bennici [Sun, 17 Mar 2019 18:31:04 +0000 (19:31 +0100)]
staging: rtl8723bs: Remove Dead Code in `struct mlme_priv`

Some Lines in the struct mlme_priv are Dead due to CONFIG_INTEL_WIDI.
This Fix removes the dead code.

Signed-off-by: Emanuel Bennici <benniciemanuel78@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: gdm724x: Do not break expressions
Ignacio Losiggio [Mon, 18 Mar 2019 01:15:36 +0000 (22:15 -0300)]
staging: gdm724x: Do not break expressions

When the entire expression can be shown in the same line breaking it
makes it more difficult to read.

Signed-off-by: Ignacio Losiggio <iglosiggio@dc.uba.ar>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8188eu: core: rtw_recv.c: Fix a tab indent issue
Sanjana Sanikommu [Sun, 17 Mar 2019 15:29:38 +0000 (20:59 +0530)]
staging: rtl8188eu: core: rtw_recv.c: Fix a tab indent issue

Fix a tab indent issue which removed warnings after compilation using
checkpatch.pl.

[WARNING]:No extra space at the beginning of a line.

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8188eu: core: rtw_recv.c: fix a paranthesis issue around pframe -> list...
Sanjana Sanikommu [Sun, 17 Mar 2019 15:35:02 +0000 (21:05 +0530)]
staging: rtl8188eu: core: rtw_recv.c: fix a paranthesis issue around pframe -> list in rtw_recv.c

Remove unnecessary parathesis issue around '->' .
Issue found by checkpatch.pl semantic patch results for
rtw_recv.c

CHECK: Remove unnecessary paranthesis around pframe -> list

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8188eu: core: rtw_recv.c: fix a space issue
Sanjana Sanikommu [Sun, 17 Mar 2019 15:39:29 +0000 (21:09 +0530)]
staging: rtl8188eu: core: rtw_recv.c: fix a space issue

Fix a space issue around the below operators mentioned in CHECK.
Issue found by checkpatch.pl semantic patch results for
rtw_recv.c..

CHECK: spaces preferred around '%' (ctx:VxV)
CHECK: spaces preferred around '&' (ctx:VxV)
CHECK: spaces preferred around '+' (ctx:VxV)
CHECK: spaces preferred around '<<' (ctx:VxV)

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: gasket: gasket_interuppt.c: Fix string split issue.
Sanjana Sanikommu [Sun, 17 Mar 2019 18:13:28 +0000 (23:43 +0530)]
staging: gasket: gasket_interuppt.c: Fix string split issue.

Remove string " across the line.
Issue found by checkpatch.pl semantic patch results for rtw_ap.c

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: greybus: remove extern prototypes
Payal Kshirsagar [Sun, 17 Mar 2019 18:36:55 +0000 (00:06 +0530)]
staging: greybus: remove extern prototypes

Functions are implicitly declared with "extern". The compiler doesn't
need it.  extern removed from drivers/staging/greybus/audio_codec.h
file.

Signed-off-by: Payal Kshirsagar <payal.s.kshirsagar.98@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: speakup: fix line over 80 characters.
Jules Irenge [Sun, 17 Mar 2019 12:31:10 +0000 (12:31 +0000)]
staging: speakup: fix line over 80 characters.

Fix coding style issues which solves checkpatch.pl warning:
    "WARNING: line over 80 characters".

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8188eu: core: rtw_ap.c: Fix multiple blank lines
Sanjana Sanikommu [Sun, 17 Mar 2019 12:29:15 +0000 (17:59 +0530)]
staging: rtl8188eu: core: rtw_ap.c: Fix multiple blank lines

Remove multiple blank lines.
Issue found by checkpatch.pl semantic patch results for rtw_ap.c

CHECK: Please dont use multiple blank lines.

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Modify the return type of the function msdc_do_command
Bhanusree Pola [Sun, 17 Mar 2019 11:01:34 +0000 (16:31 +0530)]
staging: mt7621-mmc: Modify the return type of the function msdc_do_command

Modify the return type of the function msdc_do_command to static int
as the member error of the structure struct mmc_command is of type int.

Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: Fix ret.cooci warnings
Bhanusree Pola [Sun, 17 Mar 2019 11:01:33 +0000 (16:31 +0530)]
staging: mt7621-mmc: Fix ret.cooci warnings

Return the value directly instead of storing it in local variable err.
Remove the unused local variable err.
The semantic patch that fixes the first part of the problem is as follows

// <smpl>
@@
local idexpression ret;
expression e;
@@

-ret =
+return
     e;
-return ret;
// </smpl>

Signed-off-by: Bhanusree Pola <bhanusreemahesh@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Change type of variables and return type
Madhumitha Prabakaran [Sun, 17 Mar 2019 21:33:55 +0000 (16:33 -0500)]
staging: rtl8723bs: Change type of variables and return type

Change type of local variables 'res' and return type of functions
'rtw_init_cmd_pri' and 'rtw_init_evt_priv', as function's
return types are defined for standard error codes _SUCCESS and _FAIL.
Also, change return type of functions declarations corresponding to
change made in function definitions.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: Remove wrapper functions and change function names
Madhumitha Prabakaran [Sun, 17 Mar 2019 21:33:54 +0000 (16:33 -0500)]
staging: rtl8723bs: Remove wrapper functions and change function names

Drop wrappers rtw_init_cmd_priv and rtw_init_evt_priv and remove their
function declarations, as their only purpose is to call other functions.
Change function names from _rtw_init_cmd_priv to
rtw_init_cmd_priv, and _rtw_init_evt_priv to rtw_init_evt_priv in
function definitions and function declarations. Also, remove externs in
function declarations rtw_init_cmd_priv and rtw_init_evt_priv, to
maintain Linux kernel coding style.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: os_dep: Fix space in pointer definition
Guilherme Tadashi Maeoka [Sat, 9 Mar 2019 18:30:46 +0000 (15:30 -0300)]
staging: rtl8723bs: os_dep: Fix space in pointer definition

Pointer definition "foo * bar" should be "foo *bar".

Signed-off-by: Guilherme Tadashi Maeoka <gui.maeoka@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: os_dep: Fix assignment in if condition
Guilherme Tadashi Maeoka [Sat, 9 Mar 2019 18:30:43 +0000 (15:30 -0300)]
staging: rtl8723bs: os_dep: Fix assignment in if condition

Fix an assignment in if condition.

Signed-off-by: Guilherme Tadashi Maeoka <gui.maeoka@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: vt6655: rxtx.c: Remove unnecessary space after a cast
Sanjana Sanikommu [Sun, 17 Mar 2019 10:47:51 +0000 (16:17 +0530)]
staging: vt6655: rxtx.c: Remove unnecessary space after a cast

Issue found by checkpatch.pl semantic patch results for
rxtx.c. Remove unnecessary space after the cast.

CHECK:
No necessary space is required after a cast.

Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: remove unused code
Colin Ian King [Sat, 16 Mar 2019 22:26:57 +0000 (22:26 +0000)]
staging: rtl8723bs: remove unused code

There are two final hunks of code that are only built
if SDIO_DYNAMIC_ALLOC_MEM is defined however this is never
defined and if it was the code would lead to dereferencing
an uninitialized pointer oldmem.  It appears that a previous
commit 1babeb0c3e59 ("Staging: rtl8723bs: Remove dead code")
removed some of the dead code but not all of it. Clean this
up by removing the final hunks.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192u: Add required spaces before open parenthesis
Sidong Yang [Tue, 12 Mar 2019 14:11:50 +0000 (14:11 +0000)]
staging: rtl8192u: Add required spaces before open parenthesis

Fix error reported by checkpatch.pl

Signed-off-by: Sidong Yang <realwakka@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: vc04_services: Use tabs instead of spaces
Alejandro Ismael Silva [Tue, 12 Mar 2019 12:35:27 +0000 (09:35 -0300)]
staging: vc04_services: Use tabs instead of spaces

This patch fixes the checkpatch.pl error:

WARNING: please, no spaces at the start of a line

in the interface/vchi directory.

Signed-off-by: Alejandro Ismael Silva <silva.alejandro.ismael@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8712 - fix up non-kerneldoc comments
Valdis Klētnieks [Tue, 12 Mar 2019 12:09:20 +0000 (08:09 -0400)]
staging: rtl8712 - fix up non-kerneldoc comments

Building with W=1 reports (among other things)

  CC [M]  drivers/staging/rtl8712/rtl871x_ioctl_linux.o
drivers/staging/rtl8712/rtl871x_ioctl_linux.c:1106: warning: Cannot understand  *
 on line 1106 - I thought it was a doc line

  CC [M]  drivers/staging/rtl8712/os_intfs.o
drivers/staging/rtl8712/os_intfs.c:366: warning: Cannot understand  *
 on line 366 - I thought it was a doc line
drivers/staging/rtl8712/os_intfs.c:434: warning: Cannot understand  *
 on line 434 - I thought it was a doc line

Fix up the non-kerneldoc comments

Signed-off-by: Valdis Kletnieks <valdis.kletnieks@vt.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: erofs: add a new line after variable declaration
Julio Bianco [Sat, 9 Mar 2019 17:08:53 +0000 (14:08 -0300)]
staging: erofs: add a new line after variable declaration

Add a new line after variable declaration

Signed-off-by: Julio Bianco <juliobianco@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: octeon-usb octeon-hcd: Fix several typos.
Laura Lazzati [Sat, 9 Mar 2019 18:18:27 +0000 (15:18 -0300)]
staging: octeon-usb octeon-hcd: Fix several typos.

I found that the comments had several typos such as "aenable", "internaly" and some others.
I fixed them all.

Signed-off-by: Laura Lazzati <laura.lazzati.15@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: fbtft: fixed format-string errors.
Jeremy Sowden [Fri, 8 Mar 2019 08:49:51 +0000 (08:49 +0000)]
staging: fbtft: fixed format-string errors.

Added __printf attribute to declaration of fbtft_dbg_hex and fixed
mismatches between format-specifiers and arguments in several function
calls.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8188eu: remove unnecessary braces from if/else
Hugo Lange [Sun, 3 Mar 2019 20:24:39 +0000 (20:24 +0000)]
staging: rtl8188eu: remove unnecessary braces from if/else

Removed unnecessary braces for single blocks.

Signed-off-by: Hugo Lange <hugolange7000@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: use kernel_read() instead of open-coded version
Jann Horn [Fri, 1 Mar 2019 20:11:24 +0000 (21:11 +0100)]
staging: rtl8723bs: use kernel_read() instead of open-coded version

Replace the manual call to f_op->read() under KERNEL_DS with kernel_read().
This also reduces the number of users of the legacy alias get_ds() for
KERNEL_DS.

Signed-off-by: Jann Horn <jannh@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Modify comparison to constant in rtl8723bs_xmit.c
Himadri Pandya [Sun, 17 Mar 2019 06:15:01 +0000 (11:45 +0530)]
staging: rtl8723bs: hal: Modify comparison to constant in rtl8723bs_xmit.c

Modify comparison to true in file rtl8723bs_xmit in order to follow
kernel coding style of constant should appear on the right side.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Modify comparison to constant in hal_btcoex.c
Himadri Pandya [Sun, 17 Mar 2019 06:15:00 +0000 (11:45 +0530)]
staging: rtl8723bs: hal: Modify comparison to constant in hal_btcoex.c

Modify comparison to 0 in file hal_btcoex.c in order to follow kernel
coding style of constant should appear on right side.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in hal_btcoex.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:59 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in hal_btcoex.c

- Remove comparison to NULL in file hal_btcoex.c. Suggested by
Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in hal_com.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:58 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in hal_com.c

Remove comparison to NULL in file hal_com.c. Suggested by Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in hal_com_phycfg.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:57 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in hal_com_phycfg.c

Remove comparison to NULL in file hal_com_phycfg.c. Suggested by Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_cmd.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:56 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_cmd.c

Remove comparison to NULL in file rtl8723b_cmd.c. Suggested by
Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal:Remove comparison to NULL in rtl8723bs_xmit.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:55 +0000 (11:44 +0530)]
staging: rtl8723bs: hal:Remove comparison to NULL in rtl8723bs_xmit.c

Remove comparison to NULL in file rtl8723bs_xmit.c. Suggested by Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_dm.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:54 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_dm.c

Remove comparison to NULL in file rtl8723b_dm.c. Suggested by Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_hal_init.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:53 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_hal_init.c

Remove comparison to NULL in file rtl8723b_hal_init.c. Suggested by
Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in rtl8723bs_recv.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:52 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in rtl8723bs_recv.c

Remove comparison to NULL in file rtl8723bs_recv.c. Suggested by
Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_rxdesc.c
Himadri Pandya [Sun, 17 Mar 2019 06:14:51 +0000 (11:44 +0530)]
staging: rtl8723bs: hal: Remove comparison to NULL in rtl8723b_rxdesc.c

Remove comparison to null in file rtl8723b_rxdesc.c. Suggested by Coccinelle.

Signed-off-by: Himadri Pandya <himadri18.07@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: sm750fb: Remove an unnecessary local variable in a function
Madhumitha Prabakaran [Wed, 13 Mar 2019 15:32:05 +0000 (10:32 -0500)]
staging: sm750fb: Remove an unnecessary local variable in a function

Remove an unnecessary local variable in  sm750_format_pll_reg function
and replace its initialization directly in return type.
Issue suggested by Coccinelle using ret.cocci.

Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-mmc: delele extra blank lines
Wentao Cai [Thu, 14 Mar 2019 18:25:14 +0000 (11:25 -0700)]
staging: mt7621-mmc: delele extra blank lines

Delete extra blank lines to mute checkpatch.pl check:
CHECK: Please don't use multiple blank lines

Signed-off-by: Wentao Cai <etsai042@gmail.com>
Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtlwifi: Replace 0 with false.
Bhagyashri Dighole [Thu, 14 Mar 2019 12:18:03 +0000 (17:48 +0530)]
staging: rtlwifi: Replace 0 with false.

Replace 0 with false, Bool intializations should use true or false.
Detected by Coccinelle semantic patch boolinit.cocci.

Signed-off-by: Bhagyashri Dighole <digholebhagyashri@gmail.com>
Acked-by: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: pi433: Remove unused variable.
Bhagyashri Dighole [Wed, 13 Mar 2019 19:07:17 +0000 (00:37 +0530)]
staging: pi433: Remove unused variable.

Remove unused variable which is used to store return value,
Detected by Coccinelle semantic patch returnvar.cocci

Signed-off-by: Bhagyashri Dighole <digholebhagyashri@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192e: remove boilerplate license text
Jules Irenge [Wed, 13 Mar 2019 14:53:20 +0000 (14:53 +0000)]
staging: rtl8192e: remove boilerplate license text

Remove boilerplate license text.

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8192e: add SPDX GPL-2.0 license identifier
Jules Irenge [Wed, 13 Mar 2019 14:53:19 +0000 (14:53 +0000)]
staging: rtl8192e: add SPDX GPL-2.0 license identifier

Add the SPDX GPL-2.0 license identifier to fix checkpatch.pl warning
Issue found by checkpatch.pl warning:
"WARNING: Missing or malformed SPDX-License-Identifier tag in line 1"

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: fbtft: line over 80 characters
Bhagyashri Dighole [Wed, 13 Mar 2019 08:15:09 +0000 (13:45 +0530)]
staging: fbtft: line over 80 characters

Fix the checkpatch.pl warning
WARNING: line over 80 characters

Signed-off-by: Bhagyashri Dighole <digholebhagyashri@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: gasket: Replace macro __ATTR with __ATTR_NULL
Wentao Cai [Wed, 13 Mar 2019 08:14:53 +0000 (01:14 -0700)]
staging: gasket: Replace macro __ATTR with __ATTR_NULL

Replace macro __ATTR with __ATTR_NULL since the former one is
unnecessary for gasket_sysfs_attr array terminator. Also silence the
checkpatch.pl warning:
WARNING:Symbolic permissions 'S_IRUGO' are not preferred. Consider using
octal
permissions '0444'.

Signed-off-by: Wentao Cai <etsai042@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-dts: update ethernet settings.
NeilBrown [Mon, 11 Mar 2019 23:09:37 +0000 (10:09 +1100)]
staging: mt7621-dts: update ethernet settings.

The ethernet in mt7621 is now supported by
 drivers/net/ethernet/mediatek/

which provides support for the integrated switch through DSA.

This requires some devicetree changes, and particularly allows
a board dts to identify which switch ports are present.

The second CPU interface - gmac1 - doesn't work yet, so the device
tree information may not be correct.  The phy (which is present on the
gnubee-pc2) can negotiate and report connection speed etc, but no
traffic flows.

The gnubee-pc1 has two network ports which are 'black' and 'blue'.
There are connected to switch ports 0 and 4 respectively.

Signed-off-by: NeilBrown <neil@brown.name>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: remove mt7621-eth
NeilBrown [Mon, 11 Mar 2019 23:09:37 +0000 (10:09 +1100)]
staging: remove mt7621-eth

driver/net/ethernet/mediatek/ now supports this hardware,
so we don't need a separate driver.

Signed-off-by: NeilBrown <neil@brown.name>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: mt7621-pci: IF statement expression comparing to NULL
Branden Bonaby [Tue, 12 Mar 2019 18:06:15 +0000 (14:06 -0400)]
staging: mt7621-pci: IF statement expression comparing to NULL

Remove comparison to NULL in the if statement expression
to match the Linux Kernel coding style.
CHECK: Comparison to NULL could be written "res"

Signed-off-by: Branden Bonaby <brandonbonaby94@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: rtl8712: line over 80 characters.
Bhagyashri Dighole [Tue, 12 Mar 2019 07:16:55 +0000 (12:46 +0530)]
staging: rtl8712: line over 80 characters.

Fix coding style issues, which solves checkpatch.pl warning:
"WARNING: line over 80 characters".

Signed-off-by: Bhagyashri Dighole <digholebhagyashri@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: octeon: Lines should not end with a '('
Branden Bonaby [Tue, 12 Mar 2019 02:48:06 +0000 (22:48 -0400)]
staging: octeon: Lines should not end with a '('

Start function arguments immediately after opening bracket.
CHECK: Lines should not end with a '('.

Signed-off-by: Branden Bonaby <brandonbonaby94@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
5 years agostaging: most: Replace '--help--' in Kconfig file
Wentao Cai [Tue, 12 Mar 2019 08:10:39 +0000 (01:10 -0700)]
staging: most: Replace '--help--' in Kconfig file

Replace '--help--' with 'help' to silence checkpatch.pl warning:
WARNING: prefer 'help' over '---help---' for new help texts

Signed-off-by: Wentao Cai <etsai042@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>