[PATCH -next] staging: mt7621-gpio: mt7621: make symbol gc_map static

2018-03-21 Thread Wei Yongjun
Fixes the following sparse warning: drivers/staging/mt7621-gpio/gpio-mt7621.c:47:3: warning: symbol 'gc_map' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/staging/mt7621-gpio/gpio-mt7621.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] drm/meson: Fix potential NULL dereference in meson_drv_bind_master()

2018-03-20 Thread Wei Yongjun
, t, n); + if (!res) + return -EINVAL; ... when != res == NULL e = devm_ioremap(e1, res->start, e2); Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/meson/meson_drv.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/meson/meson_drv.c b/dr

[PATCH -next] drm/meson: Fix potential NULL dereference in meson_drv_bind_master()

2018-03-20 Thread Wei Yongjun
, t, n); + if (!res) + return -EINVAL; ... when != res == NULL e = devm_ioremap(e1, res->start, e2); Signed-off-by: Wei Yongjun --- drivers/gpu/drm/meson/meson_drv.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/meson/meson_drv.c b/drivers/gpu/drm/meson/meson_dr

[PATCH -next] phy: stm32: fix using 0 as NULL pointer warnings

2018-03-20 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/phy/st/phy-stm32-usbphyc.c:331:42: warning: Using plain integer as NULL pointer drivers/phy/st/phy-stm32-usbphyc.c:344:52: warning: Using plain integer as NULL pointer Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/phy/st/phy

[PATCH -next] phy: stm32: fix using 0 as NULL pointer warnings

2018-03-20 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/phy/st/phy-stm32-usbphyc.c:331:42: warning: Using plain integer as NULL pointer drivers/phy/st/phy-stm32-usbphyc.c:344:52: warning: Using plain integer as NULL pointer Signed-off-by: Wei Yongjun --- drivers/phy/st/phy-stm32-usbphyc.c | 4 ++-- 1

[PATCH -next] phy: phy-mtk-tphy: fix missing clk_disable_unprepare() on error in mtk_phy_init()

2018-03-20 Thread Wei Yongjun
Fix the missing clk_disable_unprepare() before return from mtk_phy_init() in the error handling case. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/phy/mediatek/phy-mtk-tphy.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/phy/me

[PATCH -next] phy: phy-mtk-tphy: fix missing clk_disable_unprepare() on error in mtk_phy_init()

2018-03-20 Thread Wei Yongjun
Fix the missing clk_disable_unprepare() before return from mtk_phy_init() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/phy/mediatek/phy-mtk-tphy.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/phy/mediatek/phy-mtk-tphy.c b/drivers

[PATCH -next] ASoC: wm8400: Use devm_snd_soc_register_component()

2018-03-16 Thread Wei Yongjun
Since the remove callback is removed, the snd_soc_unregister_component() is missing when remove device. Using devm_snd_soc_register_component() instead of snd_soc_register_component(). Fixes: 10dc44c6462d ("ASoC: wm8400: replace codec to component") Signed-off-by: Wei Yongjun

[PATCH -next] ASoC: wm8400: Use devm_snd_soc_register_component()

2018-03-16 Thread Wei Yongjun
Since the remove callback is removed, the snd_soc_unregister_component() is missing when remove device. Using devm_snd_soc_register_component() instead of snd_soc_register_component(). Fixes: 10dc44c6462d ("ASoC: wm8400: replace codec to component") Signed-off-by: Wei Yongjun ---

[PATCH -next] ASoC: da7210: Use devm_snd_soc_register_component()

2018-03-16 Thread Wei Yongjun
Since the remove callback is removed, the snd_soc_unregister_component() is missing when remove device. Using devm_snd_soc_register_component() instead of snd_soc_register_component(). Fixes: d06f33aed85c ("ASoC: da7210: replace codec to component") Signed-off-by: Wei Yongjun

[PATCH -next] ASoC: da7210: Use devm_snd_soc_register_component()

2018-03-16 Thread Wei Yongjun
Since the remove callback is removed, the snd_soc_unregister_component() is missing when remove device. Using devm_snd_soc_register_component() instead of snd_soc_register_component(). Fixes: d06f33aed85c ("ASoC: da7210: replace codec to component") Signed-off-by: Wei Yongjun ---

