[PATCH -next v2] tools/testing/nvdimm: Make symbol '__nfit_test_ioremap' static

2021-04-20 Thread Zou Wei
The sparse tool complains as follows: tools/testing/nvdimm/test/iomap.c:65:14: warning: symbol '__nfit_test_ioremap' was not declared. Should it be static? This symbol is not used outside of iomap.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Zou Wei --- tools

[PATCH -next] tools/testing/nvdimm: Make symbol '__nfit_test_ioremap' static

2021-04-07 Thread Zou Wei
The sparse tool complains as follows: tools/testing/nvdimm/test/iomap.c:65:14: warning: symbol '__nfit_test_ioremap' 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-off-by: Zou Wei --- tools

[PATCH -next] agp: uninorth: Make some symbols static

2021-03-25 Thread Zou Wei
outside of uninorth-agp.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Zou Wei --- 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 62de7f4

[PATCH -next] misc: sgi-xp: xp_main: make some symbols static

2021-03-24 Thread Zou Wei
of xp_main.c, so this commit marks them static. Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/misc/sgi-xp/xp_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/misc/sgi-xp/xp_main.c b/drivers/misc/sgi-xp/xp_main.c index cf2965aa5c05..87d156c15f35 100644

[PATCH -next] mtd: rawnand: rockchip: Use flexible-array member instead of zero-length array

2021-03-23 Thread Zou Wei
Suppresses the following coccinelle warning: drivers/mtd/nand/raw/rockchip-nand-controller.c:162:4-8: WARNING use flexible-array member instead Signed-off-by: Zou Wei --- drivers/mtd/nand/raw/rockchip-nand-controller.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] drm/vmwgfx/vmwgfx_validation: Use flexible-array member instead of zero-length array

2021-03-23 Thread Zou Wei
Suppresses the following coccinelle warning: drivers/gpu/drm/vmwgfx/vmwgfx_validation.c:85:15-22: WARNING use flexible-array member instead Signed-off-by: Zou Wei --- drivers/gpu/drm/vmwgfx/vmwgfx_validation.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm

[PATCH -next] coresight: etm-perf: Mark format_attr_contextid with static keyword

2021-03-23 Thread Zou Wei
Fix the following sparse warning: drivers/hwtracing/coresight/coresight-etm-perf.c:61:25: warning: symbol 'format_attr_contextid' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/hwtracing/coresight/coresight-etm-perf.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH -next] drm/nouveau/core/client: Mark nvkm_uclient_sclass with static keyword

2021-03-23 Thread Zou Wei
Fix the following sparse warning: drivers/gpu/drm/nouveau/nvkm/core/client.c:64:1: warning: symbol 'nvkm_uclient_sclass' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/gpu/drm/nouveau/nvkm/core/client.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH -next] drm: Markdrm_send_event_helper with static keyword

2021-03-23 Thread Zou Wei
Fix the following sparse warning: drivers/gpu/drm/drm_file.c:789:6: warning: symbol 'drm_send_event_helper' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/gpu/drm/drm_file.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm

[PATCH -next] soc / drm: mediatek: Mark mtk_mutex_driver with static keyword

2021-02-18 Thread Zou Wei
Fix the following sparse warning: drivers/soc/mediatek/mtk-mutex.c:464:24: warning: symbol 'mtk_mutex_driver' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/soc/mediatek/mtk-mutex.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/soc

[PATCH -next] ACPI: tables: Mark acpi_init_fpdt with static keyword

2021-01-28 Thread Zou Wei
Fix the following sparse warning: drivers/acpi/acpi_fpdt.c:230:6: warning: symbol 'acpi_init_fpdt' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/acpi/acpi_fpdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/acpi_fpdt.c b/drivers

[PATCH -next] usb: core: Remove unused including

2021-01-25 Thread Zou Wei
Fix the following versioncheck warning: drivers/usb/core/hcd.c:14:1: unused including Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/usb/core/hcd.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index 3f03813..53bc93d 100644

[PATCH -next] usb: gadget: Remove unused including

2021-01-25 Thread Zou Wei
Fix the following versioncheck warning: drivers/usb/gadget/udc/aspeed-vhub/hub.c:33:1: unused including Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/usb/gadget/udc/aspeed-vhub/hub.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/gadget/udc/aspeed-vhub/hub.c b

[PATCH -next] kernel/sys.c: Remove unused including

2021-01-25 Thread Zou Wei
Fix the following versioncheck warning: kernel/sys.c:42:1: unused including Reported-by: Hulk Robot Signed-off-by: Zou Wei --- kernel/sys.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/sys.c b/kernel/sys.c index e6fd8b8..8ecd3c5 100644 --- a/kernel/sys.c +++ b/kernel/sys.c

