[PATCH -next] regulator: bd718x7: Make some variable static

c? drivers/regulator/bd718x7-regulator.c:902:28: warning: symbol 'bd71837_swcontrol_ops' was not declared. Should it be static? drivers/regulator/bd718x7-regulator.c:913:28: warning: symbol 'bd71837_hwcontrol_ops' was not declared. Should it be static? Reported-by: Hulk Robot

[PATCH -next] drm/amd/display: Fix possible memleak in dp_trigger_hotplug()

If parse_write_buffer_into_params() fails, we should free wr_buf before return. Fixes: 6f77b2ac6280 ("drm/amd/display: Add connector HPD trigger debugfs entry") Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 4 +++- 1 file changed, 3 insert

[PATCH -next] drm/amd/display: Create trigger_hotplug entry

Add trigger_hotplug debugfs entry. Fixes: 6f77b2ac6280 ("drm/amd/display: Add connector HPD trigger debugfs entry") Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/display

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

= { ^ Move it to ifdef block. 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/kfd_device.c b/drivers/gpu/drm/amd/amdkfd/kfd_device.c index 0e71a0543f98..cae4df259e26 100644 --- a

[PATCH -next] drm/ttm/agp: Fix Wunused-variable warning

to ifdef block to fix this. Signed-off-by: YueHaibing --- drivers/gpu/drm/radeon/radeon_ttm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c index 31c63d339629..449e77eb75f9 100644 --- a/drivers/gpu

[PATCH -next] soundwire: intel: Remove ununsed function

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] Move this to #ifdef block. Signed-off-by: YueHaibing --- drivers/soundwire/intel.c | 137 +++--- 1 fi

[PATCH -next] thermal: ti-soc-thermal: Remove unused function ti_thermal_get_temp()

commit b263b473bf62 ("thermal: ti-soc-thermal: Remove redundant code") left behind this, remove it. Signed-off-by: YueHaibing --- drivers/thermal/ti-soc-thermal/ti-thermal-common.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/thermal/ti-soc-thermal/ti-thermal-c

[PATCH -next] usb: host: ehci-sched: Remove ununsed function tt_start_uframe()

commit b35c5009bbf6 ("USB: EHCI: create per-TT bandwidth tables") left behind this, remove it. Signed-off-by: YueHaibing --- drivers/usb/host/ehci-sched.c | 20 1 file changed, 20 deletions(-) diff --git a/drivers/usb/host/ehci-sched.c b/drivers/usb/host/ehci-sch

[PATCH] qtnfmac: Remove unused macro QTNF_DMP_MAX_LEN

There is no caller in tree, so remove it. Signed-off-by: YueHaibing --- drivers/net/wireless/quantenna/qtnfmac/core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/quantenna/qtnfmac/core.c b/drivers/net/wireless/quantenna/qtnfmac/core.c index 6aafff9d4231

[PATCH -next] media: rkvdec: Remove redundant platform_get_irq error message

platform_get_irq() will call dev_err() itself on failure, so there is no need for the driver to also do this. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/staging/media/rkvdec/rkvdec.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers

[PATCH] ath10k: Remove unused macro ATH10K_ROC_TIMEOUT_HZ

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- drivers/net/wireless/ath/ath10k/mac.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c index fed8682f266f..5991d0ac41ad 100644 --- a

[PATCH 04/19] ALSA: pcm: Remove unused inline function snd_mask_sizeof

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- include/sound/pcm_params.h | 5 - 1 file changed, 5 deletions(-) diff --git a/include/sound/pcm_params.h b/include/sound/pcm_params.h index 36f94735d23d..ba184f49f7e1 100644 --- a/include/sound/pcm_params.h +++ b

[PATCH] wlcore: Remove unused macro WL1271_SUSPEND_SLEEP

commit 45aa7f071b06 ("wlcore: Use generic runtime pm calls for wowlan elp configuration") left behind this, remove it. Signed-off-by: YueHaibing --- drivers/net/wireless/ti/wlcore/main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/ti/wlcore/main.c b/d

[PATCH -next] scsi: aic94xx: Remove unused inline function

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- drivers/scsi/aic94xx/aic94xx.h | 8 1 file changed, 8 deletions(-) diff --git a/drivers/scsi/aic94xx/aic94xx.h b/drivers/scsi/aic94xx/aic94xx.h index c23bbb609126..98978bc199ff 100644 --- a/drivers/scsi

[PATCH -next] serial: 8250_pci: Remove unused function get_pci_irq()

It is not used since commit 8428413b1d14 ("serial: 8250_pci: Implement MSI(-X) support") Signed-off-by: YueHaibing --- drivers/tty/serial/8250/8250_pci.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/tty/serial/8250/8250_pci.c b/drivers/tty/serial/8250/8250_p

