[PATCH -next] mtd: nand: omap2: fix return value check in omap_nand_probe()

From: Wei Yongjun In case of error, the function dma_request_chan() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: aa7abd312c11 ('mtd: nand: omap2: Support parsing dma channel information from DT') Signed-off-by: Wei Yon

[PATCH -next] module: use kmemdup rather than duplicating its implementation

From: Wei Yongjun Use kmemdup rather than duplicating its implementation. Generated by: scripts/coccinelle/api/memdup.cocci Signed-off-by: Wei Yongjun --- kernel/module.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/kernel/module.c b/kernel/module.c index beaebea

[PATCH -next] drm/mediatek/mtk_mipi_tx: remove redundant dev_err call in mtk_mipi_tx_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_mipi_tx.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers

[PATCH -next] drm/hisilicon: Fix return value check in ade_dts_parse()

From: Wei Yongjun In case of error, the function devm_clk_get() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 12 ++-- 1 file changed,

[PATCH -next] drm: atmel-hlcdc: fix non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c:390:6: warning: symbol 'atmel_hlcdc_crtc_reset' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 2 +- 1 file changed, 1

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

From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c:107:5: warning: symbol 'ade_get_channel_formats' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 2 +- 1 file changed,

[PATCH -next] drm/hisilicon: Remove redundant dev_err call in ade_dts_parse()

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/hisilicon/kirin/kirin_drm_ade.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/dr

[PATCH -next] cxl: Use for_each_compatible_node() macro

From: Wei Yongjun Use for_each_compatible_node() macro instead of open coding it. Generated by Coccinelle. Signed-off-by: Wei Yongjun --- drivers/misc/cxl/base.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/misc/cxl/base.c b/drivers/misc/cxl/base.c index e6

[PATCH -next] regulator: act8865: Fix missing of_node_put() in act8865_pdata_from_dt()

From: Wei Yongjun This node pointer is returned by of_get_child_by_name() with refcount incremented in this function. of_node_put() is missing when exitting this function while invalid device type. Fix it by move of_get_child_by_name() code after device type check. Found by Coccinelle. Signed-o

[PATCH -next] clocksource: cadence_ttc: Fix error handling in ttc_setup_clockevent()

From: Wei Yongjun resources alloc in this function should be release in the error handling, otherwise it will cause resource leak. Signed-off-by: Wei Yongjun --- drivers/clocksource/cadence_ttc_timer.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/

[PATCH -next] nvme-rdma: fix the return value of nvme_rdma_reinit_request()

From: Wei Yongjun PTR_ERR should be applied before its argument is reassigned, otherwise the return value will be set to 0, not error code. Signed-off-by: Wei Yongjun --- drivers/nvme/host/rdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nvme/host/rdma.c b/dri

