- 11 Mar, 2013 40 commits
-
-
Eduardo Valentin authored
On OMAP54xx ES2.0 there is no single read and only one mode: continuous mode. For this reason, there is no point in defining register fields for these operations. Signed-off-by:
Eduardo Valentin <eduardo.valentin@ti.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Eduardo Valentin authored
In order to keep same nomenclature across the register definition, this change will make all 'MM' suffixes to be named 'GPU'. Signed-off-by:
-
Radhesh Fadnis authored
This patch updates the ADC conversion table for OMAP5430 ES2.0 devices. Signed-off-by:
Radhesh Fadnis <radhesh.fadnis@ti.com> Signed-off-by:
-
Radhesh Fadnis authored
The clock to Bandgap module is SW controlled on some version of OMAP silicon (OMAP44xx). But on OMAP54xx ES2.0 onwards this is HW-Auto controlled. Hence introduce a feature flag to use/not-to-use SW enable/disable of BG clock. Signed-off-by:
-
Ruslan Ruslichenko authored
To indicate that board was shut down due to TSHUT temperature reached it is good to print some information message before shutting down. Signed-off-by:
Ruslan Ruslichenko <x0191366@ti.com> Signed-off-by:
-
Dan Carpenter authored
I'm not sure where these results come from, but it can't hurt to add a sanity check the array offset. The .results[] array on the next line has HFA384x_CHINFORESULT_MAX (16) elements. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Dan Carpenter authored
This function is ugly because it hits against the 80 character limit. This patch does several things to clean it up. 1) Introduces "result" instead of inf->info.chinforesult.result[n]. 2) Reverses the ".scanchannels & (1 << i)" so everthing can be pulled in one indent level. 3) Use "chan" instead of "channel". 4) Tweaks the line breaks to the call to pr_debug(). Signed-off-by:
-
Laurent Navet authored
- WARNING: braces {} are not necessary for single statement blocks Signed-off-by:Laurent Navet <laurent.navet@gmail.com> Reviewed-by:
Stefan Hajnoczi <stefanha@gmail.com> Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes the following checkpatch warning: -WARNING: line over 80 characters Signed-off-by:
Kurt Kanzenbach <ly80toro@cip.cs.fau.de> Signed-off-by:
-
Stefan Reif authored
This patch fixes the following checkpatch errors: -ERROR: space required after that ',' -ERROR: spaces required around that '=' -ERROR: space prohibited before that close parenthesis -WARNING: please, no space before tabs Signed-off-by:
Stefan Reif <ke42caxa@cip.cs.fau.de> Signed-off-by:
-
Stefan Reif authored
This patch fixes the following checkpatch warning: -WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: -
Kurt Kanzenbach authored
This patch fixes the following checkpatch warning: -WARNING: missing space after enum definition Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes the following checkpatch error: -ERROR: do not use assignment in if condition Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes some of the following checkpatch warnings: -WARNING: line over 80 characters We did not split format strings for readability. Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes the following checkpatch error: -ERROR: "(foo*)" should be "(foo *)" Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes the following checkpatch error: -ERROR: spaces required around that '=' Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes the following checkpatch errors: -ERROR: do not initialise statics to 0 or NULL -ERROR: do not initialise globals to 0 or NULL Signed-off-by:
-
Kurt Kanzenbach authored
This patch fixes the following checkpatch warning: -ERROR: code indent should use tabs where possible -WARNING: suspect code indent for conditional statements Signed-off-by:
-
Syam Sidhardhan authored
kfree on NULL pointer is a no-op. Signed-off-by:
Syam Sidhardhan <s.syam@samsung.com> Signed-off-by:
-
Alexandru Gheorghiu authored
Fixed coding style issue. Signed-off-by:
Alexandru Gheorghiu <gheorghiuandru@gmail.com> Signed-off-by:
-
Randy Dunlap authored
Fix all sparse warning in drivers/staging/silicom/bypasslib/, e.g.: drivers/staging/silicom/bypasslib/bypass.c:471:21: warning: non-ANSI function declaration of function 'init_lib_module' drivers/staging/silicom/bypasslib/bypass.c:478:25: warning: non-ANSI function declaration of function 'cleanup_lib_module' drivers/staging/silicom/bypasslib/bypass.c:137:5: warning: symbol 'is_bypass_dev' was not declared. Should it be static? drivers/staging/silicom/bypasslib/bypass.c:182:5: warning: symbol 'is_bypass' was not declared. Should it be static? drivers/staging/silicom/bypasslib/bypass.c:192:5: warning: symbol 'get_bypass_slave' was not declared. Should it be static? drivers/staging/silicom/bypasslib/bypass.c:197:5: warning: symbol 'get_bypass_caps' was not declared. Should it be static? drivers/staging/silicom/bypasslib/bypass.c:202:5: warning: symbol 'get_wd_set_caps' was not declared. Should it be static? etc. Signed-off-by:
Randy Dunlap <rdunlap@infradead.org> Signed-off-by:
-
Peter Huewe authored
The first branch of the if statement is ended with a return thus there is no need for an else if, and thus we can move the duplicated code to the top and use it for the other two branches. Signed-off-by:
Peter Huewe <peterhuewe@gmx.de> Signed-off-by:
-
Peter Huewe authored
len is never read after assignment, thus can be removed. Signed-off-by:
-
Peter Huewe authored
sdio_boot.c and netlink_k.c both have a corresponding header file with their function prototypes but fail to include them, which leads to the following sparse warnings: sdio_boot.c:135:5: warning: symbol 'sdio_boot' was not declared. Should it be static? netlink_k.c:89:13: warning: symbol 'netlink_init' was not declared. Should it be static? netlink_k.c:109:6: warning: symbol 'netlink_exit' was not declared. Should it be static? netlink_k.c:114:5: warning: symbol 'netlink_send' was not declared. Should it be static? -> Add the include files and silence the warning Signed-off-by:
-
Joe Perches authored
Eliminate some warnings by casting to unsigned long before casting a dma_addr_t value to a pointer. btw: Does slicoss always work on x86-32? Is a pshmem guaranteed to be accessible by a 32 bit address? Signed-off-by:
Joe Perches <joe@perches.com> Signed-off-by:
-
Peter Huewe authored
Instead of assigning the pm_ops fields individually we can simply use SIMPLE_DEV_PM_OPS. Signed-off-by:
Javier Muñoz <jmunhoz@igalia.com> Signed-off-by:
-
Peter Huewe authored
Instead of performing the hash calculation for the mac address by ourself, we can simply reuse ether_crc and shift only the result according to our needs. The code was tested against the previous implementation by verifying both implementations against each other in userspace for 16200000000 different mac addresses, changing the vendor bits of the mac address first. Signed-off-by:
-
Peter Huewe authored
smatch complains about a possible buffer overflow slicoss.c:3651 slic_card_locate() error: buffer overflow 'physcard->adapter' 4 <= 4 If the for loop is not exited prematurely i++ is executed after the last iteration and thus i can be 4, which is out of bounds for physcard->adapter. -> Add check for this condition and simplify the if statement by inverting the condition. Signed-off-by:
-
Peter Huewe authored
gcc complains about an undefined operation: slicoss.c:1417:19: warning: operation on 'rspq->pageindex' may be undefined [-Wsequence-point] The intended operation was (probably) to retrieve the pageindex + 1 and let it wrap around if it reaches the num_pages. Signed-off-by:
-
Peter Huewe authored
skbtype is assigned once to NORMAL_ETHFRAME and then checked if it is NORMAL_ETHFRAME -> remove the checks. This also gets rid of the (false positive) smatch warning: slicoss.c:2829 slic_xmit_start() error: potential NULL dereference 'hcmd'. Signed-off-by:
-
Peter Huewe authored
Smatch complains that the variable adapter is dereferenced before it is checked: slicoss.c:906 slic_timer_load_check() warn: variable dereferenced before check 'adapter' (see line 904) -> move the assignment after the check. Signed-off-by:
-
Andres More authored
Removed includes and added linux/types.h instead when needed. Signed-off-by:
Andres More <more.andres@gmail.com> Signed-off-by:
-
Andres More authored
No checkpatch findings were resolved. sed -i 's/\bULONG_PTR\b/u32/g' drivers/staging/vt6656/*.[ch] sed -i 's/\bDWORD_PTR\b/u32/g' drivers/staging/vt6656/*.[ch] Signed-off-by:
-
Andres More authored
Checkpatch findings were not resolved. sed -i 's/\bDWORD\b/u32/g' drivers/staging/vt6656/*.[ch] sed -i 's/\bPDWORD\b/u32 */g' drivers/staging/vt6656/*.[ch] Signed-off-by:
-
Andres More authored
Checkpatch findings were not resolved. sed -i 's/\bWORD\b/u16/g' drivers/staging/vt6656/*.[ch] sed -i 's/\bPWORD\b/u16 */g' drivers/staging/vt6656/*.[ch] Signed-off-by:
-
Andres More authored
Checkpatch findings were not resolved, only direct replacement. sed -i 's/\bBYTE\b/u8/g' drivers/staging/vt6656/*.[ch] sed -i 's/\bPBYTE\b/u8 */g' drivers/staging/vt6656/*.[ch] Signed-off-by:
-
Kevin McKinney authored
This patch fixes a spelling error in PHSModule.c Signed-off-by:
Kevin McKinney <klmckinney1@gmail.com> Signed-off-by:
-
Kevin McKinney authored
This patch properly formats comments, and removes them as needed in PHSModule.c. Signed-off-by:
-
Kevin McKinney authored
This patch formats braces in PHSModule.c as reported by checkpatch.pl. Signed-off-by:
-
Kevin McKinney authored
This patch fixes all white space issues in PHSModule.c as reported by checkpatch.pl. Signed-off-by:
-
