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

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/sun4i/sun4i_backend.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu

[PATCH -next] staging: fsl-mc: 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/fsl-mc/bus/fsl-mc-allocator.c |

[PATCH -next] staging: fsl-mc: 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/fsl-mc/bus/fsl-mc-allocator.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging

[PATCH -next] staging: fsl-mc: use list_del_init instead of list_del/INIT_LIST_HEAD

From: Wei Yongjun <weiyongj...@huawei.com> Using list_del_init() instead of list_del() + INIT_LIST_HEAD(). Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/fsl-mc/bus/fsl-mc-allocator.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --g

[PATCH -next] staging: fsl-mc: use list_del_init instead of list_del/INIT_LIST_HEAD

From: Wei Yongjun Using list_del_init() instead of list_del() + INIT_LIST_HEAD(). Signed-off-by: Wei Yongjun --- drivers/staging/fsl-mc/bus/fsl-mc-allocator.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/fsl-mc/bus/fsl-mc-allocator.c b/drivers

[PATCH -next] staging: android: ion: Fix return value check in hi6220_ion_probe()

From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function ion_device_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

[PATCH -next] staging: android: ion: Fix return value check in hi6220_ion_probe()

From: Wei Yongjun In case of error, the function ion_device_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/android/ion/hisilicon/hi6220_ion.c | 4 ++-- 1 file changed

[PATCH -next] drm/mediatek: Remove redundant dev_err call in mtk_drm_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/gpu/drm/mediatek/mtk_drm_drv.c | 8 ++-- 1 fil

[PATCH -next] drm/mediatek: Remove redundant dev_err call in mtk_drm_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/gpu/drm/mediatek/mtk_drm_drv.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

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

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warning: drivers/gpu/drm/tilcdc/tilcdc_drv.c:64:5: warning: symbol 'tilcdc_atomic_check' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/tilcdc/tilc

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

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

[PATCH -next] staging: fsl-mc: convert to use ATTRIBUTE_GROUPS macro

From: Wei Yongjun <weiyongj...@huawei.com> Use ATTRIBUTE_GROUPS macro to reduce the number of lines of code. Generated by Coccinelle semantic patch. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/fsl-mc/bus/mc-bus.c | 8 +--- 1 file changed, 1 ins

[PATCH -next] staging: fsl-mc: convert to use ATTRIBUTE_GROUPS macro

From: Wei Yongjun Use ATTRIBUTE_GROUPS macro to reduce the number of lines of code. Generated by Coccinelle semantic patch. Signed-off-by: Wei Yongjun --- drivers/staging/fsl-mc/bus/mc-bus.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/staging/fsl-mc/bus

[PATCH -next] staging: rtl8188eu: fix missing unlock on error in rtw_resume_process()

From: Wei Yongjun <weiyongj...@huawei.com> Add the missing unlock before return from function rtw_resume_process() in the error handling case. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/staging/rtl8188eu/os_dep/usb_intf.c | 4 +++- 1 file changed, 3 inse

[PATCH -next] staging: rtl8188eu: fix missing unlock on error in rtw_resume_process()

From: Wei Yongjun Add the missing unlock before return from function rtw_resume_process() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/staging/rtl8188eu/os_dep/usb_intf.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu

[PATCH -next] drm/sun4i: Remove redundant dev_err call in sun4i_tcon_init_regmap()

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_tcon.c | 4 +--- 1 file changed,

[PATCH -next] drm/sun4i: Remove redundant dev_err call in sun4i_tcon_init_regmap()

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/sun4i/sun4i_tcon.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu

Re: [PATCH -next] ARM: pxa: remove duplicated include from spitz.c

Hi Robert, On 08/24/2016 12:24 AM, Robert Jarzmik wrote: > Wei Yongjun <weiyj...@gmail.com> writes: > >> Remove duplicated include. > How so duplicated ? Can you elaborate please ? > > Moreover, how do you do think symbol_get() can stay in spitz.c without having &

