shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-06-01 08:46:36 +09:00
Code Issues Packages Projects Releases Wiki Activity
691,586 Commits 55 Branches 2,486 Tags
dcecf46f3d30fc72d3744ca3106c483e30fe1b0e
Commit Graph

691586 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
David Kershner
dcecf46f3d staging: unisys: visorbus: Remove unused define for visorchipset. 
The define CURRENT_FILE_PC was not being used, so just remove it.

Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Mark Foresta
ce0e7435fe staging: unisys: visorbus: cleaned up include block of visorchipset.c 
Removed 4 unneeded includes netdevice, nls, fs, and ctype.

Signed-off-by: Mark Foresta <Mark.Foresta@Unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Mark Foresta
8649375a51 staging: unisys: visorbus: Removed unused define from visorbus_main.c 
Removed unused #define CURRENT_FILE_PC.

Signed-off-by: Mark Foresta <Mark.Foresta@Unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Mark Foresta
54e655d9fb staging: unisys: visorbus: removed blank line in viorbus_main.c 
Removed blank line between #defines.

Signed-off-by: Mark Foresta <Mark.Foresta@Unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Erik Arfvidson
a35e3268da staging: unisys: visorchipset: remove local_addr in handle_command 
In the function handle_command, the variable local_addr is always false. So
this patch is getting rid of logic when it is true.

Signed-off-by: Erik Arfvidson <erik.arfvidson@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Erik Arfvidson
460ea8ada0 staging: unisys: visorbus: remove target_hostname comment 
This patch simply removes TARGET_HOSTNAME comment that is no
longer relevant.

Signed-off-by: Erik Arfvidson <erik.arfvidson@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Erik Arfvidson
e3b2ed6664 staging: unisys: visorbus: rename fix_vbus_dev_info 
Rename fix_vbus_dev_info to something clearer: publish_vbus_dev_info.

Signed-off-by: Erik Arfvidson <erik.arfvidson@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Zachary Dremann
661a215bc3 staging: unisys: visorbus: Fix memory leak 
The name of a visor_device was never freed, which was allocated in
visorbus_configure.

It is expected that visorbus_device_destroy will not be called on the
same visor_device again, or this would be a double free.

Signed-off-by: Zachary Dremann <dremann@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Sameer Wadgaonkar
403043c4e1 staging: unisys: moved visor_check_channel from include/channel.h to visorbus/visorbus_main.c 
Moved the function visor_check_channel() from include/channel.h to
visorbus/visorbus_main.c.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:42 +02:00
Sameer Wadgaonkar
790627cdde staging: unisys: include: Remove unused CHANNEL_OK defines. 
The VISOR_VNIC_CHANNEL_OK_CLIENT and VISOR_HBA_CHANNEL_OK_CLIENT macros
were not being used. They can be removed.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Sameer Wadgaonkar
0f6b44ad89 staging: unisys: remove unused define VISOR_VSWITCH_CHANNEL_VERSIONID 
The VSWITCH channel is not used in Linux guests, so remove the VERSIONID.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Sameer Wadgaonkar
c2d00b218e staging: unisys: visorbus: add checks for probe, remove, pause and resume in visorbus_register_visor_driver 
Added checks for functions probe, remove, pause and resume in
visorbus_register_visor_driver() and removed these checks from
the individual functions itself.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Sameer Wadgaonkar
a3276bf32a staging: unisys: visorbus: visorbus_main.c: remove check from typename_show 
Remove check for xbus in typename_show() function in
visorbus_main.c.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Sameer Wadgaonkar
baf9b70540 staging: unisys: visorbus: visorbus_main.c: put function name and return value on same line. 
This patch makes function definitions more consistent by having function
name and return values on the same line.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Sameer Wadgaonkar
d72e1a1c48 staging: unisys: visorbus: visorbus_main.c: remove extra checks for dev->visorchannel 
Removed checks for dev->visorchannel in visorbus_release_device()
and visorbus_remove_instance() since it is also checked in the
visorchannel_destroy() function.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Sameer Wadgaonkar
55b7bd03f8 staging: unisys: visorbus: convert VMCALL_CONTROLVM_ADDR enum to #define 
VMCALL_CONTROLVM_ADDR is the only element left in enum
vmcall_monitor_interface_method_tuple. Converting it to a #define.

Signed-off-by: Sameer Wadgaonkar <sameer.wadgaonkar@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Alex Curtin
0330a357ff staging: unisys: include: iochannel.h: removed VISOR_VSWITCH_CHANNEL_SIGNATURE 
VISOR_VSWITCH_CHANNEL_SIGNATURE is a redundant definition that points to
VISOR_CHANNEL_SIGNATURE. It wasn't being used, so it has been removed.

Signed-off-by: Alex Curtin <alexander.curtin@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Alex Curtin
b4459db633 staging: unisys: include: iochannel.h: removed VISOR_VNIC_CHANNEL_SIGNATURE 
VISOR_VNIC_CHANNEL_SIGNATURE is a redundant definition that points to
VISOR_CHANNEL_SIGNATURE. This replaces that definition with
VISOR_CHANNEL_SIGNATURE.

