[PATCH 00/40] [Set 11] Rid W=1 warnings from GPU

Lee Jones lee.jones at linaro.org
Fri Jan 8 20:14:17 UTC 2021


This set is part of a larger effort attempting to clean-up W=1
kernel builds, which are currently overwhelmingly riddled with
niggly little warnings.

300 out of 5000 left!

Lee Jones (40):
  drm/nouveau/nvkm/subdev/bios/init: Demote obvious abuse of kernel-doc
  drm/nouveau/dispnv50/disp: Remove unused variable 'ret'
  drm/msm/dp/dp_display: Remove unused variable 'hpd'
  drm/amd/amdgpu/amdgpu_ih: Update 'amdgpu_ih_decode_iv_helper()'s
    function header
  drm/amd/amdgpu/vega20_ih: Add missing descriptions for 'ih' and fix
    spelling error
  drm/amd/pm/powerplay/hwmgr/process_pptables_v1_0: Provide description
    of 'call_back_func'
  drm/amd/pm/powerplay/hwmgr/ppatomctrl: Fix documentation for
    'mpll_param'
  drm/amd/pm/powerplay/hwmgr/vega12_hwmgr: Fix legacy function header
    formatting
  drm/amd/pm/powerplay/hwmgr/vega20_hwmgr: Fix legacy function header
    formatting
  drm/amd/pm/powerplay/hwmgr/smu7_hwmgr: Fix formatting and spelling
    issues
  drm/amd/pm/powerplay/hwmgr/hwmgr: Move prototype into shared header
  drm/amd/pm/powerplay/hwmgr/vega10_hwmgr: Fix a bunch of kernel-doc
    formatting issues
  drm/amd/display/dc/basics/conversion: Demote obvious kernel-doc abuse
  drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs: Demote non-kernel-doc
    comment blocks
  drm/amd/display/dc/bios/command_table_helper: Fix kernel-doc
    formatting
  drm/amd/display/dc/bios/command_table_helper2: Fix legacy formatting
    problems
  drm/amd/display/dc/bios/bios_parser: Make local functions static
  drm/amd/display/dc/bios/bios_parser: Fix a whole bunch of legacy doc
    formatting
  drm/amd/display/dc/bios/command_table: Remove unused variable
  include: drm: drm_atomic: Make use of 'new_plane_state'
  drm/amd/display/dc/calcs/dce_calcs: Remove unused variables
    'v_filter_init_mode' and 'sclk_lvl'
  drm/amd/display/dc/bios/bios_parser2: Fix some formatting issues and
    missing parameter docs
  drm/amd/display/dc/dce/dce_audio: Make function invoked by reference
    static
  drm/amd/display/dc/dce/dce_stream_encoder: Remove unused variable
    'regval'
  drm/amd/display/dc/dce/dce_link_encoder: Make functions invoked by
    reference static
  drm/amd/display/dc/dce/dce_clock_source: Fix formatting/spelling of
    worthy function headers
  drm/amd/pm/powerplay/hwmgr/vega10_hwmgr: Fix worthy function headers,
    demote barely documented one
  drm/amd/display/dc/dce/dce_transform: Remove 3 unused/legacy variables
  drm/amd/display/dc/dce/dce_dmcu: Staticify local function call
    'dce_dmcu_load_iram'
  drm/amd/display/dc/dce/dce_dmcu: Move 'abm_gain_stepsize' to only
    source file it's used in
  drm/amd/display/dc/dce/dce_opp: Make local functions and ones invoked
    by reference static
  drm/amd/display/dc/dce/dce_aux: Remove unused function
    'get_engine_type'
  drm/nouveau/nvkm/subdev/volt/gk20a: Demote non-conformant kernel-doc
    headers
  drm/amd/display/dc/bios/bios_parser: Fix misspelling of function
    parameter
  drm/amd/display/dc/bios/command_table: Remove unused variable and
    associated comment
  drm/amd/display/dc/dce/dce_i2c_hw: Make functions called by reference
    static
  drm/amd/display/dc/dce/dce_i2c_sw: Make a bunch of local functions
    static
  drm/amd/display/dc/dce/dce_panel_cntl: Remove unused variables
    'bl_pwm_cntl' and 'pwm_period_cntl'
  drm/amd/display/dc/dce/dmub_psr: Demote non-conformant kernel-doc
    headers
  drm/amd/display/dc/gpio/hw_factory: Delete unused function
    'dal_hw_factory_destroy'

 drivers/gpu/drm/amd/amdgpu/amdgpu_ih.c        |   2 +
 drivers/gpu/drm/amd/amdgpu/vega20_ih.c        |   5 +-
 .../amd/display/amdgpu_dm/amdgpu_dm_debugfs.c |   4 +-
 .../drm/amd/display/dc/basics/conversion.c    |   9 +-
 .../gpu/drm/amd/display/dc/bios/bios_parser.c | 119 +++++-----
 .../drm/amd/display/dc/bios/bios_parser2.c    |  29 ++-
 .../drm/amd/display/dc/bios/command_table.c   |  16 +-
 .../display/dc/bios/command_table_helper.c    |  20 +-
 .../display/dc/bios/command_table_helper2.c   |  20 +-
 .../gpu/drm/amd/display/dc/calcs/dce_calcs.c  |   8 +-
 .../gpu/drm/amd/display/dc/dce/dce_audio.c    |   2 +-
 drivers/gpu/drm/amd/display/dc/dce/dce_aux.c  |   6 -
 .../drm/amd/display/dc/dce/dce_clock_source.c |  57 +++--
 drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.c |   6 +-
 drivers/gpu/drm/amd/display/dc/dce/dce_dmcu.h |   2 -
 .../gpu/drm/amd/display/dc/dce/dce_i2c_hw.c   |   4 +-
 .../gpu/drm/amd/display/dc/dce/dce_i2c_sw.c   |   9 +-
 .../drm/amd/display/dc/dce/dce_link_encoder.c |   6 +-
 drivers/gpu/drm/amd/display/dc/dce/dce_opp.c  |   8 +-
 .../drm/amd/display/dc/dce/dce_panel_cntl.c   |   8 +-
 .../amd/display/dc/dce/dce_stream_encoder.c   |   3 +-
 .../drm/amd/display/dc/dce/dce_transform.c    |  13 --
 drivers/gpu/drm/amd/display/dc/dce/dmub_psr.c |  22 +-
 .../gpu/drm/amd/display/dc/gpio/hw_factory.c  |  14 --
 .../gpu/drm/amd/pm/powerplay/hwmgr/hwmgr.c    |   2 +-
 .../drm/amd/pm/powerplay/hwmgr/ppatomctrl.c   |   4 +-
 .../powerplay/hwmgr/process_pptables_v1_0.c   |   1 +
 .../drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c   |   4 +-
 .../drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.c | 137 ++++++------
 .../drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.h |   1 +
 .../drm/amd/pm/powerplay/hwmgr/vega12_hwmgr.c |  11 +-
 .../drm/amd/pm/powerplay/hwmgr/vega20_hwmgr.c |  11 +-
 drivers/gpu/drm/msm/dp/dp_display.c           |   3 -
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |   3 +-
 .../gpu/drm/nouveau/nvkm/subdev/bios/init.c   | 204 ++++++------------
 .../gpu/drm/nouveau/nvkm/subdev/volt/gk20a.c  |   4 +-
 include/drm/drm_atomic.h                      |   3 +-
 37 files changed, 325 insertions(+), 455 deletions(-)