[PATCH -next] ASoC: mediatek: mt2701: drop unnessary snd_soc_unregister_component()

2018-03-16 Thread Wei Yongjun
It's not necessary to unregister a component registered with devm_snd_soc_register_component(). Fixes: f1b5bf07365d ("ASoC: mt2701/mt8173: replace platform to componen") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- sound/soc/mediatek/mt2701/mt2701-afe-pcm.c | 2 -- 1

[PATCH -next] ASoC: mediatek: mt2701: drop unnessary snd_soc_unregister_component()

2018-03-16 Thread Wei Yongjun
It's not necessary to unregister a component registered with devm_snd_soc_register_component(). Fixes: f1b5bf07365d ("ASoC: mt2701/mt8173: replace platform to componen") Signed-off-by: Wei Yongjun --- sound/soc/mediatek/mt2701/mt2701-afe-pcm.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH -next] mtd: ubi: wl: Fix error return code in ubi_wl_init()

2018-01-18 Thread Wei Yongjun
Fix to return error code -ENOMEM from the kmem_cache_alloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/mtd/ubi/wl.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH -next] mtd: ubi: wl: Fix error return code in ubi_wl_init()

2018-01-18 Thread Wei Yongjun
Fix to return error code -ENOMEM from the kmem_cache_alloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/mtd/ubi/wl.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/ubi/wl.c b/drivers/mtd

[PATCH -next v2] ipmi/powernv: Fix error return code in ipmi_powernv_probe()

2018-01-17 Thread Wei Yongjun
Fix to return a negative error code from the request_irq() error handling case instead of 0, as done elsewhere in this function. Fixes: dce143c3381c ("ipmi/powernv: Convert to irq event interface") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- v1 -> v2: add fixes. ---

[PATCH -next v2] ipmi/powernv: Fix error return code in ipmi_powernv_probe()

2018-01-17 Thread Wei Yongjun
Fix to return a negative error code from the request_irq() error handling case instead of 0, as done elsewhere in this function. Fixes: dce143c3381c ("ipmi/powernv: Convert to irq event interface") Signed-off-by: Wei Yongjun --- v1 -> v2: add fixes. --- drivers/char/ipmi/ipmi_

[PATCH -next] slimbus: qcom: remove redundant dev_err call in qcom_slim_probe()

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/slimbus/qcom-ctrl.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/s

[PATCH -next] slimbus: qcom: remove redundant dev_err call in qcom_slim_probe()

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/slimbus/qcom-ctrl.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/slimbus/qcom-ctrl.c b/drivers

[PATCH -next] memory: ti-emif-sram: remove redundant dev_err call in ti_emif_probe()

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/memory/ti-emif-pm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/memory/ti-emif-pm.c b/d

[PATCH -next] memory: ti-emif-sram: remove redundant dev_err call in ti_emif_probe()

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/memory/ti-emif-pm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/memory/ti-emif-pm.c b/drivers/memory/ti-emif-pm.c

[PATCH -next] phy: usb: phy-brcm-usb: Remove redundant return value check of platform_get_resource()

2018-01-17 Thread 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 <weiyongj...@huawei.com> --- drivers/phy/broadcom/phy-brcm-usb.c | 4 1 file changed, 4 deletions(-) diff --git a/drive

[PATCH -next] phy: usb: phy-brcm-usb: Remove redundant return value check of platform_get_resource()

2018-01-17 Thread 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/phy/broadcom/phy-brcm-usb.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/phy/broadcom/phy-brcm-usb.c b

[PATCH -next] perf: hisi: Remove redundant dev_err call

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/perf/hisilicon/hisi_uncore_ddrc_pmu.c | 4 +--- drivers/perf/hisilicon/hisi_uncore_hha_pmu

[PATCH -next] perf: hisi: Remove redundant dev_err call

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/perf/hisilicon/hisi_uncore_ddrc_pmu.c | 4 +--- drivers/perf/hisilicon/hisi_uncore_hha_pmu.c | 4 +--- drivers/perf/hisilicon

[PATCH -next] ipmi/powernv: Fix error return code in ipmi_powernv_probe()

