[PATCH -next] CLK: clk-twl6040: fix return value check in twl6040_clk_probe()

2012-10-31 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 -next] Staging: silicom: bypasslib: remove unused including linux/version.h

2012-10-31 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/silicom/bypasslib/bplibk.h | 1

[PATCH -next] Staging: silicom: remove unused including linux/version.h

2012-10-31 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/silicom/bp_mod.c | 1 - 1 file

[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] 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] 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] 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

[PATCH -next] ACPI: fix missing unlock on error in acpi_memory_remove_memory()

2012-11-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing mutex_unlock() before return from function acpi_memory_remove_memory() in the error handling case. Introduce by commit 85fcb3758c10e063a2a30dfad75017097999deed 'ACPI / memory-hotplug: introduce a mutex lock to protect the list

[PATCH -next] ASoC: cs42l52: fix the return value of cs42l52_set_fmt()

2012-11-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix the return value of cs42l52_set_fmt() when clock inversion is not allowed and also remove the useless variable ret. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

Re: [PATCH -next] ACPI: fix missing unlock on error in acpi_memory_remove_memory()

2012-11-07 Thread Wei Yongjun
Hi Wen, On 11/08/2012 09:31 AM, Wen Congyang wrote: Hi, wei At 11/07/2012 08:38 PM, Wei Yongjun Wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing mutex_unlock() before return from function acpi_memory_remove_memory() in the error handling case. Introduce by commit

Re: [PATCH] rtc: avoid calling platform_device_put() twice in test_init()

2012-11-07 Thread Wei Yongjun
On 11/07/2012 08:51 AM, Andrew Morton wrote: On Tue, 23 Oct 2012 13:08:41 +0800 Wei Yongjun weiyj...@gmail.com wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function test_init() need to call platform_device_del() instead of platform_device_unregister

[PATCH -next] staging: comedi: usbduxfast: remove unused variable in usbduxfastsub_ai_Irq()

2012-11-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable 'p' 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 --- drivers

[PATCH -next] mtip32xx: fix potential NULL pointer dereference in mtip_timeout_function()

2012-11-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to port 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/block/mtip32xx/mtip32xx.c | 3

[PATCH] dmaengine: imx-dma: fix missing unlock on error in imxdma_xfer_desc()

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

[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

[PATCH] ALSA: hda - remove unused variable in azx_position_ok()

2012-10-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable stream 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] aoe: remove unused variable in resend()

2012-10-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable ah 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 --- drivers

[PATCH] aoe: remove unused variable in resend()

2012-10-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable ah 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 --- drivers

[PATCH] Drivers: hv: remove unused variable from channel_mgmt.c

2012-10-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variables guidtype, guidinstance and initiate 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_

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

2012-10-22 Thread Wei Yongjun
On 10/22/2012 09:50 PM, Mark Brown wrote: On Thu, Oct 18, 2012 at 10:49:19PM +0800, Wei Yongjun wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable oldstatus is initialized but never used otherwise, so remove the unused variable. Applied, thanks. However for changes like

[PATCH] rtc: avoid calling platform_device_put() twice in test_init()

2012-10-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function test_init() need to call platform_device_del() instead of platform_device_unregister(). Otherwise, we may call platform_device_put() twice. dpatch engine is used to auto generate this patch. (https://github.com/weiyj

[PATCH] pc8736x_gpio: use platform_device_unregister in pc8736x_gpio_cleanup()

2012-10-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn platform_device_unregister() only calls platform_device_del() and platform_device_put(), thus use platform_device_unregister() to simplify the code. Also the documents in platform.c shows that platform_device_del and platform_device_put must _only_

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] 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] 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] ARM i.MX: fix error-valued pointer dereference in clk_register_gate2()

2012-10-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The error-valued pointer clk is used for the arg of kfree, it should be kfree(gate) if clk_register() return ERR_PTR(). dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_

[PATCH -next] ARM: at91: remove duplicated include from pinctrl-at91.c

2012-10-26 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-at91.c | 1 - 1 file changed, 1 deletion(-) diff

[PATCH -next] ARM: at91: using for_each_set_bit to simplify the code

2012-10-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using for_each_set_bit() to simplify the code. 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-at91.c | 4 +--- 1 file

[PATCH -next] ARM: at91: remove unused variable in at91_dt_node_to_map()

2012-10-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable pin 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 --- drivers

[PATCH -next] mtip32xx: remove unused variable from mtip32xx.c

2012-10-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variables fis, reply 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 -next] GPIO: clps711x: use platform_device_unregister in gpio_clps711x_init()

2012-10-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn platform_device_unregister() only calls platform_device_del() and platform_device_put(), thus use platform_device_unregister() to simplify the code. Also the documents in platform.c shows that platform_device_del and platform_device_put must _only_

[PATCH -next] staging: comedi: use module_comedi_driver to simplify the code

2012-10-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_comedi_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 -next] staging: comedi: skel: use module_comedi_driver to simplify the code

2012-10-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use the module_comedi_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 -next] virtio: balloon: fix missing unlock on error in fill_balloon()