[PATCH -next v2] ASoC: mediatek: mt2701: fix non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: sound/soc/mediatek/mt2701/mt2701-afe-pcm.c:72:5: warning: symbol 'mt2701_dai_num_to_i2s' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- sound/soc/mediatek/mt2701/mt2701-afe-pcm.c | 2 +- 1 file changed, 1 insertion(

[PATCH -next] ASoC: mediatek: mt2701: cHECK-fix non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: sound/soc/mediatek/mt2701/mt2701-afe-pcm.c:72:5: warning: symbol 'mt2701_dai_num_to_i2s' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- sound/soc/mediatek/mt2701/mt2701-afe-pcm.c | 2 +- 1 file changed, 1 insertion(

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

From: Wei Yongjun Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. Generated by coccinelle. Signed-off-by: Wei Yongjun --- drivers/memory/atmel-ebi.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/memory/atmel-ebi.c b/drivers/memory/atmel-ebi.

[PATCH -next] orangefs: use vzalloc() instead of vmalloc()/memset(0)

From: Wei Yongjun Use vzalloc() instead of vmalloc() and memset(0). Signed-off-by: Wei Yongjun --- fs/orangefs/devorangefs-req.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/fs/orangefs/devorangefs-req.c b/fs/orangefs/devorangefs-req.c index a287a66..e839073 100644 --

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

From: Wei Yongjun Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. Signed-off-by: Wei Yongjun --- drivers/memory/atmel-ebi.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/memory/atmel-ebi.c b/drivers/memory/atmel-ebi.c index f87ad6f..b5ed3bd 1

[PATCH -next] nvmet: fix return value check in nvmet_subsys_alloc()

From: Wei Yongjun In case of error, the function kstrndup() 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/nvme/target/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

[PATCH -next] ASoC: cs35l33: Remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- sound/soc/codecs/cs35l33.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/cs35l33.c b/sound/soc/codecs/cs35l33.c index 622a111..c20edd4 100644 --- a/sound/soc/codecs/cs35l33.c +++ b/soun

[PATCH -next] tracing: Using for_each_set_bit to simplify the code

From: Wei Yongjun Using for_each_set_bit() to simplify the code. Signed-off-by: Wei Yongjun --- kernel/trace/trace.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 3d9f31b..48fce20 100644 --- a/kernel/trace/trace.c

[PATCH -next] ASoC: sunxi: remove redundant dev_err call in sun4i_i2s_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 --- sound/soc/sunxi/sun4i-i2s.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sound/soc/sunxi/sun4i-

[PATCH -next] mfd: hi655x: Fix return value check in hi655x_pmic_probe()

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(). Also remove unneeded error handling of platform_get_resource(). Signed-off-by: Wei Yongjun --- drivers/mf

[PATCH -next] drm/hisilicon: Fix return value check in ade_dts_parse()

From: Wei Yongjun In case of error, the function devm_clk_get() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 12 ++-- 1 file changed,

[PATCH] ring-buffer: Fix return value check in test_ringbuffer()

From: Wei Yongjun In case of error, the function kthread_run() 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 --- kernel/trace/ring_buffer.c | 8 1 file changed, 4 insertions(+), 4 deletion

[PATCH -next] ASoC: cs53l30: Fix non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: sound/soc/codecs/cs53l30.c:182:20: warning: symbol 'input1_sel_values' was not declared. Should it be static? sound/soc/codecs/cs53l30.c:202:20: warning: symbol 'input2_sel_values' was not declared. Should it be static? sound/soc/codecs/cs

[PATCH -next] ASoC: max9860: fix non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: sound/soc/codecs/max9860.c:120:28: warning: symbol 'max9860_regmap' was not declared. Should it be static? sound/soc/codecs/max9860.c:596:25: warning: symbol 'max9860_pm_ops' was not declared. Should it be static? Signed-off-by: Wei Yongj

[PATCH -next] mtd: mtk-nor: remove duplicated include from mtk-quadspi.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/mtd/spi-nor/mtk-quadspi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mtd/spi-nor/mtk-quadspi.c b/drivers/mtd/spi-nor/mtk-quadspi.c index 21c31b3..e661877 100644 --- a/drivers/mtd/spi-nor/mtk-qu

[PATCH -next] jump_label: Fix non static symbol warning

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

[PATCH] MIPS: ath79: Fix return value check in ath79_reg_ffclk()

From: Wei Yongjun In case of error, the function clk_register_fixed_factor() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- arch/mips/ath79/clock.c | 2 +- 1 file changed, 1 insertion(+), 1 dele

[PATCH] drm/i915: Fix missing unlock on error in i915_ppgtt_info()

From: Wei Yongjun Add the missing unlock before return from function i915_ppgtt_info() in the error handling case. Fixes: 1d2ac403ae3b(drm: Protect dev->filelist with its own mutex) Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH] phy: rockchip-dp: fix return value check in rockchip_dp_phy_probe()

From: Wei Yongjun In case of error, the function devm_kzalloc() 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/phy/phy-rockchip-dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH -next] rcutorture: Fix error return code in rcu_perf_init()

From: Wei Yongjun Fix to return a negative error code -ENOMEM from kcalloc() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- kernel/rcu/rcuperf.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/kernel/rcu/rcuperf.c b/k

[PATCH] ASoC: sti: fix return value check in uni_player_parse_dt_audio_glue()

From: Wei Yongjun In case of error, the function syscon_regmap_lookup_by_phandle() 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 --- sound/soc/sti/uniperif_player.c | 4 ++-- 1 file changed, 2 inse