2018-01-17 Thread Wei Yongjun
Fix to return a negative error code from the request_irq() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/char/ipmi/ipmi_powernv.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH -next] ipmi/powernv: Fix error return code in ipmi_powernv_probe()

2018-01-17 Thread Wei Yongjun
Fix to return a negative error code from the request_irq() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/char/ipmi/ipmi_powernv.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/char/ipmi

[PATCH -next] mtd: onenand: omap2: Remove redundant dev_err call in omap2_onenand_probe()

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/mtd/onenand/omap2.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drive

[PATCH -next] mtd: onenand: omap2: Remove redundant dev_err call in omap2_onenand_probe()

2018-01-17 Thread Wei Yongjun
There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/mtd/onenand/omap2.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/mtd/onenand/omap2.c b/drivers/mtd

[PATCH -next] dm crypt: fix error return code in crypt_ctr()

2018-01-17 Thread Wei Yongjun
Fix to return error code -ENOMEM from the mempool_create_kmalloc_pool() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/md/dm-crypt.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/md/dm-cry

[PATCH -next] dm crypt: fix error return code in crypt_ctr()

2018-01-17 Thread Wei Yongjun
Fix to return error code -ENOMEM from the mempool_create_kmalloc_pool() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/md/dm-crypt.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/md/dm-crypt.c b/drivers/md/dm-crypt.c

[PATCH -next] firmware: arm_sdei: Fix return value check in sdei_present_dt()

2018-01-15 Thread Wei Yongjun
In case of error, the function of_platform_device_create() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 677a60bd2003 ("firmware: arm_sdei: Discover SDEI support via ACPI") Signed-off-by: Wei Yongjun

[PATCH -next] firmware: arm_sdei: Fix return value check in sdei_present_dt()

2018-01-15 Thread Wei Yongjun
In case of error, the function of_platform_device_create() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 677a60bd2003 ("firmware: arm_sdei: Discover SDEI support via ACPI") Signed-off-by: Wei Yongjun --

[PATCH -next] drm/etnaviv: make local symbols static

2018-01-11 Thread Wei Yongjun
? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/etnaviv/etnaviv_iommu.c | 2 +- drivers/gpu/drm/etnaviv/etnaviv_iommu_v2.c | 2 +- 2 file changed, 2 insertion(+), 2 deletion(-) diff --git a/drivers/gpu/drm/etnaviv/etnaviv_iommu.c b/drivers/gpu/drm/etnaviv/etnaviv_iommu.c

[PATCH -next] drm/etnaviv: make local symbols static

2018-01-11 Thread Wei Yongjun
? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/etnaviv/etnaviv_iommu.c | 2 +- drivers/gpu/drm/etnaviv/etnaviv_iommu_v2.c | 2 +- 2 file changed, 2 insertion(+), 2 deletion(-) diff --git a/drivers/gpu/drm/etnaviv/etnaviv_iommu.c b/drivers/gpu/drm/etnaviv/etnaviv_iommu.c index 7a8c947..4b9b11c 100644

[PATCH -next] test_firmware: make local symbol test_fw_config static

2018-01-11 Thread Wei Yongjun
Fixes the following sparse warnings: lib/test_firmware.c:99:20: warning: symbol 'test_fw_config' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- lib/test_firmware.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] test_firmware: make local symbol test_fw_config static

2018-01-11 Thread Wei Yongjun
Fixes the following sparse warnings: lib/test_firmware.c:99:20: warning: symbol 'test_fw_config' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- lib/test_firmware.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/test_firmware.c b/lib

[PATCH -next] test_firmware: fix missing unlock on error in config_num_requests_store()

2018-01-11 Thread Wei Yongjun
Add the missing unlock before return from function config_num_requests_store() in the error handling case. Fixes: c92316bf8e94 ("test_firmware: add batched firmware tests") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- lib/test_firmware.c | 1 + 1 file changed, 1 in

[PATCH -next] test_firmware: fix missing unlock on error in config_num_requests_store()

2018-01-11 Thread Wei Yongjun
Add the missing unlock before return from function config_num_requests_store() in the error handling case. Fixes: c92316bf8e94 ("test_firmware: add batched firmware tests") Signed-off-by: Wei Yongjun --- lib/test_firmware.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/test_