Re: [PATCH -next] ARM: pxa: remove duplicated include from spitz.c

Hi Robert, On 08/24/2016 12:24 AM, Robert Jarzmik wrote: > Wei Yongjun writes: > >> Remove duplicated include. > How so duplicated ? Can you elaborate please ? > > Moreover, how do you do think symbol_get() can stay in spitz.c without having > this include ? file linux

[PATCH -next] ASoC: samsung: i2s: Add missing clk_disable_unprepare() on error in samsung_i2s_probe()

Add the missing clk_disable_unprepare() before return from samsung_i2s_probe() in the error handling case. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- sound/soc/samsung/i2s.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sound/soc/samsung/i2s.c b/sou

[PATCH -next] ASoC: samsung: i2s: Add missing clk_disable_unprepare() on error in samsung_i2s_probe()

Add the missing clk_disable_unprepare() before return from samsung_i2s_probe() in the error handling case. Signed-off-by: Wei Yongjun --- sound/soc/samsung/i2s.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sound/soc/samsung/i2s.c b/sound/soc/samsung/i2s.c index

[PATCH -next] f2fs: fix non static symbol warning

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warning: fs/f2fs/data.c:969:12: warning: symbol 'f2fs_grab_bio' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- fs/f2fs/data.c | 4 ++-- 1 file changed, 2 inse

[PATCH -next] f2fs: fix non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: fs/f2fs/data.c:969:12: warning: symbol 'f2fs_grab_bio' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- fs/f2fs/data.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/f2fs/data.c b/fs/f2fs

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

Remove duplicated include. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- arch/arm/mach-pxa/corgi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-pxa/corgi.c b/arch/arm/mach-pxa/corgi.c index 10bfdb1..183cd34 100644 --- a/arch/arm/mach-pxa/corgi.c +++ b/arch/arm/ma

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

Remove duplicated include. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- arch/arm/mach-pxa/spitz.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-pxa/spitz.c b/arch/arm/mach-pxa/spitz.c index 2c150bf..67d66c7 100644 --- a/arch/arm/mach-pxa/spitz.c +++ b/arch/arm/ma

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

Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-pxa/corgi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-pxa/corgi.c b/arch/arm/mach-pxa/corgi.c index 10bfdb1..183cd34 100644 --- a/arch/arm/mach-pxa/corgi.c +++ b/arch/arm/mach-pxa/corgi.c @@ -35,7

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

Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/arm/mach-pxa/spitz.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-pxa/spitz.c b/arch/arm/mach-pxa/spitz.c index 2c150bf..67d66c7 100644 --- a/arch/arm/mach-pxa/spitz.c +++ b/arch/arm/mach-pxa/spitz.c @@ -31,7

[PATCH -next] staging: comedi: dt2811: fix non static symbol warnings

From: Wei Yongjun <weiyongj...@huawei.com> Fixes the following sparse warnings: drivers/staging/comedi/drivers/dt2811.c:99:20: warning: symbol 'dt2811_clk_dividers' was not declared. Should it be static? drivers/staging/comedi/drivers/dt2811.c:103:20: warning: symbol 'dt2811_clk_multi

[PATCH -next] staging: comedi: dt2811: fix non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: drivers/staging/comedi/drivers/dt2811.c:99:20: warning: symbol 'dt2811_clk_dividers' was not declared. Should it be static? drivers/staging/comedi/drivers/dt2811.c:103:20: warning: symbol 'dt2811_clk_multipliers' was not declared. Should

[PATCH -next] ASoC: max98371: Add terminate entry for i2c_device_id tables

From: Wei Yongjun <weiyongj...@huawei.com> Make sure i2c_device_id tables are NULL terminated. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- sound/soc/codecs/max98371.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/codecs/max98371.c b/sound/soc/codecs/max9

[PATCH -next] ASoC: max98371: Add terminate entry for i2c_device_id tables