[PATCH 09/19] ALSA: pci/asihpi: Remove unused function hpi_stream_group_get_map()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- sound/pci/asihpi/asihpi.c | 9 - 1 file changed, 9 deletions(-) diff --git a/sound/pci/asihpi/asihpi.c b/sound/pci/asihpi/asihpi.c index 35e76480306e..8bacd4f47540 100644 --- a/sound/pci/asihpi/asihpi.c +++ b

[PATCH -next] btrfs: Remove unused function calc_global_rsv_need_space()

It is not used since commit 0096420adb03 ("btrfs: do not account global reserve in can_overcommit") Signed-off-by: YueHaibing --- fs/btrfs/space-info.c | 5 - 1 file changed, 5 deletions(-) diff --git a/fs/btrfs/space-info.c b/fs/btrfs/space-info.c index b733718f45d3..0f16a2ce5

[PATCH] ath11k: Remove unused inline function htt_htt_stats_debug_dump()

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

[PATCH v4] scsi: libfc: Fix passing zero to 'PTR_ERR' warning

drivers/scsi/libfc/fc_disc.c:304 fc_disc_error() warn: passing zero to 'PTR_ERR' fp maybe NULL in fc_disc_error(), use PTR_ERR_OR_ZERO to handle this. Signed-off-by: YueHaibing --- v4: fix error print type v3: use PTR_ERR_OR_ZERO v2: use use IS_ERR in fc_disc_error() --- drivers/

[PATCH -next] timers: Remove unused inline funtion debug_timer_free()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- kernel/time/timer.c | 5 - 1 file changed, 5 deletions(-) diff --git a/kernel/time/timer.c b/kernel/time/timer.c index a50364df1054..7da4af49d1d7 100644 --- a/kernel/time/timer.c +++ b/kernel/time/timer.c

[PATCH -next] hrtimer: Remove unused inline function debug_hrtimer_free()

There is no caller in tree, so can remove it. Signed-off-by: YueHaibing --- kernel/time/hrtimer.c | 5 - 1 file changed, 5 deletions(-) diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c index 95b6a708b040..75d0cc1a2081 100644 --- a/kernel/time/hrtimer.c +++ b/kernel/time/hrtimer.c

[PATCH -next] mtd: rawnand: vf610_nfc: Remove unused function vf610_nfc_transfer_size()

commit ecc40b8df59a ("mtd: rawnand: vf610_nfc: remove old hooks") left behind this, remove it. Signed-off-by: YueHaibing --- drivers/mtd/nand/raw/vf610_nfc.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/mtd/nand/raw/vf610_nfc.c b/drivers/mtd/nand/raw/vf610_n

[PATCH -next] PCI: dwc: unexport dw_pcie_link_set_max_speed

This function has been made static, which causes warning: WARNING: modpost: "dw_pcie_link_set_max_speed" [vmlinux] is a static EXPORT_SYMBOL_GPL Fixes: 3af45d34d30c ("PCI: dwc: Centralize link gen setting") Signed-off-by: YueHaibing --- drivers/pci/controller/dwc/pcie-d

[PATCH -next] drm/panel: s6e63m0: Add missing MODULE_LICENSE

Kbuild warns when this file is built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/gpu/drm/panel/panel-samsung-s6e63m0.o Add the missing license/author/description tags. Fixes: b7b23e447687 ("drm/panel: s6e63m0: Break out SPI transport") Signed-off-by:

[PATCH -next] media: av7110_v4l: Fix build error

ot;media: av7110_v4l: avoid a typecast") Signed-off-by: YueHaibing --- drivers/media/pci/ttpci/av7110_v4l.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/pci/ttpci/av7110_v4l.c b/drivers/media/pci/ttpci/av7110_v4l.c index 6d9c908be713..ba745b161cba 100644 --- a/

Re: [PATCH] mwifiex: pcie: Fix -Wunused-const-variable warnings

commit 77dacc8fc64c ("mwifiex: pcie: Move tables to the only place they're used") do the same things. On 2020/9/7 16:34, Kalle Valo wrote: > YueHaibing wrote: > >> These variables only used in pcie.c, move them to .c file >> can silence these warnings: &g

[PATCH v2 -next] remoteproc: k3-dsp: Fix return value check in k3_dsp_rproc_of_get_memories()

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: 6edbe024ba17 ("remoteproc: k3-dsp: Add a remoteproc driver of K3 C66x DSPs") Signed-off-by: YueHaibing Acked-by:

[PATCH -next] Bluetooth: btmtksdio: use NULL instead of zero

