[PATCH -next] ASoC: Intel: Atom: add terminate entry for dmi_system_id tables

From: Wei Yongjun Make sure dmi_system_id tables are NULL terminated. Signed-off-by: Wei Yongjun --- sound/soc/intel/atom/sst/sst_acpi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/intel/atom/sst/sst_acpi.c b/sound/soc/intel/atom/sst/sst_acpi.c index ba5c0d7..0a88537 100644

[PATCH -next] dlm: fix error return code in sctp_accept_from_sock()

From: Wei Yongjun <weiyongj...@huawei.com> 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 <weiyongj...@huawei.com> --- fs/dlm/lowcomms.c | 3 ++- 1 file changed, 2 insertions(+), 1 delet

[PATCH -next] dlm: fix error return code in sctp_accept_from_sock()

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/dlm/lowcomms.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c

[PATCH -next] phy: meson8b-usb2: fix missing clk_disable_unprepare() on error

From: Wei Yongjun <weiyongj...@huawei.com> Fix the missing clk_disable_unprepare() before return from phy_meson8b_usb2_power_on() in the error handling case. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/phy/phy-meson8b-usb2.c | 3 +++ 1 file changed, 3 inserti

[PATCH -next] phy: meson8b-usb2: fix missing clk_disable_unprepare() on error

From: Wei Yongjun Fix the missing clk_disable_unprepare() before return from phy_meson8b_usb2_power_on() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/phy/phy-meson8b-usb2.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/phy/phy-meson8b-usb2.c b/drivers

[PATCH -next] drm/i915/gvt: fix return value check

From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function i915_gem_object_create() returns ERR_PTR() not NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/i915/gvt/

[PATCH -next] drm/i915/gvt: fix return value check

From: Wei Yongjun In case of error, the function i915_gem_object_create() returns ERR_PTR() not NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/gvt/cmd_parser.c | 8 1 file changed, 4 insertions(+), 4

[tip:mm/urgent] mm/numa: Remove duplicated include from mprotect.c

Commit-ID: 137baabe351e0554d06c6d5c84059fe343e2791e Gitweb: http://git.kernel.org/tip/137baabe351e0554d06c6d5c84059fe343e2791e Author: Wei Yongjun <weiyongj...@huawei.com> AuthorDate: Mon, 17 Oct 2016 15:47:39 + Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:mm/urgent] mm/numa: Remove duplicated include from mprotect.c

Commit-ID: 137baabe351e0554d06c6d5c84059fe343e2791e Gitweb: http://git.kernel.org/tip/137baabe351e0554d06c6d5c84059fe343e2791e Author: Wei Yongjun AuthorDate: Mon, 17 Oct 2016 15:47:39 + Committer: Thomas Gleixner CommitDate: Wed, 19 Oct 2016 17:28:48 +0200 mm/numa: Remove

[PATCH -next] greybus: es2: fix error return code in ap_probe()

From: Wei Yongjun <weiyongj...@huawei.com> Fix to return a negative error code from the es2_arpc_in_enable() error handling case instead of 0, as done elsewhere in this function. Fixes: 9d9d3777a9db ("greybus: es2: Add a new bulk in endpoint for APBridgeA RPC") Signed-off

[PATCH -next] greybus: es2: fix error return code in ap_probe()

From: Wei Yongjun Fix to return a negative error code from the es2_arpc_in_enable() error handling case instead of 0, as done elsewhere in this function. Fixes: 9d9d3777a9db ("greybus: es2: Add a new bulk in endpoint for APBridgeA RPC") Signed-off-by: Wei Yongjun --- drivers/stagi

Re: [PATCH -next] staging: i4l: pcbit: remove duplicated include from capi.c

Hi Alexander, On 10/18/2016 11:30 PM, Alexander Alemayhu wrote: > On Tue, Oct 18, 2016 at 02:37:10PM +0000, Wei Yongjun wrote: >> From: Wei Yongjun <weiyongj...@huawei.com> >> >> Remove duplicated include. >> > How did you discover that? Just curious if y

Re: [PATCH -next] staging: i4l: pcbit: remove duplicated include from capi.c

Hi Alexander, On 10/18/2016 11:30 PM, Alexander Alemayhu wrote: > On Tue, Oct 18, 2016 at 02:37:10PM +0000, Wei Yongjun wrote: >> From: Wei Yongjun >> >> Remove duplicated include. >> > How did you discover that? Just curious if you used any tooling. > I found

