[PATCH] TTY: hvcs: fix missing unlock on error in hvcs_initialize()

2012-10-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock on the error handling path in function hvcs_initialize(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/tty/hvc/hvcs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/tty

[PATCH] ARM: PCI: fix missing unlock on error

2012-10-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock on the error handle path in function nanoengine_read_config() and nanoengine_write_config(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/mach-sa1100/pci-nanoengine.c | 8 ++-- 1 file changed, 6

[PATCH] ARM: PCI: fix missing unlock on error

2012-10-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock on the error handle path in function nanoengine_read_config() and nanoengine_write_config(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/mach-sa1100/pci-nanoengine.c | 8 ++-- 1 file changed, 6

[PATCH] pinctrl: fix missing unlock on error in pinctrl_groups_show()

2012-10-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock on the error handle path in function pinctrl_groups_show(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers

[PATCH] Staging: vt6655: fix missing unlock on error in vCommandTimer()

2012-10-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock on the error handle path in function vCommandTimer. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/vt6655/wcmd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/vt6655/wcmd.c

[PATCH] Staging: ced1401: fix missing unlock on error in FreeCircBlock()

2012-10-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock on the error handle path in function FreeCircBlock(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/ced1401/ced_ioc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

Re: CONFIG_MODULE_SIG breaks out-of-tree modules in modpost.

2012-10-18 Thread Wei Yongjun
On 10/15/2012 08:19 PM, David Howells wrote: Valdis Kletnieks valdis.kletni...@vt.edu wrote: /bin/sh: -c: line 0: syntax error near unexpected token `;' /bin/sh: -c: line 0: `set -e; ; echo 'cmd_/usr/src/valdis/NVIDIA-Linux-x86_64-304.51/kernel/nvidia.ko := '

[PATCH] ALSA: snd-usb: remove unused variable in init_pitch_v2()

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable ep is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/usb

[PATCH] ASoC: jack: remove unused variable in snd_soc_jack_report()

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable oldstatus is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound

[PATCH] fuse: remove unused variable in fuse_try_move_page()

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variables mapping,index are initialized but never used otherwise, so remove the unused variables. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

[PATCH] dlm: remove unused variable in *dlm_lowcomms_get_buffer()

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable users is initialized but never used otherwise, so remove the unused variable. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/dlm

[PATCH] vme: vme_tsi148.c: use module_pci_driver to simplify the code

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] vme: vme_ca91cx42.c: use module_pci_driver to simplify the code

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] vme: vme_vmivme7805.c: use module_pci_driver to simplify the code

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] Staging: ipack/bridges/tpci200: use module_pci_driver to simplify the code

2012-10-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] CRIS: locking: fix the return value of arch_read_trylock()

2012-10-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn arch_write_trylock() should return 'ret' instead of always return 1. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/cris/include/arch-v32/arch

[PATCH] pinctrl: remove duplicated include from pinctrl-xway.c

2012-10-10 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/pinctrl-xway.c | 2 -- 1 file changed, 2 deletions

[PATCH] block, sx8: use module_pci_driver to simplify the code

2012-10-10 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] pch_dma: use module_pci_driver to simplify the code

2012-10-10 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] virtio-pci: use module_pci_driver to simplify the code

2012-10-10 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_pci_driver() macro to make the code simpler by eliminating module_init and module_exit calls. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH] ubi: use list_move_tail instead of list_del/list_add_tail

2012-10-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mtd/ubi/fastmap.c | 3 +-- 1

[PATCH] hwmon: ina2xx: use module_i2c_driver to simplify the code

2012-10-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/hwmon/ina2xx.c

[PATCH] powerpc/windfarm: use module_i2c_driver to simplify the code

2012-10-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/macintosh

[PATCH] w1/ds2482: use module_i2c_driver to simplify the code

2012-10-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/w1/masters

[PATCH] mfd: use module_i2c_driver to simplify the code

2012-10-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_i2c_driver() macro to make the code smaller and a bit simpler. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/adp5520.c

[PATCH v2] gma500: medfield: drop bogus NULL check in mdfld_dsi_output_init()

2012-10-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Drop the NULL test for dev since it never be NULL. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/gma500/mdfld_dsi_output.c | 2

