[PATCH] mfd: twl6040: drop devm_free_irq of devm_ allocated irq

2013-09-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The devm_request_irq function allocates irq that is released when a driver detaches. Thus, there is no reason to explicitly call devm_free_irq in probe or remove functions. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd

[PATCH] mfd: ezx-pcap: drop devm_free_irq of devm_ allocated irq

2013-09-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The devm_request_irq function allocates irq that is released when a driver detaches. Thus, there is no reason to explicitly call devm_free_irq in probe or remove functions. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd

[PATCH -next] dma: mmp_pdma: use list_move instead of list_del/list_add

2013-09-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Using list_move() instead of list_del() + list_add(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/mmp_pdma.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/dma/mmp_pdma.c b/drivers/dma

[PATCH] i2o: add missing destroy_workqueue() on error in i2o_driver_register()

2013-09-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing destroy_workqueue() before return from i2o_driver_register() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/message/i2o/driver.c | 16 ++-- 1 file changed, 10 insertions

[PATCH] tps65090-charger: drop devm_free_irq of devm_ allocated irq

2013-09-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The devm_request_irq function allocates irq that is released when a driver detaches. Thus, there is no reason to explicitly call devm_free_irq in probe or remove functions. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/power

[PATCH -next] staging: r8188eu: remove needless check before usb_free_urb()

2013-09-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn usb_free_urb(NULL) is safe and this check is not required. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/rtl8188eu/os_dep/recv_linux.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[PATCH] mmc: cb710: drop free_irq for devm_request_irq allocated irq

2013-09-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn irq allocated with devm_request_irq should not be freed using free_irq, because doing so causes a dangling pointer, and a subsequent double free. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/misc/cb710/core.c | 2 +- 1 file

[PATCH -next] drivers: CCI: fix the error handle in cci_pmu_probe()

2013-09-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn This patch fix the error handle of function cci_pmu_probe(): - using IS_ERR() instead of NULL test for the return value of devm_ioremap_resource() since it nerver return NULL. - remove kfree() for devm_kzalloc allocated memory - remove dev_warn

[PATCH] drm/omap: fix error return code in omap_dmm_probe()

2013-09-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the refill memory alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 1 + 1 file changed, 1

[PATCH] MFD: ucb1x00-core: fix error return code in ucb1x00_probe()

2013-09-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the irq alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/ucb1x00-core.c | 1 + 1 file changed, 1 insertion

[PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EINVAL in the version check error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/hv/connection.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH] drm/msm: fix potential NULL pointer dereference

2013-09-11 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to 'pdata' should be moved below the NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/msm/msm_gpu.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm

Re: [PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-11 Thread Wei Yongjun
On 09/12/2013 04:03 AM, KY Srinivasan wrote: -Original Message- From: Wei Yongjun [mailto:weiyj...@gmail.com] Sent: Wednesday, September 11, 2013 4:20 AM To: KY Srinivasan; Haiyang Zhang Cc: yongjun_...@trendmicro.com.cn; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org

[PATCH] drm/msm: fix return value check in ERR_PTR()

2013-09-10 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function drm_prime_pages_to_sg() 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/gpu/drm

[PATCH] VMCI: fix to pass correct device identity to free_irq()

2013-09-06 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn free_irq() expects the same device identity that was passed to corresponding request_irq(), otherwise the IRQ is not freed. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/misc/vmw_vmci/vmci_guest.c | 2 +- 1 file changed, 1

[PATCH] drm/vmwgfx: fix error return code in vmw_driver_load()

2013-08-26 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the fence manager 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/vmwgfx/vmwgfx_drv.c | 4 +++- 1 file changed, 3

[PATCH] vme: vme_ca91cx42.c: fix to pass correct device identity to free_irq()

2013-08-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn free_irq() expects the same device identity that was passed to corresponding request_irq(), otherwise the IRQ is not freed. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/vme/bridges/vme_ca91cx42.c | 6 +- 1 file changed

[PATCH -next v2] drm/rcar-du: remove redundant dev_err call in rcar_du_lvdsenc_get_resources()

2013-08-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Also remove the dev_err call to avoid redundant error message and check

[PATCH -next] block: fix error return code in parse_parts()

2013-08-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EINVAL in the parts parse error handling case instead of 0(may overwrite to 0 by parse_subpart()), as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- block/cmdline-parser.c | 1 + 1 file

[PATCH -next] dma: cppi41: fix error return code in cppi41_dma_probe()

2013-08-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EINVAL in the irq parse and map error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/cppi41.c | 4 +++- 1 file changed, 3 insertions(+), 1

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

2013-08-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the kset create and add error handling case instead of 0, as done elsewhere in this function. Introduced by commit b59d0bae6ca30c496f298881616258f9cde0d9c6. (f2fs: add sysfs support for controlling the gc_thread) Signed

[PATCH -next] drm/rcar-du: fix return value check in rcar_du_lvdsenc_get_resources()

2013-08-22 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Also remove the dev_err call to avoid redundant error message. Signed-off

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

2013-08-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/pxa/mmp-pcm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/pxa/mmp-pcm.c b/sound/soc/pxa/mmp-pcm.c index 9a97843..8235e23 100644

[PATCH -next] mm/page_alloc.c: remove duplicated include from page_alloc.c

2013-08-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- mm/page_alloc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/page_alloc.c b/mm/page_alloc.c index efb2ffa..4751901 100644 --- a/mm/page_alloc.c +++ b

[PATCH -next] ASoC: rcar: fix return value check in rsnd_gen1_probe()

2013-07-29 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(), and also remove the dev_err call to avoid redundant error message. Signed-off

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

2013-07-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/x86/platform/ce4100/ce4100.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/platform/ce4100/ce4100.c b/arch/x86/platform/ce4100/ce4100.c

[PATCH -next] staging: comedi: dt9812: remove duplicated include from dt9812.c

2013-07-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/comedi/drivers/dt9812.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/dt9812.c b/drivers/staging/comedi

[PATCH -next] staging: gdm724x: remove duplicated include from gdm_lte.c

2013-07-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/gdm724x/gdm_lte.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/gdm724x/gdm_lte.c b/drivers/staging/gdm724x/gdm_lte.c

[PATCH -next] staging: xillybus: remove duplicated include from xillybus_core.c

2013-07-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/xillybus/xillybus_core.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/xillybus/xillybus_core.c b/drivers/staging

[PATCH] drm/cirrus: fix error handling in cirrus_device_init()

2013-07-27 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix the error handling in function cirrus_device_init() to avoid resources leak in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/cirrus/cirrus_main.c | 19 --- 1 file changed

[PATCH -next] staging: gdm724x: use GFP_ATOMIC under spin lock

2013-07-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn A spin lock is taken here so we should use GFP_ATOMIC. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/gdm724x/gdm_mux.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gdm724x

[PATCH] ASoC: imx-sgtl5000: fix error return code in imx_sgtl5000_probe()

2013-07-16 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 --- sound/soc/fsl/imx-sgtl5000.c | 4 +++- 1 file changed, 3 insertions

[PATCH -next] clk: sunxi: fix return value check

2013-07-16 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function clk_register_composite() and clk_register_factors() 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

[PATCH] pch_dma: fix error return code in pch_dma_probe()

2013-07-16 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENODEV when no proper base address found error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/pch_dma.c | 1 + 1 file changed, 1 insertion

[PATCH -next] mtd: orion_nand: convert to use devm_* APIs

2013-07-16 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Convert to use devm_* APIs to avoid resources leak on error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mtd/nand/orion_nand.c | 29 + 1 file changed, 9 insertions(+), 20 deletions

[PATCH RESEND] PM / AVS: SmartReflex: remove redundant dev_err call in omap_sr_probe()

2013-07-11 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn Acked-by: Nishanth Menon n...@ti.com --- cc to linux-pm with Nishanth Menon's ACK --- drivers/power/avs

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

2013-07-04 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function c2port_device_register() 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/misc

[PATCH] ncpfs: fix error return code in ncp_parse_options()

2013-07-04 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -EINVAL from the option parse error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/ncpfs/inode.c | 12 +--- 1 file changed, 9 insertions(+), 3

[PATCH] ARM: u300: fix return value check in __u300_init_boardpower()

2013-07-04 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function syscon_node_to_regmap() 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 --- arch/arm/mach

[PATCH] ASoC: db1200: add .owner to struct snd_soc_card

2013-07-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/au1x/db1200.c | 4 1 file changed, 4 insertions(+) diff

[PATCH] ASoC: mop500: add .owner to struct snd_soc_card

2013-07-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/ux500/mop500.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH] ASoC: imx_mc13783: add .owner to struct snd_soc_card

2013-07-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/fsl/imx-mc13783.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH] ASoC: brownstone: add .owner to struct snd_soc_card