[tip:efi/core] efi: Remove unused include of

Commit-ID: 10714dd37aa7ebb165677b9a793448950ff366ad Gitweb: http://git.kernel.org/tip/10714dd37aa7ebb165677b9a793448950ff366ad Author: Wei Yongjun <weiyongj...@huawei.com> AuthorDate: Tue, 18 Oct 2016 15:33:12 +0100 Committer: Ingo Molnar <mi...@kernel.org> CommitDate: Tue,

[tip:efi/core] efi: Remove unused include of

Commit-ID: 10714dd37aa7ebb165677b9a793448950ff366ad Gitweb: http://git.kernel.org/tip/10714dd37aa7ebb165677b9a793448950ff366ad Author: Wei Yongjun AuthorDate: Tue, 18 Oct 2016 15:33:12 +0100 Committer: Ingo Molnar CommitDate: Tue, 18 Oct 2016 17:11:16 +0200 efi: Remove unused include

[PATCH -next] staging: ks7010: ks_wlan_net: Use setup_timer instead of init_timer and data fields

From: Wei Yongjun <weiyongj...@huawei.com> Use setup_timer function instead of initializing timer with the function and data fields Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/ks7010/ks_wlan_net.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)

[PATCH -next] staging: ks7010: ks_wlan_net: Use setup_timer instead of init_timer and data fields

From: Wei Yongjun Use setup_timer function instead of initializing timer with the function and data fields Signed-off-by: Wei Yongjun --- drivers/staging/ks7010/ks_wlan_net.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/staging/ks7010/ks_wlan_net.c b

[PATCH -next] staging: i4l: pcbit: drv: remove duplicated include from drv.c

From: Wei Yongjun <weiyongj...@huawei.com> Remove duplicated include. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/i4l/pcbit/drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/i4l/pcbit/drv.c b/drivers/staging/i4l/pcbit/drv.c i

[PATCH -next] staging: i4l: pcbit: drv: remove duplicated include from drv.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/staging/i4l/pcbit/drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/i4l/pcbit/drv.c b/drivers/staging/i4l/pcbit/drv.c index 87b8522..89b0b5b 100644 --- a/drivers/staging/i4l/pcbit/drv.c

[PATCH -next] staging: ks7010: convert list_for_each to entry variant

From: Wei Yongjun <weiyongj...@huawei.com> convert list_for_each() to list_for_each_entry() where applicable. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/ks7010/ks_hostif.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/st

[PATCH -next] staging: i4l: pcbit: remove duplicated include from capi.c

From: Wei Yongjun <weiyongj...@huawei.com> Remove duplicated include. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/i4l/pcbit/capi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/i4l/pcbit/capi.c b/drivers/staging/i4l/pcbit/capi.c i

[PATCH -next] staging: ks7010: remove unused including

From: Wei Yongjun <weiyongj...@huawei.com> Remove including that don't need it. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/ks7010/ks_wlan.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan.h b/drivers/staging/ks7010/ks_

[PATCH -next] staging: ks7010: convert list_for_each to entry variant

From: Wei Yongjun convert list_for_each() to list_for_each_entry() where applicable. Signed-off-by: Wei Yongjun --- drivers/staging/ks7010/ks_hostif.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.c b/drivers/staging/ks7010

[PATCH -next] staging: i4l: pcbit: remove duplicated include from capi.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/staging/i4l/pcbit/capi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/i4l/pcbit/capi.c b/drivers/staging/i4l/pcbit/capi.c index 91f2994..a6c4e00 100644 --- a/drivers/staging/i4l/pcbit

[PATCH -next] staging: ks7010: remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- drivers/staging/ks7010/ks_wlan.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan.h b/drivers/staging/ks7010/ks_wlan.h index 5851714..6981170 100644 --- a/drivers/staging

[PATCH -next] staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe()

From: Wei Yongjun <weiyongj...@huawei.com> sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: 2eb9d8cbb3c3 ("staging: rts5208: rtsx.c: Alloc sizeof struct") Signed-off-by: Wei Yongjun <weiyongj...@huawe

[PATCH -next] staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe()

From: Wei Yongjun sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: 2eb9d8cbb3c3 ("staging: rts5208: rtsx.c: Alloc sizeof struct") Signed-off-by: Wei Yongjun --- drivers/staging/rts5208/rtsx.c | 2 +- 1 file

