- 02 Oct, 2015 40 commits
-
-
Mike Rapoport authored
The clear methods of lynxfb_crtc and lynxfb_output are empty and therefore they can be removed. Signed-off-by:
Mike Rapoport <mike.rapoport@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Mike Rapoport authored
The proc_checkMode of lynxfb_output is set to function that always returns 0. Calling that function and checking for its return value is meaningless. Signed-off-by:
-
Mike Rapoport authored
The lynxfb_ops_check_var function has two switch statements, the first one checks for validity of var->bits_per_pixel and it is is immediatly followed by another one that sets parameters according to var->bits_per_pixel value. These switch statements can be merged into one. Signed-off-by:
-
Iban Rodriguez authored
Correct two errors reported by checkpatch.pl because space prohibited before ','. Also split both lines into multiple lines as they are longer than 80 characters Signed-off-by:
Iban Rodriguez <iban.rodriguez@ono.com> Signed-off-by:
-
Andy Shevchenko authored
...instead of custom approach Cc: Joshua Clayton <stillcompiling@gmail.com> Signed-off-by:
Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by:
-
Bryan Paul authored
Fixed style issue if comparison should place constant on right of test Signed-off-by:
Bryan Paul <bryan.paul@yahoo.com> Signed-off-by:
-
Luis de Bethencourt authored
wstats->qual.noise is of type uint8, so it shouldn't be assigned a negative number. Assigning it 0x100 - 100, which is the equivalent to -100 dBm when IW_QUAL_DBM is set. Signed-off-by:
Luis de Bethencourt <luisbg@osg.samsung.com> Tested-by:
Mateusz Kulikowski <mateusz.kulikowski@gmail.com> Signed-off-by:
-
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:
-
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:
-