Fix sparse warnings: drivers/bluetooth/btmtksdio.c:499:57: warning: Using plain integer as NULL pointer drivers/bluetooth/btmtksdio.c:533:57: warning: Using plain integer as NULL pointer Signed-off-by: YueHaibing --- drivers/bluetooth/btmtksdio.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH -next] RDMA/bnxt_re: Remove set but not used variable 'qplib_ctx'

drivers/infiniband/hw/bnxt_re/main.c:1012:25: warning: variable ‘qplib_ctx’ set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/infiniband/hw/bnxt_re/main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers

Re: [PATCH -next] perf bench: fix return value check in do_run_multi_threaded()

On 2020/9/4 2:55, Arnaldo Carvalho de Melo wrote: > Em Thu, Sep 03, 2020 at 03:54:51PM -0300, Arnaldo Carvalho de Melo escreveu: >> Em Thu, Sep 03, 2020 at 03:53:01PM -0300, Arnaldo Carvalho de Melo escreveu: >>> Em Wed, Sep 02, 2020 at 10:05:26PM +0800, YueHaibing escreve

[PATCH] mwifiex: sdio: Fix -Wunused-const-variable warnings

not used [-Wunused-const-variable=] static const struct mwifiex_sdio_device mwifiex_sdio_sd8786 = { ^~~ Signed-off-by: YueHaibing --- drivers/net/wireless/marvell/mwifiex/sdio.c | 427 drivers/net/wireless/marvell

[PATCH] mwifiex: wmm: Fix -Wunused-const-variable warnings

header file to fix these warnings. Signed-off-by: YueHaibing --- drivers/net/wireless/marvell/mwifiex/main.c | 2 ++ drivers/net/wireless/marvell/mwifiex/wmm.c | 15 +++ drivers/net/wireless/marvell/mwifiex/wmm.h | 18 ++ 3 files changed, 19 insertions(+), 16

[PATCH] mwifiex: pcie: Fix -Wunused-const-variable warnings

mwifiex_pcie8766 = { ^~~~ Signed-off-by: YueHaibing --- drivers/net/wireless/marvell/mwifiex/pcie.c | 149 drivers/net/wireless/marvell/mwifiex/pcie.h | 149 2 files changed, 149 insertions(+), 149 deletions

[PATCH -next] crypto: pkcs7: Use match_string() helper to simplify the code

match_string() returns the array index of a matching string. Use it instead of the open-coded implementation. Signed-off-by: YueHaibing --- crypto/asymmetric_keys/pkcs7_verify.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/crypto/asymmetric_keys/pkcs7_verify.c b

[PATCH -next] perf bench: fix return value check in do_run_multi_threaded()

In case of error, the function perf_session__new() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR() Fixes: 13edc237200c ("perf bench: Add a multi-threaded synthesize benchmark") Signed-off-by: YueHaibing --- tools/

[PATCH -next] remoteproc: k3-dsp: Fix return value check in k3_dsp_rproc_of_get_memories()

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: 87218f96c21a ("remoteproc: k3-dsp: Add support for C71x DSPs") Signed-off-by: YueHaibing --- drivers/

[PATCH net-next] net/mlx5e: kTLS, Avoid kzalloc(GFP_KERNEL) under spinlock

A spin lock is held before kzalloc, it may sleep with holding the spinlock, so we should use GFP_ATOMIC instead. This is detected by coccinelle. Fixes: 0419d8c9d8f8 ("net/mlx5e: kTLS, Add kTLS RX resync support") Signed-off-by: YueHaibing --- drivers/net/ethernet/mellanox/mlx5/cor

[PATCH net-next] net: sungem: Remove unneeded cast from memory allocation

Remove dma_alloc_coherent return value cast. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/net/ethernet/sun/sungem.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/ethernet/sun/sungem.c b/drivers/net/ethernet/sun/sungem.c index

[PATCH net-next] liquidio: Remove unneeded cast from memory allocation

Remove unneeded return value cast. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/net/ethernet/cavium/liquidio/octeon_droq.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/net/ethernet/cavium/liquidio/octeon_droq.c b/drivers/net

[PATCH v3 -next] memory: omap-gpmc: Fix -Wunused-function warnings

them to #ifdef CONFIG_OF block to fix this. Signed-off-by: YueHaibing --- v3: move the functions to #ifdef block v2: update commit log --- drivers/memory/omap-gpmc.c | 114 ++--- 1 file changed, 57 insertions(+), 57 deletions(-) diff --git a/drivers/memory/omap

Re: [PATCH -next] memory: omap-gpmc: Fix -Wunused-function warnings

Pls ignore this. On 2020/9/1 15:07, YueHaibing wrote: > drivers/memory/omap-gpmc.c:987:12: warning: ‘gpmc_cs_remap’ defined but not > used [-Wunused-function] > static int gpmc_cs_remap(int cs, u32 base) > ^ > drivers/memory/omap-gpmc.c:926:20: warning: ‘g

[PATCH v2 -next] memory: omap-gpmc: Fix -Wunused-function warnings

them as __maybe_unused to fix this. Signed-off-by: YueHaibing --- v2: update commit log --- drivers/memory/omap-gpmc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c index ac0f577a51a1..24372254986e 100644 --- a

[PATCH -next] memory: omap-gpmc: Fix -Wunused-function warnings

. Signed-off-by: YueHaibing --- drivers/memory/omap-gpmc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c index ac0f577a51a1..24372254986e 100644 --- a/drivers/memory/omap-gpmc.c +++ b/drivers/memory/omap-gpmc.c @@ -916,14

Re: [PATCH -next] memory: omap-gpmc: Fix -Wunused-function warnings

On 2020/9/1 14:52, Krzysztof Kozlowski wrote: > On Tue, Sep 01, 2020 at 11:56:42AM +0800, YueHaibing wrote: >> drivers/memory/omap-gpmc.c:987:12: warning: ‘gpmc_cs_remap’ defined but not >> used [-Wunused-function] >> static int gpmc_cs_r

[PATCH -next] ACPI: Remove unused inline functions

There is no callers in tree. Signed-off-by: YueHaibing --- include/linux/acpi.h | 9 - 1 file changed, 9 deletions(-) diff --git a/include/linux/acpi.h b/include/linux/acpi.h index 17e80e182802..dd9b961628eb 100644 --- a/include/linux/acpi.h +++ b/include/linux/acpi.h @@ -979,8 +979,6

Re: [PATCH] x86/platform/intel-mid: Fix build error without CONFIG_ACPI

On 2020/9/1 12:07, Randy Dunlap wrote: > On 8/31/20 8:58 PM, YueHaibing wrote: >> arch/x86/pci/intel_mid_pci.c: In function ‘intel_mid_pci_init’: >> arch/x86/pci/intel_mid_pci.c:303:2: error: implicit declaration of function >> ‘acpi_noirq_set’; did you mean ‘acpi_irq_get’?

[PATCH] x86/platform/intel-mid: Fix build error without CONFIG_ACPI

: a912a7584ec3 ("x86/platform/intel-mid: Move PCI initialization to arch_init()") Signed-off-by: YueHaibing --- arch/x86/pci/intel_mid_pci.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/pci/intel_mid_pci.c b/arch/x86/pci/intel_mid_pci.c index 00c62115f39c..0aaf31917061 100644 ---

[PATCH -next] staging: spmi: hisi-spmi-controller: Use proper format in call to dev_err()

The correct format string for a size_t argument should be %zu. Signed-off-by: YueHaibing --- drivers/staging/hikey9xx/hisi-spmi-controller.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/hikey9xx/hisi-spmi-controller.c b/drivers/staging/hikey9xx

[PATCH -next] memory: omap-gpmc: Fix -Wunused-function warnings

. Signed-off-by: YueHaibing --- drivers/memory/omap-gpmc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c index ac0f577a51a1..24372254986e 100644 --- a/drivers/memory/omap-gpmc.c +++ b/drivers/memory/omap-gpmc.c @@ -916,14

[PATCH] ath11k: Mark two variables as __maybe_unused

Fix -Wunused-variable warnings: drivers/net/wireless/ath/ath11k/debug.c:36:20: warning: ‘htt_bp_lmac_ring’ defined but not used [-Wunused-variable] drivers/net/wireless/ath/ath11k/debug.c:15:20: warning: ‘htt_bp_umac_ring’ defined but not used [-Wunused-variable] Signed-off-by: YueHaibing

[PATCH -next] staging: rtl8723bs: os_dep: return errcode in cfg80211_rtw_leave_ibss()

We should return error to caller in cfg80211_rtw_leave_ibss(). Signed-off-by: YueHaibing --- drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs

[PATCH] libertas_tf: Remove unused macro QOS_CONTROL_LEN

There is no caller in tree. Signed-off-by: YueHaibing --- drivers/net/wireless/marvell/libertas_tf/main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/marvell/libertas_tf/main.c b/drivers/net/wireless/marvell/libertas_tf/main.c index 02bd7c99b358..cb494b59de93 100644

[PATCH net-next] net/wan/fsl_ucc_hdlc: Add MODULE_DESCRIPTION

Add missing MODULE_DESCRIPTION. Signed-off-by: YueHaibing --- drivers/net/wan/fsl_ucc_hdlc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/wan/fsl_ucc_hdlc.c b/drivers/net/wan/fsl_ucc_hdlc.c index 9edd94679283..dca97cd7c4e7 100644 --- a/drivers/net/wan/fsl_ucc_hdlc.c +++ b

[PATCH net-next] net: dl2k: Remove unused macro DRV_NAME

There is no caller in tree any more. Signed-off-by: YueHaibing --- drivers/net/ethernet/dlink/dl2k.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/dlink/dl2k.c b/drivers/net/ethernet/dlink/dl2k.c index be6d8a9ada27..e8e563d6e86b 100644 --- a/drivers/net/ethernet/dlink

[PATCH net-next] net: hns: Remove unused macro AE_NAME_PORT_ID_IDX

There is no caller in tree. Signed-off-by: YueHaibing --- drivers/net/ethernet/hisilicon/hns/hns_ae_adapt.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/ethernet/hisilicon/hns/hns_ae_adapt.c b/drivers/net/ethernet/hisilicon/hns/hns_ae_adapt.c index b43dec0560a8

[PATCH net-next] net: wan: slic_ds26522: Remove unused macro DRV_NAME

There is no caller in tree any more. Signed-off-by: YueHaibing --- drivers/net/wan/slic_ds26522.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/wan/slic_ds26522.c b/drivers/net/wan/slic_ds26522.c index 29053bec694e..8e3b1c717c10 100644 --- a/drivers/net/wan/slic_ds26522.c

[PATCH] lib80211: Remove unused macro DRV_NAME

There is no caller in tree any more. Signed-off-by: YueHaibing --- net/wireless/lib80211.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/net/wireless/lib80211.c b/net/wireless/lib80211.c index cc7b9fd5c166..d66a913027e0 100644 --- a/net/wireless/lib80211.c +++ b/net/wireless/lib80211.c

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

There is no caller in tree any more. Signed-off-by: YueHaibing --- net/tipc/link.c | 5 - 1 file changed, 5 deletions(-) diff --git a/net/tipc/link.c b/net/tipc/link.c index b7362556da95..a2989f22ebb6 100644 --- a/net/tipc/link.c +++ b/net/tipc/link.c @@ -216,11 +216,6 @@ enum { #define

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

There is no caller in tree any more. Signed-off-by: YueHaibing --- net/tipc/socket.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/tipc/socket.c b/net/tipc/socket.c index 2679e97e0389..fd5bfaab8661 100644 --- a/net/tipc/socket.c +++ b/net/tipc/socket.c @@ -52,7 +52,6 @@ #define

[PATCH net-next] mptcp: Remove unused macro MPTCP_SAME_STATE

There is no caller in tree any more. Signed-off-by: YueHaibing --- net/mptcp/protocol.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/net/mptcp/protocol.c b/net/mptcp/protocol.c index 1aad411a0e46..e6216c4f308c 100644 --- a/net/mptcp/protocol.c +++ b/net/mptcp/protocol.c @@ -24,8 +24,6

[PATCH net-next] netfilter: xt_HMARK: Use ip_is_fragment() helper

Use ip_is_fragment() to simpify code. Signed-off-by: YueHaibing --- net/netfilter/xt_HMARK.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/netfilter/xt_HMARK.c b/net/netfilter/xt_HMARK.c index 713fb38541df..8928ec56c388 100644 --- a/net/netfilter/xt_HMARK.c +++ b/net

[PATCH v3] memory: omap-gpmc: Fix build error without CONFIG_OF

eed with random/invalid settings. Fixes: a758f50f10cf ("mtd: onenand: omap2: Configure driver from DT") Signed-off-by: YueHaibing --- v3: zero gpmc_settings v2: add gpmc_read_settings_dt() stub --- drivers/memory/omap-gpmc.c | 4 1 file changed, 4 insertions(+) diff --git a/dri

Re: [PATCH] libnvdimm/e820: Fix build error without MEMORY_HOTPLUG

Pls ignore this It has be fixed in commit dbb8733abb1c ("mm-memory_hotplug-introduce-default-phys_to_target_node-implementation-fix") On 2020/8/26 20:18, YueHaibing wrote: > If MEMORY_HOTPLUG is n, build fails: > > drivers/nvdimm/e820.c: In function ‘e820_register_one’: >

[PATCH v2] memory: omap-gpmc: Fix build error without CONFIG_OF

onfigure driver from DT") Signed-off-by: YueHaibing --- v2: add gpmc_read_settings_dt() stub --- drivers/memory/omap-gpmc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c index cd9e80748591..02c0c694ecaf 100644 --- a/drivers/memo

[PATCH] libnvdimm/e820: Fix build error without MEMORY_HOTPLUG

->start); ^~~ set_page_node Fixes: 7b27a8622f80 ("libnvdimm/e820: Retrieve and populate correct 'target_node' info") Signed-off-by: YueHaibing --- drivers/nvdimm/e820.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/

[PATCH] net: cdc_ncm: Fix build error

hook into set_rx_mode to admit multicast traffic") Signed-off-by: YueHaibing --- drivers/net/usb/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/usb/Kconfig b/drivers/net/usb/Kconfig index a7fbc3ccd29e..c7bcfca7d70b 100644 --- a/drivers/net/usb/Kconfig +++ b/drivers/net/usb/Kc

[PATCH -next] btrfs: Remove unused variable 'fs_info'

m device in btrfs_rm_dev_replace_free_srcdev") Reported-by: Hulk Robot Signed-off-by: YueHaibing --- fs/btrfs/volumes.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c index ec62fc0ec113..b775ed5d3737 100644 --- a/fs/btrfs/volumes.c +++ b/fs/bt

[PATCH -next] irqchip/ti-sci-inta: Fix unsigned comparison to zero

ti_sci_inta_xlate_irq() return -ENOENT on fail, p_hwirq should be int type. Fixes: 5c4b585d2910 ("irqchip/ti-sci-inta: Add support for INTA directly connecting to GIC") Signed-off-by: YueHaibing --- drivers/irqchip/irq-ti-sci-inta.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH -next] irqchip/ti-sci-intr: Fix unsigned comparison to zero

ti_sci_intr_xlate_irq() return -ENOENT on fail, p_hwirq should be int type. Fixes: a5b659bd4bc7 ("irqchip/ti-sci-intr: Add support for INTR being a parent to INTR") Signed-off-by: YueHaibing --- drivers/irqchip/irq-ti-sci-intr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletion

[PATCH v3] scsi: libfc: Fix passing zero to 'PTR_ERR' warning

drivers/scsi/libfc/fc_disc.c:304 fc_disc_error() warn: passing zero to 'PTR_ERR' fp maybe NULL in fc_disc_error(), use PTR_ERR_OR_ZERO to handle this. Signed-off-by: YueHaibing --- v3: use PTR_ERR_OR_ZERO v2: use use IS_ERR in fc_disc_error() --- drivers/scsi/libfc/fc_disc.c | 2

Re: [PATCH v2] scsi: libfc: Fix passing zero to 'PTR_ERR' warning

On 2020/8/21 10:21, Martin K. Petersen wrote: > > YueHaibing, > >> diff --git a/drivers/scsi/libfc/fc_disc.c b/drivers/scsi/libfc/fc_disc.c >> index d8cbc9c0e766..574e842cefed 100644 >> --- a/drivers/scsi/libfc/fc_disc.c >> +++ b/drivers/scsi/libfc/fc_disc.c &

Re: [PATCH v2] net: stmmac: Fix signedness bug in stmmac_probe_config_dt()

On 2020/8/19 1:04, Andrew Lunn wrote: > On Tue, Aug 18, 2020 at 11:15:00PM +0800, YueHaibing wrote: >> The "plat->phy_interface" variable is an enum and in this context GCC >> will treat it as an unsigned int so the error handling is never >> triggered. >&

[PATCH v2] scsi: libfc: Fix passing zero to 'PTR_ERR' warning

drivers/scsi/libfc/fc_disc.c:304 fc_disc_error() warn: passing zero to 'PTR_ERR' fp maybe NULL in fc_disc_error(), use IS_ERR to handle this. Signed-off-by: YueHaibing --- v2: use IS_ERR in fc_disc_error() --- drivers/scsi/libfc/fc_disc.c | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH] net: stmmac: Fix signedness bug in stmmac_probe_config_dt()