[PATCH -next] greybus: arche-platform: Add missing of_node_put() in arche_platform_change_state()

From: Wei Yongjun <weiyongj...@huawei.com> This node pointer is returned by of_find_compatible_node() with refcount incremented in this function. of_node_put() on it before exitting this function. This is detected by Coccinelle semantic patch. Signed-off-by: Wei Yongjun <weiyongj...@h

[PATCH -next] greybus: arche-platform: Add missing of_node_put() in arche_platform_change_state()

From: Wei Yongjun This node pointer is returned by of_find_compatible_node() with refcount incremented in this function. of_node_put() on it before exitting this function. This is detected by Coccinelle semantic patch. Signed-off-by: Wei Yongjun --- drivers/staging/greybus/arche-platform.c

[PATCH -next] libnvdimm: fix error return code in __blk_label_update()

From: Wei Yongjun <weiyongj...@huawei.com> Fix to return error code -ENXIO from the nd_label_alloc_slot() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/nvdimm/label.c | 4 +++- 1 file changed,

[PATCH -next] libnvdimm: fix error return code in __blk_label_update()

From: Wei Yongjun Fix to return error code -ENXIO from the nd_label_alloc_slot() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/nvdimm/label.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/nvdimm

[PATCH -next] mm: numa: remove duplicated include from mprotect.c

From: Wei Yongjun <weiyongj...@huawei.com> Remove duplicated include. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- mm/mprotect.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/mprotect.c b/mm/mprotect.c index bcdbe62..1193652 100644 --- a/mm/mprotect.c +++ b/m

[PATCH -next] mm: numa: remove duplicated include from mprotect.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- mm/mprotect.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/mprotect.c b/mm/mprotect.c index bcdbe62..1193652 100644 --- a/mm/mprotect.c +++ b/mm/mprotect.c @@ -25,7 +25,6 @@ #include #include #include

[PATCH -next] auxdisplay: img-ascii-lcd: use setup_timer instead of init_timer and data fields

From: Wei Yongjun <weiyongj...@huawei.com> Use setup_timer function instead of initializing timer with the function and data fields Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/auxdisplay/img-ascii-lcd.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)

[PATCH -next] auxdisplay: img-ascii-lcd: use setup_timer instead of init_timer and data fields

From: Wei Yongjun Use setup_timer function instead of initializing timer with the function and data fields Signed-off-by: Wei Yongjun --- drivers/auxdisplay/img-ascii-lcd.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/auxdisplay/img-ascii-lcd.c b/drivers

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

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warning: sound/soc/rockchip/rk3399_gru_sound.c:41:14: warning: symbol 'rt5514_dmic_delay' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- sound/soc/rockchip/rk3399

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

From: Wei Yongjun Fixes the following sparse warning: sound/soc/rockchip/rk3399_gru_sound.c:41:14: warning: symbol 'rt5514_dmic_delay' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- sound/soc/rockchip/rk3399_gru_sound.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH -next] staging: bcm2708_vchiq: remove .owner field for driver

From: Wei Yongjun <weiyongj...@huawei.com> 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 <weiyongj...@huawei.com> --- drivers/staging/vc04_services/interface/vchiq_arm

[PATCH -next] staging: bcm2708_vchiq: 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/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 1 - 1 file changed, 1 deletion(-) diff --git

[PATCH -next] soc/qman_test: Use platform_device_unregister in allocate_frame_data()

From: Wei Yongjun <weiyongj...@huawei.com> platform_device_unregister() only calls platform_device_del() and platform_device_put(), thus use platform_device_unregister() to simplify the code. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soc/fsl/qbman/qman_test

[PATCH -next] soc/qman: Use platform_device_unregister in qman_destroy_portal()

From: Wei Yongjun <weiyongj...@huawei.com> platform_device_unregister() only calls platform_device_del() and platform_device_put(), thus use platform_device_unregister() to simplify the code. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soc/fsl/qbman/qman.c | 3

[PATCH -next] soc/qman_test: Use platform_device_unregister in allocate_frame_data()

From: Wei Yongjun platform_device_unregister() only calls platform_device_del() and platform_device_put(), thus use platform_device_unregister() to simplify the code. Signed-off-by: Wei Yongjun --- drivers/soc/fsl/qbman/qman_test_stash.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions

[PATCH -next] soc/qman: Use platform_device_unregister in qman_destroy_portal()