[PATCH -next] ASoC: stm32: fix a typo in stm32_adfsdm_probe()

2018-01-11 Thread Wei Yongjun
Fix a typo, we should return PTR_ERR(priv->iio_cb) instead of PTR_ERR(priv->iio_ch). Fixes: 55da094824c4 ("ASoC: stm32: add DFSDM DAI support") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- sound/soc/stm/stm32_adfsdm.c | 2 +- 1 file changed, 1 insertion(+), 1 d

[PATCH -next] ASoC: stm32: fix a typo in stm32_adfsdm_probe()

2018-01-11 Thread Wei Yongjun
Fix a typo, we should return PTR_ERR(priv->iio_cb) instead of PTR_ERR(priv->iio_ch). Fixes: 55da094824c4 ("ASoC: stm32: add DFSDM DAI support") Signed-off-by: Wei Yongjun --- sound/soc/stm/stm32_adfsdm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] clocksource/drivers/stm32: fix error return code in stm32_timer_init()

2018-01-10 Thread Wei Yongjun
Fix to return error code -ENOMEM from the kzalloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/clocksource/timer-stm32.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/d

[PATCH -next] clocksource/drivers/stm32: fix error return code in stm32_timer_init()

2018-01-10 Thread Wei Yongjun
Fix to return error code -ENOMEM from the kzalloc error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/clocksource/timer-stm32.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/clocksource/timer-stm32.c b

[PATCH -next] meson-mx-socinfo: Make local function meson_mx_socinfo_init() static

2018-01-10 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/soc/amlogic/meson-mx-socinfo.c:107:12: warning: symbol 'meson_mx_socinfo_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soc/amlogic/meson-mx-socinfo.c | 2 +- 1 file changed, 1 ins

[PATCH -next] meson-mx-socinfo: Make local function meson_mx_socinfo_init() static

2018-01-10 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/soc/amlogic/meson-mx-socinfo.c:107:12: warning: symbol 'meson_mx_socinfo_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/soc/amlogic/meson-mx-socinfo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH -next] meson-gx-socinfo: make local function meson_gx_socinfo_init static

2018-01-10 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/soc/amlogic/meson-gx-socinfo.c:100:12: warning: symbol 'meson_gx_socinfo_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soc/amlogic/meson-gx-socinfo.c | 2 +- 1 file changed, 1 ins

[PATCH -next] meson-gx-socinfo: make local function meson_gx_socinfo_init static

2018-01-10 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/soc/amlogic/meson-gx-socinfo.c:100:12: warning: symbol 'meson_gx_socinfo_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/soc/amlogic/meson-gx-socinfo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH -next] ASoC: mediatek: mt2701: fix return value check in mt2701_afe_pcm_dev_probe()

2018-01-07 Thread Wei Yongjun
In case of error, the function syscon_node_to_regmap() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: dfa3cbb83e09 ("ASoC: mediatek: modify MT2701 AFE driver to adapt mfd device") Signed-off-by: Wei Yongjun

[PATCH -next] ASoC: mediatek: mt2701: fix return value check in mt2701_afe_pcm_dev_probe()

2018-01-07 Thread Wei Yongjun
In case of error, the function syscon_node_to_regmap() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: dfa3cbb83e09 ("ASoC: mediatek: modify MT2701 AFE driver to adapt mfd device") Signed-off-by: Wei Yongjun

[PATCH -next] f2fs: make local functions static

2018-01-05 Thread Wei Yongjun
'f2fs_get_projid' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- fs/f2fs/segment.c | 4 ++-- fs/f2fs/super.c | 2 +- 2 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index b03f65e..36bfd53 100644 --- a/f

[PATCH -next] f2fs: make local functions static

2018-01-05 Thread Wei Yongjun
'f2fs_get_projid' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- fs/f2fs/segment.c | 4 ++-- fs/f2fs/super.c | 2 +- 2 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index b03f65e..36bfd53 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs

[PATCH -next] um: vector: fix missing unlock on error in vector_net_open()

