[PATCH -next] mm/slab: use list_move instead of list_del/list_add

From: Wei Yongjun Using list_move() instead of list_del() + list_add(). Signed-off-by: Wei Yongjun --- mm/slab.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/mm/slab.c b/mm/slab.c index 32c2296..cc6d816 100644 --- a/mm/slab.c +++ b/mm/slab.c @@ -3452,8 +3452,7

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

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 | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/x86/platform/ce4100/ce4100.c b/arch/x86/platform/c

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

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/x86/platform/ce4100/ce4100.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/x86/platform/ce4100/ce4100.c b/arch/x86/platform/ce4100/ce4100.c index 7ab4d3f..b27bccd 100644 --- a/arch/x86/platform

[PATCH -next] drm/arc: Remove redundant dev_err call in arcpgu_load()

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/gpu/drm/arc/arcpgu_drv.c | 4 +--- 1

[PATCH -next] drm/arc: Remove redundant dev_err call in arcpgu_load()

From: 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 --- drivers/gpu/drm/arc/arcpgu_drv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm

[PATCH -next] drm/amdgpu: use list_move instead of list_del/list_add

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/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[PATCH -next] x86, apic: Remove duplicated include from probe_64.c

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Remove duplicated include. Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- arch/x86/kernel/apic/probe_64.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/kernel/apic/probe_64.c b/arch/x86/kernel/apic/prob

[PATCH -next] drm/amdgpu: use list_move instead of list_del/list_add