[PATCH -next] mtd: nand: sunxi: fix return value check in sunxi_nfc_dma_op_prepare()

From: Wei Yongjun In case of error, the function dmaengine_prep_slave_sg() 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/mtd/nand/sunxi_nand.c | 4 ++-- 1 file changed, 2 insertions(+),

[PATCH] MIPS: pci-mt7620: Fix return value check in mt7620_pci_probe()

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(). Signed-off-by: Wei Yongjun --- arch/mips/pci/pci-mt7620.c | 8 1 file changed, 4 insertions(+),

[PATCH] ARM: zx: Fix return value check in zx296702_pd_probe()

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(). Signed-off-by: Wei Yongjun --- arch/arm/mach-zx/zx296702-pm-domain.c | 4 ++-- 1 file changed, 2 insertio

[PATCH] QE: Use GFP_ATOMIC under spin lock

From: Wei Yongjun The function cpm_muram_alloc_common() is called from several places, in some of which, such as cpm_muram_alloc(), a lock is held here, so we should use GFP_ATOMIC when a lock is held. Signed-off-by: Wei Yongjun --- drivers/soc/fsl/qe/qe_common.c | 2 +- 1 file changed, 1 inse

[PATCH] nbd: Fix return value check for debugfs_create_*()

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. Signed-off-by: Wei Yongjun --- drivers/block/nbd.c | 50 -

[PATCH -next] ASoC: Intel: sst_hsw: remove kfree for memory allocated with devm_kzalloc

From: Wei Yongjun It's not necessary to free memory allocated with devm_kzalloc and using kfree leads to a double free. Signed-off-by: Wei Yongjun --- sound/soc/intel/haswell/sst-haswell-ipc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/intel/haswell/sst-haswell-ipc.c b/sound

[PATCH -next] ASoC: Intel: sst_byt: remove kfree for memory allocated with devm_kzalloc

From: Wei Yongjun It's not necessary to free memory allocated with devm_kzalloc and using kfree leads to a double free. Signed-off-by: Wei Yongjun --- sound/soc/intel/baytrail/sst-baytrail-ipc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/intel/baytrail/sst-baytrail-ipc.c b/s

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

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- kernel/extable.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/extable.c b/kernel/extable.c index c98f926..e820cce 100644 --- a/kernel/extable.c +++ b/kernel/extable.c @@ -18,7 +18,6 @@ #include #include

[PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_audio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c index 2396cc7..d00d488 100644 --- a/drivers/gpu/drm/i915/in

[PATCH -next] ipmi: Remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- drivers/char/ipmi/ipmi_ssif.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/ipmi/ipmi_ssif.c b/drivers/char/ipmi/ipmi_ssif.c index 40ae7f2..cdcde23 100644 --- a/drivers/char/ipmi/ipmi_ssif.

[PATCH -next] staging: unisys: virtpci: Remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- drivers/staging/unisys/virtpci/virtpci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/unisys/virtpci/virtpci.c b/drivers/staging/unisys/virtpci/virtpci.c index d5ad017..397e967 100644

[PATCH -next] kdbus: remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- ipc/kdbus/metadata.c | 1 - 1 file changed, 1 deletion(-) diff --git a/ipc/kdbus/metadata.c b/ipc/kdbus/metadata.c index 06e0a54..882bc99 100644 --- a/ipc/kdbus/metadata.c +++ b/ipc/kdbus/metadata.c @@ -29,

[PATCH -next] ASoC: cs35l32: Remove unused including

From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- sound/soc/codecs/cs35l32.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/cs35l32.c b/sound/soc/codecs/cs35l32.c index 60598b2..8f40025 100644 --- a/sound/soc/codecs/cs35l32.c +++ b/soun

[PATCH] drm/sti: fix return value check in sti_dvo_probe()

From: Wei Yongjun In case of error, the function devm_ioremap_nocache() returns NULL 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/sti/sti_dvo.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH] powerpc/4xx: Fix return value check in hsta_msi_probe()