On 2020/8/18 22:51, Andreas Schwab wrote: > On Aug 18 2020, YueHaibing wrote: > >> The "plat->phy_interface" variable is an enum and in this context GCC >> will treat it as an unsigned int so the error handling is never >> triggered. >> >> Fixes

[PATCH v2] net: stmmac: Fix signedness bug in stmmac_probe_config_dt()

The "plat->phy_interface" variable is an enum and in this context GCC will treat it as an unsigned int so the error handling is never triggered. Fixes: b9f0b2f634c0 ("net: stmmac: platform: fix probe for ACPI devices") Signed-off-by: YueHaibing --- v2: use rc to do err ha

[PATCH] net: stmmac: Fix signedness bug in stmmac_probe_config_dt()

The "plat->phy_interface" variable is an enum and in this context GCC will treat it as an unsigned int so the error handling is never triggered. Fixes: b9f0b2f634c0 ("net: stmmac: platform: fix probe for ACPI devices") Signed-off-by: YueHaibing --- drivers/net/

[PATCH -next] crypto: sa2ul - Fix pointer-to-int-cast warning

definition of macro ‘dev_dbg’ dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \ ^~~ Use %p to print rctx pointer. Signed-off-by: YueHaibing --- drivers/crypto/sa2ul.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] memory: omap-gpmc: Fix build error without CONFIG_OF

