[tip: sched/core] sched/fair: Move update_nohz_stats() to the CONFIG_NO_HZ_COMMON block to simplify the code & fix an unused function warning

The following commit has been merged into the sched/core branch of tip: Commit-ID: 3f5ad91488e813026f8c5f46b839e91a83912703 Gitweb: https://git.kernel.org/tip/3f5ad91488e813026f8c5f46b839e91a83912703 Author:YueHaibing AuthorDate:Mon, 29 Mar 2021 22:40:29 +08:00 Committer

[PATCH -next 1/5] crypto: atmel-tdes - Remove redundant dev_err call in atmel_tdes_probe()

There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: YueHaibing --- drivers/crypto/atmel-tdes.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/crypto/atmel-tdes.c b/drivers/crypto/atmel-tdes.c

[PATCH -next 5/5] crypto: ccree - Remove redundant dev_err call in init_cc_resources()

There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: YueHaibing --- drivers/crypto/ccree/cc_driver.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/crypto/ccree/cc_driver.c b

[PATCH -next 0/5] Remove redundant dev_err call

This patchset remove some redundant dev_err calls YueHaibing (5): crypto: atmel-tdes - Remove redundant dev_err call in atmel_tdes_probe() crypto: img-hash - Remove redundant dev_err call in img_hash_probe() crypto: ux500 - Remove redundant dev_err calls crypto: keembay - Remove

[PATCH -next 3/5] crypto: ux500 - Remove redundant dev_err calls

There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: YueHaibing --- drivers/crypto/ux500/cryp/cryp_core.c | 1 - drivers/crypto/ux500/hash/hash_core.c | 1 - 2 files changed, 2 deletions(-) diff --git a

[PATCH -next 2/5] crypto: img-hash - Remove redundant dev_err call in img_hash_probe()

devm_platform_ioremap_resource() and devm_ioremap_resource() will print err msg while failing, so the redundant dev_err call can be removed. Signed-off-by: YueHaibing --- drivers/crypto/img-hash.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/crypto/img-hash.c b/drivers/crypto

[PATCH -next 4/5] crypto: keembay - Remove redundant dev_err calls

There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: YueHaibing --- drivers/crypto/keembay/keembay-ocs-aes-core.c | 4 +--- drivers/crypto/keembay/keembay-ocs-hcu-core.c | 4 +--- 2 files changed, 2 insertions

[PATCH v2 -next] dmaengine: at_xdmac: Remove unused inline function at_xdmac_csize()

commit 765c37d87669 ("dmaengine: at_xdmac: rework slave configuration part") left behind this, so can remove it. Signed-off-by: YueHaibing Reviewed-by: Tudor Ambarus --- v2: Fix commit log --- drivers/dma/at_xdmac.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/d

[PATCH -next] dmaengine: at_xdmac: Remove unused inline function at_xdmac_csize()

765c37d87669 ("dmaengine: at_xdmac: rework slave configuration part") left behind this, so can remove it. Signed-off-by: YueHaibing --- drivers/dma/at_xdmac.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c index fe

[PATCH -next] PM: runtime: Replace inline function pm_runtime_callbacks_present()

commit 9a7875461fd0 ("PM: runtime: Replace pm_runtime_callbacks_present()") forget to change the inline version. Fixes: 9a7875461fd0 ("PM: runtime: Replace pm_runtime_callbacks_present()") Signed-off-by: YueHaibing --- include/linux/pm_runtime.h | 2 +- 1 file change

[PATCH -next] ide: Remove unused inline function ide_dma_verbose()

commit 3ab7efe8e2cb ("ide: DMA reporting and validity checking fixes (take 3)") left behind this, so can remove it. Signed-off-by: YueHaibing --- include/linux/ide.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/linux/ide.h b/include/linux/ide.h index 6bc6ba57b505..02

[PATCH -next] mfd: twl: Remove unused inline function twl4030charger_usb_en()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- include/linux/mfd/twl.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/include/linux/mfd/twl.h b/include/linux/mfd/twl.h index 089e8942223a..8871cc5188a0 100644 --- a/include/linux/mfd/twl.h +++ b/include/linux

[PATCH -next] freezer: Remove unused inline function try_to_freeze_nowarn()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- include/linux/freezer.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/linux/freezer.h b/include/linux/freezer.h index 27828145ca09..0621c5f86c39 100644 --- a/include/linux/freezer.h +++ b/include/linux

[PATCH -next] RDMA/uverbs: Fix -Wunused-function warning