[PATCH -next] net/mlx5: Remove unused including

2021-01-25 Thread Zou Wei
Fix the following versioncheck warning: drivers/net/ethernet/mellanox/mlx5/core/main.c:53:1: unused including Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net/ethernet/mellanox/mlx5/core/main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox

[PATCH -next] scsi: gdth: Remove unused including

2021-01-25 Thread Zou Wei
Fix the following versioncheck warning: drivers/scsi/gdth.c:85:1: unused including Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/scsi/gdth.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/scsi/gdth.c b/drivers/scsi/gdth.c index f43bd53..f949001 100644 --- a/drivers

[PATCH -next] media: venus: Mark bufreq_enc with static keyword

2021-01-14 Thread Zou Wei
Fix the following sparse warning: drivers/media/platform/qcom/venus/hfi_plat_bufs_v6.c:1242:5: warning: symbol 'bufreq_enc' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/media/platform/qcom/venus/hfi_plat_bufs_v6.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH -next] phy: mediatek: Mark mtk_mipi_tx_driver with static keyword

2021-01-11 Thread Zou Wei
Fix the following sparse warning: drivers/phy/mediatek/phy-mtk-mipi-dsi.c:237:24: warning: symbol 'mtk_mipi_tx_driver' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/phy/mediatek/phy-mtk-mipi-dsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] media: ti-vpe: cal: Mark cal_camerarx_media_ops with static keyword

2021-01-11 Thread Zou Wei
Fix the following sparse warning: drivers/media/platform/ti-vpe/cal-camerarx.c:783:32: warning: symbol 'cal_camerarx_media_ops' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/media/platform/ti-vpe/cal-camerarx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] vdpa: Mark macaddr_buf with static keyword

2020-12-17 Thread Zou Wei
Fix the following sparse warning: drivers/vdpa/vdpa_sim/vdpa_sim_net.c:36:4: warning: symbol 'macaddr_buf' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/vdpa/vdpa_sim/vdpa_sim_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] usb: xhci: remove unused variable

2020-12-14 Thread Zou Wei
Fix variable set but not used compilation warnings: drivers/usb/host/xhci.c:1338:15: warning: variable ‘len’ set but not used [-Wunused-but-set-variable] unsigned int len; ^~~ Signed-off-by: Zou Wei --- drivers/usb/host/xhci.c | 9 - 1 file changed, 4 insertions(+), 5

[PATCH -next] lib/ubsan.c: Mark type_check_kinds with static keyword

2020-12-10 Thread Zou Wei
Fix the following sparse warning: lib/ubsan.c:20:12: warning: symbol 'type_check_kinds' was not declared. Should it be static? Signed-off-by: Zou Wei --- lib/ubsan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/ubsan.c b/lib/ubsan.c index cb9af3f6..3cfe902 100644

[PATCH -next v2] net/mlx5_core: remove unused including

2020-12-08 Thread Zou Wei
Remove including that don't need it. Fixes: 17a7612b99e6 ("net/mlx5_core: Clean driver version and name") Signed-off-by: Zou Wei --- drivers/net/ethernet/mellanox/mlx5/core/en_rep.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c

[PATCH -next] usb: cdnsp: Mark cdnsp_gadget_ops with static keyword

2020-12-08 Thread Zou Wei
Fix the following sparse warning: drivers/usb/cdns3/cdnsp-gadget.c:1546:29: warning: symbol 'cdnsp_gadget_ops' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/usb/cdns3/cdnsp-gadget.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb

[PATCH -next] platform/x86: ISST: Mark mmio_range_devid_0 and mmio_range_devid_1 with static keyword

2020-12-08 Thread Zou Wei
. Should it be static? Signed-off-by: Zou Wei --- drivers/platform/x86/intel_speed_select_if/isst_if_mmio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/intel_speed_select_if/isst_if_mmio.c b/drivers/platform/x86/intel_speed_select_if/isst_if_mmio.c

[PATCH -next] net/mlx5_core: remove unused including

2020-12-07 Thread Zou Wei
Remove including that don't need it. Signed-off-by: Zou Wei --- drivers/net/ethernet/mellanox/mlx5/core/en_rep.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c b/drivers/net/ethernet/mellanox/mlx5/core/en_rep.c index 989c70c..82ecc161

[PATCH -next] regulator: da9121: Mark some symbols with static keyword

2020-12-03 Thread Zou Wei
: symbol 'status_event_handling' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/regulator/da9121-regulator.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/regulator/da9121-regulator.c b/drivers/regulator/da9121-regulator.c index