From: Wei Yongjun platform_device_unregister() only calls platform_device_del() and platform_device_put(), thus use platform_device_unregister() to simplify the code. Signed-off-by: Wei Yongjun --- drivers/soc/fsl/qbman/qman.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

[PATCH -next] soc/bman: Use resource_size instead of computation

From: Wei Yongjun <weiyongj...@huawei.com> Use resource_size function on resource object instead of explicit computation. Generated by: scripts/coccinelle/api/resource_size.cocci Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soc/fsl/qbman/bman_ccsr.c | 3 +-- 1

[PATCH -next] soc/bman: Use resource_size instead of computation

From: Wei Yongjun Use resource_size function on resource object instead of explicit computation. Generated by: scripts/coccinelle/api/resource_size.cocci Signed-off-by: Wei Yongjun --- drivers/soc/fsl/qbman/bman_ccsr.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH -next] staging: bcm2708_vchiq: fix return value check in vchiq_init_state()

From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function kthread_create() 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 <weiyongj...@huawei.com> --- drivers/staging/v

[PATCH -next] staging: bcm2708_vchiq: fix return value check in vchiq_init_state()

From: Wei Yongjun In case of error, the function kthread_create() 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/vc04_services/interface/vchiq_arm/vchiq_core.c | 6 +++--- 1

[PATCH -next] staging: bcm2708_vchiq: fix return value check in vchiq_platform_conn_state_changed()

From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function kthread_create() 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 <weiyongj...@huawei.com> --- drivers/staging/v

[PATCH -next] staging: bcm2708_vchiq: fix return value check in vchiq_platform_conn_state_changed()

From: Wei Yongjun In case of error, the function kthread_create() 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/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +- 1 file

[PATCH -next] cachefiles: Remove duplicated include from namei.c

From: Wei Yongjun <weiyongj...@huawei.com> Remove duplicated include. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- fs/cachefiles/namei.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/cachefiles/namei.c b/fs/cachefiles/namei.c index 41df8a2..f08253d 100644 --- a/f

[PATCH -next] cachefiles: Remove duplicated include from namei.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- fs/cachefiles/namei.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/cachefiles/namei.c b/fs/cachefiles/namei.c index 41df8a2..f08253d 100644 --- a/fs/cachefiles/namei.c +++ b/fs/cachefiles/namei.c @@ -20,7

[PATCH -next] irqchip/eznps: drop pointless static qualifier in nps400_of_init()

From: Wei Yongjun <weiyongj...@huawei.com> There is no need to have the 'struct irq_domain *nps400_root_domain' variable static since new value always be assigned before use it. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/irqchip/irq-eznps.c | 2 +- 1 fil

[PATCH -next] irqchip/eznps: drop pointless static qualifier in nps400_of_init()

From: Wei Yongjun There is no need to have the 'struct irq_domain *nps400_root_domain' variable static since new value always be assigned before use it. Signed-off-by: Wei Yongjun --- drivers/irqchip/irq-eznps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] auxdisplay: img-ascii-lcd: Add terminate entry for of_device_id tables

From: Wei Yongjun <weiyongj...@huawei.com> Make sure of_device_id tables are NULL terminated. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/auxdisplay/img-ascii-lcd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/auxdisplay/img-ascii-lcd.c b/driver

[PATCH -next] auxdisplay: img-ascii-lcd: Add terminate entry for of_device_id tables

From: Wei Yongjun Make sure of_device_id tables are NULL terminated. Signed-off-by: Wei Yongjun --- drivers/auxdisplay/img-ascii-lcd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/auxdisplay/img-ascii-lcd.c b/drivers/auxdisplay/img-ascii-lcd.c index bf43b5d..e37a4b0 100644

[PATCH] staging: rtl8188eu: fix double unlock error in rtw_resume_process()

r <dan.carpen...@oracle.com> Signed-off-by: Wei Yongjun <weiyj...@gmail.com> diff --git a/drivers/staging/rtl8188eu/os_dep/usb_intf.c b/drivers/staging/rtl8188eu/os_dep/usb_intf.c index b2bc09e..68e1e6b 100644 --- a/drivers/staging/rtl8188eu/os_dep/usb_intf.c +++ b/drivers/staging/rt

[PATCH] staging: rtl8188eu: fix double unlock error in rtw_resume_process()

