[tip: x86/sgx] x86/sgx: Mark sgx_vepc_vm_ops static

The following commit has been merged into the x86/sgx branch of tip: Commit-ID: 523caed9efbb049339706b124185c9358c1b6477 Gitweb: https://git.kernel.org/tip/523caed9efbb049339706b124185c9358c1b6477 Author:Wei Yongjun AuthorDate:Mon, 12 Apr 2021 16:00:23 Committer

[tip: timers/core] clocksource/drivers/ingenic_ost: Fix return value check in ingenic_ost_probe()

The following commit has been merged into the timers/core branch of tip: Commit-ID: 2a65f7e2772613debd03fa2492e76a635aa04545 Gitweb: https://git.kernel.org/tip/2a65f7e2772613debd03fa2492e76a635aa04545 Author:Wei Yongjun AuthorDate:Mon, 08 Mar 2021 12:30:31 Committer

[PATCH -next v2] coresight: trbe: Fix return value check in arm_trbe_register_coresight_cpu()

In case of error, the function devm_kasprintf() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- v1 -> v2: remove fixes tag. --- drivers/hwtracing/coresight/coresight-trb

[PATCH -next v2] coresight: core: Make symbol 'csdev_sink' static

, this change marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- v1 -> v2: remove fixes tag and description. --- drivers/hwtracing/coresight/coresight-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwtracing/coresight/coresight-core.c

[PATCH -next] gfs2: use kmem_cache_free() instead of kfree()

memory allocated by kmem_cache_alloc() should be freed using kmem_cache_free(), not kfree(). Fixes: 7d6eec37a105 ("gfs2: Allocate bufdata object before taking log lock") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- fs/gfs2/trans.c | 6 +++--- 1 file changed, 3 insert

[PATCH -next] coresight: core: Make symbol 'csdev_sink' static

core: Add support for dedicated percpu sinks") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/hwtracing/coresight/coresight-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwtracing/coresight/coresight-core.c b/drivers/hwtracing/coresight

[PATCH -next] coresight: trbe: Fix return value check in arm_trbe_register_coresight_cpu()

In case of error, the function devm_kasprintf() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 3fbf7f011f24 ("coresight: sink: Add TRBE driver") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers

[tip: x86/core] x86/kprobes: Move 'inline' to the beginning of the kprobe_is_ss() declaration

The following commit has been merged into the x86/core branch of tip: Commit-ID: 2304d14db6595bea5292bece06c4c625b12d8f89 Gitweb: https://git.kernel.org/tip/2304d14db6595bea5292bece06c4c625b12d8f89 Author:Wei Yongjun AuthorDate:Wed, 24 Mar 2021 14:45:02 Committer

[PATCH -next] x86/kprobes: Fix old-style-declaration warning

From: Wei Yongjun Gcc reports build warning as follows: arch/x86/kernel/kprobes/core.c:940:1: warning: 'inline' is not at beginning of declaration [-Wold-style-declaration] 940 | static int nokprobe_inline kprobe_is_ss(struct kprobe_ctlblk *kcb) | ^~ This commit fix it by moving

[PATCH -next] libnvdimm/security: Make symbol '__nvdimm_security_overwrite_query' static

From: Wei Yongjun The sparse tool complains as follows: drivers/nvdimm/security.c:416:6: warning: symbol '__nvdimm_security_overwrite_query' was not declared. Should it be static? This symbol is not used outside of security.c, so this commit marks it static. Reported-by: Hulk Robot Signed

[PATCH -next] perf/arm_dmc620_pmu: Fix error return code in dmc620_pmu_device_probe()

From: Wei Yongjun Fix to return negative error code -ENOMEM from the error handling case instead of 0, as done elsewhere in this function. Fixes: 53c218da220c ("driver/perf: Add PMU driver for the ARM DMC-620 memory controller") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --

[PATCH -next] mfd: Make symbol 'atc260x_i2c_of_match' static