[tip: x86/platform] x86/platform/uv: Make uv_pcibus_kset and uv_hubs_kset static

2020-12-03 Thread tip-bot2 for Zou Wei
The following commit has been merged into the x86/platform branch of tip: Commit-ID: 0c683e9de0c78ee53e220eac9ee3604ca662737a Gitweb: https://git.kernel.org/tip/0c683e9de0c78ee53e220eac9ee3604ca662737a Author:Zou Wei AuthorDate:Mon, 30 Nov 2020 19:11:53 +08:00 Committer

[PATCH -next v2] firmware: xilinx: Mark pm_api_features_map with static keyword

2020-12-01 Thread Zou Wei
Fix the following sparse warning: drivers/firmware/xilinx/zynqmp.c:32:1: warning: symbol 'pm_api_features_map' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/firmware/xilinx/zynqmp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] firmware: xilinx: Mark pm_api_features_map with static keyword

2020-12-01 Thread Zou Wei
Fix the following sparse warning: drivers/firmware/xilinx/zynqmp.c:32:1: warning: symbol 'pm_api_features_map' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/firmware/xilinx/zynqmp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] x86/platform/uv: Mark some symbols with static keyword

2020-11-30 Thread Zou Wei
Fix the following sparse warnings: drivers/platform/x86/uv_sysfs.c:22:13: warning: symbol 'uv_pcibus_kset' was not declared. Should it be static? drivers/platform/x86/uv_sysfs.c:23:13: warning: symbol 'uv_hubs_kset' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers

[PATCH -next v2] sched/topology: Mark some symbols with static keyword

2020-11-24 Thread Zou Wei
ology,schedutil: Wrap sched domains rebuild") Signed-off-by: Zou Wei --- kernel/sched/topology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/sched/topology.c b/kernel/sched/topology.c index 5d3675c..2676687 100644 --- a/kernel/sched/topology.c +++ b/kernel/sched/

[PATCH -next] sched/topology: Mark some symbols with static keyword

2020-11-24 Thread Zou Wei
Fix the following sparse warnings: kernel/sched/topology.c:211:1: warning: symbol 'sched_energy_mutex' was not declared. Should it be static? kernel/sched/topology.c:212:6: warning: symbol 'sched_energy_update' was not declared. Should it be static? Signed-off-by: Zou Wei --- kernel/sched

[PATCH -next] mm/swap.c: Mark sort_page_lruvec with static keyword

2020-11-24 Thread Zou Wei
Fix the following sparse warning: mm/swap.c:1018:5: warning: symbol 'sort_page_lruvec' was not declared. Should it be static? Signed-off-by: Zou Wei --- mm/swap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/swap.c b/mm/swap.c index c787b38..5022dfe 100644 --- a/mm

[PATCH -next] pinctrl: pinctrl-microchip-sgpio: Mark some symbols with static keyword

2020-11-24 Thread Zou Wei
/pinctrl/pinctrl-microchip-sgpio.c:73:31: warning: symbol 'properties_sparx5' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/pinctrl/pinctrl-microchip-sgpio.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/pinctrl/pinctrl-microchip-sgpio.c

[PATCH -next] scsi: lpfc: Mark lpfc_nvmet_prep_abort_wqe with static keyword

2020-11-17 Thread Zou Wei
Fix the following sparse warning: drivers/scsi/lpfc/lpfc_nvmet.c:3340:1: warning: symbol 'lpfc_nvmet_prep_abort_wqe' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/scsi/lpfc/lpfc_nvmet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH -next] btrfs: remove unused variable

2020-11-17 Thread Zou Wei
] u64 zone_size; ^ Signed-off-by: Zou Wei --- fs/btrfs/ctree.c | 3 --- fs/btrfs/zoned.c | 2 -- 2 files changed, 5 deletions(-) diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index 32a57a7..e5a0941 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c @@ -1578,13 +1578,10 @@ int

[PATCH -next] powerpc/powernv/sriov: Fix unsigned comparison to zero

2020-11-16 Thread Zou Wei
Fixes coccicheck warnings: ./arch/powerpc/platforms/powernv/pci-sriov.c:443:7-10: WARNING: Unsigned expression compared with zero: win < 0 ./arch/powerpc/platforms/powernv/pci-sriov.c:462:7-10: WARNING: Unsigned expression compared with zero: win < 0 Signed-off-by: Zou Wei --- arch/p

[PATCH -next] ASoC: samsung: fix platform_no_drv_owner.cocci warnings