:930:1: warning: ‘_uverbs_get_const_signed’ defined but not used [-Wunused-function] _uverbs_get_const_signed(s64 *to, const struct uverbs_attr_bundle *attrs_bundle, ^~~~ Make these functions inline to fix this warnings. Signed-off-by: YueHaibing --- include/rdma

Re: [PATCH -next] powerpc/eeh: Remove unused inline function eeh_dev_phb_init_dynamic()

On 2021/3/26 13:08, Daniel Axtens wrote: > Hi, > >> commit 475028efc708 ("powerpc/eeh: Remove eeh_dev_phb_init_dynamic()") >> left behind this, so can remove it. > > I had a look: the inline that you are removing here is for the > !CONFIG_EEH case, which explains why it was missed the first time.

[PATCH -next] powerpc/eeh: Add correct inline functions

pseries_eeh_add_device_early()/pseries_eeh_add_device_tree_early() is never used since adding, however pseries_eeh_init_edev() and pseries_eeh_init_edev_recursive() need their inline versions. Fixes: b6eebb093cad ("powerpc/eeh: Make early EEH init pseries specific") Signed-off-by:

[PATCH -next] misc/pvpanic: Make some symbols static

Fix sparse warnings: drivers/misc/pvpanic/pvpanic.c:28:18: warning: symbol 'pvpanic_list' was not declared. Should it be static? drivers/misc/pvpanic/pvpanic.c:29:12: warning: symbol 'pvpanic_lock' was not declared. Should it be static? Signed-off-by: YueHaibing ---

[PATCH -next] ASoC: dwc: Fix -Wmissing-prototypes warnings

) ^~~ Change these to inline functions to fix the warnings. Signed-off-by: YueHaibing --- sound/soc/dwc/local.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/dwc/local.h b/sound/soc/dwc/local.h index 91dc70a826f8..1c361eb6127e 100644 --- a/sound/soc/dwc/local.h

[PATCH -next] ASoC: amd: acp-da7219-max98357a: Fix -Wunused-variable warning

: ‘cz_card’ defined but not used [-Wunused-variable] static struct snd_soc_card cz_card = { Use #ifdef block to guard this. Signed-off-by: YueHaibing --- sound/soc/amd/acp-da7219-max98357a.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/sound/soc/amd/acp-da7219

[PATCH v2 -next] sched/fair: Fix unused function warning

: 0826530de3cb ("sched/fair: Remove update of blocked load from newidle_balance") Signed-off-by: YueHaibing Reviewed-by: Vincent Guittot --- v2: Add Fixes tag, also move update_nohz_stats() closer to its only caller _nohz_idle_balance() --- kernel/sched/f

[PATCH -next] sched/fair: Fix unused function warning

-by: YueHaibing --- kernel/sched/fair.c | 40 ++-- 1 file changed, 18 insertions(+), 22 deletions(-) diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 6d73bdbb2d40..c7a7ef97d167 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -8395,28

[PATCH -next] IB/srpt: Fix passing zero to 'PTR_ERR'

Fix smatch warning: drivers/infiniband/ulp/srpt/ib_srpt.c:2341 srpt_cm_req_recv() warn: passing zero to 'PTR_ERR' Use PTR_ERR_OR_ZERO instead of PTR_ERR Fixes: 847462de3a0a ("IB/srpt: Fix srpt_cm_req_recv() error path (1/2)") Signed-off-by: YueHaibing --- drivers

[PATCH -next] powerpc/smp: Remove unused inline functions

commit 441c19c8a290 ("powerpc/kvm/book3s_hv: Rework the secondary inhibit code") left behind this, so can remove it. Signed-off-by: YueHaibing --- arch/powerpc/include/asm/smp.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/include/asm/smp.h b/arch/powerpc/include

[PATCH -next] powerpc/eeh: Remove unused inline function eeh_dev_phb_init_dynamic()

commit 475028efc708 ("powerpc/eeh: Remove eeh_dev_phb_init_dynamic()") left behind this, so can remove it. Signed-off-by: YueHaibing --- arch/powerpc/include/asm/eeh.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/e

[PATCH net-next] netfilter: conntrack: Remove unused variable declaration

commit e97c3e278e95 ("tproxy: split off ipv6 defragmentation to a separate module") left behind this. Signed-off-by: YueHaibing --- include/net/netfilter/ipv6/nf_conntrack_ipv6.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/net/netfilter/ipv6/nf_conntrack_ipv6.h b/i

Re: [PATCH v2 -next] IB/mlx5: Add missing error code

On 2021/2/22 21:23, Leon Romanovsky wrote: > On Mon, Feb 22, 2021 at 08:23:43PM +0800, YueHaibing wrote: >> Set err to -ENOMEM if kzalloc fails instead of 0. >> >> Fixes: 759738537142 ("IB/mlx5: Enable subscription for device events over >> DEVX") >> Si

[PATCH v2 -next] IB/mlx5: Add missing error code

Set err to -ENOMEM if kzalloc fails instead of 0. Fixes: 759738537142 ("IB/mlx5: Enable subscription for device events over DEVX") Signed-off-by: YueHaibing --- drivers/infiniband/hw/mlx5/devx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/infiniba

[PATCH -next] IB/mlx5: Add missing error code

Set err to -ENOMEM if kzalloc fails instead of 0. Fixes: 759738537142 ("IB/mlx5: Enable subscription for device events over DEVX") Signed-off-by: YueHaibing --- drivers/infiniband/hw/mlx5/devx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/infiniba

[PATCH] cifs: Fix inconsistent IS_ERR and PTR_ERR

ned-off-by: YueHaibing --- fs/cifs/cifs_swn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/cifs/cifs_swn.c b/fs/cifs/cifs_swn.c index d35f599aa00e..f2d730fffccb 100644 --- a/fs/cifs/cifs_swn.c +++ b/fs/cifs/cifs_swn.c @@ -272,7 +272,7 @@ static struct cifs_swn_reg *cifs_fi

[PATCH] nfp: Fix passing zero to 'PTR_ERR'

nfp_cpp_from_nfp6000_pcie() returns ERR_PTR() and never returns NULL. The NULL test should be removed, also return correct err. Fixes: 63461a028f76 ("nfp: add the PF driver") Signed-off-by: YueHaibing --- drivers/net/ethernet/netronome/nfp/nfp_main.c | 4 +--- 1 file changed, 1 inser

[PATCH] net: dsa: sja1105: Fix return value check in sja1105_ptp_clock_register()

quot;) Signed-off-by: YueHaibing --- drivers/net/dsa/sja1105/sja1105_ptp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/dsa/sja1105/sja1105_ptp.c b/drivers/net/dsa/sja1105/sja1105_ptp.c index 1b90570b257b..1e41d491c854 100644 --- a/drivers/net/dsa/sja1105/sja1105_ptp.c +