From: Wei Yongjun The sparse tool complains as follows: drivers/mfd/atc260x-i2c.c:45:27: warning: symbol 'atc260x_i2c_of_match' was not declared. Should it be static? This symbol is not used outside of atc260x-i2c.c, so this commit marks it static. Fixes: f7cb7fe34db9 ("mfd: Add MFD d

[PATCH -next] ASoC: rt715-sdca: Fix return value check in rt715_sdca_sdw_probe()

From: Wei Yongjun In case of error, the function devm_regmap_init_sdw_mbq() and devm_regmap_init_sdw() returns ERR_PTR() not NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: 393c52d2d109 ("ASoC: rt715-sdca: Add RT715 sdca vendor-specific driver"

[PATCH -next] ASoC: rt715-sdca: Remove unused including

From: Wei Yongjun Remove including that don't need it. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- sound/soc/codecs/rt715-sdca.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/codecs/rt715-sdca.c b/sound/soc/codecs/rt715-sdca.c index 92ad6fa408ec..20528afbdc57

[PATCH -next] coresight: etm: perf: Make symbol 'format_attr_contextid' static

From: Wei Yongjun The sparse tool complains as follows: drivers/hwtracing/coresight/coresight-etm-perf.c:61:25: warning: symbol 'format_attr_contextid' was not declared. Should it be static? This symbol is not used outside of coresight-etm-perf.c, so this commit marks it static. Reported

[PATCH -next RESEND] mfd: ene-kb3930: Make symbol 'kb3930_power_off' static

From: Wei Yongjun The sparse tool complains as follows: drivers/mfd/ene-kb3930.c:36:15: warning: symbol 'kb3930_power_off' was not declared. Should it be static? This symbol is not used outside of ene-kb3930.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Wei

[PATCH -next] clocksource/drivers/ingenic_ost: Fix return value check in ingenic_ost_probe()

From: Wei Yongjun In case of error, the function device_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: ca7b72b5a5f2 ("clocksource: Add driver for the Ingenic JZ47xx OST") Reported-by: Hulk Rob

[PATCH -next] regulator: rt4831: Fix return value check in rt4831_regulator_probe()

From: Wei Yongjun In case of error, the function dev_get_regmap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 9351ab8b0cb6 ("regulator: rt4831: Adds support for Richtek RT4831 DSV regulator") Reported-by:

[PATCH -next] phy: ingenic: Fix a typo in ingenic_usb_phy_probe()

From: Wei Yongjun Fix the return value check typo which testing the wrong variable in ingenic_usb_phy_probe(). Fixes: 31de313dfdcf ("PHY: Ingenic: Add USB PHY driver using generic PHY framework.") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/phy/ingenic/phy-ing

[PATCH -next] phy: ralink: phy-mt7621-pci: fix return value check in mt7621_pci_phy_probe()

From: Wei Yongjun Fix the return value check which testing the wrong variable in mt7621_pci_phy_probe(). Fixes: d87da32372a0 ("phy: ralink: Add PHY driver for MT7621 PCIe PHY") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/phy/ralink/phy-mt7621-pci.c | 4 ++-- 1 fi

[PATCH -next] mtd: parsers: ofpart: make symbol 'bcm4908_partitions_quirks' static

From: Wei Yongjun The sparse tool complains as follows: drivers/mtd/parsers/ofpart_core.c:25:32: warning: symbol 'bcm4908_partitions_quirks' was not declared. Should it be static? This symbol is not used outside of ofpart_core.c, so this commit marks it static. Fixes: 457da931b608 (&quo

[PATCH -next] afs: Remove unused variable in afs_req_issue_op()

: 799fbdf96cd51 ("afs: Use new fscache read helper API") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- fs/afs/file.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/afs/file.c b/fs/afs/file.c index af162d7dab5b..cf2b664a68a5 100644 --- a/fs/afs/file.c +++ b/fs/afs/file.c @@ -28

[PATCH -next] ASoC: rt1316: Fix return value check in rt1316_sdw_probe()

In case of error, the function devm_regmap_init_sdw() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: a262057df513 ("ASoC: rt1316: Add RT1316 SDCA vendor-specific driver") Reported-by: Hulk Robot Signed-o

[PATCH -next] drm/amd/display: Fix unused variable warning

hen CONFIG_DRM_AMD_DC_DCN is set, this commit fix the warning. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/

[PATCH -next] sound: n64: Fix return value check in n64audio_probe()

In case of error, the function devm_platform_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: 1448f8acf4cc ("sound: Add n64 driver") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun ---

[PATCH -next] nvmem: Fix return value check in rmem_read()

In case of error, the function memremap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 5a3fa75a4d9c ("nvmem: Add driver to expose reserved memory as nvmem") Reported-by: Hulk Robot Signed-off-by: W

[PATCH -next] soc: ti: knav_qmss_queue: Make symbol 'knav_acc_firmwares' static

-off-by: Wei Yongjun --- drivers/soc/ti/knav_qmss_queue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c index 2ac3856b8d42..7985c4197cf7 100644 --- a/drivers/soc/ti/knav_qmss_queue.c +++ b/drivers/soc/ti

[PATCH -next] soc: mediatek: Make symbol 'mtk_mutex_driver' static

The sparse tool complains as follows: drivers/soc/mediatek/mtk-mutex.c:464:24: warning: symbol 'mtk_mutex_driver' was not declared. Should it be static? This symbol is not used outside of mtk-mutex.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun

[PATCH -next] mfd: arizona: Make some symbols static

, so this commit marks them static. Fixes: e933836744a2 ("mfd: arizona: Add support for ACPI enumeration of WM5102 connected over SPI") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/mfd/arizona-spi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH -next] drm/rockchip: cdn-dp: Mark cdn_dp_resume as __maybe_unused

device *dev) |^ Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/gpu/drm/rockchip/cdn-dp-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/cdn-dp-core.c b/drivers/gpu/drm/rockchip/cdn-dp-core.c index

[PATCH -next] NTB: Drop kfree for memory allocated with devm_kzalloc

It's not necessary to free memory allocated with devm_kzalloc and using kfree leads to a double free. Fixes: 363baf7d6051 ("NTB: Add support for EPF PCI-Express Non-Transparent Bridge") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/ntb/hw/epf/ntb_hw_epf.c | 1

[tip: core/rcu] locktorture: Make function torture_percpu_rwsem_init() static

The following commit has been merged into the core/rcu branch of tip: Commit-ID: d49bed9abc3454bd123cbe974ecbeae119701b92 Gitweb: https://git.kernel.org/tip/d49bed9abc3454bd123cbe974ecbeae119701b92 Author:Wei Yongjun AuthorDate:Fri, 03 Jul 2020 13:05:27 +08:00 Committer

[tip: core/rcu] scftorture: Make symbol 'scf_torture_rand' static

The following commit has been merged into the core/rcu branch of tip: Commit-ID: 9a52a574676f8d4aa55f69319231ce6c343b00bb Gitweb: https://git.kernel.org/tip/9a52a574676f8d4aa55f69319231ce6c343b00bb Author:Wei Yongjun AuthorDate:Thu, 02 Jul 2020 09:56:50 -07:00 Committer

[tip: core/rcu] smp: Make symbol 'csd_bug_count' static

The following commit has been merged into the core/rcu branch of tip: Commit-ID: 2b722160f1a7929f38dfb648c7bbb45f96e65a5b Gitweb: https://git.kernel.org/tip/2b722160f1a7929f38dfb648c7bbb45f96e65a5b Author:Wei Yongjun AuthorDate:Mon, 06 Jul 2020 21:49:41 +08:00 Committer

[PATCH -next] soc: ti: pruss: Fix return value check

In case of error, the function of_device_get_match_data() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: ba59c9b43c86 ("soc: ti: pruss: support CORECLK_MUX and IEPCLK_MUX") Reported-by: Hulk Robot Signed-o

[PATCH -next] mfd: ene-kb3930: Make symbol 'kb3930_power_off' static

B3930 Embedded Controller") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/mfd/ene-kb3930.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/ene-kb3930.c b/drivers/mfd/ene-kb3930.c index 1c32ff586816..eb7312bd6361 100644 --- a/drivers/mfd/ene-kb39

[PATCH -next] riscv/mm/fault: fix old-style-declaration warning

' after 'static'. Signed-off-by: Wei Yongjun --- arch/riscv/mm/fault.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/riscv/mm/fault.c b/arch/riscv/mm/fault.c index a23eaf5ce95c..a173432ccf82 100644 --- a/arch/riscv/mm/fault.c +++ b/arch/riscv/mm/fault.c @@ -78,7 +78,7

[PATCH -next] binderfs: make symbol 'binderfs_fs_parameters' static

t api") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/android/binderfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/android/binderfs.c b/drivers/android/binderfs.c index 7b76fefde3f8..7b4f154f07e6 100644 --- a/drivers/android/binderfs.c +++ b/drive

[PATCH] kernel/relay.c: fix memleak on destroy relay channel

alloced in relay_open() by alloc_percpu() but not free while destroy the relay channel. Fix it by adding free_percpu() before return from relay_destroy_channel(). Fixes: 017c59c042d0 ("relay: Use per CPU constructs for the relay channel buffer pointers") Reported-by: Hulk Robot Signed-off-by:

[PATCH] memory: jz4780-nemc: Fix return value check in jz4780_nemc_probe()

In case of error, the function devm_ioremap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: f046e4a3f0b9 ("memory: jz4780_nemc: Only request IO memory the driver will use") Reported-by: Hulk Robot Signed-o

[PATCH -next] habanalabs: make some functions static

are not used outside of source file, so this commit marks them static. Fixes: a04b7cd97eef ("habanalabs: create internal CB pool") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/misc/habanalabs/gaudi/gaudi.c | 2 +- drivers/misc/habanalabs/goya/goya.c | 2 +- 2 file changed, 2

[PATCH -next] irqchip/imx-intmux: Fix irqdata regs save in imx_intmux_runtime_suspend()

on the stack in imx_intmux_runtime_suspend(), which means a nop. this commit fix to save regs to the right place. Fixes: bb403111e017 ("irqchip/imx-intmux: Implement intmux runtime power management") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/irqchip/irq-imx-int

[PATCH -next] powerpc/powernv/sriov: Remove unused but set variable 'phb'

-by: Hulk Robot Signed-off-by: Wei Yongjun --- arch/powerpc/platforms/powernv/pci-sriov.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/platforms/powernv/pci-sriov.c b/arch/powerpc/platforms/powernv/pci-sriov.c index 8404d8c3901d..7894745fd4f8 100644 --- a/arch/powerpc/platforms

[PATCH -next] drm: xlnx: Fix typo in parameter description

Fix typo in parameter description. Fixes: d76271d22694 ("drm: xlnx: DRM/KMS driver for Xilinx ZynqMP DisplayPort Subsystem") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/gpu/drm/xlnx/zynqmp_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[PATCH -next] mtd: fix missing unlock on error in mtdchar_compat_ioctl()

Add the missing unlock before return from function mtdchar_compat_ioctl() in the error handling case. Fixes: 210bec567936 ("mtd: properly check all write ioctls for permissions") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/mtd/mtdchar.c | 6 -- 1 file

[PATCH -next] drm/nouveau/kms/nvd9-: Fix file release memory leak

When using single_open() for opening, single_release() should be used instead of seq_release(), otherwise there is a memory leak. Fixes: 12885ecbfe62 ("drm/nouveau/kms/nvd9-: Add CRC support") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/gpu/drm/nouveau/dispnv50

[PATCH -next] mtd: rawnand: pasemi: Make pasemi_device_ready() static

The sparse tool complains as follows: drivers/mtd/nand/raw/pasemi_nand.c:71:5: warning: symbol 'pasemi_device_ready' was not declared. Should it be static? This function is not used outside of pasemi_nand.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun

[PATCH -next] soc: TI knav_qmss: make symbol 'knav_acc_range_ops' static

The sparse tool complains as follows: drivers/soc/ti/knav_qmss_acc.c:453:23: warning: symbol 'knav_acc_range_ops' was not declared. Should it be static? 'knav_acc_range_ops' is not used outside of knav_qmss_acc.c, so marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun

[PATCH -next] bootconfig: Make symbol 'xbc_namebuf' static

Fix sparse build warning: init/main.c:305:6: warning: symbol 'xbc_namebuf' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- init/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/init/main.c b/init/main.c index 9127b240fd26..6a24981c727f 100644

[PATCH -next] libnvdimm/security: Make __nvdimm_security_overwrite_query() static

-by: Wei Yongjun --- drivers/nvdimm/security.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/nvdimm/security.c b/drivers/nvdimm/security.c index 89b85970912d..11fb5ada70ad 100644 --- a/drivers/nvdimm/security.c +++ b/drivers/nvdimm/security.c @@ -413,7 +413,7 @@ static int

[PATCH -next] irqchip: mips-gic: Make some symbols static

-gic.c:634:5: warning: symbol 'gic_ipi_domain_match' was not declared. Should it be static? Those symbols are not used outside of irq-mips-gic.c, so marks them static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/irqchip/irq-mips-gic.c | 10 +- 1 file changed, 5

Re: [PATCH v2 net] rtnetlink: Fix memory(net_device) leak when ->newlink fails

On 2020/7/14 15:32, Weilong Chen wrote: > When vlan_newlink call register_vlan_dev fails, it might return error > with dev->reg_state = NETREG_UNREGISTERED. The rtnl_newlink should > free the memory. But currently rtnl_newlink only free the memory which > state is NETREG_UNINITIALIZED. > >

[PATCH -next] iommu: Make some functions static

of iommu.c, so mark them static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/iommu/iommu.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c index 1ed1e14a1f0c..40947f5bc6c5 100644 --- a/drivers/iommu/iommu.c +++ b

[PATCH -next] ftrace: Make ftrace_profile_pages_init() static

The sparse tool complains as follows: kernel/trace/ftrace.c:587:5: warning: symbol 'ftrace_profile_pages_init' was not declared. Should it be static? ftrace_profile_pages_init() is not used out of ftrace.c, so marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- kernel

[PATCH -next] device-dax: make dev_dax_kmem_probe() static

sparse report warning as follows: drivers/dax/kmem.c:22:5: warning: symbol 'dev_dax_kmem_probe' was not declared. Should it be static? dev_dax_kmem_probe() is not used outside of kmem.c, so marks it static. Signed-off-by: Wei Yongjun --- drivers/dax/kmem.c | 2 +- 1 file changed, 1 insertion

[PATCH -next] lib/test_lockup.c: make symbol 'test_works' static

Fix sparse build warning: lib/test_lockup.c:403:1: warning: symbol '__pcpu_scope_test_works' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- lib/test_lockup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/test_lockup.c b

[PATCH -next] smp: Make symbol 'csd_bug_count' static

The sparse tool complains as follows kernel/smp.c:107:10: warning: symbol 'csd_bug_count' was not declared. Should it be static? This variable is not used outside of smp.c, s this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- kernel/smp.c | 2 +- 1 file

[PATCH -next] smp: Fix unused-but-set-variable warning

Robot Signed-off-by: Wei Yongjun --- kernel/smp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/smp.c b/kernel/smp.c index 6ec6c9578225..0130bdcf6d26 100644 --- a/kernel/smp.c +++ b/kernel/smp.c @@ -123,10 +123,10 @@ static void csd_lock_record(call_single_data_t *csd

[PATCH -next] locktorture: make function torture_percpu_rwsem_init() static

The sparse tool complains as follows: kernel/locking/locktorture.c:569:6: warning: symbol 'torture_percpu_rwsem_init' was not declared. Should it be static? And this function is not used outside of locktorture.c, so this commit marks it static. Signed-off-by: Wei Yongjun --- kernel/locking

[PATCH -next] ASoC: ti: j721e-evm: Fix missing unlock on error in j721e_audio_hw_params()

Add the missing unlock before return from function j721e_audio_hw_params() in the error handling case. Fixes: 6748d0559059 ("ASoC: ti: Add custom machine driver for j721e EVM (CPB and IVI)") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- sound/soc/ti/j721e-evm.c | 2 +- 1 fi

[PATCH -next] dm zoned: fix unused but set variable warnings

:24: warning: variable nr_unmap_rnd set but not used [-Wunused-but-set-variable] 504 | unsigned int p_unmap, nr_unmap_rnd = 0, nr_rnd = 0; |^~~~ Fixes: f97809aec589 ("dm zoned: per-device reclaim") Signed-off-by: Wei Yongjun --- drivers/m

[PATCH -next] lib/test_bits: make some functions static

' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- lib/test_bits.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/test_bits.c b/lib/test_bits.c index 89e0ea83511f..c9368a2314e7 100644 --- a/lib/test_bits.c +++ b/lib/test_bits.c @@ -7,7

[PATCH -next] scftorture: Make symbol 'scf_torture_rand' static

Fix sparse build warning: kernel/scftorture.c:124:1: warning: symbol '__pcpu_scope_scf_torture_rand' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- kernel/scftorture.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel

[PATCH -next] kcov: make some symbols static

'kcov_remote_softirq_stop' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- kernel/kcov.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/kcov.c b/kernel/kcov.c index 6afae0bcbac4..6b8368be89c8 100644 --- a/kernel/kcov.c +++ b/kernel

[PATCH -next] dm: remove unused variable 'md'

o avoid build warning. Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/md/dm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/md/dm.c b/drivers/md/dm.c index a8d83d76fa9f..a02842afe358 100644 --- a/drivers/md/dm.c +++ b/drivers/md/dm.c @@ -1293,7 +1293,6 @@ static blk_q

[PATCH -next] f2fs: make __allocate_new_segment() static

From: Hulk Robot From: Hulk Robot Fix sparse build warning: fs/f2fs/segment.c:2736:6: warning: symbol '__allocate_new_segment' was not declared. Should it be static? Signed-off-by: Hulk Robot --- fs/f2fs/segment.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] drm/panel: otm8009a: Drop unnessary backlight_device_unregister()

It's not necessary to unregister backlight device which registered with devm_backlight_device_register(). Fixes: 12a6cbd4f3f1 ("drm/panel: otm8009a: Use new backlight API") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/gpu/drm/panel/panel-orisetech-otm8009a.c | 1

[PATCH -next] ASoC: mmp-sspa: Fix return value check in asoc_mmp_sspa_probe()

In case of error, the function devm_ioremap() 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 --- sound/soc/pxa/mmp-sspa.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH -next] bus: arm-integrator-lm: Fix return value check in integrator_ap_lm_probe()