2020-11-16 Thread Zou Wei
./sound/soc/samsung/midas_wm1811.c:534:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Zou Wei --- sound/soc/samsung/midas_wm1811.c | 1 - 1

[PATCH -next] mm/page_alloc: Mark some symbols with static keyword

2020-11-16 Thread Zou Wei
Fix the following sparse warnings: mm/page_alloc.c:3040:6: warning: symbol '__drain_all_pages' was not declared. Should it be static? mm/page_alloc.c:6349:6: warning: symbol '__zone_set_pageset_high_and_batch' was not declared. Should it be static? Signed-off-by: Zou Wei --- mm/page_alloc.c

[PATCH -next] cxgb4: Remove unused variable ret

2020-11-14 Thread Zou Wei
This patch fixes below warning reported by coccicheck: ./drivers/net/ethernet/chelsio/cxgb4/t4_hw.c:3284:5-8: Unneeded variable: "ret". Return "0" on line 3301 Signed-off-by: Zou Wei --- drivers/net/ethernet/chelsio/cxgb4/t4_hw.c | 4 +--- 1 file changed, 1 insertion(+),

[PATCH -next] mfd: altera-sysmgr: Use resource_size function on resource object

2020-11-14 Thread Zou Wei
./drivers/mfd/altera-sysmgr.c:155:36-39: WARNING: Suspicious code. resource_size is maybe missing with res Generated by: scripts/coccinelle/api/resource_size.cocci Signed-off-by: Zou Wei --- drivers/mfd/altera-sysmgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] mailbox: pcc: fix platform_no_drv_owner.cocci warning

2020-11-14 Thread Zou Wei
./drivers/mailbox/pcc.c:580:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Zou Wei --- drivers/mailbox/pcc.c | 1 - 1 file changed, 1

[PATCH -next] auxdisplay: fix platform_no_drv_owner.cocci warning

2020-11-14 Thread Zou Wei
./drivers/auxdisplay/lcd2s.c:373:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: Zou Wei --- drivers/auxdisplay/lcd2s.c | 1 - 1 file

[PATCH -next] scsi: be2iscsi: Mark beiscsi_attrs with static keyword

2020-11-13 Thread Zou Wei
Fix the following sparse warning: ./be_main.c:167:25: warning: symbol 'beiscsi_attrs' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/scsi/be2iscsi/be_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/be2iscsi/be_main.c b/drivers

[PATCH -next] drm/virtio: Make virtgpu_dmabuf_ops with static keyword

2020-11-13 Thread Zou Wei
Fix the following sparse warning: ./virtgpu_prime.c:46:33: warning: symbol 'virtgpu_dmabuf_ops' was not declared. Should it be static? Signed-off-by: Zou Wei --- drivers/gpu/drm/virtio/virtgpu_prime.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/virtio

[PATCH -next] perf intel-bts: use true for bool variables

2020-11-09 Thread Zou Wei
Fix coccicheck warnings: ./intel-bts.c:472:3-13: WARNING: Assignment of 0/1 to bool variable ./intel-bts.c:515:3-13: WARNING: Assignment of 0/1 to bool variable Reported-by: Hulk Robot Signed-off-by: Zou Wei --- tools/perf/util/intel-bts.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH -next] cpumask: use false and true for bool variables

2020-11-09 Thread Zou Wei
Fix coccicheck warnings: ./lib/cpumask.c:342:6-13: WARNING: Comparison of 0/1 to bool variable ./lib/cpumask.c:351:33-40: WARNING: Comparison of 0/1 to bool variable ./lib/cpumask.c:406:3-11: WARNING: Assignment of 0/1 to bool variable Reported-by: Hulk Robot Signed-off-by: Zou Wei --- lib

[PATCH -next] mmc: owl-mmc: use true and false for bool variables

2020-11-08 Thread Zou Wei
Fix coccicheck warnings: ./owl-mmc.c:524:2-18: WARNING: Assignment of 0/1 to bool variable ./owl-mmc.c:528:2-18: WARNING: Assignment of 0/1 to bool variable Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/mmc/host/owl-mmc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH -next v2] ASoC: mediatek: mt8192: Make some symbols static

2020-11-05 Thread Zou Wei
Signed-off-by: Zou Wei --- sound/soc/mediatek/mt8192/mt8192-afe-gpio.c | 2 +- sound/soc/mediatek/mt8192/mt8192-afe-pcm.c | 4 ++-- sound/soc/mediatek/mt8192/mt8192-dai-i2s.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/sound/soc/mediatek/mt8192/mt8192-afe-gpio.c b

[PATCH -next] ASoC: mediatek: mt8192: Make some symbols static

