[PATCH] spi: coldfire-qspi: fix error return code in mcfqspi_probe()

2013-05-15 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn If pdev-dev.platform_data is not set, mcfqspi_probe() will return 0 and release all the resources, in this case, we should return a error code instead of 0. This patch fix to return -ENOENT in this case and move the check for pdev-dev.platform_data

[PATCH v2] NVMe: fix error return code in nvme_submit_bio_queue()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENOMEM in the nvem iod alloc error handling case instead of 0(possible overwrite to 0 by above nvme_submit_flush_data()), as done elsewhere in this function. Signed-off-by: Wei Yongjun --- v1 -> v2: send to mail list from MAINTAINERS --- drivers/block/n

[PATCH] mtd: fsmc_nand: fix error return code in fsmc_nand_probe()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENODEV in the dma channel request error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/mtd/nand/fsmc_nand.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mtd/nand/fsmc_nand.c b/drivers/mtd

[PATCH] rtc: rtc-vr41xx: fix error return code in rtc_probe()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun Fix to return -EBUSY in the platform irq get error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/rtc/rtc-vr41xx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/rtc/rtc-vr41xx.c b

[PATCH] staging: slicoss: fix error return code in slic_entry_probe()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/staging/slicoss/slicoss.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/drivers

[PATCH] hwmon: fix error return code in abituguru_probe()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/hwmon/abituguru.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/drivers/hwmon

[PATCH] hwmon: fix error return code in abituguru_probe()

2013-05-13 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/hwmon/abituguru.c | 16 ++-- 1 file changed, 10

[PATCH] staging: slicoss: fix error return code in slic_entry_probe()

2013-05-13 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/slicoss/slicoss.c | 18 +++--- 1 file

[PATCH] rtc: rtc-vr41xx: fix error return code in rtc_probe()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EBUSY in the platform irq get error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/rtc/rtc-vr41xx.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH] mtd: fsmc_nand: fix error return code in fsmc_nand_probe()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENODEV in the dma channel request error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mtd/nand/fsmc_nand.c | 1 + 1 file changed, 1 insertion

[PATCH v2] NVMe: fix error return code in nvme_submit_bio_queue()

2013-05-13 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the nvem iod alloc error handling case instead of 0(possible overwrite to 0 by above nvme_submit_flush_data()), as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- v1 - v2: send

[PATCH] gdrom: fix error return code in probe_gdrom()

2013-05-12 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENOMEM in the memory alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/cdrom/gdrom.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/cdrom/gdrom.c b/drivers

[PATCH] NVMe: fix error return code in nvme_submit_bio_queue()

2013-05-12 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENOMEM in the nvem iod alloc error handling case instead of 0(possible overwrite to 0 by above nvme_submit_flush_data()), as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/block/nvme-core.c | 6 -- 1 file changed, 4 insertions

[PATCH] NVMe: fix error return code in nvme_submit_bio_queue()

2013-05-12 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the nvem iod alloc error handling case instead of 0(possible overwrite to 0 by above nvme_submit_flush_data()), as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block

[PATCH] gdrom: fix error return code in probe_gdrom()

2013-05-12 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in 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/cdrom/gdrom.c | 8 ++-- 1 file changed, 6 insertions(+), 2

[PATCH] uio: fix error return code in uio_dev_add_attributes()

2013-05-08 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENOMEM in the kobject_create_and_add() and kzalloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/uio/uio.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/uio/uio.c b/drivers/uio

[PATCH] remoteproc: fix error return code in rproc_fw_boot()

2013-05-08 Thread Wei Yongjun
From: Wei Yongjun Fix to return -EINVAL in the rproc_find_loaded_rsc_table() error handling case instead of 0, as done elsewhere in this function. Introduced by commit a2b950ac7b1e6442919ee9e79c4963e134698869 (remoteproc: perserve resource table data). Signed-off-by: Wei Yongjun --- drivers

[PATCH] remoteproc: fix error return code in rproc_fw_boot()