In case of error, the function of_find_matching_node() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: ccea5e8a5918 ("bus: Add driver for Integrator/AP logic modules") Reported-by: Hulk Robot Signed-off-by: W

[PATCH -next] iommu/sun50i: Fix return value check in sun50i_iommu_probe()

In case of error, the function devm_platform_ioremap_resource() returns ERR_PTR() not NULL. The NULL test in the return value check must be replaced with IS_ERR(). Fixes: 4100b8c229b3 ("iommu: Add Allwinner H6 IOMMU driver") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- dri

[PATCH -next] ASoC: SOF: core: fix error return code in sof_probe_continue()

Fix to return negative error code -ENOMEM from the IPC init error handling case instead of 0, as done elsewhere in this function. Fixes: c16211d6226d ("ASoC: SOF: Add Sound Open Firmware driver core") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- sound/soc/sof/core.c | 1

[PATCH -next] mm/hmm/test: fix missing unlock on error in dmirror_migrate_finalize_and_map()

Add the missing unlock before return from function dmirror_migrate_finalize_and_map() in the error handling case. Fixes: 5d5e54be8a1e ("mm/hmm/test: add selftest driver for HMM") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- lib/test_hmm.c | 4 +++- 1 file changed, 3 insert

[PATCH -next] mm/hmm/test: fix error return code in hmm_dmirror_init()