2020-11-05 Thread Zou Wei
-by: Zou Wei --- sound/soc/mediatek/mt8192/mt8192-dai-i2s.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/mediatek/mt8192/mt8192-dai-i2s.c b/sound/soc/mediatek/mt8192/mt8192-dai-i2s.c index 53c560e..5b29340 100644 --- a/sound/soc/mediatek/mt8192/mt8192-dai-i2s.c +++ b

[PATCH -next] clk: imx: scu: Make pd_np with static keyword

2020-11-04 Thread Zou Wei
Fix the following sparse warning: ./clk-scu.c:23:20: warning: symbol 'pd_np' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/clk/imx/clk-scu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/imx/clk-scu.c b

[PATCH -next] dpaa_eth: use false and true for bool variables

2020-11-03 Thread Zou Wei
Fix coccicheck warnings: ./dpaa_eth.c:2549:2-22: WARNING: Assignment of 0/1 to bool variable ./dpaa_eth.c:2562:2-22: WARNING: Assignment of 0/1 to bool variable Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net/ethernet/freescale/dpaa/dpaa_eth.c | 4 ++-- 1 file changed, 2

[PATCH -next] IB/isert: Fix warning Comparison to bool

2020-11-03 Thread Zou Wei
Fix below warning reported by coccicheck: ./ib_isert.c:1104:12-24: WARNING: Comparison to bool Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/infiniband/ulp/isert/ib_isert.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/ulp/isert/ib_isert.c

[PATCH -next] clk: imx: gate2: Remove unused variable ret

2020-11-03 Thread Zou Wei
This patch fixes below warning reported by coccicheck: ./clk-gate2.c:57:5-8: Unneeded variable: "ret". Return "0" on line 68 Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/clk/imx/clk-gate2.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a

[PATCH -next] drm/nouveu: remove unused variable

2020-11-02 Thread Zou Wei
device *dev; ^~~ Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/nouveau/nouveau_bo.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c index 8133377..96f00b5 100644 --- a/drivers

[PATCH -next] drm/amd/pm: remove unused variable

2020-11-02 Thread Zou Wei
Fix variable set but not used compilation warning: ./vangogh_ppt.c:397:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable] int ret = 0; ^~~ Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c | 2 -- 1 file changed

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

2020-11-02 Thread Zou Wei
Fixes the following W=1 kernel build warning: ./panfrost_job.c:617:28: warning: unused variable ‘js’ [-Wunused-variable] struct panfrost_job_slot *js = pfdev->js; ^~ Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/panfrost/panfrost_job.c

[PATCH -next v2] usb: Mark sync_all_pins() with static keyword

2020-10-30 Thread Zou Wei
GPIO to be used for 7211 USB signals") Reported-by: Hulk Robot Signed-off-by: Zou Wei --- v2: - Make title more precise. - Shorten the warning path. - Add period. drivers/usb/misc/brcmstb-usb-pinmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/misc/brcmstb-usb-p

[PATCH -next v3] platform/x86/dell-wmi-sysman: Make some symbols static

2020-10-30 Thread Zou Wei
: warning: symbol 'po_mechanism' was not declared. Should it be static? ./passobj-attributes.c:129:23: warning: symbol 'po_role' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- v2: - put all of them in a way that each occupies only a single line v3

[PATCH -next] usb: Make sync_all_pins static

2020-10-30 Thread Zou Wei
GPIO to be used for 7211 USB signals") Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/usb/misc/brcmstb-usb-pinmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/misc/brcmstb-usb-pinmap.c b/drivers/usb/misc/brcmstb-usb-pinmap.c index 02144c3..cc96

[PATCH -next v2] platform/x86/dell-wmi-sysman: Make some symbols static

2020-10-29 Thread Zou Wei
/x86/dell-wmi-sysman/passobj-attributes.c:129:23: warning: symbol 'po_role' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- v2: - put all of them in a way that each occupies only a single line drivers/platform/x86/dell-wmi-sysman/passobj-attributes.c | 14

[PATCH -next] dma-buf: heaps: Make sys_heap static

2020-10-29 Thread Zou Wei
Fix the following sparse warning: drivers/dma-buf/heaps/system_heap.c:23:17: warning: symbol 'sys_heap' was not declared. Should it be static? sys_heap has only call within system_heap.c It should be static Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/dma-buf/heaps

[PATCH -next] drm/amdgpu/swsmu: Remove unused static struct 'navi10_i2c_algo'