[PATCH] IB/srpt: Fix passing zero to 'PTR_ERR'

Fix smatch warning: drivers/infiniband/ulp/srpt/ib_srpt.c:2341 srpt_cm_req_recv() warn: passing zero to 'PTR_ERR' Use PTR_ERR_OR_ZERO instead of PTR_ERR Fixes: 847462de3a0a ("IB/srpt: Fix srpt_cm_req_recv() error path (1/2)") Signed-off-by: YueHaibing --- drivers

[PATCH] nbd: Fix passing zero to 'PTR_ERR' warning

Fix smatch warning: drivers/block/nbd.c:698 nbd_read_stat() warn: passing zero to 'ERR_PTR' sock_xmit() never return 0, remove check 0 to fix this warning. Signed-off-by: YueHaibing --- drivers/block/nbd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/b

[PATCH] loop: Fix passing zero to 'PTR_ERR' warning

Fix smatch warning: drivers/block/loop.c:799 loop_attr_backing_file_show() warn: passing zero to 'PTR_ERR' file_path() never returns 0, so use IS_ERR instead of IS_ERR_OR_NULL to fix this. Signed-off-by: YueHaibing --- drivers/block/loop.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH] net: macb: Fix passing zero to 'PTR_ERR'

Check PTR_ERR with IS_ERR to fix this. Fixes: cd5afa91f078 ("net: macb: Add null check for PCLK and HCLK") Signed-off-by: YueHaibing --- drivers/net/ethernet/cadence/macb_main.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/net/ethern

[PATCH net-next] net/mlx5: Fix passing zero to 'PTR_ERR'

never returns NULL, so NULL test should be removed. Signed-off-by: YueHaibing --- drivers/net/ethernet/mellanox/mlx5/core/esw/acl/egress_lgcy.c | 2 +- drivers/net/ethernet/mellanox/mlx5/core/esw/acl/egress_ofld.c | 2 +- drivers/net/ethernet/mellanox/mlx5/core/esw/acl/ingress_lgcy.c | 2 +-

[PATCH v2] drm/bridge: tpd12s015: Fix irq registering in tpd12s015_probe

e83f ("drm/bridge: Add driver for the TI TPD12S015 HDMI level shifter") Signed-off-by: YueHaibing --- v2: Add checking for >= 0 and update commit message --- drivers/gpu/drm/bridge/ti-tpd12s015.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/b

Re: [PATCH] drm/bridge: tpd12s015: Fix irq registering in tpd12s015_probe

On 2020/11/2 14:57, Tomi Valkeinen wrote: > On 31/10/2020 09:19, Sam Ravnborg wrote: >> Hi YueHaibing >> >> Thanks for the fix. Appreciated but please update as per comments below. >> >> On Sat, Oct 31, 2020 at 11:16:48AM +0800, YueHaibing wrote: >>> gpiod

Re: [PATCH] usb: dwc2: Fix unused label warning