2018-01-04 Thread Wei Yongjun
Add the missing unlock before return from function vector_net_open() in the error handling case. Fixes: ad1f62ab2bd4 ("High Performance UML Vector Network Driver") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- arch/um/drivers/vector_kern.c | 4 +++- 1 file changed, 3

[PATCH -next] um: vector: fix missing unlock on error in vector_net_open()

2018-01-04 Thread Wei Yongjun
Add the missing unlock before return from function vector_net_open() in the error handling case. Fixes: ad1f62ab2bd4 ("High Performance UML Vector Network Driver") Signed-off-by: Wei Yongjun --- arch/um/drivers/vector_kern.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)

[PATCH -next] drm/panel: ili9322:Make local symbols static

2018-01-02 Thread Wei Yongjun
? Also change ili9322_inputs to 'const char * const' to avoid chackpatch warning. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/panel/panel-ilitek-ili9322.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili

[PATCH -next] drm/panel: ili9322:Make local symbols static

2018-01-02 Thread Wei Yongjun
? Also change ili9322_inputs to 'const char * const' to avoid chackpatch warning. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/panel/panel-ilitek-ili9322.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9322.c b/drivers/gpu/drm/panel

[PATCH -next] irqchip/ompic: fix return value check in ompic_of_init()

2018-01-02 Thread Wei Yongjun
In case of error, the function ioremap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 9b54470afd83 ("irqchip: add initial support for ompic") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- dri

[PATCH -next] irqchip/ompic: fix return value check in ompic_of_init()

2018-01-02 Thread Wei Yongjun
In case of error, the function ioremap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 9b54470afd83 ("irqchip: add initial support for ompic") Signed-off-by: Wei Yongjun --- drivers/irqchip/irq-ompic.c | 4 ++

[PATCH -next] dm raid: make local symbol raid_sets static

2018-01-01 Thread Wei Yongjun
Fixes the following sparse warning: drivers/md/dm-raid.c:33:1: warning: symbol 'raid_sets' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/md/dm-raid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] dm raid: make local symbol raid_sets static

2018-01-01 Thread Wei Yongjun
Fixes the following sparse warning: drivers/md/dm-raid.c:33:1: warning: symbol 'raid_sets' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/md/dm-raid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/md/dm-raid.c b/drivers/md/dm-raid.c

[PATCH -next] tee: shm: make local function __tee_shm_alloc() static

2018-01-01 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/tee/tee_shm.c:115:16: warning: symbol '__tee_shm_alloc' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/tee/tee_shm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH -next] tee: shm: make local function __tee_shm_alloc() static

2018-01-01 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/tee/tee_shm.c:115:16: warning: symbol '__tee_shm_alloc' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/tee/tee_shm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/tee/tee_shm.c b

[PATCH -next] soundwire: Fix typo in return value check of sdw_read()

2018-01-01 Thread Wei Yongjun
Fix the typo, 'status' should be instead of 'status2'. Fixes: b0a9c37b0178 ("soundwire: Add slave status handling") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/soundwire/bus.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/d

[PATCH -next] soundwire: Fix typo in return value check of sdw_read()

2018-01-01 Thread Wei Yongjun
Fix the typo, 'status' should be instead of 'status2'. Fixes: b0a9c37b0178 ("soundwire: Add slave status handling") Signed-off-by: Wei Yongjun --- drivers/soundwire/bus.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/soundwire/bus.c b/drivers

[PATCH -next] slimbus: qcom: Make some local functions static

2018-01-01 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/slimbus/qcom-ctrl.c:151:6: warning: symbol 'slim_ack_txn' was not declared. Should it be static? drivers/slimbus/qcom-ctrl.c:304:6: warning: symbol 'slim_alloc_txbuf' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyo

[PATCH -next] slimbus: qcom: Make some local functions static

2018-01-01 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/slimbus/qcom-ctrl.c:151:6: warning: symbol 'slim_ack_txn' was not declared. Should it be static? drivers/slimbus/qcom-ctrl.c:304:6: warning: symbol 'slim_alloc_txbuf' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers

[PATCH -next] slimbus: Use GFP_ATOMIC under spin lock