2020-10-29 Thread Zou Wei
= { ^~~ Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c index ef1a62e..bec63f2 100644

[PATCH -next] platform/surface: remove status assignment without reading

2020-10-29 Thread Zou Wei
The status local variable is assigned but never read: drivers/platform/surface/surface3-wmi.c:60:14: warning: variable ‘status’ set but not used [-Wunused-but-set-variable] acpi_status status; ^~ Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/platform/surface

[PATCH -next] platform/x86/dell-wmi-sysman: Make wmi_sysman_kobj_sysfs_ops static

2020-10-29 Thread Zou Wei
Fix the following sparse warning: drivers/platform/x86/dell-wmi-sysman/sysman.c:258:24: warning: symbol 'wmi_sysman_kobj_sysfs_ops' was not declared. Should it be static? wmi_sysman_kobj_sysfs_ops has only call within sysman.c It should be static Reported-by: Hulk Robot Signed-off-by: Zou Wei

[PATCH -next] platform/x86/dell-wmi-sysman: Make some symbols static

2020-10-29 Thread Zou Wei
Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/platform/x86/dell-wmi-sysman/passobj-attributes.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/platform/x86/dell-wmi-sysman/passobj-attributes.c b/drivers/platform/x86/dell-wmi-sysman/passobj

[PATCH -next] net: stmmac: platform: remove useless if/else

2020-10-28 Thread Zou Wei
Fix the following coccinelle report: ./drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c:233:6-8: WARNING: possible condition with no effect (if == else) Both branches are the same, so remove the else if/else altogether. Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net

[PATCH -next] net: nvidia: forcedeth: remove useless if/else

2020-10-28 Thread Zou Wei
Fix the following coccinelle report: ./drivers/net/ethernet/nvidia/forcedeth.c:3479:8-10: WARNING: possible condition with no effect (if == else) Both branches are the same, so remove the else if/else altogether. Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net/ethernet/nvidia

[PATCH -next] coresight: core: Remove unneeded semicolon

2020-10-28 Thread Zou Wei
Fixes coccicheck warning: ./drivers/hwtracing/coresight/coresight-core.c:421:4-5: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/hwtracing/coresight/coresight-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwtracing/coresight

[PATCH -next] drm/i915: Remove unused variable ret

2020-10-28 Thread Zou Wei
This patch fixes below warnings reported by coccicheck ./drivers/gpu/drm/i915/i915_debugfs.c:789:5-8: Unneeded variable: "ret". Return "0" on line 1012 Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/i915/i915_debugfs.c | 3 +-- 1 file changed, 1 inser

[PATCH -next] mm/hugetlb: Make is_aligned_hugepage_range static

2020-10-28 Thread Zou Wei
Fix the following sparse warning: arch/mips/mm/hugetlbpage.c:64:5: warning: symbol 'is_aligned_hugepage_range' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- arch/mips/mm/hugetlbpage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH -next] phy: cpcap-usb: Use IRQF_ONESHOT

2020-10-28 Thread Zou Wei
Fixes coccicheck error: ./drivers/phy/motorola/phy-cpcap-usb.c:365:9-34: ERROR: Threaded IRQ with no primary handler requested without IRQF_ONESHOT Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/phy/motorola/phy-cpcap-usb.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion

[PATCH -next] mm, page_alloc: Fix old-style function definition

2020-10-28 Thread Zou Wei
Fix warning: mm/page_alloc.c:763:6: warning: old-style function definition [-Wold-style-definition] void init_mem_debugging() ^~ Reported-by: Hulk Robot Signed-off-by: Zou Wei --- mm/page_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm

[PATCH -next] drm/amd/display: remove useless if/else

2020-10-28 Thread Zou Wei
heses in dce_calcs.c") Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c b/drivers/gpu/drm/amd/display/dc/calcs/d

[PATCH -next] firmware: tegra: Use memdup_user() as a cleanup

2020-10-28 Thread Zou Wei
Hulk Robot Signed-off-by: Zou Wei --- drivers/firmware/tegra/bpmp-debugfs.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/firmware/tegra/bpmp-debugfs.c b/drivers/firmware/tegra/bpmp-debugfs.c index c1bbba9..7234136 100644 --- a/drivers/firmware/tegra/bpmp

[PATCH -next] net: wan: sdla: Use bitwise instead of arithmetic

2020-10-28 Thread Zou Wei
Fix the following coccinelle warnings: ./drivers/net/wan/sdla.c:841:38-39: WARNING: sum of probable bitmasks, consider | Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net/wan/sdla.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/wan/sdla.c b

[PATCH -next] drm/tilcdc: fix platform_no_drv_owner.cocci warnings

2020-10-28 Thread Zou Wei
s (v5)") Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/tilcdc/tilcdc_panel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c index 00efc30..4235780 100644 --- a/drivers/gpu/drm/tilcdc/tilc

