- 02 Oct, 2015 40 commits
-
-
Larry Finger authored
In commit 0a0796eb, there is no include to mon.h, a newly created header file. This omission leads to the following Sparse warnings: CHECK drivers/staging/rtl8188eu/os_dep/mon.c drivers/staging/rtl8188eu/os_dep/mon.c:81:6: warning: symbol 'rtl88eu_mon_recv_hook' was not declared. Should it be static? drivers/staging/rtl8188eu/os_dep/mon.c:113:6: warning: symbol 'rtl88eu_mon_xmit_hook' was not declared. Should it be static? drivers/staging/rtl8188eu/os_dep/mon.c:167:19: warning: symbol 'rtl88eu_mon_init' was not declared. Should it be static? drivers/staging/rtl8188eu/os_dep/mon.c:188:6: warning: symbol 'rtl88eu_mon_deinit' was not declared. Should it be static? Signed-off-by:
Larry Finger <Larry.Finger@lwfinger.net> Cc: Jakub Sitnicki <jsitnicki@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Arjun Krishna Babu authored
Two lines of code that were over 80 characters long is fixed by splitting them across multiple lines. The lines of code are now easier to comprehend. Issue found by checkpatch. Signed-off-by:
Arjun Krishna Babu <arjunkrishnababu96@gmail.com> Signed-off-by:
-
Ian Abbott authored
Expand the descriptions of the functions and document the return values. Signed-off-by:
Ian Abbott <abbotti@mev.co.uk> Reviewed-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Signed-off-by:
-
Ian Abbott authored
Expand the descriptions of the functions and document the return values. Signed-off-by:
-
Ian Abbott authored
Fix the 'Return' tags in the kernel-doc comments as they currently say 'Returns', which is not recognized by kernel-doc. Signed-off-by:
-
Rohit kumar authored
This patch fixes error handling case when buffer->pages allocation fails. Also, it removes unreachable code of checking ret variable although it is not updated. Signed-off-by:
Rohit kumar <rohit.kr@samsung.com> Reviewed-by:
Laura Abbott <labbott@redhat.com> Suggested-by:
Pintu Kumar <pintu.k@samsung.com> Reviewed-by:
Gioh Kim <gioh.kim@lge.com> Signed-off-by:
-
Tim Sell authored
No need for serio.h to be included. Signed-off-by:
Tim Sell <Timothy.Sell@unisys.com> Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
-
Tim Sell authored
The most-noticed key that wasn't being mapped correctly was Right-Alt, which is the AltGr key on many non-US keyboards, used to select many extended characters. Signed-off-by:
-
Benjamin Romer authored
Properly handle the return value from queue_delayed_work() - it's a bool, not an int, so using a less than comparison isn't appropriate. This mistake was found by David Binderman <dcb314@hotmail.com>. Signed-off-by:
-
David Kershner authored
We don't need the list of devices, we can loop through the one provided by the network api and filter on ours. Signed-off-by:
David Kershner <david.kershner@unisys.com> Signed-off-by:
-
David Kershner authored
devnum pool and devnum are no longer needed. Signed-off-by:
-
David Kershner authored
The net device already has a name, so use that instead. Signed-off-by:
-
Shivani Bhardwaj authored
Fixed the warning generated by sparse that 'cast truncates bits from constant value' by typecasting unsigned values to u8 as their logical operation is being performed with and stored in a u8 type variable. Signed-off-by:
Shivani Bhardwaj <shivanib134@gmail.com> Signed-off-by:
-
Cristina Moraru authored
Move final */ to the next line, in accordance with the suggestion of checkpatch Signed-off-by:
Cristina Moraru <cristina.moraru09@gmail.com> Signed-off-by:
-
Cristina Moraru authored
Fixed 'suspect code indent for conditional statements (8, 24)' Warning Signed-off-by:
-
Aybuke Ozdemir authored
This patch "uint*_t" type instead of "u*" type was used. checkpatch.pl issue in octeon driver. Signed-off-by:
Aybuke Ozdemir <aybuke.147@gmail.com> Reviewed-by:
Arnd Bergmann <arnd@arndb.de> Signed-off-by:
-
Shraddha Barke authored
Remove debug messages related to fbtft_par_dbg(DEBUG_BACKLIGHT.. ) as this info can be obtained using kernel function tracer Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Shraddha Barke authored
Remove debug messages related to fbtft_par_dbg(DEBUG_WRITE_VMEM.. ) as this info can be obtained using kernel function tracer Signed-off-by:
-
Ioana Ciornei authored
Update indentation of block comments to follow linux coding style Signed-off-by:
Ioana Ciornei <ciorneiioana@gmail.com> Signed-off-by:
-
Shraddha Barke authored
Declare structure as static since it is not accessed anywhere apart from this particular file. Signed-off-by:
-
Oleg Drokin authored
Now that ldlm_flock code no longer uses it. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Oleg Drokin authored
This is server-side code that cannot work on the client (vfs would do this check on the local node). Signed-off-by:
-
Oleg Drokin authored
Since there are no server namespaces, just replace it with ldlm_cli_inactive_namespace_list pointer. Also make ldlm_cli_inactive_namespace_list static as it's only used in ldlm_resource.c Signed-off-by:
-
Oleg Drokin authored
This rather large patch prunes all unused EXPORT_SYMBOLS and marks functions only used locally as static lustre ldlm module. The only two remaining nonstatic functions that should be static now are: ldlm_cancel_lru_local ldlm_resource_putref_locked But some bigger code shuffling around is needed to achieve that, so it's left for a future patch. Signed-off-by:
-
Oleg Drokin authored
Server-side scanning is not really used in the client code, so it's ok to drop it. Signed-off-by:
-
Oleg Drokin authored
Since the code we have is Lustre-client only, this function always returns 0, so drop it and amend all the callsites to drop dead code. One of the places also sets LDLM_FL_NS_SRV to indicate a lock is in a server namespace. This too cannot happen in this code, so drop all such checks as well. Signed-off-by:
-
Oleg Drokin authored
Since staging tree code is just the client, ns_is_client is always true, so change all callers as such and drop all the dead code for when it's false. Signed-off-by:
-
Oleg Drokin authored
All users are gone, and they were used on the server anyway. Signed-off-by:
-
Oleg Drokin authored
It was only used on the server. Signed-off-by:
-
Oleg Drokin authored
They are only used on the server. Also remove helper functions and cleanup callsites. Signed-off-by:
-
Oleg Drokin authored
This includes ldlm_register_intent(), ns_policy field in the namespace and all of it's users, as this could only happen on the server. Signed-off-by:
-
Oleg Drokin authored
Nothing adds locks into waiting list on the client, so no point in retaining those. Signed-off-by:
-
Oleg Drokin authored
Signed-off-by:
-
Oleg Drokin authored
Signed-off-by:
-
Oleg Drokin authored
It was directly used only on the server. Client side part was split into smaller chunks to avoid deadlocks. Signed-off-by:
-
Oleg Drokin authored
It's not used anywhere. Signed-off-by:
-
Oleg Drokin authored
Only used on the server. Signed-off-by:
-
Oleg Drokin authored
This particular incarnation is only used on the server. Signed-off-by:
-
Oleg Drokin authored
Not used anywhere Signed-off-by:
-
Oleg Drokin authored
And all supporting export infrastructure. There's no use for it all on client. Signed-off-by:
-