Fix to return negative error code -ENOMEM from the alloc_page() error handling case instead of 0, as done elsewhere in this function. Fixes: 5d5e54be8a1e ("mm/hmm/test: add selftest driver for HMM") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- lib/test_hmm.c | 4 +++- 1 fi

[tip: locking/kcsan] kcsan: Use GFP_ATOMIC under spin lock

The following commit has been merged into the locking/kcsan branch of tip: Commit-ID: 52785b6ae8eded7ac99d65c92d989b702e5b4376 Gitweb: https://git.kernel.org/tip/52785b6ae8eded7ac99d65c92d989b702e5b4376 Author:Wei Yongjun AuthorDate:Fri, 17 Apr 2020 02:58:37 Committer

[PATCH -next] gnss: sirf: fix error return code in sirf_probe()

Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Fixes: d2efbbd18b1e ("gnss: add driver for sirfstar-based receivers") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/gnss/sirf.c | 8 ++-- 1 file

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

Using devm_snd_soc_register_component() can make the code shorter and cleaner. Signed-off-by: Wei Yongjun --- sound/soc/codecs/rt5677-spi.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/sound/soc/codecs/rt5677-spi.c b/sound/soc/codecs/rt5677-spi.c index

[PATCH -next] ipack: tpci200: fix error return code in tpci200_register()