From: Wei Yongjun Using list_move() instead of list_del() + list_add(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c

[PATCH -next] x86, apic: Remove duplicated include from probe_64.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/x86/kernel/apic/probe_64.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/kernel/apic/probe_64.c b/arch/x86/kernel/apic/probe_64.c index d4880a3..c303054 100644 --- a/arch/x86/kernel/apic/probe_64.c

[PATCH -next] mtd: atmel-quadspi: remove redundant dev_err call in atmel_qspi_probe()

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/mtd/spi-nor/atmel-quadspi.c | 2 -- 1

[PATCH -next] mtd: atmel-quadspi: remove redundant dev_err call in atmel_qspi_probe()

From: 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 --- drivers/mtd/spi-nor/atmel-quadspi.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/mtd/spi-nor/atmel

[PATCH -next] drm/rockchip: analogix_dp: add missing clk_disable_unprepare() on error

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Add the missing clk_disable_unprepare() before return in the error handling case. Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 ++ 1 file changed, 2 insertions(+)

[PATCH -next] drm/rockchip: analogix_dp: add missing clk_disable_unprepare() on error

From: Wei Yongjun Add the missing clk_disable_unprepare() before return in the error handling case. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c b/drivers

[PATCH -next] drm/rockchip: analogix_dp: remove .owner field for driver

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Remove .owner field if calls are used which set it automatically. Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- drivers/gpu/drm/rockchip/analogix_d

[PATCH -next] drm/hisilicon: Fix error handling of ade_power_up()

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Fix the reset_control_deassert() fail and clk_prepare_enable() fail error handling of ade_power_up(). Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 10 -- 1

[PATCH -next] drm/rockchip: analogix_dp: remove .owner field for driver

From: Wei Yongjun Remove .owner field if calls are used which set it automatically. Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Wei Yongjun --- drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu

[PATCH -next] drm/hisilicon: Fix error handling of ade_power_up()

From: Wei Yongjun Fix the reset_control_deassert() fail and clk_prepare_enable() fail error handling of ade_power_up(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm

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

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Remove duplicated include. Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- drivers/hwtracing/coresight/coresight-etm4x.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/hwtracing/coresight/coresight-etm4x.

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

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/hwtracing/coresight/coresight-etm4x.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/hwtracing/coresight/coresight-etm4x.c b/drivers/hwtracing/coresight/coresight-etm4x.c index 0d8ebfa..6e639b9

[PATCH -next] ARM: ux500: remove duplicated include from cpu-db8500.c

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Remove duplicated include. Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- arch/arm/mach-ux500/cpu-db8500.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/

[PATCH -next] ARM: ux500: remove duplicated include from cpu-db8500.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-ux500/cpu-db8500.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c index 46b1da1..c0c36b97 100644 --- a/arch/arm/mach-ux500/cpu

[tip:locking/core] locking/static_keys: Fix non static symbol Sparse warning

Commit-ID: 885885f6b88d22f81e67ee6a61561e480b27d27a Gitweb: http://git.kernel.org/tip/885885f6b88d22f81e67ee6a61561e480b27d27a Author: Wei Yongjun <yongjun_...@trendmicro.com.cn> AuthorDate: Fri, 17 Jun 2016 17:19:40 + Committer: Ingo Molnar <mi...@kernel.org> CommitDate

[tip:locking/core] locking/static_keys: Fix non static symbol Sparse warning

Commit-ID: 885885f6b88d22f81e67ee6a61561e480b27d27a Gitweb: http://git.kernel.org/tip/885885f6b88d22f81e67ee6a61561e480b27d27a Author: Wei Yongjun AuthorDate: Fri, 17 Jun 2016 17:19:40 + Committer: Ingo Molnar CommitDate: Thu, 7 Jul 2016 09:06:46 +0200 locking/static_keys: Fix non

Re: [PATCH -next] memory: atmel-ebi: use PTR_ERR_OR_ZERO() to simplify the code

Hi, On 07/06/2016 08:18 PM, Alexandre Belloni wrote: > Hi, > > On 06/07/2016 at 12:08:05 +, weiyj...@163.com wrote : >> From: Wei Yongjun <yongjun_...@trendmicro.com.cn> >> >> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. >> > I'm guessi

Re: [PATCH -next] memory: atmel-ebi: use PTR_ERR_OR_ZERO() to simplify the code

Hi, On 07/06/2016 08:18 PM, Alexandre Belloni wrote: > Hi, > > On 06/07/2016 at 12:08:05 +, weiyj...@163.com wrote : >> From: Wei Yongjun >> >> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. >> > I'm guessing you found that using coccinnelle

[PATCH -next] clocksource: keystone: Fix return value check in keystone_timer_init()

From: Wei Yongjun In case of error, function of_clk_get() 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 --- drivers/clocksource/timer-keystone.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] clocksource: keystone: Fix return value check in keystone_timer_init()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, function of_clk_get() 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/clocksource/timer

[tip:x86/kaslr] x86, kaslr: Remove unused including

Commit-ID: 19259943f0954dcd1817f94776376bf51c6a46d5 Gitweb: http://git.kernel.org/tip/19259943f0954dcd1817f94776376bf51c6a46d5 Author: Wei Yongjun AuthorDate: Sat, 7 Dec 2013 21:02:36 +0800 Committer: H. Peter Anvin CommitDate: Tue, 14 Jan 2014 10:45:56 -0800 x86, kaslr: Remove unused

[tip:x86/kaslr] x86, kaslr: Remove unused including linux/ version.h

Commit-ID: 19259943f0954dcd1817f94776376bf51c6a46d5 Gitweb: http://git.kernel.org/tip/19259943f0954dcd1817f94776376bf51c6a46d5 Author: Wei Yongjun yongjun_...@trendmicro.com.cn AuthorDate: Sat, 7 Dec 2013 21:02:36 +0800 Committer: H. Peter Anvin h...@linux.intel.com CommitDate: Tue, 14

[PATCH -next] ion: Fix sparse non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: drivers/staging/android/ion/ion_dummy_driver.c:26:19: warning: symbol 'idev' was not declared. Should it be static? drivers/staging/android/ion/ion_dummy_driver.c:27:17: warning: symbol 'heaps' was not declared. Should it be static

[PATCH -next] ion: Fix sparse non static symbol warnings

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warnings: drivers/staging/android/ion/ion_dummy_driver.c:26:19: warning: symbol 'idev' was not declared. Should it be static? drivers/staging/android/ion/ion_dummy_driver.c:27:17: warning: symbol 'heaps

[PATCH -next] ASoC: wm_adsp: Fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: sound/soc/codecs/wm_adsp.c:1500:6: warning: symbol 'wm_adsp2_boot_work' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- sound/soc/codecs/wm_adsp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] ASoC: wm_adsp: Fix sparse non static symbol warning

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warning: sound/soc/codecs/wm_adsp.c:1500:6: warning: symbol 'wm_adsp2_boot_work' was not declared. Should it be static? Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/codecs/wm_adsp.c | 2 +- 1

[PATCH -next] ASoC: ux500: Fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: sound/soc/ux500/ux500_msp_i2s.c:649:5: warning: symbol 'ux500_msp_i2s_of_init_msp' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- sound/soc/ux500/ux500_msp_i2s.c | 6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH -next] ASoC: ux500: Fix sparse non static symbol warning

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warning: sound/soc/ux500/ux500_msp_i2s.c:649:5: warning: symbol 'ux500_msp_i2s_of_init_msp' was not declared. Should it be static? Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- sound/soc/ux500