2012-11-12 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function fill_balloon() in the error handling case. Introduced by 9864a8(virtio_balloon: introduce migration primitives to balloon pages) dpatch engine is used to auto generate this patch. (https

[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

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 -next] ALSA: hda - fix error return code in patch_alc662()

2013-04-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/pci/hda/patch_realtek.c | 3 ++- 1 file changed, 2 insertions

[PATCH -next] spi: omap2-mcspi: fix error return code in omap2_mcspi_probe()

2013-04-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/spi/spi-omap2-mcspi.c | 3 ++- 1 file changed, 2 insertions

[PATCH -next] drbd: fix error return code in drbd_init()

2013-04-04 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/drbd/drbd_main.c | 2 ++ 1 file changed, 2 insertions

[PATCH -next] virtio_console: make local symbols static

2013-04-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Those symbols only used within this file, and should be static. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/char/virtio_console.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/char

[PATCH -next v2] drbd: fix error return code in drbd_init()

2013-04-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/drbd/drbd_main.c | 4 +--- 1 file changed, 1

[PATCH -next] workqueue: use kmem_cache_free() instead of kfree()

2013-04-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn memory allocated by kmem_cache_alloc() should be freed using kmem_cache_free(), not kfree(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/workqueue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel

[PATCH -next] mei: convert to use simple_open()

2013-04-09 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn This removes an open coded simple_open() function and replaces file operations references to the function with simple_open() instead. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/misc/mei/debugfs.c | 10 ++ 1 file

[PATCH -next] f2fs: fix error return code in f2fs_fill_super()

2013-04-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Introduce by commit c0d39e(f2fs: fix return values from validate superblock) Signed-off-by: Wei Yongjun yongjun_

[PATCH -next] perf: fix error return code

2013-04-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/events/core.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH -next] mxs/spi: fix error return code in mxs_spi_probe()

2013-04-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/spi/spi-mxs.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH -next] audit: remove duplicated include from audit.c

2013-04-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- kernel/audit.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/audit.c b/kernel/audit.c index 488f85f..9377913 100644 --- a/kernel/audit.c +++ b

[PATCH] pinctrl: exynos5440: fix invalid use of sizeof in exynos5440_pinctrl_probe()

2013-03-14 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/pinctrl-exynos5440.c | 2 +- 1 file changed, 1 insertion

[PATCH] xtensa: net: fix invalid use of sizeof in iss_net_setup()

2013-03-14 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/xtensa/platforms/iss/network.c | 2 +- 1 file changed, 1 insertion

[PATCH -next] Staging: netlogic: remove unused variable in xlr_net_start_xmit()

2013-03-14 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable 'qmap' is initialized but never used otherwise, so remove the unused variable. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/netlogic/xlr_net.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH] cciss: fix invalid use of sizeof in cciss_find_cfgtables()

2013-03-14 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/cciss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] UBIFS: fix missing unlock on error in ubifs_remount_rw()

2013-03-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function ubifs_remount_rw() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/ubifs/super.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs

[PATCH -next] mtip32xx: fix error return code in mtip_pci_probe()

2013-03-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/mtip32xx/mtip32xx.c | 4 +++- 1 file changed, 3

[PATCH -next] Drivers: hv: balloon: make local functions static

2013-03-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn local functions that could be static. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/hv/hv_balloon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/hv/hv_balloon.c b/drivers/hv/hv_balloon.c

[PATCH -next] loopdev: fix error return code in loop_add()

2013-03-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/loop.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers

[PATCH -next] rpmsg: fix error return code in rpmsg_probe()

2013-03-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/rpmsg/virtio_rpmsg_bus.c | 4 +++- 1 file changed, 3

[PATCH -next] mailbox: fix invalid use of sizeof in mailbox_msg_send()

2013-03-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mailbox/mailbox.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH -next v2] mailbox: fix invalid use of sizeof in mailbox_msg_send()

2013-03-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mailbox/mailbox.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH -next] ASoC: arizona: remove duplicated include from arizona.c

2013-03-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/codecs/arizona.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/arizona.c b/sound/soc/codecs/arizona.c index 26e1579..c979ff2

[PATCH -next] ocfs2: fix error return code in ocfs2_info_handle_freefrag()

2013-03-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/ocfs2/ioctl.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH -next] mg_disk: fix error return code in mg_probe()

2013-03-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/mg_disk.c | 4 +++- 1 file changed, 3 insertions

[PATCH -next] staging: sync: fix return value check in sync_fence_alloc()

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function anon_inode_getfile() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging

[PATCH -next] sh-pfc: fix return value check in sh_pfc_register_pinctrl()

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function pinctrl_register() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/sh-pfc

[PATCH -next] char/tpm/tpm_i2c_stm_st33: remove duplicated include from tpm_i2c_stm_st33.c

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/char/tpm/tpm_i2c_stm_st33.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/char/tpm/tpm_i2c_stm_st33.c b/drivers/char/tpm

[PATCH -next] pinctrl: sirf: using for_each_set_bit to simplify the code

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using for_each_set_bit() to simplify the code. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/pinctrl-sirf.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/pinctrl/pinctrl

[PATCH -next] powerpc: use for_each_compatible_node() macro

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use for_each_compatible_node() macro instead of open coding it. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/powerpc/include/asm/parport.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/powerpc

  1   2   3   4   5   6   7   >