2018-01-01 Thread Wei Yongjun
A spin lock is taken here so we should use GFP_ATOMIC. Fixes: afbdcc7c384b ("slimbus: Add messaging APIs to slimbus framework") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/slimbus/messaging.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

[PATCH -next] slimbus: Use GFP_ATOMIC under spin lock

2018-01-01 Thread Wei Yongjun
A spin lock is taken here so we should use GFP_ATOMIC. Fixes: afbdcc7c384b ("slimbus: Add messaging APIs to slimbus framework") Signed-off-by: Wei Yongjun --- drivers/slimbus/messaging.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/slimbus/messaging.c

[PATCH -next] slimbus: qcom: Fix return value check in qcom_slim_probe()

2018-01-01 Thread Wei Yongjun
In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: ad7fcbc308b0 ("slimbus: qcom: Add Qualcomm Slimbus controller driver") Signed-off-by: Wei Yongjun

[PATCH -next] slimbus: Fix missing unlock on error in slim_msg_response()

2018-01-01 Thread Wei Yongjun
Add the missing unlock before return from function slim_msg_response() in the error handling case. Fixes: afbdcc7c384b ("slimbus: Add messaging APIs to slimbus framework") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/slimbus/messaging.c | 1 + 1 file chan

[PATCH -next] slimbus: qcom: Fix return value check in qcom_slim_probe()

2018-01-01 Thread Wei Yongjun
In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: ad7fcbc308b0 ("slimbus: qcom: Add Qualcomm Slimbus controller driver") Signed-off-by: Wei Yongjun --- drive

[PATCH -next] slimbus: Fix missing unlock on error in slim_msg_response()

2018-01-01 Thread Wei Yongjun
Add the missing unlock before return from function slim_msg_response() in the error handling case. Fixes: afbdcc7c384b ("slimbus: Add messaging APIs to slimbus framework") Signed-off-by: Wei Yongjun --- drivers/slimbus/messaging.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH -next] xen/pvcalls: use GFP_ATOMIC under spin lock

2017-12-27 Thread Wei Yongjun
A spin lock is taken here so we should use GFP_ATOMIC. Fixes: 9774c6cca266 ("xen/pvcalls: implement accept command") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/xen/pvcalls-front.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/xen

[PATCH -next] xen/pvcalls: use GFP_ATOMIC under spin lock

2017-12-27 Thread Wei Yongjun
A spin lock is taken here so we should use GFP_ATOMIC. Fixes: 9774c6cca266 ("xen/pvcalls: implement accept command") Signed-off-by: Wei Yongjun --- drivers/xen/pvcalls-front.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/xen/pvcalls-front.c b/drivers/x

[PATCH -next] mtd: sharpslpart: make local function sharpsl_nand_cleanup_ftl() static

2017-12-19 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/mtd/parsers/sharpslpart.c:222:6: warning: symbol 'sharpsl_nand_cleanup_ftl' was not declared. Should it be static? Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/mtd/parsers/sharpslpart.c | 2 +- 1 file changed, 1 insertion

[PATCH -next] mtd: sharpslpart: make local function sharpsl_nand_cleanup_ftl() static

2017-12-19 Thread Wei Yongjun
Fixes the following sparse warnings: drivers/mtd/parsers/sharpslpart.c:222:6: warning: symbol 'sharpsl_nand_cleanup_ftl' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/mtd/parsers/sharpslpart.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[tip:irq/urgent] irqchip/exiu: Fix return value check in exiu_init()

2017-11-14 Thread tip-bot for Wei Yongjun
Commit-ID: 0e54705b0e01dcaf3eb2a496bb66d5f05012056b Gitweb: https://git.kernel.org/tip/0e54705b0e01dcaf3eb2a496bb66d5f05012056b Author: Wei Yongjun <weiyongj...@huawei.com> AuthorDate: Tue, 14 Nov 2017 06:57:28 + Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:irq/urgent] irqchip/exiu: Fix return value check in exiu_init()

2017-11-14 Thread tip-bot for Wei Yongjun
Commit-ID: 0e54705b0e01dcaf3eb2a496bb66d5f05012056b Gitweb: https://git.kernel.org/tip/0e54705b0e01dcaf3eb2a496bb66d5f05012056b Author: Wei Yongjun AuthorDate: Tue, 14 Nov 2017 06:57:28 + Committer: Thomas Gleixner CommitDate: Tue, 14 Nov 2017 11:27:22 +0100 irqchip/exiu: Fix