Re: [PATCH] clk: fix return value check in bcm2835_init_clocks()

2012-10-08 Thread Wei Yongjun
On 10/09/2012 10:31 AM, Stephen Warren wrote: On 10/07/2012 08:02 AM, Wei Yongjun wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_register_fixed_rate() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should

[PATCH] dgrp: remove unused including linux/version.h

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove including linux/version.h that don't need it. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/dgrp/dgrp_sysfs.c | 1 - 1 file

[PATCH] ARM: sa11x0: remove duplicated include from hackkit.c

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/mach-sa1100/hackkit.c | 1 - 1 file changed, 1 deletion(-) diff

[PATCH] extcon: remove duplicated include from extcon-gpio.c

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/extcon/extcon-gpio.c | 1 - 1 file changed, 1 deletion(-) diff

[PATCH] drm/nouveau: remove duplicated include from nouveau_drm.c

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/nouveau/nouveau_drm.c | 1 - 1 file changed, 1 deletion

[PATCH] pinctrl: remove duplicated include from pinctrl-bcm2835.c

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/pinctrl-bcm2835.c | 1 - 1 file changed, 1 deletion

[PATCH] staging: csr: remove unused including linux/version.h

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove including linux/version.h that don't need it. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/csr/unifi_wext.h | 1

[PATCH] gma500: medfield: fix potential NULL pointer dereference in mdfld_dsi_output_init()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/gma500/mdfld_dsi_output.c | 7

[PATCH] pinctrl: fix return value in bcm2835_pinctrl_probe()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function pinctrl_register() returns NULL not ERR_PTR(). The PTR_ERR() in the return value should be replaced with error no. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off

[PATCH] clk: fix return value check in sirfsoc_of_clk_init()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_register() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch

[PATCH] clk: fix return value check in bcm2835_init_clocks()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_register_fixed_rate() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). dpatch engine is used to auto generate this patch. (https://github.com

[PATCH] perf tools: remove duplicated include from trace-event-python.c

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- tools/perf/util/scripting-engines/trace-event-python.c | 1 - 1 file

[PATCH] drivers:misc: ti-st: fix potential NULL pointer dereference in st_register()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/misc/ti-st/st_core.c | 3 ++- 1 file

[PATCH] staging: r8712u: fix potential NULL pointer dereference in r871x_wps_start()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/rtl8712/rtl871x_ioctl_linux.c

[PATCH v2] drivers:misc: ti-st: fix potential NULL pointer dereference in st_register()

2012-10-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove the pointless NULL dereference above the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- v1 - v2: drop the NULL dereference

[PATCH] omfs: convert to use beXX_add_cpu()

2012-09-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Convert cpu_to_beXX(beXX_to_cpu(E1) + E2) to use beXX_add_cpu(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/omfs/file.c | 5 ++--- 1 file

[PATCH] hpfs: convert to use leXX_add_cpu()

2012-09-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Convert cpu_to_leXX(leXX_to_cpu(E1) + E2) to use leXX_add_cpu(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/hpfs/dnode.c | 30

[PATCH] ARM: imx: fix return value check in imx3_init_l2x0()

2012-09-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function ioremap() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off

[PATCH] staging: tidspbridge: fix return value check in dsp_wdt_init()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_get() returns ERR_PTR() and never returns NULL pointer. The NULL test in the error handling should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj/dpatch

[PATCH] davinci: fix return value check by using IS_ERR in tnetv107x_devices_init()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_get() returns ERR_PTR() not NULL pointer. The NULL test in the error handling should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj/dpatch) Signed-off

[PATCH] ARM: imx: fix the return value check in imx_clk_busy_divider()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_register() returns ERR_PTR() no NULL pointer. The NULL test in the return value check should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj/dpatch) Signed

[PATCH] clk: fix return value check in of_fixed_clk_setup()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_register_fixed_rate() returns ERR_PTR() not NULL pointer. The NULL test in the return value check should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj

[PATCH] c2port: fix return value check in duramar2150_c2port_init()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function c2port_device_register() returns ERR_PTR() not NULL pointer. The NULL test in the return value check should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj

[PATCH] spi: sh-hspi: fix return value check in hspi_probe()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_get() returns ERR_PTR() and never returns NULL pointer. The NULL test in the error handling should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj/dpatch

[PATCH] ARM: pxa: fix return value check in pxa2xx_drv_pcmcia_probe()

2012-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_get() returns ERR_PTR() and never returns NULL pointer. The NULL test in the error handling should be replaced with IS_ERR(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj/dpatch

[PATCH] hppfs: fix the return value of get_inode()

2012-09-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function get_inode() returns ERR_PTR(). But the users hppfs_lookup() and hppfs_fill_super() use NULL test for check the return value, not IS_ERR(), so we'd better change the return value of get_inode() to NULL instead

[PATCH] ufs: fix return value check in ufs_alloc_lastblock()

2012-09-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function ufs_get_locked_page() returns ERR_PTR() or NULL pointer. The IS_ERR() test in the error handling should be replaced with IS_ERR_OR_NULL(). dpatch engine is used to auto generated this patch. (https://github.com/weiyj

[PATCH] gpio_msm: using for_each_set_bit to simplify the code

2012-09-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using for_each_set_bit() to simplify the code. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpio/gpio-msm-v2.c | 4 +--- 1 file changed, 1

[PATCH] ARM: at91/gpio: using for_each_set_bit to simplify the code

2012-09-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using for_each_set_bit() to simplify the code. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/mach-at91/gpio.c | 4 +--- 1 file changed, 1

[PATCH] ARM: mmp: using for_each_set_bit to simplify the code

2012-09-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using for_each_set_bit() to simplify the code. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/mach-mmp/irq.c | 4 +--- 1 file changed, 1

[PATCH] gpio: pxa: using for_each_set_bit to simplify the code

2012-09-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using for_each_set_bit() to simplify the code. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpio/gpio-pxa.c | 4 +--- 1 file changed, 1

[PATCH] ocfs2: move the dereference below the NULL test

2012-09-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/ocfs2/move_extents.c | 3 ++- fs/ocfs2/journal.c

[PATCH] irqdomain: move the dereference below the NULL test

2012-09-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/irq/irqdomain.c | 3 ++- 1 file changed, 2

[PATCH] ARM: davinci: dm644x: move the dereference below the NULL test

2012-09-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/mach-davinci/board-dm644x-evm.c | 5

[PATCH] hwspinlock/core: move the dereference below the NULL test

2012-09-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/hwspinlock/hwspinlock_core.c | 3 ++- 1

[PATCH] memstick: move the dereference below the NULL test

2012-09-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/memstick/host/r592.c | 3 ++- 1 file

[PATCH] configfs: move the dereference below the NULL test

2012-09-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/configfs/dir.c | 5 +++-- 1 file changed, 3

[PATCH] powerpc/powernv: move the dereference below the NULL test

2012-09-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/powerpc/platforms/powernv/pci.c | 8

[PATCH] staging: tidspbridge: move the dereference below the NULL test

2012-09-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/tidspbridge/core/chnl_sm.c | 3

[PATCH] block: move the dereference below the NULL test

2012-09-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. spatch with a semantic match is used to found this. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/swim3.c | 3 ++- 1 file changed, 2

[PATCH] JFS: use list_move instead of list_del/list_add

2012-09-06 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/jfs/jfs_txnmgr.c | 9 +++-- 1 file

[PATCH] iommu/exynos: use list_del_init instead of list_del/INIT_LIST_HEAD

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_del_init() instead of list_del() + INIT_LIST_HEAD(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/iommu/exynos-iommu.c | 3

[PATCH] staging: gdm72xx: use list_move instead of list_del/list_add

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/gdm72xx/gdm_usb.c | 3

[PATCH] staging: wlags49_h2: use list_move instead of list_del/list_add

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/wlags49_h2/wl_netdev.c | 3

[PATCH] clockevents: use list_move instead of list_del/list_add

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/time/clockevents.c | 6 ++ 1

[PATCH] audit: use list_move instead of list_del/list_add in audit-tree

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/audit_tree.c | 12 1

[PATCH] ALSA: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/drivers/opl4/opl4_synth.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/sound/drivers

[PATCH] mm: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- mm/rmap.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/mm/rmap.c b/mm/rmap.c index 0f3b7cd

[PATCH] staging: ozwpan: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/ozwpan/ozhcd.c

[PATCH] staging: gdm72xx: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/gdm72xx

[PATCH] ALSA: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/pci/emu10k1/memory.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/pci/emu10k1

[PATCH] dma: tegra: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/tegra20-apb-dma.c

[PATCH] drbd: use list_move_tail instead of list_del/list_add_tail

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move_tail() instead of list_del() + list_add_tail(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/drbd/drbd_worker.c

[PATCH] Staging: bcm: fix possible memory leak in bcm_char_ioctl()

2012-09-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn psFwInfo has been allocated in this function and should be freed before leaving from the error handling cases. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_

[PATCH] parport: fix possible memory leak in parport_gsc_probe_port()

2012-09-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn ops has been allocated in this function and should be freed before leaving from the error handling cases. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_

[PATCH] staging: csr: fix possible memory leak in do_patch_convert_download()

2012-09-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn pfw has been allocated in function xbv_to_patch() and should be freed before leaving from the error handling cases. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_

[PATCH] kdb: fix possible memory leak in kdb_defcmd2()

2012-09-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In the error handling case of kdb_defcmd2() s-command has been set to NULL but the save_command remain no free, this will cause memory leak. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off

[PATCH] ALSA: fix possible memory leak in snd_mixer_oss_build_input()

2012-09-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn uinfo has been allocated in this function and should be freed before leaving from the error handling cases. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_

[PATCH] mtd: fix possible memory leak in logfs_mtd_can_write_buf()

2012-09-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn buf has been allocated in this function and should be freed before leaving from the error handling case. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_

[PATCH] staging: rtl8192e: remove pointless conditional before kfree_skb()

2012-08-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove pointless conditional before kfree_skb(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging

[PATCH] staging: ozwpan: remove pointless conditional before kfree_skb()

2012-08-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove pointless conditional before kfree_skb(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/ozwpan/ozpd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/ozwpan/ozpd.c b

[PATCH] audit: remove pointless conditional before kfree_skb()

2012-08-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove pointless conditional before kfree_skb(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/audit.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/kernel/audit.c b/kernel/audit.c index ea3b7b6

[PATCH] cciss: remove useless set memory to zero use memset()

2012-08-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The memory return by kzalloc() or kmem_cache_zalloc() has already be set to zero, so remove useless memset(0). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_

[PATCH] vmwgfx: remove useless set memory to zero use memset()

2012-08-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The memory return by kzalloc() or kmem_cache_zalloc() has already be set to zero, so remove useless memset(0). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 1 - drivers/gpu/drm/vmwgfx

[PATCH] xen/blkback: use kmem_cache_zalloc instead of kmem_cache_alloc/memset

2012-08-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using kmem_cache_zalloc() instead of kmem_cache_alloc() and memset(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/xen-blkback

[PATCH] dmaengine: use kmem_cache_zalloc instead of kmem_cache_alloc/memset

2012-08-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using kmem_cache_zalloc() instead of kmem_cache_alloc() and memset(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/ioat/dma_v2.c

[PATCH] staging: wlags49_h2: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/wlags49_h2/wl_wext.c | 3 ++- 1 file changed

[PATCH] et131x: using is_zero_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_zero_ether_addr() to simplify the code Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/et131x/et131x.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/et131x/et131x.c

[PATCH] staging: rtl8187se: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun

[PATCH] Staging: rtl8192u: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun

[PATCH] Staging: vt6655: using is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() to simplify the code. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/vt6655/hostap.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/vt6655

[PATCH] Staging: vt6656: using is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() to simplify the code. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/vt6656/hostap.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/staging/vt6656

[PATCH] staging: rtl8192e: use is_broadcast_ether_addr() instead of memcmp()

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() instead of directly use memcmp() to determine if the ethernet address is broadcast address. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun

[PATCH] staging: r8712u: use is_broadcast_ether_addr() to simplify the code

2012-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using is_broadcast_ether_addr() to simplify the code. spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/rtl8712/ethernet.h

<    1   2   3   4   5   6   7   >