linux/drivers/staging/bcm, branch v3.2.31Linux kernel source treehttps://git.amat.us/linux/atom/drivers/staging/bcm?h=v3.2.312011-10-23T08:16:34ZStaging: bcm: Fix three initialization errors in InterfaceDld.c2011-10-23T08:16:34ZKevin McKinneyklmckinney1@gmail.com2011-10-20T12:15:23Zurn:sha1:a86581829fdd646655116f89cb8af2f22b5fbf9e
This patch fixes three initialization errors.
One is an incorrect initialization of a static
variable. The other two are incorrect
initializations in an if statement. These
errors were found by checkpatch.pl.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Fix coding style issues in InterfaceDld.c2011-10-23T08:16:34ZKevin McKinneyklmckinney1@gmail.com2011-10-20T12:15:22Zurn:sha1:a5e4b81e274d7b455531a5e33ce1523caaf31a84
This patch fixes multiple coding style issues in file,
InterfaceDld.c, found by checkpatch.pl.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Fix coding style issues in InterfaceMisc.c2011-10-11T16:18:58ZKevin McKinneyklmckinney1@gmail.com2011-10-11T01:01:50Zurn:sha1:54ced00639aa90c92e0ce399cc84ca6fc6a2eebb
This patch cleans up several code style issues found
in InterfaceMisc.c reported by checkpatch.pl.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Remove assignment from if conditions reported by checkpatch.pl2011-10-11T16:18:57ZKevin McKinneyklmckinney1@gmail.com2011-10-11T01:01:49Zurn:sha1:ca45e70029b3d96923953992c05a2515dd3d9f24
This patch removes an assignment from three if
conditions. In all three cases, the line of code was
attempting to allocate memory, and check
if the memory was allocated in the if statement.
This issue was reported by checkpatch.pl.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Fix coding style issues reported by checkpatch.pl2011-10-11T16:18:57ZKevin McKinneyklmckinney1@gmail.com2011-10-11T01:01:48Zurn:sha1:f969f84ed92e55040dbf51398ba2d8f5d23bf498
This patch cleans up several code style issues found
in InterfaceInit.c reported by checkpatch.pl.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Fix assignment issue in if statement reported by checkpatch.pl.2011-10-11T16:18:21ZKevin McKinneyklmckinney1@gmail.com2011-10-07T22:50:12Zurn:sha1:d491061b98bf3d2d036b9896f884403cdc2c0327
This patch fixes an error where an assignment "="
was being used in an if statement to determine if
Firmware was downloaded. This patch removes that
assignment, and places it above the if statement.
The if statement then evaluates the status to
verify if "0" successful, or != 0 failure.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Remove assignment in if condition reported by checkpatch.pl2011-10-11T16:18:21ZKevin McKinneyklmckinney1@gmail.com2011-10-07T22:50:11Zurn:sha1:9d4f1d0c9d1cf2fc4a73b6fff30aa129b9d1586e
There is a case where an assignment was being done
in an if condition. This patch removes the assignment
from the if condition and places the assignment above
the if statement; thereby improving the readability of
the code.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Replace dated variable __FUNCTION__ reported by checkpatch.pl2011-10-11T16:18:21ZKevin McKinneyklmckinney1@gmail.com2011-10-07T22:50:10Zurn:sha1:a8a1cdd6d4daeeb7a8016fbaa2cb79f37b14ea9a
This patch replaces the obsolete variable, __FUNCTION__, that
holds the name of the current function with variable, __func__.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Fix coding style issues reported by checkpatch.pl2011-10-11T16:18:20ZKevin McKinneyklmckinney1@gmail.com2011-10-07T22:50:09Zurn:sha1:6a4ef5f9b39cc84550d2f031d3546d67c6584d90
This patch cleans up several hundred code style issues found
in Misc.c reported by checkpatch.pl.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Staging: bcm: Alter LOC for readability/understandability purposes2011-09-30T00:34:51ZKevin McKinneyklmckinney1@gmail.com2011-09-28T01:28:11Zurn:sha1:2505aa6ce42a686b2d3db95ccdcc7bc100e7b8c0
This patch alters a line of code to make it more readable
and easier to understand. The purpose of the original line
of code was to compute the amount of memory to request from
kmalloc. This mulit-step algorithm was being done in one
line of code, thus making it more difficult to understand.
Therefore, I split this algorithm into three logical steps.
Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>