[PATCH -next] drivers/rtc/rtc-isl12057.c: remove duplicated include

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/rtc/rtc-isl12057.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rtc/rtc-isl12057.c b/drivers/rtc/rtc-isl12057.c index 7854a65..7e5ead9 100644 --- a/drivers/rtc/rtc-isl12057.c +++ b/drivers/rtc

[PATCH -next] mtd: nuc900_nand: remove redundant return value check of platform_get_resource()

From: Wei Yongjun Remove unneeded error handling on the result of a call to platform_get_resource() when the value is passed to devm_ioremap_resource(). Signed-off-by: Wei Yongjun --- drivers/mtd/nand/nuc900_nand.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/mtd/nand

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

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/x86/xen/enlighten.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index a4e2f30..b6d61c3 100644 --- a/arch/x86/xen/enlighten.c +++ b/arch/x86/xen

[PATCH -next] xen/evtchn_fifo: fix error return code in evtchn_fifo_setup()

From: Wei Yongjun Fix to return -ENOMEM from the error handling case instead of 0 (overwrited to 0 by the HYPERVISOR_event_channel_op call), otherwise the error condition cann't be reflected from the return value. Signed-off-by: Wei Yongjun --- drivers/xen/events/events_fifo.c | 6 -- 1

[PATCH -next] isp1704_charger: remove useless check in isp1704_charger_probe()

From: Wei Yongjun Neither devm_usb_get_phy_by_phandle() nor devm_usb_get_phy() can return a NULL result, so remove the useless !isp->phy check. Signed-off-by: Wei Yongjun --- drivers/power/isp1704_charger.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/power/isp1704_charger.

[PATCH -next] xen-platform: fix error return code in platform_pci_init()

From: Wei Yongjun Fix to return a negative error code from the error handling case instead of 0, otherwise the error condition cann't be reflected from the return value. Signed-off-by: Wei Yongjun --- drivers/xen/platform-pci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

[PATCH -next] misc: genwqe: fix return value check in genwqe_device_create()

From: Wei Yongjun In case of error, the function device_create_with_groups() 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 --- drivers/misc/genwqe/card_dev.c | 4 ++-- 1 file changed, 2 insertions

[PATCH -next] misc: genwqe: fix return value check in genwqe_device_create()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function device_create_with_groups() 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

[PATCH -next] xen-platform: fix error return code in platform_pci_init()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return a negative error code from the error handling case instead of 0, otherwise the error condition cann't be reflected from the return value. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/xen/platform-pci.c | 3

[PATCH -next] isp1704_charger: remove useless check in isp1704_charger_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Neither devm_usb_get_phy_by_phandle() nor devm_usb_get_phy() can return a NULL result, so remove the useless !isp-phy check. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/power/isp1704_charger.c | 2 -- 1 file changed, 2

[PATCH -next] xen/evtchn_fifo: fix error return code in evtchn_fifo_setup()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM from the error handling case instead of 0 (overwrited to 0 by the HYPERVISOR_event_channel_op call), otherwise the error condition cann't be reflected from the return value. Signed-off-by: Wei Yongjun yongjun_

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

From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- arch/x86/xen/enlighten.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index a4e2f30..b6d61c3 100644