Fix to return negative error code -ENOMEM from the ioremap() error handling case instead of 0, as done elsewhere in this function. Fixes: 43986798fd50 ("ipack: add error handling for ioremap_nocache") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/ipack/carriers/tpc

[PATCH -next] phy: ti: j721e-wiz: Fix some error return code in wiz_probe()

Fix to return negative error code from some error handling cases instead of 0, as done elsewhere in this function. Fixes: 091876cc355d ("phy: ti: j721e-wiz: Add support for WIZ module present in TI J721E SoC") Reported-by: Hulk Robot Signed-off-by: Wei Yongjun --- drivers/phy/ti

[PATCH -next] visorbus: fix error return code in visorchipset_init()

Fix to return negative error code -ENODEV from the visor_check_channel() error handling case instead of 0. Also change the error code to -ENOMEM in kzalloc() error case. Signed-off-by: Wei Yongjun --- drivers/visorbus/visorchipset.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions

[PATCH -next] soc: mediatek: Missing platform_device_unregister() on error in mtk_mmsys_probe()

Add the missing platform_device_unregister() before return from mtk_mmsys_probe() in the error handling case. Fixes: 667c769246b0 ("soc / drm: mediatek: Fix mediatek-drm device probing") Signed-off-by: Wei Yongjun --- drivers/soc/mediatek/mtk-mmsys.c | 4 +++- 1 file changed, 3 insert