From: Wei Yongjun Make sure i2c_device_id tables are NULL terminated. Signed-off-by: Wei Yongjun --- sound/soc/codecs/max98371.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/codecs/max98371.c b/sound/soc/codecs/max98371.c index 3053879..781be9ba 100644 --- a/sound/soc/codecs

[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_hotplug.c:480:6: warning: symbol 'i915_hpd_poll_init_work' was not declared. Should it be static? Also move the '{' to new line. Signed-off-by: Wei Yongjun <weiyongj...@h

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

From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/i915/intel_hotplug.c:480:6: warning: symbol 'i915_hpd_poll_init_work' was not declared. Should it be static? Also move the '{' to new line. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_hotplug.c | 3 ++- 1

[PATCH -next] irqchip/irq-jcore-aic: fix non static symbol warning

Fixes the following sparse warning: drivers/irqchip/irq-jcore-aic.c:47:12: warning: symbol 'aic_irq_of_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/irqchip/irq-jcore-aic.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

[PATCH -next] irqchip/irq-jcore-aic: fix non static symbol warning

Fixes the following sparse warning: drivers/irqchip/irq-jcore-aic.c:47:12: warning: symbol 'aic_irq_of_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/irqchip/irq-jcore-aic.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] drm/mgag200: fix error return code in mgag200fb_create()

Fix to return error code -ENOMEM from the vmalloc() error handling case instead of 0, as done elsewhere in this function. Fixes: aec9e12953e7 ("drm/mgag200: Fix error handling paths in fbdev driver") Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/gpu/drm/mgag200/

[PATCH -next] drm/mgag200: fix error return code in mgag200fb_create()

Fix to return error code -ENOMEM from the vmalloc() error handling case instead of 0, as done elsewhere in this function. Fixes: aec9e12953e7 ("drm/mgag200: Fix error handling paths in fbdev driver") Signed-off-by: Wei Yongjun --- drivers/gpu/drm/mgag200/mgag200_fb.c | 4 +++- 1 file

[PATCH -next] ASoC: Intel: Skylake: Fix error return code in skl_probe()

Fix to return error code -ENODEV from the error handling case instead of 0, as done elsewhere in this function. Fixes: 87b2bdf02278 ("ASoC: Intel: Skylake: Initialize NHLT table") Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- sound/soc/intel/skylake/skl.c | 4 +++- 1

[PATCH -next] ASoC: Intel: Skylake: Fix error return code in skl_probe()

Fix to return error code -ENODEV from the error handling case instead of 0, as done elsewhere in this function. Fixes: 87b2bdf02278 ("ASoC: Intel: Skylake: Initialize NHLT table") Signed-off-by: Wei Yongjun --- sound/soc/intel/skylake/skl.c | 4 +++- 1 file changed, 3 insertions(+),

Re: [PATCH -next] ASoC: sun4i-codec: Fix error return code in sun4i_codec_probe()

Hi On 08/10/2016 10:04 PM, Chen-Yu Tsai wrote: > On Wed, Aug 10, 2016 at 9:42 PM, Wei Yongjun <weiyj...@gmail.com> wrote: >> Fix to return error code -ENOMEM instead of 0 when create card >> failed, as done elsewhere in this function. >> >> Fixes: 45fb6b

Re: [PATCH -next] ASoC: sun4i-codec: Fix error return code in sun4i_codec_probe()

Hi On 08/10/2016 10:04 PM, Chen-Yu Tsai wrote: > On Wed, Aug 10, 2016 at 9:42 PM, Wei Yongjun wrote: >> Fix to return error code -ENOMEM instead of 0 when create card >> failed, as done elsewhere in this function. >> >> Fixes: 45fb6b6f2aa3 ("ASoC: sunxi: add

[PATCH v2] w1: fix typo in parameter description

Fix typo in parameter description. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- v1 -> v2: make it as one line, comments from Evgeniy Polyakov --- drivers/w1/w1.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index bb34362

[PATCH v2] w1: fix typo in parameter description

Fix typo in parameter description. Signed-off-by: Wei Yongjun --- v1 -> v2: make it as one line, comments from Evgeniy Polyakov --- drivers/w1/w1.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index bb34362..4bd898b 100644 --- a/driv

[PATCH -next] ASoC: atmel-pdmic: add missing clk_disable_unprepare() on error in atmel_pdmic_cpu_dai_startup()

Add the missing clk_disable_unprepare() before return from atmel_pdmic_cpu_dai_startup() in the error handling case. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- sound/soc/atmel/atmel-pdmic.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/soc/atmel

[PATCH -next] ASoC: atmel-pdmic: add missing clk_disable_unprepare() on error in atmel_pdmic_cpu_dai_startup()

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

[PATCH -next] ASoC: sun4i-codec: Fix error return code in sun4i_codec_probe()

Fix to return error code -ENOMEM instead of 0 when create card failed, as done elsewhere in this function. Fixes: 45fb6b6f2aa3 ("ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs") Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- sound/soc/sunxi/sun4i-

[PATCH -next] ASoC: sun4i-codec: Fix error return code in sun4i_codec_probe()

Fix to return error code -ENOMEM instead of 0 when create card failed, as done elsewhere in this function. Fixes: 45fb6b6f2aa3 ("ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs") Signed-off-by: Wei Yongjun --- sound/soc/sunxi/sun4i-codec.c | 1 + 1 file

[PATCH -next] ASoC: topology: Fix error return code in soc_tplg_dapm_widget_create()

Fix to return error code -ENOMEM instead of 0 when failed to create widget, as done elsewhere in this function. Fixes: 8a9782346dcc ("ASoC: topology: Add topology core") Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- sound/soc/soc-topology.c | 1 + 1 file changed, 1 insertio

[PATCH -next] ASoC: topology: Fix error return code in soc_tplg_dapm_widget_create()

Fix to return error code -ENOMEM instead of 0 when failed to create widget, as done elsewhere in this function. Fixes: 8a9782346dcc ("ASoC: topology: Add topology core") Signed-off-by: Wei Yongjun --- sound/soc/soc-topology.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sou

Re: [PATCH] w1: fix timeout_us parameter description

Hi On 08/10/2016 06:54 AM, Evgeniy Polyakov wrote: > Hi > > 08.08.2016, 16:52, "Wei Yongjun" <weiyj...@gmail.com>: >> Fix 'timeout_us' parameter description. >> >> Signed-off-by: Wei Yongjun <weiyj...@gmail.com> >> --- >> drivers/w1/w

Re: [PATCH] w1: fix timeout_us parameter description

Hi On 08/10/2016 06:54 AM, Evgeniy Polyakov wrote: > Hi > > 08.08.2016, 16:52, "Wei Yongjun" : >> Fix 'timeout_us' parameter description. >> >> Signed-off-by: Wei Yongjun >> --- >> drivers/w1/w1.c | 4 ++-- >> 1 file changed, 2 insertions(

[PATCH] w1: fix timeout_us parameter description

Fix 'timeout_us' parameter description. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/w1/w1.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index bb34362..e213c67 100644 --- a/drivers/w1/w1.c +++ b/drivers/w1/w1.c @@

[PATCH] w1: fix timeout_us parameter description

Fix 'timeout_us' parameter description. Signed-off-by: Wei Yongjun --- drivers/w1/w1.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index bb34362..e213c67 100644 --- a/drivers/w1/w1.c +++ b/drivers/w1/w1.c @@ -53,8 +53,8 @@ int

[PATCH] NTB: ntb_hw_intel: Fix typo in module parameter descriptions

Fix typo in module parameter descriptions. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/ntb/hw/intel/ntb_hw_intel.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/ntb/hw/intel/ntb_hw_intel.c b/drivers/ntb/hw/intel/ntb_hw_intel.c

[PATCH] NTB: ntb_hw_intel: Fix typo in module parameter descriptions

Fix typo in module parameter descriptions. Signed-off-by: Wei Yongjun --- drivers/ntb/hw/intel/ntb_hw_intel.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/ntb/hw/intel/ntb_hw_intel.c b/drivers/ntb/hw/intel/ntb_hw_intel.c index 0d5c29a..1ee61d9 100644

[PATCH] ntb_pingpong: Fix db_init parameter description

Fix 'db_init' parameter description. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/ntb/test/ntb_pingpong.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ntb/test/ntb_pingpong.c b/drivers/ntb/test/ntb_pingpong.c index 7d31179..4358611

[PATCH] ntb_pingpong: Fix db_init parameter description

Fix 'db_init' parameter description. Signed-off-by: Wei Yongjun --- drivers/ntb/test/ntb_pingpong.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ntb/test/ntb_pingpong.c b/drivers/ntb/test/ntb_pingpong.c index 7d31179..4358611 100644 --- a/drivers/ntb/test

[PATCH -next v2] virtio: fix possible memory leak in virtqueue_add()

'desc' is malloced in virtqueue_add() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/virtio/virtio_ring.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/

[PATCH -next v2] virtio: fix possible memory leak in virtqueue_add()

'desc' is malloced in virtqueue_add() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Signed-off-by: Wei Yongjun --- drivers/virtio/virtio_ring.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/virtio/virtio_ring.c b/drivers

[PATCH -next] virtio: fix possible memory leak in virtqueue_add()

desc may malloced in virtqueue_add() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/virtio/virtio_ring.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/

[PATCH -next] virtio: fix possible memory leak in virtqueue_add()

desc may malloced in virtqueue_add() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Signed-off-by: Wei Yongjun --- drivers/virtio/virtio_ring.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/virtio/virtio_ring.c b/drivers

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

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 <weiyj...@gmail.com> --- drivers/mailbox/bcm-pdc-mailbox.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mailbox/bcm-pdc-mai

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

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/bcm-pdc-mailbox.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mailbox/bcm-pdc-mailbox.c b/drivers/mailbox

[PATCH -next] drm/amdgpu: use kmemdup rather than duplicating its implementation

Use kmemdup rather than duplicating its implementation. Generated by: scripts/coccinelle/api/memdup.cocci Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/

[PATCH -next] drm/amdgpu: use kmemdup rather than duplicating its implementation

Use kmemdup rather than duplicating its implementation. Generated by: scripts/coccinelle/api/memdup.cocci Signed-off-by: Wei Yongjun --- drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c b

[PATCH -next] drm/arm: mali-dp: Fix error return code in malidp_bind()

Fix to return error code -EINVAL from the error handling case instead of 0, as done elsewhere in this function. Fixes: 3c31760e760c ('drm/arm: mali-dp: Set crtc.port to the port instead of the endpoint') Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/gpu/drm/arm/malidp_drv

[PATCH -next] drm/arm: mali-dp: Fix error return code in malidp_bind()

Fix to return error code -EINVAL from the error handling case instead of 0, as done elsewhere in this function. Fixes: 3c31760e760c ('drm/arm: mali-dp: Set crtc.port to the port instead of the endpoint') Signed-off-by: Wei Yongjun --- drivers/gpu/drm/arm/malidp_drv.c | 4 +++- 1 file changed, 3

[PATCH -next] iommu/amd: Fix error return code in irq_remapping_alloc()

Fix to return a negative error code from the alloc_irq_index() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/iommu/amd_iommu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/iommu/amd_iom

[PATCH -next] iommu/amd: Fix error return code in irq_remapping_alloc()

Fix to return a negative error code from the alloc_irq_index() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/iommu/amd_iommu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c

[PATCH -next] iommu/amd: Fix non static symbol warning

Fixes the following sparse warning: drivers/iommu/amd_iommu.c:106:1: warning: symbol '__pcpu_scope_flush_queue' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/iommu/amd_iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH -next] iommu/amd: Fix non static symbol warning

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

[PATCH -next] drm/arm: mali-dp: Remove redundant dev_err call in malidp_bind()

There is a error message within devm_ioremap_resource already, so remove the DRM_ERROR call to avoid redundant error message. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/gpu/drm/arm/malidp_drv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drive

[PATCH -next] drm/arm: mali-dp: Remove redundant dev_err call in malidp_bind()

There is a error message within devm_ioremap_resource already, so remove the DRM_ERROR call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/arm/malidp_drv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/arm/malidp_drv.c b

[PATCH -next] rtc: asm9260: remove .owner field for driver

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 <weiyj...@gmail.com> --- drivers/rtc/rtc-asm9260.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rtc/rtc-asm9260.c b/d

[PATCH -next] rtc: asm9260: remove .owner field for driver

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/rtc/rtc-asm9260.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rtc/rtc-asm9260.c b/drivers/rtc/rtc-asm9260.c

[PATCH -next] drm/fsl-dcu: use PTR_ERR_OR_ZERO() to simplify the code

Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. Generated by: scripts/coccinelle/api/ptr_ret.cocci Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/gpu/drm/fsl-dcu/fsl_tcon.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/f

[PATCH -next] drm/fsl-dcu: use PTR_ERR_OR_ZERO() to simplify the code

Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. Generated by: scripts/coccinelle/api/ptr_ret.cocci Signed-off-by: Wei Yongjun --- drivers/gpu/drm/fsl-dcu/fsl_tcon.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/fsl-dcu/fsl_tcon.c b/drivers

[PATCH -next] rtc: asm9260: Fix missing spin_lock_init()

The driver allocates the spinlock but not initialize it. Use spin_lock_init() on it to initialize it correctly. This is detected by Coccinelle semantic patch. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/rtc/rtc-asm9260.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH -next] rtc: asm9260: Fix missing spin_lock_init()

The driver allocates the spinlock but not initialize it. Use spin_lock_init() on it to initialize it correctly. This is detected by Coccinelle semantic patch. Signed-off-by: Wei Yongjun --- drivers/rtc/rtc-asm9260.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rtc/rtc-asm9260.c

[PATCH -next] rtc: at91sam9: Fix missing spin_lock_init()

The driver allocates the spinlock but not initialize it. Use spin_lock_init() on it to initialize it correctly. This is detected by Coccinelle semantic patch. Signed-off-by: Wei Yongjun <weiyj...@gmail.com> --- drivers/rtc/rtc-at91sam9.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH -next] rtc: at91sam9: Fix missing spin_lock_init()