[PATCH -next] mtd: nuc900_nand: remove redundant return value check of platform_get_resource()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove unneeded error handling on the result of a call to platform_get_resource() when the value is passed to devm_ioremap_resource(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mtd/nand/nuc900_nand.c | 3 --- 1 file

[PATCH -next] drivers/rtc/rtc-isl12057.c: remove duplicated include

From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/rtc/rtc-isl12057.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rtc/rtc-isl12057.c b/drivers/rtc/rtc-isl12057.c index 7854a65..7e5ead9

[PATCH -next] Staging: silicom: fix sparse non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: drivers/staging/silicom/bypasslib/bypass.c:528:12: warning: symbol 'init_lib_module' was not declared. Should it be static? drivers/staging/silicom/bypasslib/bypass.c:534:13: warning: symbol 'cleanup_lib_module' was not declared. Should

[PATCH -next] Staging: silicom: fix sparse non static symbol warnings

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warnings: drivers/staging/silicom/bypasslib/bypass.c:528:12: warning: symbol 'init_lib_module' was not declared. Should it be static? drivers/staging/silicom/bypasslib/bypass.c:534:13: warning: symbol

[PATCH -next] misc: genwqe: remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- drivers/misc/genwqe/card_sysfs.c | 1 - drivers/misc/genwqe/card_base.h | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/misc/genwqe/card_sysfs.c b/drivers/misc/genwqe/card_sysfs.c index

[PATCH -next] misc: genwqe: remove unused including linux/version.h

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/misc/genwqe/card_sysfs.c | 1 - drivers/misc/genwqe/card_base.h | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers

[PATCH -next] mfd: twl6040: fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: drivers/mfd/twl6040.c:89:20: warning: symbol 'twl6040_patch' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/mfd/twl6040.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd

[PATCH -next] mfd: twl6040: fix sparse non static symbol warning

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warning: drivers/mfd/twl6040.c:89:20: warning: symbol 'twl6040_patch' was not declared. Should it be static? Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/twl6040.c | 2 +- 1 file changed, 1

[PATCH] staging: lustre: fix return value check in capa_hmac()

From: Wei Yongjun In case of error, the function crypto_alloc_hash() 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 --- drivers/staging/lustre/lustre/obdclass/capa.c | 4 ++-- 1 file changed, 2

[PATCH] staging: sep: add missing destroy_workqueue() in sep_crypto.c

From: Wei Yongjun Add the missing destroy_workqueue() before return from sep_crypto_setup() and sep_crypto_takedown(). Signed-off-by: Wei Yongjun --- drivers/staging/sep/sep_crypto.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/sep/sep_crypto.c b/drivers/staging/sep

[PATCH -next] mfd: rtsx: fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: drivers/mfd/rtl8411.c:473:6: warning: symbol 'rtl8411_init_common_params' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/mfd/rtl8411.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] mfd: rtsx: fix sparse non static symbol warning

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warning: drivers/mfd/rtl8411.c:473:6: warning: symbol 'rtl8411_init_common_params' was not declared. Should it be static? Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/rtl8411.c | 2 +- 1

[PATCH] staging: sep: add missing destroy_workqueue() in sep_crypto.c

From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing destroy_workqueue() before return from sep_crypto_setup() and sep_crypto_takedown(). Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/sep/sep_crypto.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH] staging: lustre: fix return value check in capa_hmac()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function crypto_alloc_hash() 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] drm/i915: fix return value check of debugfs_create_file()

From: Wei Yongjun In case of error, the function debugfs_create_file() returns NULL pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/i915_debugfs.c | 12

[PATCH -next] staging: xillybus: fix error return code in xilly_probe()

From: Wei Yongjun Fix to return negative error code -EIO from the error handling case instead of 0. Signed-off-by: Wei Yongjun --- drivers/staging/xillybus/xillybus_pcie.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/xillybus/xillybus_pcie.c b/drivers

[PATCH -next] staging: xillybus: fix error return code in xilly_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return negative error code -EIO from the error handling case instead of 0. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/xillybus/xillybus_pcie.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH -next] drm/i915: fix return value check of debugfs_create_file()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function debugfs_create_file() returns NULL pointer not ERR_PTR() if debugfs is enabled. The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

[PATCH -next] ARM: pxa: fix return value check in em_x270_sensor_init()

From: Wei Yongjun In case of error, the function regulator_get() 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 --- arch/arm/mach-pxa/em-x270.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH -next] ARM: pxa: fix return value check in em_x270_sensor_init()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function regulator_get() 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-pxa/em

[PATCH -next] x86, kaslr: remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- arch/x86/boot/compressed/aslr.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/boot/compressed/aslr.c b/arch/x86/boot/compressed/aslr.c index 84be175..90a21f4 100644 --- a/arch/x86/boot

[PATCH -next] x86, kaslr: remove unused including linux/version.h

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 --- arch/x86/boot/compressed/aslr.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/boot/compressed/aslr.c b/arch/x86/boot

[PATCH -next] sysfs, kernfs: remove duplicated include from file.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- fs/kernfs/file.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/kernfs/file.c b/fs/kernfs/file.c index 990c97f..4a5863b 100644 --- a/fs/kernfs/file.c +++ b/fs/kernfs/file.c @@ -13,7 +13,6 @@ #include

[PATCH -next] mfd: sta2x11-mfd: fix return value check in sta2x11_mfd_platform_probe()