On 2020/11/2 15:04, Minas Harutyunyan wrote: > On 10/31/2020 10:03 AM, YueHaibing wrote: >> drivers/usb/dwc2/platform.c: In function ‘dwc2_driver_probe’: >> drivers/usb/dwc2/platform.c:611:1: warning: label ‘error_debugfs’ defined >> but not used [-Wunused-labe

Re: [PATCH] drm/bridge: tpd12s015: Fix irq registering in tpd12s015_probe

On 2020/10/31 15:19, Sam Ravnborg wrote: > Hi YueHaibing > > Thanks for the fix. Appreciated but please update as per comments below. > > On Sat, Oct 31, 2020 at 11:16:48AM +0800, YueHaibing wrote: >> gpiod_to_irq() return negative value in case of error, >> the ex

[PATCH] usb: dwc2: Fix unused label warning

drivers/usb/dwc2/platform.c: In function ‘dwc2_driver_probe’: drivers/usb/dwc2/platform.c:611:1: warning: label ‘error_debugfs’ defined but not used [-Wunused-label] error_debugfs: ^ Move label 'error_debugfs' to ifdef block. Signed-off-by: YueHaibing --- driver

[PATCH net-next] openvswitch: Use IS_ERR instead of IS_ERR_OR_NULL

Fix smatch warning: net/openvswitch/meter.c:427 ovs_meter_cmd_set() warn: passing zero to 'PTR_ERR' dp_meter_create() never returns NULL, use IS_ERR instead of IS_ERR_OR_NULL to fix this. Signed-off-by: YueHaibing --- net/openvswitch/meter.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH] pstore: Fix passing zero to 'PTR_ERR' warning

Fix smatch warning: fs/pstore/platform.c:320 allocate_buf_for_compression() warn: passing zero to 'PTR_ERR' crypto_alloc_comp() never return NULL, use IS_ERR instead of IS_ERR_OR_NULL to fix this. Signed-off-by: YueHaibing --- fs/pstore/platform.c | 2 +- 1 file changed, 1 inser

[PATCH] mm/zswap: Fix passing zero to 'PTR_ERR' warning

Fix smatch warning: mm/zswap.c:425 zswap_cpu_comp_prepare() warn: passing zero to 'PTR_ERR' crypto_alloc_comp() never return NULL, use IS_ERR instead of IS_ERR_OR_NULL to fix this. Fixes: f1c54846ee45 ("zswap: dynamic pool creation") Signed-off-by: YueHaibing --- mm/zs

[PATCH] spi: spi-mem: Fix passing zero to 'PTR_ERR' warning

Fix smatch warning: drivers/spi/spi-mem.c:746 spi_mem_probe() warn: passing zero to 'PTR_ERR' Fixes: 5d27a9c8ea9e ("spi: spi-mem: Extend the SPI mem interface to set a custom memory name") Signed-off-by: YueHaibing --- drivers/spi/spi-mem.c | 2 +- 1 file changed, 1 inse

[PATCH] drm/bridge: tpd12s015: Fix irq registering in tpd12s015_probe

gpiod_to_irq() return negative value in case of error, the existing code handle negative error codes wrongly. Fixes: cff5e6f7e83f ("drm/bridge: Add driver for the TI TPD12S015 HDMI level shifter") Signed-off-by: YueHaibing --- drivers/gpu/drm/bridge/ti-tpd12s015.c | 2 +- 1 file

[PATCH] sfp: Fix error handing in sfp_probe()

gpiod_to_irq() never return 0, but returns negative in case of error, check it and set gpio_irq to 0. Fixes: 73970055450e ("sfp: add SFP module support") Signed-off-by: YueHaibing --- drivers/net/phy/sfp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drive

[PATCH] serial: mctrl_gpio: Fix passing zero to 'ERR_PTR' warning

drivers/tty/serial/serial_mctrl_gpio.c:214 mctrl_gpio_init() warn: passing zero to 'ERR_PTR' gpiod_to_irq() never return 0, so remove the useless test and make code more clear. Signed-off-by: YueHaibing --- drivers/tty/serial/serial_mctrl_gpio.c | 2 +- 1 file changed, 1 inser

[PATCH net-next] net/mlx5e: Remove duplicated include

Remove duplicated include. Signed-off-by: YueHaibing --- drivers/net/ethernet/mellanox/mlx5/core/en_rx.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_rx.c index 599f5b5ebc97..58c177756dc4 100644

[PATCH net-next] net: hns3: Remove duplicated include

Remove duplicated include. Signed-off-by: YueHaibing --- drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_dcb.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_dcb.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_dcb.c index 3606240025a8

[PATCH net-next] liquidio: cn68xx: Remove duplicated include