Cc: Alex Deucher <alexander.deucher at amd.com>
Cc: amd-gfx at lists.freedesktop.org
Cc: Anthony Koo <Anthony.Koo at amd.com>
Cc: Ben Skeggs <bskeggs at redhat.com>
Cc: Bhawanpreet Lakha <Bhawanpreet.Lakha at amd.com>
Cc: Charlene Liu <Charlene.Liu at amd.com>
Cc: "Christian König" <christian.koenig at amd.com>
Cc: Daniel Vetter <daniel at ffwll.ch>
Cc: David Airlie <airlied at linux.ie>
Cc: dri-devel at lists.freedesktop.org
Cc: Eric Bernstein <eric.bernstein at amd.com>
Cc: Eryk Brol <eryk.brol at amd.com>
Cc: Evan Quan <evan.quan at amd.com>
Cc: Feifei Xu <Feifei.Xu at amd.com>
Cc: Felix Kuehling <Felix.Kuehling at amd.com>
Cc: freedreno at lists.freedesktop.org
Cc: George Shen <george.shen at amd.com>
Cc: Harry Wentland <harry.wentland at amd.com>
Cc: Hawking Zhang <Hawking.Zhang at amd.com>
Cc: Igor Kravchenko <Igor.Kravchenko at amd.com>
Cc: Krunoslav Kovac <Krunoslav.Kovac at amd.com>
Cc: Kuogee Hsieh <khsieh at codeaurora.org>
Cc: Lee Jones <lee.jones at linaro.org>
Cc: Leo Li <sunpeng.li at amd.com>
Cc: Lewis Huang <Lewis.Huang at amd.com>
Cc: linux-arm-msm at vger.kernel.org
Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
Cc: Mauro Rossi <issor.oruam at gmail.com>
Cc: Maxime Ripard <mripard at kernel.org>
Cc: Mikita Lipski <mikita.lipski at amd.com>
Cc: nouveau at lists.freedesktop.org
Cc: Qinglang Miao <miaoqinglang at huawei.com>
Cc: Rob Clark <robdclark at gmail.com>
Cc: Sean Paul <sean at poorly.run>
Cc: Thomas Zimmermann <tzimmermann at suse.de>
Cc: Wyatt Wood <wyatt.wood at amd.com>
-- 
2.25.1



More information about the amd-gfx mailing list