The driver allocates the spinlock but not initialize it. Use spin_lock_init() on it to initialize it correctly. This is detected by Coccinelle semantic patch. Signed-off-by: Wei Yongjun --- drivers/rtc/rtc-at91sam9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/rtc/rtc-at91sam9

[PATCH -next] rapidio/rio_cm: fix return value check in riocm_init()

In case of error, the function class_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 <weiyj...@gmail.com> --- drivers/rapidio/rio_cm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 del

[PATCH -next] rapidio/rio_cm: fix return value check in riocm_init()

In case of error, the function class_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/rapidio/rio_cm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[tip:x86/headers] x86/apic: Remove duplicated include from probe_64.c

Commit-ID: f6329088b3d9898d6212c410d465d096cdcd3746 Gitweb: http://git.kernel.org/tip/f6329088b3d9898d6212c410d465d096cdcd3746 Author: Wei Yongjun <yongjun_...@trendmicro.com.cn> AuthorDate: Tue, 19 Jul 2016 12:02:20 + Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:x86/headers] x86/apic: Remove duplicated include from probe_64.c

Commit-ID: f6329088b3d9898d6212c410d465d096cdcd3746 Gitweb: http://git.kernel.org/tip/f6329088b3d9898d6212c410d465d096cdcd3746 Author: Wei Yongjun AuthorDate: Tue, 19 Jul 2016 12:02:20 + Committer: Thomas Gleixner CommitDate: Tue, 19 Jul 2016 16:02:31 +0200 x86/apic: Remove

