[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <feda98c5-a677-7bf5-c1e7-2bf311ba8097@amd.com>
Date: Mon, 23 Nov 2020 12:52:42 +0100
From: Christian König <christian.koenig@....com>
To: Lee Jones <lee.jones@...aro.org>
Cc: linux-kernel@...r.kernel.org,
Alex Deucher <alexander.deucher@....com>,
amd-gfx@...ts.freedesktop.org, Ben Skeggs <bskeggs@...hat.com>,
Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...ux.ie>,
Drew Davenport <ddavenport@...omium.org>,
dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org,
Fritz Koenig <frkoenig@...gle.com>,
Hawking Zhang <Hawking.Zhang@....com>,
Huang Rui <ray.huang@....com>,
Jerome Glisse <glisse@...edesktop.org>,
Jiansong Chen <Jiansong.Chen@....com>,
Jonathan Marek <jonathan@...ek.ca>,
Kalyan Thota <kalyan_t@...eaurora.org>,
Likun Gao <Likun.Gao@....com>, linaro-mm-sig@...ts.linaro.org,
linux-arm-msm@...r.kernel.org, linux-media@...r.kernel.org,
Luben Tuikov <luben.tuikov@....com>,
nouveau@...ts.freedesktop.org,
Qinglang Miao <miaoqinglang@...wei.com>,
Rob Clark <robdclark@...il.com>, Sean Paul <sean@...rly.run>,
Sonny Jiang <sonny.jiang@....com>,
Sumit Semwal <sumit.semwal@...aro.org>,
Tao Zhou <tao.zhou1@....com>
Subject: Re: [PATCH 00/40] [Set 8] Rid W=1 warnings from GPU
Only skimmed over them, but over all looks sane to me.
Series is Acked-by: Christian König <christian.koenig@....com>
Thanks,
Christian.
Am 23.11.20 um 12:18 schrieb Lee Jones:
> 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.
>
> Only 900 (from 5000) to go!
>
> Lee Jones (40):
> drm/radeon/radeon_device: Consume our own header where the prototypes
> are located
> drm/amd/amdgpu/amdgpu_ttm: Add description for 'page_flags'
> drm/amd/amdgpu/amdgpu_ib: Provide docs for 'amdgpu_ib_schedule()'s
> 'job' param
> drm/amd/amdgpu/amdgpu_virt: Correct possible copy/paste or doc-rot
> misnaming issue
> drm/amd/amdgpu/cik_ih: Supply description for 'ih' in
> 'cik_ih_{get,set}_wptr()'
> drm/amd/amdgpu/uvd_v4_2: Fix some kernel-doc misdemeanours
> drm/amd/amdgpu/dce_v8_0: Supply description for 'async'
> drm/amd/amdgpu/cik_sdma: Supply some missing function param
> descriptions
> drm/amd/amdgpu/gfx_v7_0: Clean-up a bunch of kernel-doc related issues
> drm/msm/disp/dpu1/dpu_core_perf: Fix kernel-doc formatting issues
> drm/msm/disp/dpu1/dpu_hw_blk: Add one missing and remove an extra
> param description
> drm/msm/disp/dpu1/dpu_formats: Demote non-conformant kernel-doc header
> drm/msm/disp/dpu1/dpu_hw_catalog: Remove duplicated initialisation of
> 'max_linewidth'
> drm/msm/disp/dpu1/dpu_hw_catalog: Move definitions to the only place
> they are used
> drm/nouveau/nvkm/subdev/bios/init: Demote obvious abuse of kernel-doc
> drm/amd/amdgpu/si_dma: Fix a bunch of function documentation issues
> drm/amd/amdgpu/gfx_v6_0: Supply description for
> 'gfx_v6_0_ring_test_ib()'s 'timeout' param
> drm/msm/disp/dpu1/dpu_encoder: Fix a few parameter/member formatting
> issues
> drm/msm/disp/dpu1/dpu_hw_lm: Fix misnaming of parameter 'ctx'
> drm/msm/disp/dpu1/dpu_hw_sspp: Fix kernel-doc formatting abuse
> drm/amd/amdgpu/uvd_v3_1: Fix-up some documentation issues
> drm/amd/amdgpu/dce_v6_0: Fix formatting and missing parameter
> description issues
> drm/amd/include/vega20_ip_offset: Mark top-level IP_BASE definition as
> __maybe_unused
> drm/amd/include/navi10_ip_offset: Mark top-level IP_BASE as
> __maybe_unused
> drm/amd/include/arct_ip_offset: Mark top-level IP_BASE definition as
> __maybe_unused
> drm/amd/include/navi14_ip_offset: Mark top-level IP_BASE as
> __maybe_unused
> drm/amd/include/navi12_ip_offset: Mark top-level IP_BASE as
> __maybe_unused
> drm/amd/include/sienna_cichlid_ip_offset: Mark top-level IP_BASE as
> __maybe_unused
> drm/amd/include/vangogh_ip_offset: Mark top-level IP_BASE as
> __maybe_unused
> drm/amd/include/dimgrey_cavefish_ip_offset: Mark top-level IP_BASE as
> __maybe_unused
> drm/msm/disp/dpu1/dpu_rm: Fix formatting issues and supply
> 'global_state' description
> drm/msm/disp/dpu1/dpu_vbif: Fix a couple of function param
> descriptions
> drm/amd/amdgpu/cik_sdma: Add one and remove another function param
> description
> drm/amd/amdgpu/uvd_v4_2: Add one and remove another function param
> description
> drm/msm/disp/dpu1/dpu_plane: Fix some spelling and missing function
> param descriptions
> drm/amd/amdgpu/gmc_v7_0: Add some missing kernel-doc descriptions
> drm/amd/amdgpu/gmc_v8_0: Fix more issues attributed to copy/paste
> drm/msm/msm_drv: Make '_msm_ioremap()' static
> drm/amd/amdgpu/gmc_v9_0: Remove unused table
> 'ecc_umc_mcumc_status_addrs'
> drm/amd/amdgpu/gmc_v9_0: Suppy some missing function doc descriptions
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 1 +
> drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 1 +
> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 12 +-
> drivers/gpu/drm/amd/amdgpu/cik_ih.c | 2 +
> drivers/gpu/drm/amd/amdgpu/cik_sdma.c | 18 ++-
> drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 2 +-
> drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 1 +
> drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 1 +
> drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 33 +++--
> drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c | 7 +-
> drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 5 +
> drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 38 +----
> drivers/gpu/drm/amd/amdgpu/si_dma.c | 14 +-
> drivers/gpu/drm/amd/amdgpu/uvd_v3_1.c | 10 +-
> drivers/gpu/drm/amd/amdgpu/uvd_v4_2.c | 10 +-
> drivers/gpu/drm/amd/include/arct_ip_offset.h | 4 +-
> .../amd/include/dimgrey_cavefish_ip_offset.h | 2 +-
> .../gpu/drm/amd/include/navi10_ip_offset.h | 2 +-
> .../gpu/drm/amd/include/navi12_ip_offset.h | 2 +-
> .../gpu/drm/amd/include/navi14_ip_offset.h | 2 +-
> .../amd/include/sienna_cichlid_ip_offset.h | 2 +-
> .../gpu/drm/amd/include/vangogh_ip_offset.h | 2 +-
> .../gpu/drm/amd/include/vega20_ip_offset.h | 2 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 17 +--
> drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 15 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c | 2 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_blk.c | 2 +-
> .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 75 +++++++++-
> .../drm/msm/disp/dpu1/dpu_hw_catalog_format.h | 88 ------------
> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c | 2 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c | 4 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 19 ++-
> drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c | 5 +-
> drivers/gpu/drm/msm/disp/dpu1/dpu_vbif.c | 2 +-
> drivers/gpu/drm/msm/msm_drv.c | 4 +-
> .../gpu/drm/nouveau/nvkm/subdev/bios/init.c | 136 +++++++++---------
> drivers/gpu/drm/radeon/radeon_device.c | 1 +
> 37 files changed, 277 insertions(+), 268 deletions(-)
> delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog_format.h
>
> Cc: Alex Deucher <alexander.deucher@....com>
> Cc: amd-gfx@...ts.freedesktop.org
> Cc: Ben Skeggs <bskeggs@...hat.com>
> Cc: "Christian König" <christian.koenig@....com>
> Cc: Daniel Vetter <daniel@...ll.ch>
> Cc: David Airlie <airlied@...ux.ie>
> Cc: Drew Davenport <ddavenport@...omium.org>
> Cc: dri-devel@...ts.freedesktop.org
> Cc: freedreno@...ts.freedesktop.org
> Cc: Fritz Koenig <frkoenig@...gle.com>
> Cc: Hawking Zhang <Hawking.Zhang@....com>
> Cc: Huang Rui <ray.huang@....com>
> Cc: Jerome Glisse <glisse@...edesktop.org>
> Cc: Jiansong Chen <Jiansong.Chen@....com>
> Cc: Jonathan Marek <jonathan@...ek.ca>
> Cc: Kalyan Thota <kalyan_t@...eaurora.org>
> Cc: Likun Gao <Likun.Gao@....com>
> Cc: linaro-mm-sig@...ts.linaro.org
> Cc: linux-arm-msm@...r.kernel.org
> Cc: linux-media@...r.kernel.org
> Cc: Luben Tuikov <luben.tuikov@....com>
> Cc: nouveau@...ts.freedesktop.org
> Cc: Qinglang Miao <miaoqinglang@...wei.com>
> Cc: Rob Clark <robdclark@...il.com>
> Cc: Sean Paul <sean@...rly.run>
> Cc: Sonny Jiang <sonny.jiang@....com>
> Cc: Sumit Semwal <sumit.semwal@...aro.org>
> Cc: Tao Zhou <tao.zhou1@....com>
Powered by blists - more mailing lists