r Signed-off-by: Wei Yongjun diff --git a/drivers/staging/rtl8188eu/os_dep/usb_intf.c b/drivers/staging/rtl8188eu/os_dep/usb_intf.c index b2bc09e..68e1e6b 100644 --- a/drivers/staging/rtl8188eu/os_dep/usb_intf.c +++ b/drivers/staging/rtl8188eu/os_dep/usb_intf.c @@ -294,8 +294,10 @@ static int rtw_r

[PATCH -next] powerpc/4xx: Fix error return code in ppc4xx_msi_probe()

From: Wei Yongjun <weiyongj...@huawei.com> Fix to return a negative error code from the ppc4xx_setup_pcieh_hw() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- arch/powerpc/sysdev/ppc4xx_msi.c | 3 ++- 1

[PATCH -next] powerpc/4xx: Fix error return code in ppc4xx_msi_probe()

From: Wei Yongjun Fix to return a negative error code from the ppc4xx_setup_pcieh_hw() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- arch/powerpc/sysdev/ppc4xx_msi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[tip:irq/core] genirq: Make function __irq_do_set_handler() static

Commit-ID: b8129a1f6aaaca02d92186acf19ceb545b4b489a Gitweb: http://git.kernel.org/tip/b8129a1f6aaaca02d92186acf19ceb545b4b489a Author: Wei Yongjun <weiyongj...@huawei.com> AuthorDate: Sun, 25 Sep 2016 15:36:39 + Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:irq/core] genirq: Make function __irq_do_set_handler() static

Commit-ID: b8129a1f6aaaca02d92186acf19ceb545b4b489a Gitweb: http://git.kernel.org/tip/b8129a1f6aaaca02d92186acf19ceb545b4b489a Author: Wei Yongjun AuthorDate: Sun, 25 Sep 2016 15:36:39 + Committer: Thomas Gleixner CommitDate: Sun, 25 Sep 2016 16:46:52 -0400 genirq: Make function

[PATCH -next] staging: most: fix error return code in audio_probe_channel()

From: Wei Yongjun <weiyongj...@huawei.com> Fix to return a negative error code from the audio_set_hw_params() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/most/aim-sound/sound.c | 3

[PATCH -next] staging: most: fix error return code in audio_probe_channel()

From: Wei Yongjun Fix to return a negative error code from the audio_set_hw_params() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/staging/most/aim-sound/sound.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH -next] genirq: Make function __irq_do_set_handler() static

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warning: kernel/irq/chip.c:786:1: warning: symbol '__irq_do_set_handler' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- kernel/irq/chip.c | 2 +- 1 file changed,

[PATCH -next] genirq: Make function __irq_do_set_handler() static

From: Wei Yongjun Fixes the following sparse warning: kernel/irq/chip.c:786:1: warning: symbol '__irq_do_set_handler' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- kernel/irq/chip.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/irq/chip.c

[PATCH -next] ipmi/bt-bmc: remove redundant return value check of platform_get_resource()

From: Wei Yongjun <weiyongj...@huawei.com> 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 <weiyongj...@huawei.com> --- drivers/char/ipmi/bt-bmc.c | 5 - 1 fil

[PATCH -next] ipmi/bt-bmc: 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/char/ipmi/bt-bmc.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/char/ipmi/bt-bmc.c

[PATCH -next] gpu: ipu-v3: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

From: Wei Yongjun <weiyongj...@huawei.com> Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). Generated by: scripts/coccinelle/api/err_cast.cocci Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/ipu-v3/ipu-image-convert.c | 2 +- 1 file changed,

[PATCH -next] gpu: ipu-v3: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

From: Wei Yongjun Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). Generated by: scripts/coccinelle/api/err_cast.cocci Signed-off-by: Wei Yongjun --- drivers/gpu/ipu-v3/ipu-image-convert.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/ipu-v3

[PATCH -next] soc: mediatek: PMIC wrap: fix missing clk_disable_unprepare() on error in pwrap_probe()

From: Wei Yongjun <weiyongj...@huawei.com> Fix the missing clk_disable_unprepare() before return from pwrap_probe() in the error handling case. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soc/mediatek/mtk-pmic-wrap.c | 3 ++- 1 file changed, 2 insertions(+)

[PATCH -next] soc: mediatek: PMIC wrap: fix missing clk_disable_unprepare() on error in pwrap_probe()

From: Wei Yongjun Fix the missing clk_disable_unprepare() before return from pwrap_probe() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/soc/mediatek/mtk-pmic-wrap.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/soc/mediatek/mtk-pmic-wrap.c