[tip:x86/headers] x86/ce4100: Remove duplicated include from ce4100.c

Commit-ID: 2384d1d832efc59447b27a283080551caf922086 Gitweb: http://git.kernel.org/tip/2384d1d832efc59447b27a283080551caf922086 Author: Wei Yongjun <yongjun_...@trendmicro.com.cn> AuthorDate: Tue, 19 Jul 2016 12:02:11 + Committer: Thomas Gleixner <t...@linutronix.de> Com

[tip:x86/headers] x86/ce4100: Remove duplicated include from ce4100.c

Commit-ID: 2384d1d832efc59447b27a283080551caf922086 Gitweb: http://git.kernel.org/tip/2384d1d832efc59447b27a283080551caf922086 Author: Wei Yongjun AuthorDate: Tue, 19 Jul 2016 12:02:11 + Committer: Thomas Gleixner CommitDate: Tue, 19 Jul 2016 16:02:17 +0200 x86/ce4100: Remove

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

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Fixes the following sparse warning: drivers/gpu/drm/virtio/virtgpu_display.c:349:37: warning: symbol 'virtio_mode_config_helpers' was not declared. Should it be static? Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>

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

From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/virtio/virtgpu_display.c:349:37: warning: symbol 'virtio_mode_config_helpers' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1

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

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Fixes the following sparse warning: drivers/gpu/drm/vgem/vgem_fence.c:75:24: warning: symbol 'vgem_fence_ops' was not declared. Should it be static? Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> --- drivers/

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