quot;) Signed-off-by: YueHaibing --- drivers/memory/omap-gpmc.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c index ce0e7e2d7cff..332bb4580817 100644 --- a/drivers/memory/omap-gpmc.c +++ b/drivers/memory/omap-gpmc.c @@ -1176

[PATCH -next] dmaengine: iop-adma: Fix pointer cast warnings

] (void *) adev->dma_desc_pool); Use dma_addr_t for dma_desc_pool, and %pad to print dma_addr_t. Signed-off-by: YueHaibing --- drivers/dma/iop-adma.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/dma/iop-adma.c b/drivers/dma/iop-adma.c index

[PATCH -next] vdpa: Remove duplicate include

Remove duplicate include file Signed-off-by: YueHaibing --- drivers/vhost/vdpa.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/vhost/vdpa.c b/drivers/vhost/vdpa.c index 3fab94f88894..95e2b8307a2a 100644 --- a/drivers/vhost/vdpa.c +++ b/drivers/vhost/vdpa.c @@ -22,7 +22,6

[PATCH -next] phy: ti: j721e-wiz: Remove duplicate include

Remove duplicate include file Signed-off-by: YueHaibing --- drivers/phy/ti/phy-j721e-wiz.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/phy/ti/phy-j721e-wiz.c b/drivers/phy/ti/phy-j721e-wiz.c index 33c4cf0105a4..c9cfafe89cbf 100644 --- a/drivers/phy/ti/phy-j721e-wiz.c +++ b