Signed-off-by: Alex Curtin <alexander.curtin@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Alex Curtin
c1e1c4d101 staging: unisys: include: iochannel.h: removed VISOR_VHBA_CHANNEL_SIGNATURE 
VISOR_VHBA_CHANNEL_SIGNATURE is a redundant definition that points to
VISOR_CHANNEL_SIGNATURE. This replaces that definition with
VISOR_CHANNEL_SIGNATURE.

Signed-off-by: Alex Curtin <alexander.curtin@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Alex Curtin
e9b9275c3e staging: unisys: visorbus: vbuschannel.h: removed VISOR_VBUS_CHANNEL_SIGNATURE 
VISOR_VBUS_CHANNEL_SIGNATURE just pointed to VISOR_CHANNEL_SIGNATURE, so
this replaces the redundant definition with VISOR_CHANNEL_SIGNATURE.

Signed-off-by: Alex Curtin <alexander.curtin@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Alex Curtin
8e516683c4 staging: unisys: visorbus: controlvmchannel.h: removed VISOR_CONTROLVM_CHANNEL_SIGNATURE 
VISOR_CONTROLVM_CHANNEL_SIGNATURE points to VISOR_CHANNEL_SIGNATURE, so
this replaces the redundant definition with VISOR_CHANNEL_SIGNATURE.

Signed-off-by: Alex Curtin <alexander.curtin@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
Erik Arfvidson
c4d0a30703 staging: unisys: include: channel.h: remove unused pound defines 
This patch removes all the unused pound defines currently in
channel.h.

Signed-off-by: Erik Arfvidson <erik.arfvidson@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:41 +02:00
David Binder
0d89f94afb staging: unisys: visorhba: viosrhba_main.c: Remove unnecessary checks 
Removes unnecessary checks for a NULL pointer in a non-API function.

Signed-off-by: David Binder <david.binder@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
Erik Arfvidson
b06f5d1116 staging: unisys: visorbus: controlvmchannel.h: fix spacing 
This patch fixes space and tab aligned comments in
controlvmchannel.h.

Signed-off-by: Erik Arfvidson <erik.arfvidson@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
David Binder
920680f4a7 staging: unisys: visornic: visornic_main.c: Adjust whitespace usage 
Adjusts whitespace usage in order to create a more uniform coding style.

Signed-off-by: David Binder <david.binder@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
David Binder
3cc1db3d96 staging: unisys: visorinput: visorinput.c: Adjust whitespace usage 
Adjusts whitespace usage in order to create a more uniform coding style.

Signed-off-by: David Binder <david.binder@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
David Binder
400678b19f staging: unisys: visorhba: visorhba_main.c: Adjust whitespace usage 
Removes unnecessary blank lines to create a more uniform coding style.

Signed-off-by: David Binder <david.binder@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
David Binder
d4e8a22e87 staging: unisys: visorbus: visorbus_main.c: Adjust code layout 
Adjusts the ordering of function statements, as well as modifies
whitespace usage, in order to create a more uniform coding style.

Signed-off-by: David Binder <david.binder@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
David Binder
904ee62ac5 staging: unisys: visorbus: visorchipset.c: Adjust code layout 
Adjusts the ordering of function statements, as well as modifies whitespace
usage, in order to create a more uniform coding style.

Signed-off-by: David Binder <david.binder@unisys.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
Zachary Dremann
23d01c425b staging: unisys: visorbus: visorbus_main.c: use __func__ over hardcoded name 
As reported by checkpatch.pl, replace hard-coded usage of the current
function's name in format string with usage of __func__.

Signed-off-by: Zachary Dremann <dremann@gmail.com>
Signed-off-by: David Kershner <david.kershner@unisys.com>
Reviewed-by: David Binder <david.binder@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-18 08:48:40 +02:00
Colin Ian King
d56c160cb6 staging: lustre: fix spelling mistake, "grranted" -> "granted" 
Trivial fix to spelling mistake in CERROR error message. Also
clean up the grammar.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-17 14:38:30 +02:00
James Simmons
b6c82cc87f staging: lustre: lustre: fix all braces issues reported by checkpatch 
Cleanup all braces that was reported by checkpatch. The only
issue not fixed up is in mdc_lock.c. Removing the braces in
the case of mdc_lock.c will break the build.

Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-17 14:38:30 +02:00
Shreeya Patel
09dad31208 Staging: rtl8192u: Use __func__ instead of function name. 
Current function name is accessed using __func__.
Use '%s and __func__' instead of a function name.