Remove duplicated include. Signed-off-by: YueHaibing --- drivers/net/ethernet/cavium/liquidio/cn68xx_device.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/cavium/liquidio/cn68xx_device.c b/drivers/net/ethernet/cavium/liquidio/cn68xx_device.c index 2a6d1cadac9e

Re: [PATCH -next] drm/rockchip: cdn-dp: Mark cdn_dp_core_suspend/resume __maybe_unused

ping... On 2020/8/11 10:12, YueHaibing wrote: > If CONFIG_PM is not set, gcc warns: > > drivers/gpu/drm/rockchip/cdn-dp-core.c:1124:12: > warning: ‘cdn_dp_resume’ defined but not used [-Wunused-function] > > Mark them __maybe_unused to fix this. > > Reported-by: Hul

[tip: timers/urgent] hrtimer: Remove unused inline function debug_hrtimer_free()

The following commit has been merged into the timers/urgent branch of tip: Commit-ID: 5254cb87c0423f73c8036235795788a132e8956e Gitweb: https://git.kernel.org/tip/5254cb87c0423f73c8036235795788a132e8956e Author:YueHaibing AuthorDate:Wed, 09 Sep 2020 21:48:50 +08:00

[tip: timers/urgent] timers: Remove unused inline funtion debug_timer_free()

The following commit has been merged into the timers/urgent branch of tip: Commit-ID: 9010e3876e1c3f7b1c3769bee519d6a871589aca Gitweb: https://git.kernel.org/tip/9010e3876e1c3f7b1c3769bee519d6a871589aca Author:YueHaibing AuthorDate:Wed, 09 Sep 2020 21:47:49 +08:00

Re: [PATCH] bpfilter: Fix build error with CONFIG_BPFILTER_UMH

On 2020/10/16 3:57, Alexei Starovoitov wrote: > On Thu, Oct 15, 2020 at 12:26 PM Jakub Kicinski wrote: >> >> On Thu, 15 Oct 2020 12:03:14 -0700 Alexei Starovoitov wrote: >>> On Thu, Oct 15, 2020 at 11:56 AM Jakub Kicinski wrote: How so? It's using in-tree headers instead of system ones. >

[PATCH] bpfilter: Fix build error with CONFIG_BPFILTER_UMH

. scripts/Makefile.userprogs:43: recipe for target 'net/bpfilter/main.o' failed make[2]: *** [net/bpfilter/main.o] Error 1 Add missing include path to fix this. Signed-off-by: YueHaibing --- net/bpfilter/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ne

[tip: irq/core] arm64: Fix -Wunused-function warning when !CONFIG_HOTPLUG_CPU

The following commit has been merged into the irq/core branch of tip: Commit-ID: 9d9edb962e910552c9c008800ec907293a47852e Gitweb: https://git.kernel.org/tip/9d9edb962e910552c9c008800ec907293a47852e Author:YueHaibing AuthorDate:Fri, 18 Sep 2020 20:33:18 +08:00 Committer

[tip: irq/core] irqchip/ti-sci-intr: Fix unsigned comparison to zero

The following commit has been merged into the irq/core branch of tip: Commit-ID: 8ddf1905a904ca86d71ca1c435e4b0b2a0b70df8 Gitweb: https://git.kernel.org/tip/8ddf1905a904ca86d71ca1c435e4b0b2a0b70df8 Author:YueHaibing AuthorDate:Wed, 26 Aug 2020 11:53:21 +08:00 Committer

[tip: irq/core] irqchip/ti-sci-inta: Fix unsigned comparison to zero

The following commit has been merged into the irq/core branch of tip: Commit-ID: 4c9b1bfaa5039fee650f4de514a8e70ae976fc2f Gitweb: https://git.kernel.org/tip/4c9b1bfaa5039fee650f4de514a8e70ae976fc2f Author:YueHaibing AuthorDate:Wed, 26 Aug 2020 11:54:30 +08:00 Committer

[tip: sched/core] sched: Remove unused inline function uclamp_bucket_base_value()

The following commit has been merged into the sched/core branch of tip: Commit-ID: 51bd5121c4eb25b911f6bc1ab4de5fe865fe0dcb Gitweb: https://git.kernel.org/tip/51bd5121c4eb25b911f6bc1ab4de5fe865fe0dcb Author:YueHaibing AuthorDate:Tue, 22 Sep 2020 21:24:10 +08:00 Committer