[PATCH -next] vdpa/mlx5: Remove duplicate include

Remove duplicate include file Signed-off-by: YueHaibing --- drivers/vdpa/mlx5/net/mlx5_vnet.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/vdpa/mlx5/net/mlx5_vnet.c b/drivers/vdpa/mlx5/net/mlx5_vnet.c index 9df69d5efe8c..12fb83dc1de9 100644 --- a/drivers/vdpa/mlx5/net/mlx5_vnet.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 843f6fad3b89..fda6164db356 100644 --- a/mm/madvise.c +++ b/mm/madvise.c @@ -29,7 +29,6 @@ #include #include

[PATCH] scsi: libfc: Fix passing zero to 'PTR_ERR' warning

drivers/scsi/libfc/fc_disc.c:304 fc_disc_error() warn: passing zero to 'PTR_ERR' fc_disc_error() expect a ERR_PTR pointer, so pass ERR_PTR(err) to fix this. Signed-off-by: YueHaibing --- drivers/scsi/libfc/fc_disc.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) di

[PATCH -next] mm: slab: Remove duplicate include

Remove duplicate header which is included twice. Signed-off-by: YueHaibing --- mm/slab.h | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/slab.h b/mm/slab.h index 6cc323f1313a..95e5cc1bb2a3 100644 --- a/mm/slab.h +++ b/mm/slab.h @@ -46,7 +46,6 @@ struct kmem_cache { #include #include

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

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: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/rockchip/cdn-dp-core.c | 4

[PATCH] dpaa2-eth: Fix passing zero to 'PTR_ERR' warning

Fix smatch warning: drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c:2419 alloc_channel() warn: passing zero to 'ERR_PTR' setup_dpcon() should return ERR_PTR(err) instead of zero in error handling case. Fixes: d7f5a9d89a55 ("dpaa2-eth: defer probe on object allocate") Signe

[PATCH v2 -next] media: staging: tegra-vde: Mark tegra_vde_runtime_suspend/resume as __maybe_unused

If CONFIG_PM is not set, gcc warns: drivers/staging/media/tegra-vde/vde.c:916:12: warning: 'tegra_vde_runtime_suspend' defined but not used [-Wunused-function] Make it __maybe_unused to fix this. Signed-off-by: YueHaibing --- v2: both suspend and resume functions marked --- drive

Re: [PATCH -next] media: staging: tegra-vde: Mark tegra_vde_runtime_suspend as __maybe_unused

On 2020/8/3 22:11, Dmitry Osipenko wrote: > 03.08.2020 16:00, Yuehaibing пишет: >> On 2020/8/3 20:51, Dmitry Osipenko wrote: >>> 03.08.2020 14:59, YueHaibing пишет: >>>> If CONFIG_PM is not set, gcc warns: >>>> >>>> drive

[PATCH -next] dmaengine: iop-adma: Fix -Wint-to-pointer-cast build warning

different size [-Wint-to-pointer-cast] Use void* for dma_desc_pool_virt, dma_addr_t for dma_desc_pool, and use %pad to print dma_addr_t. Signed-off-by: YueHaibing --- drivers/dma/iop-adma.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/drivers/dma/iop-adma.c

[PATCH net-next] fib: Fix undef compile warning

net/core/fib_rules.c:26:7: warning: "CONFIG_IP_MULTIPLE_TABLES" is not defined, evaluates to 0 [-Wundef] #elif CONFIG_IP_MULTIPLE_TABLES ^ Fixes: 8b66a6fd34f5 ("fib: fix another fib_rules_ops indirect call wrapper problem") Signed-off-by:

Re: [PATCH -next] media: staging: tegra-vde: Mark tegra_vde_runtime_suspend as __maybe_unused

On 2020/8/3 20:51, Dmitry Osipenko wrote: > 03.08.2020 14:59, YueHaibing пишет: >> If CONFIG_PM is not set, gcc warns: >> >> drivers/staging/media/tegra-vde/vde.c:916:12: >> warning: 'tegra_vde_runtime_suspend' defined but not used >> [-Wunused-functio

[PATCH -next] PCI: cadence: Make cdns_pci_map_bus stub helper inline

If CONFIG_PCIE_CADENCE_HOST is n, gcc warns: drivers/pci/controller/cadence/pcie-cadence.h:486:22: warning: 'cdns_pci_map_bus' defined but not used [-Wunused-function] Make stub helper inline to fix this. Signed-off-by: YueHaibing --- drivers/pci/controller/cadence/pcie-cadence.h |

[PATCH -next] media: staging: tegra-vde: Mark tegra_vde_runtime_suspend as __maybe_unused

If CONFIG_PM is not set, gcc warns: drivers/staging/media/tegra-vde/vde.c:916:12: warning: 'tegra_vde_runtime_suspend' defined but not used [-Wunused-function] Make it __maybe_unused to fix this. Signed-off-by: YueHaibing --- drivers/staging/media/tegra-vde/vde.c | 2 +- 1 file

[PATCH -next] usb: mtu3: Remove unsused inline function is_first_entry

It is never used, so can be removed. Signed-off-by: YueHaibing --- drivers/usb/mtu3/mtu3.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/usb/mtu3/mtu3.h b/drivers/usb/mtu3/mtu3.h index 71f4f02c05c6..aef0a0bba25a 100644 --- a/drivers/usb/mtu3/mtu3.h +++ b/drivers/usb/mtu3/mtu3

[PATCH net-next] ip_vti: Fix unused variable warning

If CONFIG_INET_XFRM_TUNNEL is set but CONFIG_IPV6 is n, net/ipv4/ip_vti.c:493:27: warning: 'vti_ipip6_handler' defined but not used [-Wunused-variable] Signed-off-by: YueHaibing --- net/ipv4/ip_vti.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/ipv4/ip_vti.c b/net/ipv

[PATCH -next] soc: qcom: geni: Fix unused lable warning

pport to fix earlycon crash") Signed-off-by: YueHaibing --- drivers/soc/qcom/qcom-geni-se.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc/qcom/qcom-geni-se.c b/drivers/soc/qcom/qcom-geni-se.c index 3413129d73ef..d0e4f520cff8 100644 --- a/drivers/soc/qcom/qcom

[PATCH -next] drm/amdgpu/vcn3.0: Remove set but not used variable 'direct_poll'

drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c: In function vcn_v3_0_start_sriov: drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c:1235:3: warning: variable direct_poll set but not used [-Wunused-but-set-variable] It is never used, so can remove it. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu

[PATCH v2] tools/bpftool: Fix error handing in do_skeleton()

Fix pass 0 to PTR_ERR, also dump more err info using libbpf_strerror. Fixes: 5dc7a8b21144 ("bpftool, selftests/bpf: Embed object file inside skeleton") Signed-off-by: YueHaibing Reviewed-by: Quentin Monnet --- v2: use libbpf_strerror --- tools/bpf/bpftool/gen.c | 5 - 1 file

[PATCH -next] powerpc: Remove unneeded inline functions

Both of those functions are only called from 64-bit only code, so the stubs should not be needed at all. Suggested-by: Michael Ellerman Signed-off-by: YueHaibing --- arch/powerpc/include/asm/mmu_context.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/include/asm

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