[PATCH -next] irqchip/exiu: Fix return value check in exiu_init()

2017-11-13 Thread Wei Yongjun
In case of error, the function of_iomap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 706cffc1b912 ("irqchip/exiu: Add support for Socionext Synquacer EXIU controller") Signed-off-by: Wei Yongjun

[PATCH -next] irqchip/exiu: Fix return value check in exiu_init()

2017-11-13 Thread Wei Yongjun
In case of error, the function of_iomap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 706cffc1b912 ("irqchip/exiu: Add support for Socionext Synquacer EXIU controller") Signed-off-by: Wei Yongjun --- drive

[PATCH -next] ipmi_si_platform: Fix typo in parameter description

2017-11-03 Thread Wei Yongjun
Fix typo in parameter description. Fixes: 95e300c052fd ("ipmi: Make the DMI probe into a generic platform probe") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/char/ipmi/ipmi_si_platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[PATCH -next] ipmi_si_platform: Fix typo in parameter description

2017-11-03 Thread Wei Yongjun
Fix typo in parameter description. Fixes: 95e300c052fd ("ipmi: Make the DMI probe into a generic platform probe") Signed-off-by: Wei Yongjun --- drivers/char/ipmi/ipmi_si_platform.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/ipmi/ipmi_si_pla

[PATCH -next] ipmi watchdog: fix typo in parameter description

2017-11-03 Thread Wei Yongjun
Fix typo in parameter description. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/char/ipmi/ipmi_watchdog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/ipmi/ipmi_watchdog.c b/drivers/char/ipmi/ipmi_watchdog.c index 76b2706..09e8463

[PATCH -next] ipmi watchdog: fix typo in parameter description

2017-11-03 Thread Wei Yongjun
Fix typo in parameter description. Signed-off-by: Wei Yongjun --- drivers/char/ipmi/ipmi_watchdog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/char/ipmi/ipmi_watchdog.c b/drivers/char/ipmi/ipmi_watchdog.c index 76b2706..09e8463 100644 --- a/drivers/char/ipmi

[PATCH -next] nullb: fix error return code in null_init()

2017-10-17 Thread Wei Yongjun
Fix to return error code -ENOMEM from the null_alloc_dev() error handling case instead of 0, as done elsewhere in this function. Fixes: 2984c8684f96 ("nullb: factor disk parameters") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/block/null_blk.c | 4 +++- 1

[PATCH -next] nullb: fix error return code in null_init()

2017-10-17 Thread Wei Yongjun
Fix to return error code -ENOMEM from the null_alloc_dev() error handling case instead of 0, as done elsewhere in this function. Fixes: 2984c8684f96 ("nullb: factor disk parameters") Signed-off-by: Wei Yongjun --- drivers/block/null_blk.c | 4 +++- 1 file changed, 3 insertions(+),

[PATCH -next] iommu/ipmmu-vmsa: Fix return value check in ipmmu_find_group_dma()

2017-10-17 Thread Wei Yongjun
In case of error, the function iommu_group_get() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 3ae47292024f ("iommu/ipmmu-vmsa: Add new IOMMU_DOMAIN_DMA ops") Signed-off-by: Wei Yongjun <weiyongj.

[PATCH -next] iommu/ipmmu-vmsa: Fix return value check in ipmmu_find_group_dma()

2017-10-17 Thread Wei Yongjun
In case of error, the function iommu_group_get() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 3ae47292024f ("iommu/ipmmu-vmsa: Add new IOMMU_DOMAIN_DMA ops") Signed-off-by: Wei Yongjun --- drivers/iommu/ip

[PATCH -next] MIPS: bpf: Fix a typo in build_one_insn()

2017-10-13 Thread Wei Yongjun
Fix a typo in build_one_insn(). Fixes: b6bd53f9c4e8 ("MIPS: Add missing file for eBPF JIT.") Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- arch/mips/net/ebpf_jit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mips/net/ebpf_jit.c b/arch/mip

