[PATCH] powerpc/fsl_msi: fix error return code in fsl_of_msi_probe()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the MSI bitmap alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/powerpc/sysdev/fsl_msi.c | 5 ++--- 1 file changed

Re: [PATCH] rtc: rtc-tile: add missing platform_device_unregister() when module exit

2013-05-07 Thread Wei Yongjun
On 05/08/2013 07:28 AM, Andrew Morton wrote: On Sun, 5 May 2013 13:07:00 +0800 Wei Yongjun weiyj...@gmail.com wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn We have registered platform device when module init, and need unregister it when module exit. ... --- a/drivers/rtc/rtc

[PATCH] rtc: rtc-tile: add missing platform_device_unregister() when module exit

2013-05-04 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn We have registered platform device when module init, and need unregister it when module exit. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/rtc/rtc-tile.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rtc/rtc

[PATCH RESEND] spi-topcliff-pch: fix to use list_for_each_entry_safe() when delete list items

2013-04-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Since we will remove items off the list using list_del_init() we need to use a safe version of the list_for_each_entry() macro aptly named list_for_each_entry_safe(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/spi/spi

[PATCH -next] xen/arm: remove duplicated include from enlighten.c

2013-04-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/xen/enlighten.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c index 9a0a917..d30042e 100644

[PATCH] TTY: ehv_bytechan: add missing platform_driver_unregister() when module exit

2013-04-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn We have registered platform driver when module init, and need unregister it when module exit. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- no compile test. --- drivers/tty/ehv_bytechan.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH] w1: ds2780: fix the error handling in w1_ds2780_add_slave()

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use platform_device_put() instead of platform_device_unregister() if platform_device_add() fail, and platform_device_del() should be used in the error handling case after platform_device_add() success. Signed-off-by: Wei Yongjun yongjun_

[PATCH] w1: bq27000: fix the error handling in w1_bq27000_add_slave()

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use platform_device_put() instead of platform_device_unregister() if platform_device_add() fails, and also add the return value check of platform_device_add_data(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/w1/slaves

[PATCH] w1: ds2781: fix the error handling in w1_ds2781_add_slave()

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use platform_device_put() instead of platform_device_unregister() if platform_device_add() fail, and platform_device_del() should be used in the error handling case after platform_device_add() success. Signed-off-by: Wei Yongjun yongjun_

[PATCH] w1: ds2760: fix the error handling in w1_ds2760_add_slave()

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use platform_device_put() instead of platform_device_unregister() if platform_device_add() fail, and platform_device_del() should be used in the error handling case after platform_device_add() success. Signed-off-by: Wei Yongjun yongjun_

[PATCH] tpm_tis: missing platform_driver_unregister() on error in init_tis()

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing platform_driver_unregister() before return from init_tis() in the device register error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/char/tpm/tpm_tis.c | 19 +-- 1 file changed

[PATCH] msm: iommu: missing platform_driver_unregister() on error in msm_iommu_driver_init()

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing platform_driver_unregister() before return from msm_iommu_driver_init() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/iommu/msm_iommu_dev.c | 1 + 1 file changed, 1 insertion

[PATCH] spi-topcliff-pch: missing platform_driver_unregister() on error in pch_spi_init()

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing platform_driver_unregister() before return from pch_spi_init() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/spi/spi-topcliff-pch.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH -next] staging: ozwpan: fix error return code in oz_cdev_register()

2013-04-25 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 done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/ozwpan/ozcdev.c | 2 ++ 1 file changed, 2 insertions

[PATCH -next] rtc: rtc-pcf2123: fix error return code in pcf2123_probe()

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENODEV in the chip not found error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/rtc/rtc-pcf2123.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH -next v3] gpu: host1x: gr2d: fix error return code in gr2d_submit()

2013-04-24 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOENT in the host1x_bo lookup error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- v1 - v2: fix the patch subject --- drivers/gpu/host1x/drm/gr2d.c | 8