[PATCH -next v2] staging: kpc2000: fix error return code in kp2000_pcie_probe()

Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Also removed var 'rv' since we can use 'err' instead. Fixes: 7dc7967fc39a ("staging: kpc2000: add initial set of Daktronics drivers") Signed-off-by: Wei Yongjun --- v1

Re: [PATCH -next] staging: kpc2000: fix error return code in kp2000_pcie_probe()

On 2020/5/6 20:57, Dan Carpenter wrote: > On Wed, May 06, 2020 at 12:52:55PM +0000, Wei Yongjun wrote: >> Fix to return a negative error code from the error handling >> case instead of 0, as done elsewhere in this function. Also >> removed var 'rv' since we can use 'err'

[PATCH -next] staging: kpc2000: fix error return code in kp2000_pcie_probe()

Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Also removed var 'rv' since we can use 'err' instead. Signed-off-by: Wei Yongjun --- drivers/staging/kpc2000/kpc2000/core.c | 7 +++ 1 file changed, 3 insertions(+), 4

[PATCH -next] firmware: imx: scu: Fix possible memory leak in imx_scu_probe()

'chan_name' is malloced in imx_scu_probe() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Fixes: edbee095fafb ("firmware: imx: add SCU firmware driver support") Signed-off-by: Wei Yongjun --- drivers/firmware/imx/imx-scu.c | 1