[PATCH -next] MIPS: bpf: Fix a typo in build_one_insn()

2017-10-13 Thread Wei Yongjun
Fix a typo in build_one_insn(). Fixes: b6bd53f9c4e8 ("MIPS: Add missing file for eBPF JIT.") Signed-off-by: Wei Yongjun --- arch/mips/net/ebpf_jit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mips/net/ebpf_jit.c b/arch/mips/net/ebpf_jit.c index 01b7a8

[PATCH] drm: Fix return value check in kirin_drm_platform_probe()

2017-10-11 Thread Wei Yongjun
In case of error, the function of_graph_get_remote_node() 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 <weiyongj...@huawei.com> --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 4 ++--

[PATCH] drm: Fix return value check in kirin_drm_platform_probe()

2017-10-11 Thread Wei Yongjun
In case of error, the function of_graph_get_remote_node() 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/gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 4 ++-- 1 file changed, 2 insertions

[tip:ras/core] RAS: Make local function parse_ras_param() static

2017-05-21 Thread tip-bot for Wei Yongjun
Commit-ID: 5f0744e50db1628d6de770b92278445b3de2779f Gitweb: http://git.kernel.org/tip/5f0744e50db1628d6de770b92278445b3de2779f Author: Wei Yongjun <weiyongj...@huawei.com> AuthorDate: Fri, 19 May 2017 11:39:12 +0200 Committer: Thomas Gleixner <t...@linutronix.de> CommitDate:

[tip:ras/core] RAS: Make local function parse_ras_param() static

2017-05-21 Thread tip-bot for Wei Yongjun
Commit-ID: 5f0744e50db1628d6de770b92278445b3de2779f Gitweb: http://git.kernel.org/tip/5f0744e50db1628d6de770b92278445b3de2779f Author: Wei Yongjun AuthorDate: Fri, 19 May 2017 11:39:12 +0200 Committer: Thomas Gleixner CommitDate: Sun, 21 May 2017 21:55:12 +0200 RAS: Make local

[PATCH -next] drm/vgem: Fix return value check in vgem_init()

2017-05-20 Thread Wei Yongjun
From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function platform_device_register_simple() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: 315f0242aa2b ("drm/vgem: Convert to a struct drm_devi

[PATCH -next] drm/vgem: Fix return value check in vgem_init()

2017-05-20 Thread Wei Yongjun
From: Wei Yongjun In case of error, the function platform_device_register_simple() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: 315f0242aa2b ("drm/vgem: Convert to a struct drm_device subclass") Signed-o

[PATCH -next] drm/pl111: Fix return value check in pl111_amba_probe()

2017-05-20 Thread Wei Yongjun
From: Wei Yongjun <weiyongj...@huawei.com> In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: bed41005e617 ("drm/pl111: Initial drm/kms driver for pl111&q

[PATCH -next] drm/pl111: Fix return value check in pl111_amba_probe()

2017-05-20 Thread Wei Yongjun
From: Wei Yongjun In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: bed41005e617 ("drm/pl111: Initial drm/kms driver for pl111") Signed-off-by: W

[PATCH] goldfish_pipe: use GFP_ATOMIC under spin lock

2017-05-20 Thread Wei Yongjun
From: Wei Yongjun <weiyongj...@huawei.com> The function get_free_pipe_id_locked() is called from goldfish_pipe_open() with a lock is held, so we should use GFP_ATOMIC instead of GFP_KERNEL. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/platform/goldfish/goldfis

[PATCH] goldfish_pipe: use GFP_ATOMIC under spin lock

2017-05-20 Thread Wei Yongjun
From: Wei Yongjun The function get_free_pipe_id_locked() is called from goldfish_pipe_open() with a lock is held, so we should use GFP_ATOMIC instead of GFP_KERNEL. Signed-off-by: Wei Yongjun --- drivers/platform/goldfish/goldfish_pipe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] drm/i915: Fix return value check in kfence selftests

2017-05-18 Thread Wei Yongjun
From: Wei Yongjun <weiyongj...@huawei.com> Fix the return value check which testing the wrong variable. Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> --- drivers/gpu/drm/i915/selftests/i915_sw_fence.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dri

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