From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/vgem/vgem_fence.c:75:24: warning: symbol 'vgem_fence_ops' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/vgem/vgem_fence.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] drm/amdgpu: fix return value check in amdgpu_debugfs_ring_init()

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> In case of error, the function debugfs_create_*() 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. (defined(CONFIG_DEBUG_FS) make sure debugfs is e

[PATCH -next] drm/amdgpu: fix return value check in amdgpu_ttm_debugfs_init()

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. (defined(CONFIG_DEBUG_FS) make sure debugfs is e

[PATCH -next] drm/amdgpu: fix return value check in amdgpu_debugfs_ring_init()

From: Wei Yongjun In case of error, the function debugfs_create_*() 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. (defined(CONFIG_DEBUG_FS) make sure debugfs is enabled) Signed-off-by: Wei Yongjun

[PATCH -next] drm/amdgpu: fix return value check in amdgpu_ttm_debugfs_init()

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. (defined(CONFIG_DEBUG_FS) make sure debugfs is enabled) Signed-off-by: Wei Yongjun

[PATCH -next] drm/amdgpu: fix return value check in amdgpu_debugfs_regs_init()

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> In case of error, the function debugfs_create_*() 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. (defined(CONFIG_DEBUG_FS) make sure debugfs is e

[PATCH -next] drm/amdgpu: fix return value check in amdgpu_debugfs_regs_init()

From: Wei Yongjun In case of error, the function debugfs_create_*() 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. (defined(CONFIG_DEBUG_FS) make sure debugfs is enabled) Signed-off-by: Wei Yongjun

[PATCH -next] drm/arc: Fix some sparse warnings

From: Wei Yongjun <yongjun_...@trendmicro.com.cn> Fixes the following sparse warnings: drivers/gpu/drm/arc/arcpgu_drv.c:52:5: warning: symbol 'arcpgu_gem_mmap' was not declared. Should it be static? drivers/gpu/drm/arc/arcpgu_drv.c:134:48: warning: Using plain integer as NULL pointer d

[PATCH -next] drm/arc: Fix some sparse warnings

From: Wei Yongjun Fixes the following sparse warnings: drivers/gpu/drm/arc/arcpgu_drv.c:52:5: warning: symbol 'arcpgu_gem_mmap' was not declared. Should it be static? drivers/gpu/drm/arc/arcpgu_drv.c:134:48: warning: Using plain integer as NULL pointer drivers/gpu/drm/arc/arcpgu_drv.c:155:5

[PATCH -next] mm/slab: 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> --- mm/slab.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/mm/slab.c b/mm/slab.c index 32c2

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