Lee Jones
2020-Nov-16 17:40 UTC
[Nouveau] [PATCH v2 00/42] Rid W=1 warnings from GPU (non-Radeon)
[sending again, as `git send-email` crashed!] This set contains fixes for some "wouldn't it be nice if" issues, however most of the patches seen here have been on the MLs, but were left unreviewed. Lee Jones (42): drm/amd/amdgpu/atombios_encoders: Remove set but unused variable 'backlight_level' drm/armada/armada_overlay: Staticify local function 'armada_overlay_duplicate_state' drm/drm_dp_mst_topology: Remove set but never used variable 'len' drm/exynos/exynos7_drm_decon: Supply missing description for param 'ctx' drm/exynos/exynos_drm_fimd: Add missing description for param 'ctx' drm/exynos/exynos_drm_gsc: Supply missing description for 'num_limits' drm/mediatek/mtk_dpi: Remove unused struct definition 'mtk_dpi_encoder_funcs' drm/mediatek/mtk_disp_color: Fix formatting and provide missing member description drm/mediatek/mtk_disp_ovl: Fix formatting and provide missing member description drm/mediatek/mtk_disp_rdma: Fix formatting and supply missing struct member description drm/mediatek/mtk_drm_crtc: Demote seriously out-of-date struct header drm/mediatek/mtk_drm_drv: Staticise local function invoked by reference drm/meson/meson_venc: Make local function 'meson_venc_hdmi_get_dmt_vmode' static drm/meson/meson_vclk: Make two local functions static drm/msm/adreno/a6xx_gpu: Staticise local function 'a6xx_idle' drm/msm/disp/mdp5/mdp5_crtc: Make local function 'mdp5_crtc_setup_pipeline()' static drm/msm/disp/mdp5/mdp5_ctl: Demote non-conformant kernel-doc headers drm/msm/disp/mdp5/mdp5_kms: Make local functions 'mdp5_{en,dis}able()' static drm/msm/disp/dpu1/dpu_core_perf: Remove set but unused variable 'dpu_cstate' drm/msm/disp/dpu1/dpu_encoder: Remove a bunch of unused variables drm/nouveau/nvkm/core/firmware: Fix formatting, provide missing param description drm/pl111/pl111_display: Make local function static drm/pl111/pl111_debugfs: Make local function 'pl111_debugfs_regs()' static drm/rockchip/dw-mipi-dsi-rockchip: Demote non-conformant kernel-doc headers drm/rockchip/rockchip_rgb: Consume our own header drm/rockchip/rockchip_lvds: Fix struct document formatting drm/selftests/test-drm_mm: Mark 'hole_end' as always_unused drm/selftests/test-drm_framebuffer: Remove set but unused variable 'fb' drm/selftests/test-drm_dp_mst_helper: Place 'struct drm_dp_sideband_msg_req_body' onto the heap drm/selftests/test-drm_dp_mst_helper: Move 'sideband_msg_req_encode_decode' onto the heap drm/ttm/ttm_bo: Fix one function header - demote lots of kernel-doc abuses drm/ttm/ttm_tt: Demote kernel-doc header format abuses drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header drm/v3d/v3d_drv: Remove unused static variable 'v3d_v3d_pm_ops' drm/v3d/v3d_gem: Provide descriptions for 'v3d_lookup_bos's params drm/v3d/v3d_sched: Demote non-conformant kernel-doc header drm/vc4/vc4_hdmi_regs: Mark some data sets as __maybe_unused drm/vc4/vc4_hdmi: Remove set but unused variable 'ret' drm/vc4/vc4_v3d: Demote non-conformant kernel-doc headers drm/vc4/vc4_debugfs: Demote non-conformant kernel-doc headers gpu/ipu-v3/ipu-di: Strip out 2 unused 'di_sync_config' entries include/drm/drm_atomic: Make use of 'new_crtc_state' .../gpu/drm/amd/amdgpu/atombios_encoders.c | 3 -- drivers/gpu/drm/armada/armada_overlay.c | 2 +- drivers/gpu/drm/drm_dp_mst_topology.c | 4 +- drivers/gpu/drm/exynos/exynos7_drm_decon.c | 1 + drivers/gpu/drm/exynos/exynos_drm_fimd.c | 1 + drivers/gpu/drm/exynos/exynos_drm_gsc.c | 1 + drivers/gpu/drm/mediatek/mtk_disp_color.c | 5 ++- drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 5 ++- drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 5 ++- drivers/gpu/drm/mediatek/mtk_dpi.c | 9 ----- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 4 +- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 4 +- drivers/gpu/drm/meson/meson_vclk.c | 8 ++-- drivers/gpu/drm/meson/meson_venc.c | 4 +- drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 2 +- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 3 -- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 12 +----- drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 6 +-- drivers/gpu/drm/msm/disp/mdp5/mdp5_ctl.c | 6 +-- drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 4 +- drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 9 +++-- drivers/gpu/drm/pl111/pl111_debugfs.c | 2 +- drivers/gpu/drm/pl111/pl111_display.c | 2 +- .../gpu/drm/rockchip/dw-mipi-dsi-rockchip.c | 4 +- drivers/gpu/drm/rockchip/rockchip_lvds.c | 2 +- drivers/gpu/drm/rockchip/rockchip_rgb.c | 1 + .../drm/selftests/test-drm_dp_mst_helper.c | 40 +++++++++++++------ .../gpu/drm/selftests/test-drm_framebuffer.c | 3 +- drivers/gpu/drm/selftests/test-drm_mm.c | 2 +- drivers/gpu/drm/ttm/ttm_bo.c | 23 ++++++----- drivers/gpu/drm/ttm/ttm_range_manager.c | 2 +- drivers/gpu/drm/ttm/ttm_tt.c | 4 +- drivers/gpu/drm/v3d/v3d_drv.c | 36 ----------------- drivers/gpu/drm/v3d/v3d_gem.c | 2 + drivers/gpu/drm/v3d/v3d_sched.c | 2 +- drivers/gpu/drm/vc4/vc4_debugfs.c | 4 +- drivers/gpu/drm/vc4/vc4_hdmi.c | 3 +- drivers/gpu/drm/vc4/vc4_hdmi_regs.h | 6 +-- drivers/gpu/drm/vc4/vc4_v3d.c | 4 +- drivers/gpu/ipu-v3/ipu-di.c | 4 -- include/drm/drm_atomic.h | 3 +- 41 files changed, 105 insertions(+), 142 deletions(-) Cc: Ajay Kumar <ajaykumar.rs at samsung.com> Cc: Akshu Agarwal <akshua at gmail.com> Cc: Alex Deucher <alexander.deucher at amd.com> Cc: amd-gfx at lists.freedesktop.org Cc: Ben Skeggs <bskeggs at redhat.com> Cc: Christian Koenig <christian.koenig at amd.com> Cc: "Christian K?nig" <christian.koenig at amd.com> Cc: Chris Zhong <zyw at rock-chips.com> Cc: Chun-Kuang Hu <chunkuang.hu at kernel.org> Cc: Daniel Vetter <daniel at ffwll.ch> Cc: David Airlie <airlied at linux.ie> Cc: David Francis <David.Francis at amd.com> Cc: dri-devel at lists.freedesktop.org Cc: Eric Anholt <eric at anholt.net> Cc: Eunchul Kim <chulspro.kim at samsung.com> Cc: Fabio Estevam <festevam at gmail.com> Cc: freedreno at lists.freedesktop.org Cc: "Heiko St?bner" <heiko at sntech.de> Cc: Huang Rui <ray.huang at amd.com> Cc: Inki Dae <inki.dae at samsung.com> Cc: Jerome Brunet <jbrunet at baylibre.com> Cc: Jie Qiu <jie.qiu at mediatek.com> Cc: Jinyoung Jeon <jy0.jeon at samsung.com> Cc: Joonyoung Shim <jy0922.shim at samsung.com> Cc: Kalyan Thota <kalyan_t at codeaurora.org> Cc: Kevin Hilman <khilman at baylibre.com> Cc: Krzysztof Kozlowski <krzk at kernel.org> Cc: Kyungmin Park <kyungmin.park at samsung.com> Cc: linaro-mm-sig at lists.linaro.org Cc: linux-amlogic at lists.infradead.org Cc: linux-arm-kernel at lists.infradead.org Cc: linux-arm-msm at vger.kernel.org Cc: linux-mediatek at lists.infradead.org Cc: linux-media at vger.kernel.org Cc: linux-rockchip at lists.infradead.org Cc: linux-samsung-soc at vger.kernel.org Cc: Luben Tuikov <luben.tuikov at amd.com> Cc: Lyude Paul <lyude at redhat.com> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com> Cc: Mark Yao <mark.yao at rock-chips.com> Cc: Martin Blumenstingl <martin.blumenstingl at googlemail.com> Cc: Matthias Brugger <matthias.bgg at gmail.com> Cc: Maxime Ripard <mripard at kernel.org> Cc: Neil Armstrong <narmstrong at baylibre.com> Cc: Nickey Yang <nickey.yang at rock-chips.com> Cc: Nirmoy Das <nirmoy.aiemd at gmail.com> Cc: nouveau at lists.freedesktop.org Cc: NXP Linux Team <linux-imx at nxp.com> Cc: Pengutronix Kernel Team <kernel at pengutronix.de> Cc: Philipp Zabel <p.zabel at pengutronix.de> Cc: Rob Clark <robdclark at gmail.com> Cc: Russell King <linux at armlinux.org.uk> Cc: Sandy Huang <hjc at rock-chips.com> Cc: Sangmin Lee <lsmin.lee at samsung.com> Cc: Sascha Hauer <s.hauer at pengutronix.de> Cc: Sean Paul <sean at poorly.run> Cc: Seung-Woo Kim <sw0312.kim at samsung.com> Cc: Shawn Guo <shawnguo at kernel.org> Cc: Sumit Semwal <sumit.semwal at linaro.org> Cc: Thomas Zimmermann <tzimmermann at suse.de> Cc: YT SHEN <yt.shen at mediatek.com> -- 2.25.1
Lee Jones
2020-Nov-16 17:40 UTC
[Nouveau] [PATCH 21/42] drm/nouveau/nvkm/core/firmware: Fix formatting, provide missing param description
... and demote non-conformant kernel-doc header. Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'subdev' not described in 'nvkm_firmware_get' drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'fwname' not described in 'nvkm_firmware_get' drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'ver' not described in 'nvkm_firmware_get' drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'fw' not described in 'nvkm_firmware_get' drivers/gpu/drm/nouveau/nvkm/core/firmware.c:106: warning: Function parameter or member 'fw' not described in 'nvkm_firmware_put' Cc: Ben Skeggs <bskeggs at redhat.com> Cc: David Airlie <airlied at linux.ie> Cc: Daniel Vetter <daniel at ffwll.ch> Cc: dri-devel at lists.freedesktop.org Cc: nouveau at lists.freedesktop.org Signed-off-by: Lee Jones <lee.jones at linaro.org> --- drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/core/firmware.c b/drivers/gpu/drm/nouveau/nvkm/core/firmware.c index 8b25367917ca0..ca1f8463cff51 100644 --- a/drivers/gpu/drm/nouveau/nvkm/core/firmware.c +++ b/drivers/gpu/drm/nouveau/nvkm/core/firmware.c @@ -58,9 +58,10 @@ nvkm_firmware_load_blob(const struct nvkm_subdev *subdev, const char *base, /** * nvkm_firmware_get - load firmware from the official nvidia/chip/ directory - * @subdev subdevice that will use that firmware - * @fwname name of firmware file to load - * @fw firmware structure to load to + * @subdev: subdevice that will use that firmware + * @fwname: name of firmware file to load + * @ver: firmware version to load + * @fw: firmware structure to load to * * Use this function to load firmware files in the form nvidia/chip/fwname.bin. * Firmware files released by NVIDIA will always follow this format. @@ -98,7 +99,7 @@ nvkm_firmware_get(const struct nvkm_subdev *subdev, const char *fwname, int ver, return -ENOENT; } -/** +/* * nvkm_firmware_put - release firmware loaded with nvkm_firmware_get */ void -- 2.25.1
Daniel Vetter
2020-Nov-17 19:02 UTC
[Nouveau] [PATCH 21/42] drm/nouveau/nvkm/core/firmware: Fix formatting, provide missing param description
On Mon, Nov 16, 2020 at 05:40:51PM +0000, Lee Jones wrote:> ... and demote non-conformant kernel-doc header. > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'subdev' not described in 'nvkm_firmware_get' > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'fwname' not described in 'nvkm_firmware_get' > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'ver' not described in 'nvkm_firmware_get' > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:71: warning: Function parameter or member 'fw' not described in 'nvkm_firmware_get' > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:106: warning: Function parameter or member 'fw' not described in 'nvkm_firmware_put' > > Cc: Ben Skeggs <bskeggs at redhat.com>Ben fyi I smashed this into drm-misc-next, seemed trivial enough to not be a bother. -Daniel> Cc: David Airlie <airlied at linux.ie> > Cc: Daniel Vetter <daniel at ffwll.ch> > Cc: dri-devel at lists.freedesktop.org > Cc: nouveau at lists.freedesktop.org > Signed-off-by: Lee Jones <lee.jones at linaro.org> > --- > drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/nvkm/core/firmware.c b/drivers/gpu/drm/nouveau/nvkm/core/firmware.c > index 8b25367917ca0..ca1f8463cff51 100644 > --- a/drivers/gpu/drm/nouveau/nvkm/core/firmware.c > +++ b/drivers/gpu/drm/nouveau/nvkm/core/firmware.c > @@ -58,9 +58,10 @@ nvkm_firmware_load_blob(const struct nvkm_subdev *subdev, const char *base, > > /** > * nvkm_firmware_get - load firmware from the official nvidia/chip/ directory > - * @subdev subdevice that will use that firmware > - * @fwname name of firmware file to load > - * @fw firmware structure to load to > + * @subdev: subdevice that will use that firmware > + * @fwname: name of firmware file to load > + * @ver: firmware version to load > + * @fw: firmware structure to load to > * > * Use this function to load firmware files in the form nvidia/chip/fwname.bin. > * Firmware files released by NVIDIA will always follow this format. > @@ -98,7 +99,7 @@ nvkm_firmware_get(const struct nvkm_subdev *subdev, const char *fwname, int ver, > return -ENOENT; > } > > -/** > +/* > * nvkm_firmware_put - release firmware loaded with nvkm_firmware_get > */ > void > -- > 2.25.1 >-- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
Seemingly Similar Threads
- [PATCH 00/30] [Set 6] Rid W=1 warnings from GPU
- [RESEND 00/42] Rid W=1 warnings from GPU (non-Radeon)
- [PATCH 00/53] drm: Convert to platform remove callback returning void
- [PATCH 7/7] drm: Split out drm_probe_helper.h
- [PATCH 7/7] drm: Split out drm_probe_helper.h