- 11 Mar, 2013 40 commits
-
-
SeongJae Park authored
Signed-off-by:
SeongJae Park <sj38.park@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Guenter Roeck authored
An unnecessary typecast in dgrp_net_ops.c causes the following build error if compiled with W=1. In function ‘copy_from_user’, inlined from ‘dgrp_net_ioctl’ at drivers/staging/dgrp/dgrp_net_ops.c:3408:21: arch/x86/include/asm/uaccess_32.h:211:26: error: call to ‘copy_from_user_overflow’ declared with attribute error: copy_from_user() buffer size is not provably correct Signed-off-by:Guenter Roeck <linux@roeck-us.net> Signed-off-by:
-
Niklas Söderlund authored
Same result no matter what path is taken. Signed-off-by:
Niklas Söderlund <niso@kth.se> Acked-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Niklas Söderlund authored
Signed-off-by:
-
Niklas Söderlund authored
There is no implementation of these functions anywhere in the code. Signed-off-by:
-
Niklas Söderlund authored
Signed-off-by:
-
Eduardo Valentin authored
This patch changes the clock management code to also update the clock prepare counter, this way we won't skip the enable/disable operation due to prepare dependencies. Signed-off-by:
Eduardo Valentin <eduardo.valentin@ti.com> Signed-off-by:
-
Eduardo Valentin authored
This patch simply changes the name of files containing data structure definition. Signed-off-by:
-
Eduardo Valentin authored
This patch removes from data structure the double reference of the conversion table. It keeps the reference coming from bandgap data definition. The patch also adapts the code accordingly. Signed-off-by:
-
Eduardo Valentin authored
Update documentation with DT example for OMAP54xx devices. Signed-off-by:
-
Eduardo Valentin authored
Simple update on documentation file for DT. This patch also adds an example for OMAP4430 and 0MAP4470, and also updated OMAP4460's example. Signed-off-by:
-
Eduardo Valentin authored
For ES2.0 devices, it is not guaranteed that current DTEMP or DTEMP0 from the history buffer are going to contain correct values, due to desynchronization between BG clk and OCP clk. For this reason, this patch changes the driver to first: a. consider a feature flag, FREEZE_BIT, in order to check it is possible to freeze the history buffer or not. b. whenever reading the temperature, it will fetch from DTEMP1 instead of DTEMP or DTEMP0. This WA is applicable only for OMAP5430 ES2.0. Signed-off-by:
-
Eduardo Valentin authored
On OMAP54xx there is only one counter register. For this reason, each domain must use the same counter register. This patch changes the data definition to coupe with this. Signed-off-by:
-
Eduardo Valentin authored
This patch removes from OMAP54xx the features: . CLK_CTRL . COUNTER . MODE_CONFIG Because these features are not present in OMAP54xx ES2.0 Signed-off-by:
-
Eduardo Valentin authored
This patch updates the OMAP54xx data structure to use the right clock source name for ES2.0. Signed-off-by:
-
Eduardo Valentin authored
On OMAP54xx ES2.0 there are new features inside the bandgap device. This patch introduces the registers definition to access these features and adapts the data structures to map these new registers. The new features are: . SIDLE mode . Cumulative register . History buffer. . Buffer freeze bit . Buffer clear bit Signed-off-by:
-
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 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:
-