[PATCH -next v2] drm/mcde: dsi: Fix return value check in mcde_dsi_bind()

The of_drm_find_bridge() function returns NULL on error, it doesn't return error pointers so this check doesn't work. Fixes: 5fc537bfd000 ("drm/mcde: Add new driver for ST-Ericsson MCDE") Signed-off-by: Wei Yongjun --- v1 - > v2: add fixes and fix the subject --- drivers/gpu/drm/mc

[PATCH -next] drm/vboxvideo: Fix a NULL vs IS_ERR() check in vbox_hw_init()

The devm_gen_pool_create() function returns ERR_PTR() on error, it doesn't return NULL so this check doesn't work. Fixes: 4cc9b565454b ("drm/vboxvideo: Use devm_gen_pool_create") Signed-off-by: Wei Yongjun --- drivers/gpu/drm/vboxvideo/vbox_main.c | 4 ++-- 1 file changed, 2 insert

[PATCH -next v2] mailbox: zynqmp-ipi: Fix NULL vs IS_ERR() check in zynqmp_ipi_mbox_probe()

In case of error, the function devm_ioremap() returns NULL pointer not ERR_PTR(). So we should check whether the return value of devm_ioremap() is NULL instead of IS_ERR. Fixes: 4981b82ba2ff ("mailbox: ZynqMP IPI mailbox controller") Signed-off-by: Wei Yongjun --- v1 ->v2: fix th