Problem found by checkpatch.

Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-17 14:38:29 +02:00
Marcus Wolf
874bcba65f staging: pi433: New driver 
Added a driver for the pi433 radio module
(see https://www.pi433.de/en.html for details).

Signed-off-by: Marcus Wolf <linux@Wolf-Entwicklungen.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 16:58:13 +02:00
Craig Inches
07119e2573 Staging: Lustre Fix block statement style issue 
This fixes a block statement which didnt end with */

Signed-off-by: Craig Inches <Craig@craiginches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:03 +02:00
Craig Inches
081cb7dbaa Staging: Lustre Fixing multiline block comments in lnetst.h 
This fixes multiple block statements found not to match
style as per checkpatch

Signed-off-by: Craig Inches <Craig@craiginches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Craig Inches
e066348300 Staging: Lustre Fix up multiple Block Comments in lib-types.h 
This patch fixes some multiline comment blocks which didnt conform
to the style guide, found by checkpatch.

Signed-off-by: Craig Inches <Craig@craiginches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Craig Inches
358fc9771e Staging: Lustre Clean up line over 80Char in lib-lnet.h 
This patch fixes a warning generated by checkpatch for
a line over 80 characters.

Signed-off-by: Craig Inches <Craig@craiginches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Simon Sandström
f124a47857 staging: vt6656: Use variable instead of its type in sizeof(...) 
Use sizeof(variable) instead of sizeof(type) in memory allocations to
prevent problems if the variable type changes in the future.

Signed-off-by: Simon Sandström <simon@nikanor.nu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Simon Sandström
3781f88786 staging: vt6656: Align function parameters 
Fixes checkpatch.pl warnings "Alignment should match open parenthesis".

Signed-off-by: Simon Sandström <simon@nikanor.nu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Simon Sandström
9e6519bbca staging: vt6656: Remove unnecessary blank lines 
Fixes checkpatch.pl warning "Blank lines aren't necessary after an open
brace". Also adds braces to balance if-else statement.

Signed-off-by: Simon Sandström <simon@nikanor.nu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Simon Sandström
f21997572c staging: vt6656: Add spaces between operators 
Fixes checkpatch.pl warnings "spaces preferred around that <operator>".

Signed-off-by: Simon Sandström <simon@nikanor.nu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Janusz Lisiecki
7b379dba14 staging: ks7010: Fix cast to restricted __le16 in ks_wlan_net.c 
This patch fixes the following Sparse warnings in ks_wlan_net.c:
drivers/staging/ks7010/ks_wlan_net.c:1359:24: warning: cast to restricted __le16
link_ap_info_t structure field 'capability' has native order and is used everywhere
in the code in such way (i.e get_ap_information, get_current_ap). Both sides of
assignment are u16 (native order) so 'le16_to_cpu' is not needed and wrong.

Signed-off-by: Janusz Lisiecki <janusz.lisiecki@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Lukasz Faber
0c2310217b staging: rtl8192u: Fix braces placement and spacing 
Fixes the following checkpatch errors:
- open brace '{' following enum go on the same line
- space required after that close brace '}'

Signed-off-by: Lukasz Faber <faber@agh.edu.pl>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Arnd Bergmann
4596351d80 staging: rtl8192u: reduce stack frame size in ieee80211_rx_mgt_rsl 
I ran into a stack frame size warning in code that hasn't changed much
recently, no idea why I didn't spot this earlier despite build many
thousand randconfigs.

drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c: In function 'ieee80211_rx_mgt_rsl':
drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:2563:1: error: the frame size of 1676 bytes is larger than 1024 bytes [-Werror=frame-larger-than=]

We have a couple of other copies of the same function in the kernel,
and they simply don't put a gigantic structure on the stack, so I'm
changing this copy to do the same as the others, replacing the
stack variable with a GFP_ATOMIC allocation.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Colin Ian King
023eba2749 staging: fbtft: make const array gamma_par_mask static 
Don't populate array gamma_par_mask on the stack but instead make it
static.  Makes the object code smaller by 148 bytes:

Before:
   text	   data	    bss	    dec	    hex	filename
   2993	   1104	      0	   4097	   1001	drivers/staging/fbtft/fb_st7789v.o

After:
   text	   data	    bss	    dec	    hex	filename
   2757	   1192	      0	   3949	    f6d	drivers/staging/fbtft/fb_st7789v.o

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Rui Teng
186882700f staging: lustre: fix sparse error: incompatible types in comparison expression 
Comparing two user space addresses to avoid sparse error:

drivers/staging//lustre/lnet/selftest/conrpc.c:490:30: error:
incompatible types in comparison expression (different address spaces)

Signed-off-by: Rui Teng <rui.teng@linux.vnet.ibm.com>
Acked-by: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Bincy K Philip
187645ef1f staging: ccree: move comment to fit coding style 
Trivial fix for Line over 80 characters

Moved the comment to top of the definition

Signed-off-by: Bincy K Philip <bincy_k_philip@yahoo.co.in>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Gilad Ben-Yossef
04f682b936 staging: ccree: remove whitespace before a quoted newline 
Remove unnecessary whitespace before a quoted newline in strings.

Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:02 +02:00
Gilad Ben-Yossef
b71ad22dff staging: ccree: avoid unnecessary line continuation 
Avoid unnecessary line continuation in log function call.

Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 2017-07-16 08:41:01 +02:00