- 20 Dec, 2013 37 commits
-
-
H Hartley Sweeten authored
The "Counting Mode" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
H Hartley Sweeten authored
The "Input Select" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
The "LoadB" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
The "LoadA" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
The "Mode" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
The "SW Save" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
The "Command" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
The "AutoIncrement" registers are sequential in the enum ni_gpct_register. Replace this inline CamelCase function with a simple define. Signed-off-by:
-
H Hartley Sweeten authored
As prefered by the CodingStyle, rename all the CamelCase labels of the enum ni_gpct_register. Cleanup all the helper functions in ni_tio_internal.h that used the enum. The 'counter_index' parameter to all the functions is the ni_gpct_device 'counter_index' which is initialized when the subdevices are setup. This value is always < 4 so the default: BUG() cases can never happen. For aesthetics, rename the 'counter_idx' to simply 'idx' and fix the type in some of the helpers. Also, remove the unnecessary break statements after the return statements. Signed-off-by:
-
Gokulnath Avanashilingam authored
This patch fixes all the warning related to spacing issues found by checkpatch.pl script in InterfaceIdleMode.c Signed-off-by:
Gokulnath Avanashilingam <Gokulnath.Avanashilingam@in.bosch.com> Signed-off-by:
-
Gokulnath Avanashilingam authored
Used the time_after and time_before insted of comparing the jiffies directly.This will fix the warnings and errors found by the checkpatch.pl script. Signed-off-by:
-
Gokulnath Avanashilingam authored
Fixed all the line over 80 characters warning found by checkpatch.pl script. Signed-off-by:
-
Gokulnath Avanashilingam authored
Removed the developer debug prints BCM_DEBUG_PRINT() as per the TODO list, also removed braces for the if-statement to match coding style Signed-off-by:
-
Wei Yongjun authored
Add the missing destroy_workqueue() before return from sep_crypto_setup() and sep_crypto_takedown(). Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
-
Wei Yongjun authored
In case of error, the function crypto_alloc_hash() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by:
-
Andreas Frembs authored
Checkpatch mentioned that these macros should be defined with a 'do {...} while(0)' statement. So we used grep to find all files using these macros and we found out that these macros were only used in rtl819x_Qos.h, in the defines we removed before. Since the macros are not used anywhere else we decided to remove them from the driver. Signed-off-by:Andreas Frembs <andreas.frembs@studium.uni-erlangen.de> Signed-off-by:
Matthias Schoepe <matthias.schoepe@studium.uni-erlangen.de> Signed-off-by:
-
Andreas Frembs authored
In rtl819x_Qos.h there were several macros defined but never used. So we removed this macros. Signed-off-by:
-
Andreas Frembs authored
This patch fixes the following checkpatch errors from rtllib_endianfree.h: - ERROR: Macros with complex values should be enclosed in parenthesis Signed-off-by:
-
Michael Hoefler authored
In this patch we fix some "logical" errors in bypass.c of the silicom bypass driver (in staging). Checkpatch complains about the following errors: - unnecessary forward declarations in a source file - assignment in if condition In addition to that the __init and __exit macros were missing at the init and cleanup function. There are still two warnings left for this file related to too many leadingtabs at nested blocks. I did not touch this issue becasue the code needs really some refactoring. And since i do not have the appropriate hardware to test the code, i do not change the functionality in any way. Signed-off-by:
Michael Hoefler <michael.hoefler@studium.uni-erlangen.de> Signed-off-by:
Christoph Kohl <christoph.kohl@t-online.de> Signed-off-by:
-
Michael Hoefler authored
This patch improves bypass.c (in staging) in terms of coding style. This includes different issues some of them mentioned by checkpatch: - a c++ one line comment - parenthesis at return statementes - multiple definitions in one line - missing braces according to the style guide Signed-off-by:
-
Michael Hoefler authored
This patch addresses several problems in bypass.c found by checkpatch. Furthermore it removes/adds some empty lines to make the code more readable. The following problems are fixed: - line over 80 characters - space after logical operator ! - spaces instead of tabs - missing empty line after local declarations - empty line after { The empty line issues were not discovered by checkpatch but in my opinion these changes make perfect sense in terms of readability. Signed-off-by: -
Larry Finger authored
Each instance may bre replaced by udelay Signed-off-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Larry Finger authored
Signed-off-by:
-
Larry Finger authored
Signed-off-by:
-
Larry Finger authored
Signed-off-by:
-
Larry Finger authored
This wrapper does nothing on Linux. When calls to it were removed, a number of routines became empty and could also be removed. Signed-off-by:
-
Larry Finger authored
Signed-off-by:
-
Larry Finger authored
Again there are two - _exit_critical() and _exit_critical_ex(). Signed-off-by:
-
Larry Finger authored
There are two such wrappers - _enter_critical() and _enter_critical_ex(). Signed-off-by:
-
Larry Finger authored
With this change, a number of variables for storing flags are no longer used. Signed-off-by:
-
Larry Finger authored
Some comment lines that mentioned spin_lock_bh() are also removed. Signed-off-by:
-
Larry Finger authored
Signed-off-by:
-
Lidza Louina authored
Originally, this driver created it's own allocating function. This patch removes that function and calls kzalloc directly. This patch affects: - driver.c - driver.h - fep5.c - tty.c Signed-off-by:
Lidza Louina <lidza.louina@gmail.com> Signed-off-by:
-
stalinsrinivasan.s authored
This is a patch to the dgnc_cls.c file that fixes up comparison with jiffies usage warning found by the checkpatch.pl tool Signed-off-by:
S. Stalin Srinivasan <stalinsrinivasan.s@gmail.com> Signed-off-by:
-
stalinsrinivasan.s authored
This is a patch to the dgnc_cls.c file that fixes up volatile usage warning found by the checkpatch.pl tool Signed-off-by:
-
stalinsrinivasan.s authored
This is a patch to the dgnc_cls.c file that fixes up 80 characters per line and code indent warnings found by the checkpatch.pl tool. Signed-off-by:
-
John Stultz authored
Prior to subitting this, Colin reworked the compat_ioctl support for the ion_test driver, moving the structure to be the same size on both 32 and 64 bit architectures. Two small things were left out. The compat_ioctl ptr assignment, and the fact that despite having uniform sized types in the structure, the structure pads out to different sizes on different arches. This patch resolves this issue by adding a padding entry after the write flag, and adding the compat_ioctl ptr. Changes in v2: - Add a padding int rather then making write a u64 Acked-by:
Colin Cross <ccross@android.com> Cc: Android Kernel Team <kernel-team@android.com> Signed-off-by:
John Stultz <john.stultz@linaro.org> Signed-off-by:
-
- 19 Dec, 2013 3 commits
-
-
Masanari Iida authored
This patch fixed trailing whitespace found by checkpatch.pl within dgap_trace.c Signed-off-by:
Masanari Iida <standby24x7@gmail.com> Signed-off-by:
-
Gokulnath A authored
Fixed all the line over 80 characters warning found by checkpatch.pl script. Signed-off-by:
-
Gokulnath A authored
Removed the developer debug prints BCM_DEBUG_PRINT() as per the TODO list, also removed braces for the if-statement to match coding style Signed-off-by:
-