[PATCH -next] mailbox: Fix NULL vs IS_ERR() check in zynqmp_ipi_mbox_probe()

In case of error, the function devm_ioremap() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 4981b82ba2ff ("mailbox: ZynqMP IPI mailbox controller") Signed-off-by: Wei Yongjun --- drivers/mailbox/zynqmp-ipi

[PATCH -next] staging: pi433: fix error return code in pi433_probe()

Fix to return negative error code -ENOMEM from cdev alloc failed error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/staging/pi433/pi433_if.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/pi433/pi433_if.c b/drivers

[PATCH -next] soc: ti: knav_qmss_queue: fix error return code in knav_queue_probe()

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

[PATCH -next] drm/mcde: dsi: Fix return value check in dev_err()

In case of error, the function of_drm_find_bridge() 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/mcde/mcde_dsi.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions

[PATCH -next] stm class: dummy_stm: fix error return code in dummy_stm_init()

Fix to return error code -ENOMEM from the error handling case instead of 0(ret can be overwritted to 0 in for loop). Signed-off-by: Wei Yongjun --- drivers/hwtracing/stm/dummy_stm.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/hwtracing/stm/dummy_stm.c b

[PATCH -next] stm class: stm_heartbeat: fix error return code

Fix to return error code -ENOMEM from the error handling case instead of 0(ret can be overwritted to 0 in for loop). Signed-off-by: Wei Yongjun --- drivers/hwtracing/stm/heartbeat.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/hwtracing/stm/heartbeat.c b

[PATCH -next] soc: ti: omap-prm: fix return value check in omap_prm_probe()

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: 3e99cb214f03 ("soc: ti: add initial PRM driver with reset control support") Signed-off-by: Wei Yongjun --

[PATCH -next] phy: lantiq: vrx200-pcie: fix error return code in ltq_vrx200_pcie_phy_power_on()

Fix to return a negative error code from the error handling case instead of 0, as done elsewhere in this function. Fixes: e52a632195bf ("phy: lantiq: vrx200-pcie: add a driver for the Lantiq VRX200 PCIe PHY") Signed-off-by: Wei Yongjun --- drivers/phy/lantiq/phy-lantiq-vrx200-pcie.c

[PATCH -next] ASoC: SOF: imx8: Fix return value check in imx8_probe()

In case of error, the function devm_ioremap_wc() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Fixes: 202acc565a1f ("ASoC: SOF: imx: Add i.MX8 HW support") Signed-off-by: Wei Yongjun --- sound/soc/sof/imx/imx8.c |

[PATCH -next] staging: fsl-dpaa2/ethsw: Remove useless set memory to zero use memset()

The memory return by kzalloc() has already be set to zero, so remove useless memset(0). Signed-off-by: Wei Yongjun --- drivers/staging/fsl-dpaa2/ethsw/ethsw.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/fsl-dpaa2/ethsw/ethsw.c b/drivers/staging/fsl-dpaa2/ethsw/ethsw.c

[PATCH -next] intel_th: msu: Fix possible memory leak in mode_store()

'mode' is malloced in mode_store() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Fixes: 615c164da0eb ("intel_th: msu: Introduce buffer interface") Signed-off-by: Wei Yongjun --- drivers/hwtracing/intel_th/msu.c | 4 +++- 1 file

[PATCH -next] ASoC: SOF: debug: fix possible memory leak in sof_dfsentry_write()

'string' is malloced in sof_dfsentry_write() and should be freed before leaving from the error handling cases, otherwise it will cause memory leak. Fixes: 091c12e1f50c ("ASoC: SOF: debug: add new debugfs entries for IPC flood test") Signed-off-by: Wei Yongjun --- sound/soc/sof/d

[PATCH] unicore32: dma: fix to pass correct device identity to free_irq()

free_irq() expects the same device identity that was passed to corresponding request_irq(), otherwise the IRQ is not freed. Fixes: 10c9c10c3151 ("unicore32 core architecture: mm related: consistent device DMA handling") Signed-off-by: Wei Yongjun --- arch/unicore32/kernel/dma.c | 2

  1   2   3   4   5   6   7   8   9   10   >