- 09 Jul, 2012 18 commits
-
-
Devendra Naga authored
for if else statements having single block no braces are needed fixed the following checkpatch warning drivers/staging/rts_pstor/sd.c:140: WARNING: braces {} are not necessary for any arm of this statement Signed-off-by:Devendra Naga <devendra.aaru@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Jesper Juhl authored
I missed a few redundant newlines the first time. Signed-off-by:
Jesper Juhl <jj@chaosbits.net> Signed-off-by:
-
Jesper Juhl authored
My previous cleanup patches missed a few cases of redundant/missing/replaced curly braces. This should fix up the last ones. Signed-off-by:
-
Jesper Juhl authored
A few cases were missed in my previous cleanup, this takes care of the last cases of missing space (or too much space (as in a newline)) around operators ('=', '==', ',', '<'). Signed-off-by: -
Seth Jennings authored
This patch improves mapping performance in zsmalloc by getting usage information from the user in the form of a "mapping mode" and using it to avoid unnecessary copying for objects that span pages. Signed-off-by:
Seth Jennings <sjenning@linux.vnet.ibm.com> Signed-off-by:
-
Seth Jennings authored
Add information on the usage limits of zs_map_object() Signed-off-by:
-
Seth Jennings authored
Improve zs_unmap_object() performance by adding a fast path for objects that don't span pages. Signed-off-by:
-
Seth Jennings authored
This patch replaces the page table assisted object mapping method, which has x86 dependencies, with a arch-independent method that does a simple copy into a temporary per-cpu buffer. While a copy seems like it would be worse than mapping the pages, tests demonstrate the copying is always faster and, in the case of running inside a KVM guest, roughly 4x faster. Signed-off-by:
-
Xiao Guangrong authored
tmem_obj_find and insertion tmem-obj have the some logic, we can integrate the code Signed-off-by:
Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com> Acked-by:
Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Signed-off-by:
-
Xiao Guangrong authored
Introduce get_zcache_client to remove the common code Acked-by:
-
Xiao Guangrong authored
Cleanup the code for zcache_do_preload and zcache_put_page Acked-by:
-
Xiao Guangrong authored
zcache_do_preload is called in zcache_put_page where IRQ is disabled, so, need not care preempt Acked-by:
-
Xiao Guangrong authored
Need not set global parameters to 0 Acked-by:
-
Xiao Guangrong authored
These functions are called only when system is initializing, so mark __init for them to free memory Acked-by:
-
Xiao Guangrong authored
zcache is enabled only if one of CONFIG_CLEANCACHE and CONFIG_FRONTSWAP is enabled, see the Kconfig: depends on (CLEANCACHE || FRONTSWAP) && CRYPTO=y && X86 So, we can remove the check in the source code Acked-by:
-
Xiao Guangrong authored
Fix: drivers/staging/zcache/zcache-main.c: In function ‘zcache_comp_op’: drivers/staging/zcache/zcache-main.c:112:2: warning: ‘ret’ may be used uninitial Signed-off-by:
-
Xiao Guangrong authored
In zcache_get_pool_by_id, the refcount of zcache_host is not increased, but it is always decreased in zcache_put_pool Acked-by:
-
Emil Goode authored
Sparse is warning about non-ANSI function declaration. Add void to the parameterless function. drivers/staging/csr/csr_wifi_hip_chiphelper.c:633:31: warning: non-ANSI function declaration of function 'ChipHelper_Null' I also fixed this checkpatch error: ERROR: "foo* bar" should be "foo *bar" Signed-off-by:
Emil Goode <emilgoode@gmail.com> Signed-off-by:
-
- 06 Jul, 2012 22 commits
-
-
Andy Shevchenko authored
Signed-off-by:
Andy Shevchenko <andriy.shevchenko@linux.intel.com> Acked-by:
Kyungmin Park <kyungmin.park@samsung.com> Cc: Andrzej Pietrasiewicz <andrzej.p@samsung.com> Signed-off-by:
-
Andy Shevchenko authored
Signed-off-by:
-
Andy Shevchenko authored
Signed-off-by:
-
Andy Shevchenko authored
Signed-off-by:
Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by:
-
Andy Shevchenko authored
Signed-off-by:
-
Andy Shevchenko authored
Signed-off-by:
-
Dan Carpenter authored
Sparse complains that "arg" is not a __user pointer. The "argp" and "arg" variables are equivalent but argp is declared as a __user pointer. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-By:
Martyn Welch <martyn.welch@ge.com> Signed-off-by:
-
David Murray authored
Fixed a tiny checkpatch.pl warning. Signed-off-by:
David Murray <therealcykey@gmail.com> Signed-off-by:
-
Samuel Iglesias Gonsálvez authored
The IRQ vector should be saved in MEM space base address according to the datasheet. Signed-off-by:
Samuel Iglesias Gonsálvez <siglesias@igalia.com> Signed-off-by:
-
Marcos Paulo de Souza authored
All these macros are not used. So, remove this all. Signed-off-by:
-
Justin P. Mattock authored
Signed-off-by:
Justin P. Mattock <justinmattock@gmail.com> Signed-off-by:
-
Devendra Naga authored
this following warn is fixed up drivers/staging/ft1000/ft1000-usb/ft1000_hw.c:1754: ERROR: open brace '{' following function declarations go on the next line Signed-off-by: -
Omar Ramirez Luna authored
Dynamically allocate my_sym_buf to silence the following warning: drivers/staging/tidspbridge/dynload/cload.c: In function 'dload_symbols': drivers/staging/tidspbridge/dynload/cload.c:890: warning: the frame size of 1040 bytes is larger than 1024 bytes Signed-off-by:Omar Ramirez Luna <omar.ramirez@copitl.com> Signed-off-by:
-
Omar Ramirez Luna authored
Dynamically allocate ibuf to silence the following warning: drivers/staging/tidspbridge/dynload/cload.c: In function 'dload_data': drivers/staging/tidspbridge/dynload/cload.c:1337: warning: the frame size of 1216 bytes is larger than 1024 bytes Signed-off-by: -
Omar Ramirez Luna authored
Due to its size, this function declares too many variables, to split it a new structure has been declared to hold values as they are read from the baseimage. While at it, indentation was reduced by renaming variables and reducing blocks of code with the following structure: if (success) { ... if (success) ... } This fixes the following warning: drivers/staging/tidspbridge/core/io_sm.c: In function 'bridge_io_on_loaded': drivers/staging/tidspbridge/core/io_sm.c:777: warning: the frame size of 1032 bytes is larger than 1024 bytes Signed-off-by: -
Omar Ramirez Luna authored
And fix the following warning for passing an incorrect variable type. ../tiomap3430.c: In function 'user_va2_pa': ../tiomap3430.c:1555: warning: passing argument 1 of 'pmd_offset' from incompatible pointer type arch/arm/include/asm/pgtable-2level.h:156: note: expected 'struct pud_t *' but argument is of type 'pmdval_t (*)[2]' While at it, eliminate 'if' nesting to increase readability. Signed-off-by: -
Marcos Paulo de Souza authored
These two functions only has commented code, so remove it. Signed-off-by:
-
Marcos Paulo de Souza authored
The function crypto_sep_dump_message don't have any use in this driver. So remove it. Signed-off-by:
-
Johannes Thumshirn authored
Changed a call to strict_strtoul() into kstrtou8() in pod_set_midi_postprocess(). Signed-off-by:
Johannes Thumshirn <morbidrsa@googlemail.com> Signed-off-by:
-
Johannes Thumshirn authored
Changed strict_strtoul() to kstrtou() in pod_set_channel() to take changes in pod_send_channel() into account. Signed-off-by:
-
Johannes Thumshirn authored
Exchanged call to strict_strtoul() with kstrtou8() in pod_resolve(). Signed-off-by:
-
Johannes Thumshirn authored
Exchange strict_strtoul() with kstrtou8() and make "value" a u8 instead of a unsigned long. This is also needed for the changed line6_pod_transmit_parameter(). Signed-off-by:
-
