- 11 May, 2010 40 commits
-
-
Huang Weiyi authored
Remove unused #include <linux/version.h>('s) in drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c Signed-off-by: Huang Weiyi <weiyi.huang@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Zachary Richey authored
This patch fixes coding style issues found by checkpatch.pl, and doesnt line break string literals. Signed-off-by: Zachary Richey <zr.public@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Stewart Robertson authored
This is a patch to the ni_at_ao.c file that adds the missing KERN_INFO requested by the checkpatch.pl tool. Signed-off-by: Stewart Robertson <stewart_r@aliencamel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Darren Armstrong authored
Fixed coding style issues: 80-char width limit, KERN_ facility level Signed-off-by: Darren Armstrong <darren.armstrong85@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Gustavo Silva authored
This is a patch to the das08.c file that fixes up the following issues found by the checkpatch.pl tool. WARNING: line over 80 characters x 6 ERROR: code indent should use tabs where possible x 3 ERROR: spaces required around that '?' (ctx:VxV) x 4 ERROR: spaces required around that ':' (ctx:VxV) x 4 ERROR: that open brace { should be on the previous line x 1 WARNING: printk() should include KERN_ facility level x 9 WARNING: braces {} are not necessary for single statement blocks x 1 WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable x 2 Signed-off-by: Gustavo Silva <silvagustavo@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Gustavo Silva authored
This is a patch to the das16.c file that fixes up the following issues found by the checkpatch.pl tool. WARNING: line over 80 characters x 23 ERROR: spaces required around that '?' (ctx:VxV) x 2 ERROR: spaces required around that ':' (ctx:VxV) x 2 WARNING: printk() should include KERN_ facility level x 17 WARNING: braces {} are not necessary for single statement blocks x 8 Signed-off-by: Gustavo Silva <silvagustavo@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
David Fernández authored
Signed-off-by: Frank Mori Hess <fmhess@users.sourceforge.net> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Gorskin Ilya authored
This is a patch to the comedi.h files that fixes up errors found by the checkpatch.pl tool Signed-off-by: Gorskin Ilya <revent82@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Dan Carpenter authored
"dac_range_table" has 5 elements. Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Stephen Palmateer authored
This is a patch to the comedi_parport.c file that fixes the KERN_ facility warnings found by the checkpatch.pl tool Signed-off-by: Stephen Palmateer <stephen.palmateer@netsweeper.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
David Binderman authored
Signed-off-by: David Binderman <dcb314@hotmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Bill Pemberton authored
NetVscOnChannelCallback() was prototyped as static, but the actual declartion of the function was not static. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Bill Pemberton authored
NetVscOnChannelCallback() used a dynamic sized array that also made the frame size over 2048. Replace it with a buffer allocated from kzalloc. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Ruslan Pisarev authored
This is a patch to the vstorage.h file that fixed up a TAB and spaces Errors found by the checkpatch.pl tools, like spaces required around that ':' (ctx:VxV) Signed-off-by: Ruslan Pisarev <ruslan@rpisarev.org.ua> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed a coding style issue. Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed multiple coding style issues. Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed multiple coding style issues Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed multiple coding style issues. Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed multiple coding style issues. Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed multiple coding style issues Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Stefan Schick authored
Fixed some coding style issues Signed-off-by: Stefan Schick <stinder21@googlemail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Pavel Machek authored
(BTW, winbond/README alias winbond/TODO doesn't mention it, but another (trivial) step is going to be to change variable names and function names from CamelCase to lower_case spelling.) Signed-off-by: Pavel Machek <pavel@ucw.cz> Acked-by: Pekka Enberg <penberg@cs.helsinki.fi> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Pavel Machek authored
Remove last bits of earlysuspend support. Signed-off-by: Pavel Machek <pavel@ucw.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Pavel Machek authored
> All of the Kconfig menu items under "config DREAM" should be listed > indented under the 'DREAM' symbol, but they are not. (using xconfig) > In menuconfig, the DREAM symbol isn't listed (since it depends on BROKEN), > but the other (subordinate) symbols are still listed. Ok, this should fix it ... or at least make it better an non-issue for people with non-HTC hardware. Signed-off-by: Pavel Machek <pavel@ucw.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Pavel Machek authored
gpio_* drivers still need drivers in staging. Yes, that will need to be fixed, but at least fix compilation for now. Signed-off-by: Pavel Machek <pavel@ucw.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Andre Silva authored
This is a patch to the arlan-proc.c file that fixes up multiple coding style errors and warnings found by the checkpatch.pl tool Signed-off-by: Andre Silva <andre.beat@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
John Sheehan authored
add missing space before closing brace Signed-off-by: John Sheehan <john.d.sheehan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Robert Babilon authored
This is a patch to the ni_daq_700.c file that fixes several EXPORT_SYMBOL warnings found by the checkpatch.pl tool Signed-off-by: Robert Babilon <robert.babilon@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Wouter Van Rooy authored
This patch fixes some C99 style comments, trailing white spaces and some unnecessary 'if' braces found by checkpatch.pl Signed-off-by: Wouter Van Rooy <wouter.vanrooy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Lars Lindley authored
Signed-off-by: Lars Lindley <lindley@coyote.org> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jason Wong authored
Fixed multiple brace coding style issue. Signed-off-by: Jason Wong <tsanghan@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Arnd Bergmann authored
Signed-off-by: Arnd Bergmann <arnd@arndb.de> Cc: Frederic Weisbecker <fweisbec@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
H Hartley Sweeten authored
The wait_ibsyclr function is supposed to return the status of the I2C cycle. Currently it will always return FALSE because the IIC_CSR2 register is not re-read in order to update the cached register value. This results in the NEW_CYCLE bit still being 1. The current code actually works correctly only because the return value of {Read|Write}I2C is not checked in the driver. Fix wait_ibsyclr by actually reading the IIC_CSR2 register to get the updated status. While here, change the return type to be an actual errno instead of the private TRUE/FALSE define and remove the now obvious comments about the return value. Also, remove the local variable 'writestat' in WriteI2C and just return the result of wait_ibsyclr. Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com> Cc: Scott Smedley <ss@aao.gov.au> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Huang Weiyi authored
Remove duplicated #include('s) in drivers/staging/sep/sep_driver.c Signed-off-by: Huang Weiyi <weiyi.huang@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Huang Weiyi authored
Remove unused #include <linux/version.h>('s) in drivers/staging/dt3155/allocator.c drivers/staging/rtl8192u/ieee80211/ieee80211_tx.c drivers/staging/vme/boards/vme_vmivme7805.c Signed-off-by: Huang Weiyi <weiyi.huang@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Huang Weiyi authored
Remove duplicated #include('s) in drivers/staging/wlan-ng/p80211wext.c Signed-off-by: Huang Weiyi <weiyi.huang@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Lars Lindley authored
These patches fixes some whitspace and indentation warnings from checkpatch.pl Also these changed #includes: bc_dts_glob_lnx.h:43: WARNING: Use #include <linux/param.h> instead of <asm/param.h> rystalhd_lnx.h:45: WARNING: Use #include <linux/io.h> instead of <asm/io.h> crystalhd_lnx.h:49: WARNING: Use #include <linux/uaccess.h> instead of <asm/uaccess.h> It all compiles fine, but I don't have the hardware to test with.. Signed-off-by: Lars Lindley <lindley@coyote.org> Cc: Jarod Wilson <jarod@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Jani Nikula authored
Use IS_ERR() instead of comparing to NULL. Signed-off-by: Jani Nikula <ext-jani.1.nikula@nokia.com> Cc: Jarod Wilson <jarod@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Mike Sheldon authored
This patches mds.c to replace all the C99 style comments (//) with C89 style (/* ... */), as reported by checkpatch.pl. Signed-off-by: Mike Sheldon <mike@mikeasoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-
Dan Carpenter authored
a4_exists is an integer used as a boolean type so the original code works. But all the other conditions use && and this makes it consistent. Signed-off-by: Dan Carpenter <error27@gmail.com> Cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-