From: Wei Yongjun In case of error, the function devm_regmap_init_mmio() 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 --- drivers/mfd/sta2x11-mfd.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH -next] mfd: sec: fix sparse NULL pointer warning

From: Wei Yongjun Fixes the following sparse warning: drivers/mfd/sec-core.c:202:16: warning: Using plain integer as NULL pointer Signed-off-by: Wei Yongjun --- drivers/mfd/sec-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/sec-core.c b/drivers/mfd/sec

[PATCH -next] mfd: sec: fix sparse NULL pointer warning

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fixes the following sparse warning: drivers/mfd/sec-core.c:202:16: warning: Using plain integer as NULL pointer Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mfd/sec-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] mfd: sta2x11-mfd: fix return value check in sta2x11_mfd_platform_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function devm_regmap_init_mmio() 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/mfd

[PATCH -next] sysfs, kernfs: remove duplicated include from file.c

From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- fs/kernfs/file.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/kernfs/file.c b/fs/kernfs/file.c index 990c97f..4a5863b 100644 --- a/fs/kernfs/file.c

[PATCH -next] platform/chrome: unregister platform driver/device when module exit

From: Wei Yongjun We have registered platform driver and device when module init, and need unregister them when module exit. Signed-off-by: Wei Yongjun --- drivers/platform/chrome/chromeos_laptop.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/platform/chrome

[PATCH -next] staging: rts5208: fix error return code in rtsx_probe()

From: Wei Yongjun Fix to return -ENOMEM instead of 0 when the memory alloc fail in probe error handling path. Signed-off-by: Wei Yongjun --- drivers/staging/rts5208/rtsx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rts5208/rtsx.c b/drivers/staging

[PATCH -next] staging: rts5208: fix error return code in rtsx_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM instead of 0 when the memory alloc fail in probe error handling path. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/staging/rts5208/rtsx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

[PATCH -next] platform/chrome: unregister platform driver/device when module exit

From: Wei Yongjun yongjun_...@trendmicro.com.cn We have registered platform driver and device when module init, and need unregister them when module exit. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/platform/chrome/chromeos_laptop.c | 3 +++ 1 file changed, 3 insertions

[PATCH] dma: k3dma: add missing clk_disable_unprepare() on error in k3_dma_probe()

From: Wei Yongjun Add the missing clk_disable_unprepare() before return from k3_dma_probe() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/dma/k3dma.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/dma/k3dma.c b/drivers/dma/k3dma.c index

[PATCH] dma: k3dma: add missing clk_disable_unprepare() on error in k3_dma_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Add the missing clk_disable_unprepare() before return from k3_dma_probe() in the error handling case. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/dma/k3dma.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

Re: [PATCH] drivers: replace IS_ERR and PTR_ERR with PTR_ERR_OR_ZERO

On 11/11/2013 04:38 PM, Duan Jiong wrote: > 于 2013年11月11日 15:43, Wei Yongjun 写道: >> On 11/01/2013 06:09 PM, Duan Jiong wrote: >>> This patch fixes coccinelle error regarding usage of IS_ERR and >>> PTR_ERR instead of PTR_ERR_OR_ZERO. >&g

Re: [PATCH] drivers: replace IS_ERR and PTR_ERR with PTR_ERR_OR_ZERO

On 11/11/2013 04:38 PM, Duan Jiong wrote: 于 2013年11月11日 15:43, Wei Yongjun 写道: On 11/01/2013 06:09 PM, Duan Jiong wrote: This patch fixes coccinelle error regarding usage of IS_ERR and PTR_ERR instead of PTR_ERR_OR_ZERO. Signed-off-by: Duan Jiong duanj.f...@cn.fujitsu.com

Re: [PATCH] drivers: replace IS_ERR and PTR_ERR with PTR_ERR_OR_ZERO

On 11/01/2013 06:09 PM, Duan Jiong wrote: > This patch fixes coccinelle error regarding usage of IS_ERR and > PTR_ERR instead of PTR_ERR_OR_ZERO. > > Signed-off-by: Duan Jiong > --- > drivers/misc/carma/carma-fpga.c | 5 + > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git

[PATCH] MIPS: Alchemy: add missing platform_set_drvdata() in au1550nd_probe()

From: Wei Yongjun Add missing platform_set_drvdata() in au1550nd_probe(), otherwise calling platform_get_drvdata() in remove returns NULL. Signed-off-by: Wei Yongjun --- drivers/mtd/nand/au1550nd.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/mtd/nand/au1550nd.c b/drivers/mtd