[PATCH -next] phy: rockchip-typec: fix non static symbol warnings

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warnings: drivers/phy/phy-rockchip-typec.c:295:16: warning: symbol 'usb3_pll_cfg' was not declared. Should it be static? drivers/phy/phy-rockchip-typec.c:312:16: warning: symbol 'dp_pll_cfg' was not declared.

[PATCH -next] phy: rockchip-typec: fix non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: drivers/phy/phy-rockchip-typec.c:295:16: warning: symbol 'usb3_pll_cfg' was not declared. Should it be static? drivers/phy/phy-rockchip-typec.c:312:16: warning: symbol 'dp_pll_cfg' was not declared. Should it be static? Signed-off

[PATCH] drm/omap: fix return value check in dsi_bind()

From: Wei Yongjun <weiyongj...@huawei.com> Fix the retrn value check which testing the wrong variable in dsi_bind(). Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/omapdrm/dss/dsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dri

[PATCH] drm/omap: fix return value check in dsi_bind()

From: Wei Yongjun Fix the retrn value check which testing the wrong variable in dsi_bind(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/omapdrm/dss/dsi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/dss/dsi.c b/drivers/gpu/drm/omapdrm/dss

[PATCH -next] vme: fake: remove unexpected unlock in fake_master_set()

From: Wei Yongjun <weiyongj...@huawei.com> image->lock is unlocked in some error handling path without take the lock, so remove those unexpected unlock. Fixes: 658bcdae9c67 ("vme: Adding Fake VME driver") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/v

[PATCH -next] vme: fake: remove unexpected unlock in fake_master_set()

From: Wei Yongjun image->lock is unlocked in some error handling path without take the lock, so remove those unexpected unlock. Fixes: 658bcdae9c67 ("vme: Adding Fake VME driver") Signed-off-by: Wei Yongjun --- drivers/vme/bridges/vme_fake.c | 3 --- 1 file changed, 3 deletions(-

[PATCH -next] ASoC: sti: fix missing clk_disable_unprepare() on error in uni_player_start()

From: Wei Yongjun <weiyongj...@huawei.com> Fix the missing clk_disable_unprepare() before return from uni_player_start() in the error handling case. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- sound/soc/sti/uniperif_player.c | 4 +++- 1 file changed, 3 insertions(+)

[PATCH -next] ASoC: sti: fix missing clk_disable_unprepare() on error in uni_player_start()

From: Wei Yongjun Fix the missing clk_disable_unprepare() before return from uni_player_start() in the error handling case. Signed-off-by: Wei Yongjun --- sound/soc/sti/uniperif_player.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/soc/sti/uniperif_player.c b

[PATCH -next] ASoC: rt5663: fix sparse warnings

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warnings: sound/soc/codecs/rt5663.c:1367:14: warning: duplicate const sound/soc/codecs/rt5663.c:1577:5: warning: symbol 'rt5663_button_detect' was not declared. Should it be static? sound/soc/codecs/rt5663.c:2857:24: w

[PATCH -next] ASoC: rt5663: fix sparse warnings

From: Wei Yongjun Fixes the following sparse warnings: sound/soc/codecs/rt5663.c:1367:14: warning: duplicate const sound/soc/codecs/rt5663.c:1577:5: warning: symbol 'rt5663_button_detect' was not declared. Should it be static? sound/soc/codecs/rt5663.c:2857:24: warning: symbol

[PATCH -next] drm/i915: Fix non static symbol warning

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warning: drivers/gpu/drm/i915/intel_dp.c:1527:5: warning: symbol 'intel_dp_compute_bpp' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/i915/in

[PATCH -next] drm/i915: Fix non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/i915/intel_dp.c:1527:5: warning: symbol 'intel_dp_compute_bpp' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_dp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH -next] memory: atmel-sdramc: use builtin_platform_driver to simplify the code

From: Wei Yongjun <weiyongj...@huawei.com> Use the builtin_platform_driver() macro to make the code simpler. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/memory/atmel-sdramc.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/

[PATCH -next] memory: atmel-sdramc: use builtin_platform_driver to simplify the code

From: Wei Yongjun Use the builtin_platform_driver() macro to make the code simpler. Signed-off-by: Wei Yongjun --- drivers/memory/atmel-sdramc.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/memory/atmel-sdramc.c b/drivers/memory/atmel-sdramc.c index 12080b0

[PATCH -next] memory: atmel-ebi: fix return value check in at91_ebi_dev_disable()

From: Wei Yongjun <weiyongj...@huawei.com> Fix the retrn value check which testing the wrong variable in at91_ebi_dev_disable(). Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/memory/atmel-ebi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dr

[PATCH -next] memory: atmel-ebi: fix return value check in at91_ebi_dev_disable()

From: Wei Yongjun Fix the retrn value check which testing the wrong variable in at91_ebi_dev_disable(). Signed-off-by: Wei Yongjun --- drivers/memory/atmel-ebi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/memory/atmel-ebi.c b/drivers/memory/atmel-ebi.c index

[PATCH -next v2] ARM: orion: fix invalid use of sizeof in orion_ge00_switch_init()

From: Wei Yongjun <weiyongj...@huawei.com> sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: fe158a17c1e0 ("ARM: orion: simplify orion_ge00_switch_init") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com

[PATCH -next v2] ARM: orion: fix invalid use of sizeof in orion_ge00_switch_init()

From: Wei Yongjun sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: fe158a17c1e0 ("ARM: orion: simplify orion_ge00_switch_init") Signed-off-by: Wei Yongjun --- v1 -> v2: fix file module name in subject ---

[PATCH -next] dma: mv_xor: fix invalid use of sizeof in orion_ge00_switch_init()

From: Wei Yongjun <weiyongj...@huawei.com> sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: fe158a17c1e0 ("ARM: orion: simplify orion_ge00_switch_init") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com&

[PATCH -next] dma: mv_xor: fix invalid use of sizeof in orion_ge00_switch_init()

From: Wei Yongjun sizeof() when applied to a pointer typed expression gives the size of the pointer, not that of the pointed data. Fixes: fe158a17c1e0 ("ARM: orion: simplify orion_ge00_switch_init") Signed-off-by: Wei Yongjun --- arch/arm/plat-orion/common.c | 2 +- 1 file

[PATCH -next] staging: ks7010: remove unused including

From: Wei Yongjun <weiyongj...@huawei.com> Remove including that don't need it. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/ks7010/ks_wlan_net.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/st

[PATCH -next] staging: ks7010: remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- drivers/staging/ks7010/ks_wlan_net.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c index a35325e..c17c45d 100644

[PATCH -next] staging: rtl8712: fix double lock bug in SetPSModeWorkItemCallback()

From: Wei Yongjun <weiyongj...@huawei.com> Fix a double lock bug in SetPSModeWorkItemCallback(). Fixes: 5c2ba8b85e35 ("rtl8712: pwrctrl_priv: Replace semaphore lock with mutex") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/rtl8712/rtl871x_pw

[PATCH -next] staging: rtl8712: fix double lock bug in SetPSModeWorkItemCallback()

From: Wei Yongjun Fix a double lock bug in SetPSModeWorkItemCallback(). Fixes: 5c2ba8b85e35 ("rtl8712: pwrctrl_priv: Replace semaphore lock with mutex") Signed-off-by: Wei Yongjun --- drivers/staging/rtl8712/rtl871x_pwrctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH -next] mailbox: platform_mhu: Remove redundant dev_err call in platform_mhu_probe()

From: Wei Yongjun <weiyongj...@huawei.com> 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 <weiyongj...@huawei.com> --- drivers/mailbox/platform_mhu.c | 4 +--- 1 file changed,

[PATCH -next] mailbox: platform_mhu: Remove redundant dev_err call in platform_mhu_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/mailbox/platform_mhu.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/mailbox

[PATCH -next] nbd: fix return value check

From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function blk_mq_alloc_request() or blk_mq_init_queue() returns ERR_PTR() not NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: fd8383fd88a2 ("nbd: convert to blkmq") Signed-off

[PATCH -next] nbd: fix return value check

From: Wei Yongjun In case of error, the function blk_mq_alloc_request() or blk_mq_init_queue() returns ERR_PTR() not NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: fd8383fd88a2 ("nbd: convert to blkmq") Signed-off-by: Wei Yongjun --- dri

[PATCH -next] drm/sun4i: backend: remove redundant dev_err call in sun4i_backend_bind()

From: Wei Yongjun <weiyongj...@huawei.com> 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 <weiyongj...@huawei.com> --- drivers/gpu/drm/sun4i/sun4i_backend.c | 4 +--- 1 fil

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