[tip:x86/platform] x86/olpc/xo1/sci: Don't call input_free_device () after input_unregister_device()

2013-04-24 Thread tip-bot for Wei Yongjun
Commit-ID: 3482e664dc7f44e0ffb1fcf1de7af74977748f31 Gitweb: http://git.kernel.org/tip/3482e664dc7f44e0ffb1fcf1de7af74977748f31 Author: Wei Yongjun yongjun_...@trendmicro.com.cn AuthorDate: Wed, 24 Apr 2013 10:46:24 +0800 Committer: Ingo Molnar mi...@kernel.org CommitDate: Wed, 24 Apr

[PATCH -next] gpu: host1x: gr2d drivers/gpu/host1x/drm/gr2d.c: fix error return code in gr2d_submit()

2013-04-23 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 done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/host1x/drm/gr2d.c | 8 ++-- 1 file changed, 6

[PATCH -next] staging: cptm1217: don't call input_free_device() after input_unregister_device()

2013-04-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn input_free_device() should only be used if input_register_device() was not called yet or if it failed. Once device was unregistered use input_unregister_device() and memory will be freed once last reference to the device is dropped. Signed-off

[PATCH -next v2] gpu: host1x: gr2d: fix error return code in gr2d_submit()

2013-04-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EINVAL in the host1x_bo lookup error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- v1 - v2: fix the patch subject --- drivers/gpu/host1x/drm/gr2d.c | 8

[PATCH -next RESEND] gpu: host1x: drm: fix missing unlock on error

2013-04-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function host1x_drm_init() and host1x_drm_exit() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- rebased on the latest linux-next.git tree. --- drivers/gpu/host1x

Re: [PATCH -next RESEND] gpu: host1x: drm: fix missing unlock on error

2013-04-23 Thread Wei Yongjun
not sure whether this patch was lost. Regards, Yongjun Wei Mark On 04/24/2013 10:48 AM, Wei Yongjun wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function host1x_drm_init() and host1x_drm_exit() in the error handling case. Signed-off

[PATCH -next] mei: fix krealloc() misuse in in mei_cl_irq_read_msg()

2013-04-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn If krealloc() returns NULL, it doesn't free the original. So any code of the form 'foo = krealloc(foo, ...);' is almost certainly a bug. Introduced by commit fcb136e1ac5774909e0d85189f721b8dfa800e0f(mei: fix reading large reposnes) Signed-off

[PATCH -next] iommu/amd: fix error return code in early_amd_iommu_init()

2013-04-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM int the memory alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/iommu/amd_iommu_init.c | 1 + 1 file changed, 1 insertion