2013-07-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/pxa/brownstone.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH] ASoC: ttc_dkb: add .owner to struct snd_soc_card

2013-07-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add missing .owner of struct snd_soc_card. This prevents the module from being removed from underneath its users. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/pxa/ttc-dkb.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH -next RESEND] pinctrl: st: fix return value check

2013-06-28 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. The function syscon_regmap_lookup_by_phandle() returns ERR_PTR() and never returns NULL

[PATCH -next] pinctrl: st: fix return value check

2013-06-26 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. The function syscon_regmap_lookup_by_phandle() returns ERR_PTR() and never returns NULL

[PATCH -next] pinctrl: vt8500: wmt: remove redundant dev_err call in wmt_pinctrl_probe()

2013-06-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/pinctrl/vt8500/pinctrl-wmt.c | 4 +--- 1 file changed

[PATCH -next] PM / AVS: SmartReflex: remove redundant dev_err call in omap_sr_probe()

2013-06-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/power/avs/smartreflex.c | 4 +--- 1 file changed, 1

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

2013-06-25 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/common/edma.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index a432e6c..39ad030 100644

[PATCH -next] ASoC: mid-x86: Convert to use devm_* APIs

2013-06-24 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn devm_* APIs are device managed and make code simpler. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/mid-x86/mfld_machine.c | 29 ++--- 1 file changed, 10 insertions(+), 19 deletions(-) diff --git