From: Wei Yongjun In case of error, the functions platform_get_resource() and kmalloc() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- arch/powerpc/sysdev/ppc4xx_hsta_msi.c | 4 ++-- 1 file changed, 2 in

[PATCH] ASoC: tfa9879: Fix return value check in tfa9879_i2c_probe()

From: Wei Yongjun In case of error, the function devm_kzalloc() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- sound/soc/codecs/tfa9879.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -

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

From: Wei Yongjun Fixes the following sparse warnings: drivers/clocksource/timer-digicolor.c:66:24: warning: symbol 'dc_timer' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/clocksource/timer-digicolor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH] phy: fix return value check in armada375_usb_phy_probe()

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(). Signed-off-by: Wei Yongjun --- drivers/phy/phy-armada375-usb2.c | 4 ++-- 1 file changed, 2 insertions(+)

[PATCH -next] clk: shmobile: fix sparse NULL pointer warning

From: Wei Yongjun Fixes the following sparse warnings: drivers/clk/shmobile/clk-sh73a0.c:57:17: warning: Using plain integer as NULL pointer Signed-off-by: Wei Yongjun --- drivers/clk/shmobile/clk-sh73a0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/shmobi

[PATCH] Mailbox: Fix return value check in pcc_init()

From: Wei Yongjun In case of error, the function platform_create_bundle() 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/mailbox/pcc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deleti

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

From: Wei Yongjun Fixes the following sparse warnings: drivers/regulator/rk808-regulator.c:100:5: warning: symbol 'rk808_set_suspend_voltage' was not declared. Should it be static? drivers/regulator/rk808-regulator.c:115:5: warning: symbol 'rk808_set_suspend_enable' was not declared. Should it

[PATCH -next] ASoC: Intel: fix return value check in sst_acpi_probe()

From: Wei Yongjun In case of error, the function platform_device_register_data() 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 --- sound/soc/intel/sst/sst_acpi.c | 8 1 file changed, 4 ins

[PATCH -next] coresight-tmc: use module_amba_driver to simplify the code

From: Wei Yongjun module_amba_driver() makes the code simpler by eliminating boilerplate code. Signed-off-by: Wei Yongjun --- drivers/coresight/coresight-tmc.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/coresight/coresight-tmc.c b/drivers/coresigh

[PATCH -next] coresight-tpiu: use module_amba_driver to simplify the code

From: Wei Yongjun module_amba_driver() makes the code simpler by eliminating boilerplate code. Signed-off-by: Wei Yongjun --- drivers/coresight/coresight-tpiu.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/coresight/coresight-tpiu.c b/drivers/coresi

[PATCH -next] coresight-funnel: use module_amba_driver to simplify the code

From: Wei Yongjun module_amba_driver() makes the code simpler by eliminating boilerplate code. Signed-off-by: Wei Yongjun --- drivers/coresight/coresight-funnel.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/coresight/coresight-funnel.c b/drivers/co

[PATCH -next] coresight-etb: use module_amba_driver to simplify the code

From: Wei Yongjun module_amba_driver() makes the code simpler by eliminating boilerplate code. Signed-off-by: Wei Yongjun --- drivers/coresight/coresight-etb10.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/coresight/coresight-etb10.c b/drivers/core

[PATCH -next] ASoC: Intel: fix sparse non static symbol warnings

From: Wei Yongjun Fixes the following sparse warnings: sound/soc/intel/sst/sst_acpi.c:248:5: warning: symbol 'sst_acpi_probe' was not declared. Should it be static? sound/soc/intel/sst/sst_acpi.c:335:5: warning: symbol 'sst_acpi_remove' was not declared. Should it be static? Signed-off-by: We

[PATCH -next] extcon: Remove duplicated include from extcon-class.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/extcon/extcon-class.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/extcon/extcon-class.c b/drivers/extcon/extcon-class.c index 043dcd9..8319f25 100644 --- a/drivers/extcon/extcon-class.c +++ b/dri

[PATCH -next] clk: mmp: fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warnings: drivers/clk/mmp/clk-frac.c:113:6: warning: symbol 'clk_factor_init' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/clk/mmp/clk-frac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driv