[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

[tip:perf/urgent] perf: Fix error return code

2013-04-14 Thread tip-bot for Wei Yongjun
Commit-ID: c481420248c6730246d2a1b1773d5d7007ae0835 Gitweb: http://git.kernel.org/tip/c481420248c6730246d2a1b1773d5d7007ae0835 Author: Wei Yongjun yongjun_...@trendmicro.com.cn AuthorDate: Fri, 12 Apr 2013 11:05:54 +0800 Committer: Ingo Molnar mi...@kernel.org CommitDate: Fri, 12 Apr

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

[tip:perf/core] perf report: Remove duplicated include

2013-03-21 Thread tip-bot for Wei Yongjun
Commit-ID: db3c6bf811581c626471a6aecdf0024575b707d7 Gitweb: http://git.kernel.org/tip/db3c6bf811581c626471a6aecdf0024575b707d7 Author: Wei Yongjun yongjun_...@trendmicro.com.cn AuthorDate: Wed, 13 Mar 2013 12:24:42 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Fri

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

[PATCH] GFS2: return error if malloc failed in gfs2_rs_alloc()

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The error code in gfs2_rs_alloc() is set to ENOMEM when error but never be used, instead, gfs2_rs_alloc() always return 0. Fix to return 'error'. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/gfs2/rgrp.c | 2 +- 1 file changed, 1

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

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_get() returns ERR_PTR() not 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/clk/clk-prima2.c | 2 +- 1 file

[PATCH] ALSA: asihpi - fix potential NULL pointer dereference

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference should be moved below the NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/pci/asihpi/asihpi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/asihpi/asihpi.c b/sound/pci

[PATCH] ASoC: wm_adsp: fix possible memory leak in wm_adsp_load_coeff()

2013-03-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn 'file' is malloced in wm_adsp_load_coeff() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/codecs/wm_adsp.c | 5 +++-- 1

[PATCH] PM / devfreq: fix missing unlock on error in exynos4_busfreq_pm_notifier_event()

2013-02-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function exynos4_busfreq_pm_notifier_event() in the error handling case. This issue introduced by commit 8fa938 (PM / devfreq: exynos4_bus: honor RCU lock usage) Signed-off-by: Wei Yongjun yongjun_

[PATCH -next] firewire: net: remove unused variable in fwnet_receive_broadcast()

2012-12-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable card is initialized but never used otherwise, so remove the unused variable. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/firewire/net.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/firewire

[PATCH -next] tile/PCI: use for_each_pci_dev to simplify the code

2012-12-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use for_each_pci_dev to simplify the code. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/tile/kernel/pci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/tile/kernel/pci.c b/arch/tile/kernel/pci.c

[PATCH -next] mn10300: use for_each_pci_dev to simplify the code

2012-12-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use for_each_pci_dev to simplify the code. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/mn10300/unit-asb2305/pci-irq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mn10300/unit-asb2305/pci-irq.c b

[PATCH -next] Drivers: hv: remove unused variable in vmbus_recvpacket_raw()

2012-12-03 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variable userlen is initialized but never used otherwise, so remove the unused variable. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/hv/channel.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/hv

[PATCH -next] TTY: hvsi: use for_each_compatible_node() macro

2012-12-03 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 --- drivers/tty/hvc/hvsi.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/tty/hvc/hvsi.c b

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

2012-12-03 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/platforms/82xx/pq2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc

[PATCH -next] drm: tegra: fix missing unlock on error

2012-12-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function host1x_drm_init() and host1x_drm_exit() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/tegra/host1x.c | 2 ++ 1 file changed, 2 insertions

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

2012-12-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Use for_each_compatible_node() macro. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/xtensa/platforms/xtfpga/setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/xtensa/platforms/xtfpga/setup.c b

[PATCH -next] f2fs: remove unused variable

2012-12-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The variables node_page and page_offset are initialized but never used otherwise, so remove those unused variables. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/f2fs/file.c | 2 -- fs/f2fs/dir.c | 2 -- 2 files changed, 4

[PATCH -next] mfd: sta2x11-mfd: fix potential NULL pointer dereference in __sta2x11_mfd_mask()

2012-12-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to 'mfd' should be moved below the NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/sta2x11-mfd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mfd/sta2x11-mfd.c b

[PATCH -next] mfd: sta2x11-mfd: fix potential NULL pointer dereference in __sta2x11_mfd_mask()

2012-12-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to 'mfd' should be moved below the NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/sta2x11-mfd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mfd/sta2x11-mfd.c b

[PATCH -next] staging: sb105x: fix potential NULL pointer dereference in mp_chars_in_buffer()

2012-11-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to 'state' should be moved below the NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/sb105x/sb_pci_mp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging

Re: [PATCH] spi: omap2-mcspi: remove duplicated include from spi-omap2-mcspi.c

2012-11-15 Thread Wei Yongjun
On 11/14/2012 11:54 PM, Grant Likely wrote: On Sun, 7 Oct 2012 21:29:21 +0800, Wei Yongjun weiyj...@gmail.com wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Applied, thanks. dpatch engine is used to auto generate this patch. (https://github.com/weiyj

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

[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

<    1   2   3   4   5   6   7   >