2013-05-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EINVAL in the rproc_find_loaded_rsc_table() error handling case instead of 0, as done elsewhere in this function. Introduced by commit a2b950ac7b1e6442919ee9e79c4963e134698869 (remoteproc: perserve resource table data). Signed-off

[PATCH] uio: fix error return code in uio_dev_add_attributes()

2013-05-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the kobject_create_and_add() and kzalloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/uio/uio.c | 2 ++ 1 file changed, 2

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 wrote: > >> From: Wei Yongjun >> >> We have registered platform device when module init, and >> need unregister it when module exit. >> >> ... >> >>

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

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun 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 --- arch/powerpc/sysdev/fsl_msi.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/arch

[PATCH] drm/exynos: fix error return code in exynos_drm_load()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENOMEM in the exynos_plane_init() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu

[PATCH] drm/exynos: fix error return code in exynos_drm_ipp_set_property()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. ipp_create_cmd_work() return ERR_PTR() on error and never return NULL, so use IS_ERR() instead of IS_ERR_OR_NULL(). Signed-off-by: Wei Yongjun --- drivers/gpu

Re: [PATCH] pinctrl: single: fix error return code in pcs_parse_one_pinctrl_entry()

2013-05-07 Thread Wei Yongjun
On 05/07/2013 08:11 PM, Haojian Zhuang wrote: > On 7 May 2013 20:06, Wei Yongjun wrote: >> From: Wei Yongjun >> >> Fix to return a negative error code from the error handling >> case instead of 0, as done elsewhere in this function.

[PATCH] pinctrl: single: fix error return code in pcs_parse_one_pinctrl_entry()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Introduced by commit 9dddb4df90d136429b6d6ddefceb49a9b93f6cd1 (pinctrl: single: support generic pinconf) Signed-off-by: Wei Yongjun --- drivers/pinctrl/pinctrl

[PATCH] pstore/ram: fix error return code in ramoops_probe()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun 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 --- fs/pstore/ram.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c index 32cbd7c..1376e5a

[PATCH] ASoC: wm0010: fix error return code in wm0010_boot()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun Fix to return -ENOMEM in the memory malloc of 'out' and 'img_swap' error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- sound/soc/codecs/wm0010.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/codecs/wm0010.c b

[PATCH] ASoC: wm0010: fix error return code in wm0010_boot()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the memory malloc of 'out' and 'img_swap' error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/codecs/wm0010.c | 1 + 1 file changed

[PATCH] pstore/ram: fix error return code in ramoops_probe()

2013-05-07 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 --- fs/pstore/ram.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH] pinctrl: single: fix error return code in pcs_parse_one_pinctrl_entry()

2013-05-07 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. Introduced by commit 9dddb4df90d136429b6d6ddefceb49a9b93f6cd1 (pinctrl: single: support generic pinconf) Signed-off-by: Wei Yongjun

[PATCH] drm/exynos: fix error return code in exynos_drm_load()

2013-05-07 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the exynos_plane_init() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/exynos/exynos_drm_drv.c | 4 +++- 1 file changed, 3