[PATCH] MIPS: Alchemy: add missing platform_set_drvdata() in au1550nd_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn Add missing platform_set_drvdata() in au1550nd_probe(), otherwise calling platform_get_drvdata() in remove returns NULL. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/mtd/nand/au1550nd.c | 2 ++ 1 file changed, 2 insertions

Re: [PATCH] drivers: replace IS_ERR and PTR_ERR with PTR_ERR_OR_ZERO

On 11/01/2013 06:09 PM, Duan Jiong wrote: This patch fixes coccinelle error regarding usage of IS_ERR and PTR_ERR instead of PTR_ERR_OR_ZERO. Signed-off-by: Duan Jiong duanj.f...@cn.fujitsu.com --- drivers/misc/carma/carma-fpga.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-)

[PATCH] ASoC: fsl: set correct platform drvdata in pcm030_fabric_probe()

From: Wei Yongjun platform_set_drvdata(op, pdata) in pcm030_fabric_probe() will be overwrited when calling snd_soc_register_card(card), but cm030_fabric_remove() use drvdata as a type of struct pcm030_audio_data, so we should move platform_set_drvdata() below snd_soc_register_card() call

[PATCH] ASoC: fsl: set correct platform drvdata in pcm030_fabric_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn platform_set_drvdata(op, pdata) in pcm030_fabric_probe() will be overwrited when calling snd_soc_register_card(card), but cm030_fabric_remove() use drvdata as a type of struct pcm030_audio_data, so we should move platform_set_drvdata() below

[PATCH -next] mtd: nand: omap: fix error return code in omap_nand_probe()

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/mtd/nand/omap2.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/nand/omap2.c b/drivers

[PATCH -next] clk: keystone: fix return value check in _of_pll_clk_init()

From: Wei Yongjun clk_register_pll() returns ERR_PTR() when memory alloc fail and NULL in the other case, the user of this function should used IS_ERR_OR_NULL() to check the return value. this patch change clk_register_pll() to return only ERR_PTR(), and also fixed the user. Signed-off-by: Wei

[PATCH -next] pinctrl: imx1: fix return value check in imx1_pinctrl_core_probe()

From: Wei Yongjun In case of error, the function devm_ioremap_nocache() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- drivers/pinctrl/pinctrl-imx1-core.c | 4 ++-- 1 file changed, 2 insertions

[PATCH -next] pinctrl: imx1: fix return value check in imx1_pinctrl_core_probe()

From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function devm_ioremap_nocache() returns NULL pointer 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

[PATCH -next] clk: keystone: fix return value check in _of_pll_clk_init()

From: Wei Yongjun yongjun_...@trendmicro.com.cn clk_register_pll() returns ERR_PTR() when memory alloc fail and NULL in the other case, the user of this function should used IS_ERR_OR_NULL() to check the return value. this patch change clk_register_pll() to return only ERR_PTR(), and also fixed

[PATCH -next] mtd: nand: omap: fix error return code in omap_nand_probe()

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/mtd/nand/omap2.c | 3 ++- 1 file changed, 2 insertions(+), 1

[PATCH -next] ARM i.MX53: remove duplicated include from clk-imx51-imx53.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-imx/clk-imx51-imx53.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c index ce37af2..2828ea65 100644 --- a/arch/arm/mach-imx

[PATCH -next] ARM: ixp4xx: remove duplicated include from dsmg600-setup.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-ixp4xx/dsmg600-setup.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-ixp4xx/dsmg600-setup.c b/arch/arm/mach-ixp4xx/dsmg600-setup.c index 736dc69..7740e13 100644 --- a/arch/arm/mach

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

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/mtd/mtdcore.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c index 7189089..92311a5 100644 --- a/drivers/mtd/mtdcore.c +++ b/drivers/mtd/mtdcore.c @@ -37,7

[PATCH -next] tifm: fix error return code in tifm_7xx1_probe()

From: Wei Yongjun Fix to return ENODEV in the pci ioremap error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/misc/tifm_7xx1.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/misc/tifm_7xx1.c b/drivers

[PATCH -next] skd: fix error return code in skd_pci_probe()

From: Wei Yongjun Fix to return -ENOMEM in the skd construct error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/block/skd_main.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/block/skd_main.c b/drivers

<    2   3   4   5   6   7   8   9   10   11   >