[PATCH -next] drm/i915: Fix missing unlock on error in i915_gem_init_hw()

From: Wei Yongjun Add the missing unlock before return from function i915_gem_init_hw() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/i915_gem.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gp

[PATCH -next] ASoC: es8328: fix error return code in es8328_codec_probe()

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

[PATCH] drm/msm: Fix missing unlock on error in msm_fbdev_create()

From: Wei Yongjun Add the missing unlock before return from function msm_fbdev_create() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/msm/msm_fbdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/msm_fbdev.c b/drivers/gpu/d

[PATCH -next] MIPS: Remove duplicated include from numa.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- arch/mips/loongson/loongson-3/numa.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/mips/loongson/loongson-3/numa.c b/arch/mips/loongson/loongson-3/numa.c index ca025a6..37ed184 100644 --- a/arch/mips/loong

[PATCH] HSI: omap_ssi: Fix return value check in ssi_debug_add_ctrl()

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. Signed-off-by: Wei Yongjun --- drivers/hsi/controllers/omap_ssi.c | 8 1 file ch

[PATCH] HSI: omap_ssi_port: Fix return value check in ssi_debug_add_port()

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. Signed-off-by: Wei Yongjun --- drivers/hsi/controllers/omap_ssi_port.c | 8 1 fi

[PATCH -next] drm/i915: Remove duplicated include from intel_dsi_panel_vbt.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c index 47c7584..1439516 100644 --- a

[PATCH -next] drm: Remove duplicated include from drm_plane_helper.c

From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/drm_plane_helper.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c index 6d13314..64ce96c 100644 --- a/drivers/gpu/drm/drm_pla

[PATCH -next] clk: sunxi: sun6i-apb0: fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warnings: drivers/clk/sunxi/clk-sun6i-apb0.c:60:27: warning: symbol 'sun6i_a31_apb0_clk_dt_ids' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/clk/sunxi/clk-sun6i-apb0.c | 2 +- 1 file changed, 1 insertion(+), 1 dele

[PATCH -next] clk: sunxi: sun6i-apb0-gates: fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warnings: drivers/clk/sunxi/clk-sun6i-apb0-gates.c:82:27: warning: symbol 'sun6i_a31_apb0_gates_clk_dt_ids' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/clk/sunxi/clk-sun6i-apb0-gates.c | 2 +- 1 file changed, 1 in

[PATCH -next] ARM: mvebu: fix return value check in armada_xp_pmsu_cpufreq_init()

From: Wei Yongjun In case of error, the function clk_get() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- arch/arm/mach-mvebu/pmsu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) di

[PATCH] xen/pciback: Fix error return code in xen_pcibk_attach()

From: Wei Yongjun Fix to return -EFAULT from the error handling case instead of 0 when version mismatch with pcifront. Signed-off-by: Wei Yongjun --- drivers/xen/xen-pciback/xenbus.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/xen/xen-pciback/xenbus.c b/drivers/xen/xen-pciback/

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

From: Wei Yongjun Fixes the following sparse warnings: drivers/power/power_supply_core.c:540:5: warning: symbol '__power_supply_register' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/power/power_supply_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deleti

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

From: Wei Yongjun Fixes the following sparse warnings: drivers/misc/vexpress-syscfg.c:133:22: warning: symbol 'vexpress_syscfg_regmap_config' was not declared. Should it be static? drivers/misc/vexpress-syscfg.c:279:5: warning: symbol 'vexpress_syscfg_probe' was not declared. Should it be stat

[PATCH -next] HSI: ssi_protocol: Fix sparse non static symbol warning

From: Wei Yongjun Fixes the following sparse warning: drivers/hsi/clients/ssi_protocol.c:904:6: warning: symbol 'ssip_port_event' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/hsi/clients/ssi_protocol.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH -next] ALSA: bebob: Fix missing unlock on error in special_clk_ctl_put()

From: Wei Yongjun Add the missing unlock before return from function special_clk_ctl_put() in the error handling case. Signed-off-by: Wei Yongjun --- sound/firewire/bebob/bebob_maudio.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sound/firewire/bebob/bebob_maudio.c