[PATCH -next] firmware: ti_sci: Remove unneeded semicolon

2020-09-22 Thread Zou Wei
r ringacc, psi-l and udma") Signed-off-by: Zou Wei --- drivers/firmware/ti_sci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index 896f53e..e0c8adf 100644 --- a/drivers/firmware/ti_sci.c +++ b/drivers/firmware

[PATCH -next] dma-buf: heaps: Remove unused variable ret

2020-09-22 Thread Zou Wei
This patch fixes below warnings reported by coccicheck ./drivers/dma-buf/heaps/heap-helpers.c:202:5-8: Unneeded variable: "ret". Return "0" on line 215 Signed-off-by: Zou Wei --- drivers/dma-buf/heaps/heap-helpers.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[PATCH -next] clk: mediatek: fix platform_no_drv_owner.cocci warnings

2020-09-22 Thread Zou Wei
pport") Signed-off-by: Zou Wei --- drivers/clk/mediatek/clk-mt6765.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/clk/mediatek/clk-mt6765.c b/drivers/clk/mediatek/clk-mt6765.c index db8db1b..d77ea5a 100644 --- a/drivers/clk/mediatek/clk-mt6765.c +++ b/drivers/clk/mediatek/clk-mt6765.c

[PATCH -next] ACPICA: Remove unneeded semicolon

2020-09-22 Thread Zou Wei
Fixes coccicheck warning: ./drivers/acpi/acpica/nsalloc.c:297:2-3: Unneeded semicolon Signed-off-by: Zou Wei --- drivers/acpi/acpica/nsalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/acpi/acpica/nsalloc.c b/drivers/acpi/acpica/nsalloc.c index fe9b363

[tip: core/rcu] locktorture: Use true and false to assign to bool variables

2020-07-31 Thread tip-bot2 for Zou Wei
The following commit has been merged into the core/rcu branch of tip: Commit-ID: d02c6b52d12fa30eeabfaf5aefe12078eacb94b2 Gitweb: https://git.kernel.org/tip/d02c6b52d12fa30eeabfaf5aefe12078eacb94b2 Author:Zou Wei AuthorDate:Mon, 13 Apr 2020 20:02:59 +08:00 Committer

[tip: perf/core] libtraceevent: Remove unneeded semicolon

2020-05-08 Thread tip-bot2 for Zou Wei
The following commit has been merged into the perf/core branch of tip: Commit-ID: eebe80c982546ce1041b7dcf3c461406f1e7e88f Gitweb: https://git.kernel.org/tip/eebe80c982546ce1041b7dcf3c461406f1e7e88f Author:Zou Wei AuthorDate:Tue, 28 Apr 2020 17:12:01 +08:00 Committer

[tip: perf/core] perf report: Fix warning assignment of 0/1 to bool variable

2020-05-08 Thread tip-bot2 for Zou Wei
The following commit has been merged into the perf/core branch of tip: Commit-ID: 6fa9c3e779b89612c56f8ee0096b2252d906357a Gitweb: https://git.kernel.org/tip/6fa9c3e779b89612c56f8ee0096b2252d906357a Author:Zou Wei AuthorDate:Sun, 26 Apr 2020 20:38:03 +08:00 Committer

[tip: perf/core] perf tools: Remove unneeded semicolons

2020-05-08 Thread tip-bot2 for Zou Wei
The following commit has been merged into the perf/core branch of tip: Commit-ID: 8284bbeab75f1842dc11531356115c4d09abebc0 Gitweb: https://git.kernel.org/tip/8284bbeab75f1842dc11531356115c4d09abebc0 Author:Zou Wei AuthorDate:Tue, 28 Apr 2020 17:18:43 +08:00 Committer

[tip: perf/core] perf c2c: Remove unneeded semicolon

2020-05-08 Thread tip-bot2 for Zou Wei
The following commit has been merged into the perf/core branch of tip: Commit-ID: 2cca512ad2fe14d060838eaf8581cda600764141 Gitweb: https://git.kernel.org/tip/2cca512ad2fe14d060838eaf8581cda600764141 Author:Zou Wei AuthorDate:Tue, 28 Apr 2020 16:58:56 +08:00 Committer

[PATCH -next] drm/udl: Make udl_handle_damage static

2020-04-30 Thread Zou Wei
Fix the following sparse warning: drivers/gpu/drm/udl/udl_modeset.c:269:5: warning: symbol 'udl_handle_damage' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/udl/udl_modeset.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PATCH -next] scsi: aacraid: Make some symbols static