[PATCH -next] sched: Remove unused inline function uclamp_bucket_base_value()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- kernel/sched/core.c | 5 - 1 file changed, 5 deletions(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 2d95dc3f4644..1b06b952dbbe 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c

[PATCH net-next] netfilter: nf_tables_offload: Remove unused macro FLOW_SETUP_BLOCK

commit 9a32669fecfb ("netfilter: nf_tables_offload: support indr block call") left behind this. Signed-off-by: YueHaibing --- net/netfilter/nf_tables_offload.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/net/netfilter/nf_tables_offload.c b/net/netfilter/nf_tables_offlo

[PATCH net-next] ipvs: Remove unused macros

They are not used since commit e4ff67513096 ("ipvs: add sync_maxlen parameter for the sync daemon") Signed-off-by: YueHaibing --- net/netfilter/ipvs/ip_vs_sync.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/net/netfilter/ipvs/ip_vs_sync.c b/net/netfilter/ipvs/ip_vs_sy

[PATCH net-next] tipc: Remove unused macro CF_SERVER

It is no used any more, so can remove it. Signed-off-by: YueHaibing --- net/tipc/topsrv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/tipc/topsrv.c b/net/tipc/topsrv.c index 1489cfb941d8..5f6f86051c83 100644 --- a/net/tipc/topsrv.c +++ b/net/tipc/topsrv.c @@ -48,7 +48,6 @@ #define

[PATCH 2/5] wlcore: Remove unused function no_write_handler()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- drivers/net/wireless/ti/wlcore/debugfs.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/net/wireless/ti/wlcore/debugfs.c b/drivers/net/wireless/ti/wlcore/debugfs.c index 48adb1876ab9..cce8d75d8b81

[PATCH] ath11k: Remove unused function ath11k_htc_restore_tx_skb()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- drivers/net/wireless/ath/ath11k/htc.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/net/wireless/ath/ath11k/htc.c b/drivers/net/wireless/ath/ath11k/htc.c index e9e354fc11fa..4de2350dfbf3 100644

[PATCH -next] arm64: Fix -Wunused-function warning

If CONFIG_HOTPLUG_CPU is n, gcc warns: arch/arm64/kernel/smp.c:967:13: warning: ‘ipi_teardown’ defined but not used [-Wunused-function] static void ipi_teardown(int cpu) ^~~~ Use #ifdef guard this. Signed-off-by: YueHaibing --- arch/arm64/kernel/smp.c | 4 +++- 1 file

Re: [PATCH net-next] genetlink: Remove unused function genl_err_attr()

On 2020/9/18 2:34, Cong Wang wrote: > On Wed, Sep 16, 2020 at 9:33 AM YueHaibing wrote: >> >> It is never used, so can remove it. > > This is a bit confusing, it was actually used before, see commit > ab0d76f6823cc3a4e2. Yes,thanks for reminding, will be careful later. > . >

[PATCH v2 net-next] netdev: Remove unused functions

There is no callers in tree, so can remove it. Signed-off-by: YueHaibing Reviewed-by: Saeed Mahameed --- v2: fix title typo 'funtions' --> 'functions' --- include/linux/netdevice.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/include/linux/netd

[PATCH v2 next] fpga: dfl: n3000-nios: Make m10_n3000_info static

Fix sparse warning: drivers/fpga/dfl-n3000-nios.c:392:23: warning: symbol 'm10_n3000_info' was not declared. Should it be static? Signed-off-by: YueHaibing --- drivers/fpga/dfl-n3000-nios.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/fpga/dfl-n300

Re: [PATCH -next] net/sched: Remove unused function qdisc_queue_drop_head()

Pls drop this duplicate. On 2020/9/16 22:16, YueHaibing wrote: > It is not used since commit a09ceb0e0814 ("sched: remove qdisc->drop") > > Signed-off-by: YueHaibing > --- > include/net/sch_generic.h | 6 -- > 1 file changed, 6 deletions(-) > > dif

Re: [PATCH -next] fpga: dfl: Make m10_n3000_info static

On 2020/9/17 0:10, Moritz Fischer wrote: > Hi Yue, > > On Wed, Sep 16, 2020 at 10:25:36PM +0800, YueHaibing wrote: >> Fix sparse warning: >> >> drivers/fpga/dfl-n3000-nios.c:392:23: warning: >> symbol 'm10_n3000_info' was not declared. Should it

[PATCH net-next] net/sched: Remove unused function qdisc_queue_drop_head()

It is not used since commit a09ceb0e0814 ("sched: remove qdisc->drop") Signed-off-by: YueHaibing --- include/net/sch_generic.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/include/net/sch_generic.h b/include/net/sch_generic.h index d60e7c39d60c..6c762457122f 100644 ---

[PATCH -next] net/sched: Remove unused function qdisc_queue_drop_head()

It is not used since commit a09ceb0e0814 ("sched: remove qdisc->drop") Signed-off-by: YueHaibing --- include/net/sch_generic.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/include/net/sch_generic.h b/include/net/sch_generic.h index d60e7c39d60c..6c762457122f 100644 ---

[PATCH net-next] netfilter: nf_tables: Remove ununsed function nft_data_debug

It is never used, so can be removed. Signed-off-by: YueHaibing --- include/net/netfilter/nf_tables.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/include/net/netfilter/nf_tables.h b/include/net/netfilter/nf_tables.h index 8ceca0e419b3..c4c526507ddb 100644 --- a/include/net

[PATCH -next] btrfs: Make btrfs_sysfs_add_fs_devices static

Fix sparse warning: fs/btrfs/sysfs.c:1386:5: warning: symbol 'btrfs_sysfs_add_fs_devices' was not declared. Should it be static? Signed-off-by: YueHaibing --- fs/btrfs/sysfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/btrfs/sysfs.c b/fs/btrfs/sys

[PATCH -next] fpga: dfl: Make m10_n3000_info static

Fix sparse warning: drivers/fpga/dfl-n3000-nios.c:392:23: warning: symbol 'm10_n3000_info' was not declared. Should it be static? Signed-off-by: YueHaibing --- drivers/fpga/dfl-n3000-nios.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/fpga/dfl-n300

[PATCH net-next] genetlink: Remove unused function genl_err_attr()

It is never used, so can remove it. Signed-off-by: YueHaibing --- include/net/genetlink.h | 8 1 file changed, 8 deletions(-) diff --git a/include/net/genetlink.h b/include/net/genetlink.h index 6e5f1e1aa822..b9eb92f3fe86 100644 --- a/include/net/genetlink.h +++ b/include/net

[PATCH net-next] netdev: Remove unused funtions

There is no callers in tree, so can remove it. Signed-off-by: YueHaibing --- include/linux/netdevice.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 157e0242e9ee..909b1fbb0481 100644 --- a/include/linux/netdevice.h

[PATCH -next] Input: stmfts - Fix a & vs && typo

In stmfts_sysfs_hover_enable_write(), we should check value and sdata->hover_enabled is all true. Fixes: 78bcac7b2ae1 ("Input: add support for the STMicroelectronics FingerTip touchscreen") Signed-off-by: YueHaibing --- drivers/input/touchscreen/stmfts.c | 2 +- 1 file changed

Re: [PATCH -next] soc: renesas: r8a779a0-sysc: Make r8a779a0_sysc_info static

On 2020/9/15 23:22, Geert Uytterhoeven wrote: > Hi Yue, > > On Tue, Sep 15, 2020 at 4:58 PM YueHaibing wrote: >> Fix sparse warning: >> >> drivers/soc/renesas/r8a779a0-sysc.c:99:33: warning: symbol >> 'r8a779a0_sysc_info' was not declared. Should it

[PATCH -next] soc: renesas: r8a779a0-sysc: Make r8a779a0_sysc_info static

Fix sparse warning: drivers/soc/renesas/r8a779a0-sysc.c:99:33: warning: symbol 'r8a779a0_sysc_info' was not declared. Should it be static? Signed-off-by: YueHaibing --- drivers/soc/renesas/r8a779a0-sysc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s

[PATCH -next] clk: imx35: Make mx35_clocks_init static

Fix sparse warning: drivers/clk/imx/clk-imx35.c:251:12: warning: symbol 'mx35_clocks_init' was not declared. Should it be static? Signed-off-by: YueHaibing --- drivers/clk/imx/clk-imx35.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/imx/clk-imx35.c

[PATCH 3/3] clk: imx31: Make mx31_clocks_init static

Fix sparse warning: drivers/clk/imx/clk-imx31.c:135:12: warning: symbol 'mx31_clocks_init' was not declared. Should it be static? Signed-off-by: YueHaibing --- drivers/clk/imx/clk-imx31.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/imx/clk-imx31.c

[PATCH -next] mm/madvise: remove duplicate include

Remove duplicate header which is included twice. Signed-off-by: YueHaibing --- mm/madvise.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/madvise.c b/mm/madvise.c index 460e19d60ba3..9bd21f9019d9 100644 --- a/mm/madvise.c +++ b/mm/madvise.c @@ -30,7 +30,6 @@ #include #include

[PATCH -next] perf metric: Remove duplicate include

Remove duplicate header which is included twice. Signed-off-by: YueHaibing --- tools/perf/tests/parse-metric.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/tools/perf/tests/parse-metric.c b/tools/perf/tests/parse-metric.c index 23db8acc492d..b344decfb91b 100644 --- a/tools/perf/tests

[PATCH -next] clk: socfpga: agilex: Remove unused variable 'cntr_mux'

drivers/clk/socfpga/clk-agilex.c:24:37: warning: ‘cntr_mux’ defined but not used [-Wunused-const-variable=] static const struct clk_parent_data cntr_mux[] = { ^~~~ There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- drivers/clk

[PATCH -next] agp/uninorth: Make two variable static

Robot Signed-off-by: YueHaibing --- drivers/char/agp/uninorth-agp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/char/agp/uninorth-agp.c b/drivers/char/agp/uninorth-agp.c index 62de7f4ba864..06db04cdc377 100644 --- a/drivers/char/agp/uninorth-agp.c +++ b/drivers/cha

[PATCH -next] mfd: mt6360: Remove unused including

Remove including that don't need it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/mfd/mt6360-core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mfd/mt6360-core.c b/drivers/mfd/mt6360-core.c index e9cacc27d980..4661c1b29a72 100644 --- a/drivers/mfd/mt6360-c

[PATCH v2 -next] media: marvell-ccic: Fix -Wunused-function warnings

-driver.c:310:12: warning: ‘mmpcam_runtime_resume’ defined but not used [-Wunused-function] static int mmpcam_runtime_resume(struct device *dev) ^ Mark them as __maybe_unused to fix this. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- v2: Adjust based on

Re: [PATCH -next] media: marvell-ccic: Fix -Wunused-function warnings

On 2020/9/10 22:57, Lubomir Rintel wrote: > On Thu, Sep 10, 2020 at 05:18:15PM +0800, Yuehaibing wrote: >> On 2020/9/10 16:22, Lubomir Rintel wrote: >>> On Thu, Sep 10, 2020 at 04:09:33PM +0800, YueHaibing wrote: >>>> If CONFIG_PM is n, gcc warns: >>>> &g

Re: [PATCH -next] media: marvell-ccic: Fix -Wunused-function warnings

On 2020/9/10 16:22, Lubomir Rintel wrote: > On Thu, Sep 10, 2020 at 04:09:33PM +0800, YueHaibing wrote: >> If CONFIG_PM is n, gcc warns: >> >> drivers/media/platform/marvell-ccic/mmp-driver.c:347:12: warning: >> ‘mmpcam_resume’ defined but not used [-Wunused-function] &

[PATCH -next] media: marvell-ccic: Fix -Wunused-function warnings

to fix this. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/media/platform/marvell-ccic/mmp-driver.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/marvell-ccic/mmp-driver.c b/drivers/media/platform/marvell-ccic/mmp-driver.c

Re: [PATCH -next] drm/rockchip: cdn-dp: Mark cdn_dp_core_suspend/resume __maybe_unused

ping... On 2020/8/11 10:12, YueHaibing wrote: > If CONFIG_PM is not set, gcc warns: > > drivers/gpu/drm/rockchip/cdn-dp-core.c:1124:12: > warning: ‘cdn_dp_resume’ defined but not used [-Wunused-function] > > Mark them __maybe_unused to fix this. > > Reported-by: Hul

[PATCH v2 -next] drm/amdkfd: Fix -Wunused-const-variable warning

= { ^ As Huang Rui suggested, Raven already has the fallback path, so it should be out of IOMMU v2 flag. Suggested-by: Huang Rui Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdkfd/kfd_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdkfd

[PATCH -next] soundwire: intel: Fix -Wunused-function warning

If CONFIG_PM is not set, build warns: drivers/soundwire/intel.c:488:12: warning: 'intel_link_power_down' defined but not used [-Wunused-function] Mark the functions __maybe_unused and remove the CONFIG_PM dependency. Signed-off-by: YueHaibing --- drivers/soundwire/in

[PATCH -next] staging: regulator: hi6421v600-regulator: Remove unused including

Remove including that don't need it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/staging/hikey9xx/hi6421v600-regulator.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/hikey9xx/hi6421v600-regulator.c b/drivers/staging/hikey9xx/hi6421v600-regula

[PATCH] brcmsmac: phytbl_lcn: Remove unused variable 'dot11lcn_gain_tbl_rev1'

emove unused array 'dot11lcnphytbl_rx_gain_info_rev1'") left behind this, remove it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- .../brcm80211/brcmsmac/phy/phytbl_lcn.c | 99 --- 1 file changed, 99 deletions(-) diff --git a/drivers/net/wireles

[PATCH] brcmsmac: phy_lcn: Remove unused variable lcnphy_rx_iqcomp_table_rev0

("brcmsmac: phy_lcn: Remove a bunch of unused variables") left behind this, remove it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- .../broadcom/brcm80211/brcmsmac/phy/phy_lcn.c | 55 --- 1 file changed, 55 deletions(-) diff --git a/drivers/net/wireless/broadcom

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

Fix gcc build warning: arch/riscv/mm/fault.c:81:1: warning: ‘inline’ is not at beginning of declaration [-Wold-style-declaration] static void inline vmalloc_fault(struct pt_regs *regs, int code, unsigned long addr) ^~ Reported-by: Hulk Robot Signed-off-by: YueHaibing --- arch/riscv/mm

  1   2   3   4   5   6   7   8   9   10   >