[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 We have registered platform device when module init, and need unregister it when module exit. Signed-off-by: Wei Yongjun --- drivers/rtc/rtc-tile.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rtc/rtc-tile.c b/drivers/rtc/rtc-tile.c index 249b653..fc3dee9

[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] TTY: ehv_bytechan: add missing platform_driver_unregister() when module exit

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

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

2013-04-27 Thread Wei Yongjun
From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- 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 --- a/arch/arm/xen/enlighten.c +++ b/arch/arm/xen

[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 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 --- drivers/spi/spi-topcliff-pch.c | 12 ++-- 1 file changed, 6

[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: ds2760: fix the error handling in w1_ds2760_add_slave()

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/w1/slaves/w1_ds2760.c | 3 ++- 1

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

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/w1/slaves/w1_ds2781.c | 3 ++- 1

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

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/w1/slaves/w1_bq27000.c | 4 +++- 1 file changed, 3 insertions(+), 1

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

2013-04-26 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/w1/slaves/w1_ds2780.c | 3 ++- 1

[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 -next] rtc: rtc-pcf2123: fix error return code in pcf2123_probe()

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/rtc/rtc-pcf2123.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rtc/rtc-pcf2123.c b/drivers/rtc/rtc-pcf2123

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

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/staging/ozwpan/ozcdev.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/ozwpan/ozcdev.c b/drivers

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

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

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

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

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

2013-04-25 Thread Wei Yongjun
From: Wei Yongjun Add the missing platform_driver_unregister() before return from init_tis() in the device register error handling case. Signed-off-by: Wei Yongjun --- drivers/char/tpm/tpm_tis.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/drivers

[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

[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 AuthorDate: Wed, 24 Apr 2013 10:46:24 +0800 Committer: Ingo Molnar CommitDate: Wed, 24 Apr 2013 08:44:47 +0200 x86/olpc/xo1/sci: Don't

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

2013-04-24 Thread Wei Yongjun
From: Wei Yongjun 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 --- v1 -> v2: fix the patch subject --- drivers/gpu/host1x/drm/gr2d.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deleti

[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

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

2013-04-23 Thread Wei Yongjun
am not sure whether this patch was lost. Regards, Yongjun Wei > > Mark > On 04/24/2013 10:48 AM, Wei Yongjun wrote: >> From: Wei Yongjun >> >> Add the missing unlock before return from function host1x_drm_init() and >> host1x_drm_exit() in the error handling ca

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

2013-04-23 Thread Wei Yongjun
From: Wei Yongjun 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 --- rebased on the latest linux-next.git tree. --- drivers/gpu/host1x/drm/drm.c | 2 ++ 1 file changed, 2 insertions(+) diff

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

2013-04-23 Thread Wei Yongjun
From: Wei Yongjun 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 --- v1 -> v2: fix the patch subject --- drivers/gpu/host1x/drm/gr2d.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deleti

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

2013-04-23 Thread Wei Yongjun
From: Wei Yongjun 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-by: Wei Yongjun --- drivers

[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 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 --- drivers/gpu/host1x/drm/gr2d.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/host1x/drm

[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] iommu/amd: fix error return code in early_amd_iommu_init()

2013-04-22 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/iommu/amd_iommu_init.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/iommu/amd_iommu_init.c b/drivers/iommu

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

2013-04-22 Thread Wei Yongjun
From: Wei Yongjun 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-by: Wei Yongjun --- drivers

[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] spi: omap2-mcspi: fix error return code in omap2_mcspi_probe()

2013-04-17 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/spi/spi-omap2-mcspi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/spi/spi-omap2

[PATCH -next] ALSA: hda - fix error return code in patch_alc662()

2013-04-17 Thread Wei Yongjun
From: Wei Yongjun 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 --- sound/pci/hda/patch_realtek.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/hda

[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 AuthorDate: Fri, 12 Apr 2013 11:05:54 +0800 Committer: Ingo Molnar CommitDate: Fri, 12 Apr 2013 06:33:56 +0200 perf: Fix error return code

[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] perf: fix error return code

2013-04-11 Thread Wei Yongjun
From: Wei Yongjun 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 --- kernel/events/core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/kernel/events/core.c b/kernel

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

2013-04-11 Thread Wei Yongjun
From: Wei Yongjun 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 --- fs/f2fs/super.c | 3 ++- 1 file changed, 2

[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] mei: convert to use simple_open()

2013-04-09 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/misc/mei/debugfs.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git

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

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

[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 v2] drbd: fix error return code in drbd_init()

2013-04-05 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/block/drbd/drbd_main.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/block/drbd

[PATCH -next] virtio_console: make local symbols static

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

[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 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 --- drivers/block/drbd/drbd_main.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/block/drbd/drbd_main.c b/drivers

[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] audit: remove duplicated include from audit.c

2013-04-03 Thread Wei Yongjun
From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- 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/kernel/audit.c @@ -58,7 +58,6 @@ #ifdef CONFIG_SECURITY

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

2013-04-03 Thread Wei Yongjun
From: Wei Yongjun 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 --- drivers/spi/spi-mxs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/spi/spi-mxs.c b/drivers/spi/spi-mxs.c index

[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

<    5   6   7   8   9   10   11   12   13   14   >