2020-04-30 Thread Zou Wei
: warning: symbol 'aac_send_safw_hostttime' was not declared. Should it be static? drivers/scsi/aacraid/commsup.c:2383:5: warning: symbol 'aac_send_hosttime' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/scsi/aacraid/commsup.c | 4 ++-- drivers

[PATCH -next] drm/amdgpu: Fix warning Comparison to bool

2020-04-30 Thread Zou Wei
fix below warnings reported by coccicheck drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c:630:5-11: WARNING: Comparison to bool Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm

[PATCH -next] drm/amd/display: Fix unsigned comparison to zero

2020-04-29 Thread Zou Wei
Fixes coccicheck warning: drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c:1398:60-61: WARNING: Unsigned expression compared with zero: j >= 0 Fixes: 238387774232 ("drm/amd/display: fix rn soc bb update") Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/a

[PATCH -next] KVM: MIPS/Emulate: Remove unneeded semicolon

2020-04-29 Thread Zou Wei
Fixes coccicheck warnings: arch/mips/kvm/emulate.c:1793:3-4: Unneeded semicolon arch/mips/kvm/emulate.c:1968:3-4: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: Zou Wei --- arch/mips/kvm/emulate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/mips/kvm

[PATCH -next] KVM: MIPS/VZ: Remove unneeded semicolon

2020-04-29 Thread Zou Wei
Signed-off-by: Zou Wei --- arch/mips/kvm/vz.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/mips/kvm/vz.c b/arch/mips/kvm/vz.c index dde2088..389dd0f 100644 --- a/arch/mips/kvm/vz.c +++ b/arch/mips/kvm/vz.c @@ -1118,7 +1118,7 @@ static enum emulation_result

[PATCH -next] dlm: Remove unneeded semicolon

2020-04-29 Thread Zou Wei
Fixes coccicheck warning: fs/dlm/rcom.c:566:2-3: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: Zou Wei --- fs/dlm/rcom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/dlm/rcom.c b/fs/dlm/rcom.c index e3d9f72..4daf5dc 100644 --- a/fs/dlm/rcom.c +++ b/fs/dlm

[PATCH -next] NTFS: Remove unneeded semicolon

2020-04-29 Thread Zou Wei
Fixes coccicheck warnings: fs/ntfs/lcnalloc.c:902:2-3: Unneeded semicolon fs/ntfs/super.c:1615:2-3: Unneeded semicolon fs/ntfs/super.c:1684:2-3: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: Zou Wei --- fs/ntfs/lcnalloc.c | 2 +- fs/ntfs/super.c| 4 ++-- 2 files changed, 3

[PATCH -next] hpfs: Remove unneeded semicolon

2020-04-29 Thread Zou Wei
Fixes coccicheck warning: fs/hpfs/buffer.c:56:2-3: Unneeded semicolon Reported-by: Hulk Robot Signed-off-by: Zou Wei --- fs/hpfs/buffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/hpfs/buffer.c b/fs/hpfs/buffer.c index e285d6b..d392468 100644 --- a/fs/hpfs

[PATCH -next v3] hinic: Use ARRAY_SIZE for nic_vf_cmd_msg_handler

2020-04-29 Thread Zou Wei
fix coccinelle warning, use ARRAY_SIZE drivers/net/ethernet/huawei/hinic/hinic_sriov.c:713:43-44: WARNING: Use ARRAY_SIZE v1-->v2: remove cmd_number v2-->v3: preserve the reverse christmas tree ordering of local variables Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drive

[PATCH -next] drm/dp_mst: use false for bool variable

2020-04-29 Thread Zou Wei
Fixes coccicheck warning: drivers/gpu/drm/drm_dp_mst_topology.c:2229:6-13: WARNING: Assignment of 0/1 to bool variable Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm

[PATCH -next v2] hinic: Use ARRAY_SIZE for nic_vf_cmd_msg_handler

2020-04-28 Thread Zou Wei
fix coccinelle warning, use ARRAY_SIZE drivers/net/ethernet/huawei/hinic/hinic_sriov.c:713:43-44: WARNING: Use ARRAY_SIZE -- v1-->v2: remove cmd_number Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net/ethernet/huawei/hinic/hinic_sriov.c | 8 +++- 1 file chan

[PATCH -next] hinic: Use kmemdup instead of kzalloc and memcpy

2020-04-28 Thread Zou Wei
Fixes coccicheck warnings: drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:452:17-24: WARNING opportunity for kmemdup drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:458:23-30: WARNING opportunity for kmemdup Reported-by: Hulk Robot Signed-off-by: Zou Wei --- drivers/net/ethernet

  1   2   >