[PATCH -next] rsxx: fix error return code in rsxx_pci_probe()

2013-06-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the create_singlethread_workqueue() error handling case instead of 0, as done elsewhere in this function. Introduced by commit a3299ab18591d36ad5622f5064619123c439b779. (rsxx: Individual workqueues for interruptible events

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

2013-06-19 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/kernel/process.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/kernel/process.c b/arch/arm/kernel/process.c index 7591df5..f89ff80

[PATCH -next] drm/i915: fix potential NULL pointer dereference in i915_gem_context_get_hang_stats()

2013-06-19 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 --- drivers/gpu/drm/i915/i915_gem_context.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915

[PATCH -next] Staging: netlogic: fix missing free_netdev() on error in xlr_net_probe()

2013-06-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix missing free_netdev() before return from function xlr_net_probe() in the devm_ioremap_resource() error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- no compile test. --- drivers/staging/netlogic/xlr_net.c | 6

[PATCH] ALSA: firewire: fix error return code in scs_probe()

2013-06-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the kmalloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/firewire/scs1x.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion

[PATCH -next v2] Staging: netlogic: fix missing free_netdev() on error in xlr_net_probe()

2013-06-18 Thread Wei Yongjun
Fix missing free_netdev() before return from function xlr_net_probe() in the devm_ioremap_resource() error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- v1 - v2: remove redundant error message. --- drivers/staging/netlogic/xlr_net.c | 5 ++--- 1 file changed, 2

[PATCH] vme: vme_tsi148.c: fix error return code in tsi148_probe()

2013-06-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the tsi148_crcsr_init() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/vme/bridges/vme_tsi148.c | 3 ++- 1 file

[PATCH -next] mfd: htc-egpio: use devm_ioremap_nocache() instead of ioremap_nocache()

2013-06-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Replace probe-time ioremap_nocache() call with devm_ioremap_nocache() to avoid iounmap() missing and get rid of the corresponding iounmap() call on remove. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/htc-egpio.c | 4

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

2013-06-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/dlm/lowcomms.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 4f539dd..d90909e 100644 --- a/fs/dlm

[PATCH -next] pinctrl: core: fix missing unlock on error in pinctrl_find_gpio_range_from_pin()

2013-06-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing unlock before return from function pinctrl_find_gpio_range_from_pin() in the error handling case. Introduced by commit 2ff3477efd7086544b9e298fc63afab0645921b4. (pinctrl: add pin list based GPIO ranges) Signed-off-by: Wei Yongjun

[PATCH] staging: tidspbridge: fix error return code in omap34_xx_bridge_probe()

2013-06-06 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the class_create() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/tidspbridge/rmgr/drv_interface.c | 1 + 1

[PATCH] memstick: fix error return code in r592_probe()

2013-06-06 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/memstick/host/r592.c | 11 +++ 1 file changed, 7

[PATCH] pcmcia: pd6729: fix error return code in pd6729_pci_probe()

2013-06-06 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/pcmcia/pd6729.c | 2 ++ 1 file changed, 2 insertions(+) diff

[PATCH] UBIFS: fix error return code in ubifs_remount_rw() and ubifs_fill_super()

2013-06-06 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the kmalloc() and d_make_root() error handling case instead of 0, as done elsewhere in those functions. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/ubifs/super.c | 8 ++-- 1 file changed, 6

[PATCH -next] staging: octeon-usb: remove duplicated include from octeon-hcd.c

2013-06-05 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/octeon-usb/octeon-hcd.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon

[PATCH] pch_phub: fix error return code in pch_phub_probe()

2013-06-04 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Error code had been assigned to retval in some error handling case but never use it, 0 is always returned in those case, this patch fix them to return a negative error code. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/misc

[PATCH -next] staging/lustre: fix return value check in libcfs_sock_ioctl()

2013-06-02 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function sock_alloc_file() 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 --- no compile test

[PATCH] xen-pciback: fix error return code in pcistub_irq_handler_switch()

2013-05-31 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOENT in the pcistub_device_find() and pci_get_drvdata() error handling case instead of 0(overwrite to 0 by str_to_slot()), as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers

[PATCH] pcie-gadget-spear: fix error return code in spear_pcie_gadget_probe()

2013-05-31 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the clk_get_sys() and clk_enable() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/misc/spear13xx_pcie_gadget.c | 8

[PATCH] ALSA: sis7019: fix error return code in sis_chip_create()

2013-05-30 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the pci_set_dma_mask() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/pci/sis7019.c | 3 ++- 1 file changed, 2

Re: [PATCH -next v2] dmaengine: ste_dma40: fix error return code in d40_probe()

2013-05-30 Thread Wei Yongjun
On 05/31/2013 02:29 AM, Andy Shevchenko wrote: On Thu, May 30, 2013 at 7:32 AM, Wei Yongjun weiyj...@gmail.com wrote: In many of the error handling case, the return value 'ret' not set and 0 will be return from d40_probe() even if error, but we should return a negative error code instead

[PATCH -next v3] dmaengine: ste_dma40: fix error return code in d40_probe()

2013-05-30 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In many of the error handling case, the return value 'ret' not set and 0 will be return from d40_probe() even if error, but we should return a negative error code instead in those error handling case. This patch fixed them, and also removed useless

[PATCH -next v2] dmaengine: ste_dma40: fix error return code in d40_probe()

2013-05-29 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In many of the error handling case, the return value 'ret' not set and 0 will be return from d40_probe() even if error, but we should return a negative error code instead in those error handling case. This patch fixed them, and also removed useless

[PATCH -next] dmaengine: ste_dma40: fix error return code in d40_probe()

2013-05-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn In many of the error handling case, the return value 'ret' not set and 0 will be return from d40_probe() even if error, but we should return a negative error code instead in those error handling case. This patch fixed them, and also removed useless

[PATCH -next] dmaengine: ste_dma40: add missing iounmap() on error in d40_probe()

2013-05-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing iounmap() before return from d40_probe() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/ste_dma40.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/dma/ste_dma40.c

[PATCH -next] drm/i915: fix error return code in init_pipe_control()

2013-05-28 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the kmap() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpu/drm/i915/intel_ringbuffer.c | 6 -- 1 file changed, 4 insertions

[PATCH -next] pinctrl: sunxi: fix error return code in sunxi_pinctrl_probe()

2013-05-23 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the devm_clk_get() error handling case instead of 0, as done elsewhere in this function. Introduced by commit 950707c0eb5c7aeaa2c446a04c824f4be686d2f6 (pinctrl: sunxi: add clock support) Signed-off-by: Wei

[PATCH -next] gpio-ml-ioh: fix error return code in ioh_gpio_probe()

2013-05-21 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code in the irq descs alloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/gpio/gpio-ml-ioh.c | 1 + 1 file changed, 1

[PATCH -next] staging: btmtk_usb: remove unused including linux/version.h

2013-05-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove including linux/version.h that don't need it. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/btmtk_usb/btmtk_usb.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/btmtk_usb/btmtk_usb.c b

[PATCH] ipmi: add missing platform_device_put() on error in try_smi_init()

2013-05-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing platform_device_put() before return from try_smi_init() in the platform_device_add() fail case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/char/ipmi/ipmi_si_intf.c | 1 + 1 file changed, 1 insertion

[PATCH] ARM: etm: add missing misc_deregister() on probe error and remove

2013-05-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing misc_deregister() on probe error and remove. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/arm/kernel/etm.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/arm/kernel/etm.c b/arch

[PATCH -next] staging: lustre: fix to use list_for_each_entry_safe() when delete items

2013-05-17 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/staging

[PATCH -next] UBI: fix error return code in ubi_init()

2013-05-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the kmem_cache_create() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mtd/ubi/build.c | 4 +++- 1 file changed, 3 insertions

[PATCH -next] staging: wlags49_h2: fix error return code in wl_adapter_insert()

2013-05-17 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn When return from wl_adapter_insert() in the register_netdev() error handling case, 'ret' which is 0 is returned, but we should return a negative error code instead, so fix to return the return value of register_netdev(). Introduce by commit

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

2013-05-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 done elsewhere in this function. Change function exynos_plane_init() to return ERR_PTR(), so the caller can use the error code from it. Signed-off-by: Wei Yongjun

[PATCH] spi: ep93xx: fix error return code in ep93xx_spi_probe()

2013-05-15 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the workqueue create error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/spi/spi-ep93xx.c | 1 + 1 file changed, 1 insertion(+) diff

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

[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

<    1   2   3   4   5   6   7   >