All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: Lee Jones <lee.jones@linaro.org>
Cc: Faith <faith@valinux.com>,
	Alex Deucher <alexander.deucher@amd.com>,
	Andy Gross <andy.gross@ti.com>, David Airlie <airlied@linux.ie>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
	Rob Clark <rob.clark@linaro.org>,
	linaro-mm-sig@lists.linaro.org, Leo Li <sunpeng.li@amd.com>,
	Huang Rui <ray.huang@amd.com>,
	dri-devel@lists.freedesktop.org,
	Keith Whitwell <keith@tungstengraphics.com>,
	Jeff Hartmann <jhartmann@valinux.com>, Rob Clark <rob@ti.com>,
	Gareth Hughes <gareth@valinux.com>,
	Christian Koenig <christian.koenig@amd.com>,
	by <jhartmann@precisioninsight.com>,
	linux-media@vger.kernel.org
Subject: Re: [PATCH 00/19] [Set 2] Rid W=1 warnings from GPU
Date: Fri, 6 Nov 2020 23:26:30 +0100	[thread overview]
Message-ID: <20201106222630.GD826715@ravnborg.org> (raw)
In-Reply-To: <20201106214949.2042120-1-lee.jones@linaro.org>

Hi Lee and DRM folks.

On Fri, Nov 06, 2020 at 09:49:30PM +0000, Lee Jones wrote:
> 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.
> 
> There are 5000 warnings to work through.  It will take a couple more
> sets.  Although, ("drm/amd/display/dc/basics/fixpt31_32: Move
> variables to where they're used") does take care of 2000 of them!
> 
> Lee Jones (19):
>   drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header
>   drm/r128/ati_pcigart: Source file headers are not good candidates for
>     kernel-doc
Applied

>   drm/selftests/test-drm_dp_mst_helper: Move
>     'sideband_msg_req_encode_decode' onto the heap
>   drm/mga/mga_dma: Demote kernel-doc abusers to standard comment blocks
>   drm/mga/mga_state: Remove unused variable 'buf_priv'
Applied x2

>   drm/radeon/atom: Move prototype into shared location
>   drm/radeon/radeon_kms: Include header containing our own prototypes
>   drm/omapdrm/omap_gem: Fix misnamed and missing parameter descriptions
>   drm/omapdrm/omap_dmm_tiler: Demote abusive use of kernel-doc format
>   drm/radeon/radeon: Move prototype into shared header
>   drm/radeon/radeon_drv: Source file headers are not good candidates for
>     kernel-doc
>   drm/amd/display/dc/basics/fixpt31_32: Move variables to where they're
>     used
>   drm/radeon/radeon_drv: Move prototypes to a shared headerfile
>   drm/amd/amdgpu/amdgpu_device: Provide documentation for 'reg_addr'
>     params
>   drm/radeon: Move prototypes to shared header
>   drm/amd/amdgpu/amdgpu_kms: Remove 'struct drm_amdgpu_info_device
>     dev_info' from the stack
>   drm/radeon/radeon_kms: Fix misnaming of 'radeon_info_ioctl's dev param
>   drm/radeon/atombios_crtc: Remove description of non-existent function
>     param 'encoder'
>   drm/v3d/v3d_drv: Remove unused static variable 'v3d_v3d_pm_ops'

I have applied the three patches that has no obvious maintainer as indicated
above. I assume the respective maintaines to pick radeon, omapdrm, ttm,
amd, v3d and selftest patches.

	Sam
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

WARNING: multiple messages have this Message-ID (diff)
From: Sam Ravnborg <sam@ravnborg.org>
To: Lee Jones <lee.jones@linaro.org>
Cc: Faith <faith@valinux.com>,
	Alex Deucher <alexander.deucher@amd.com>,
	Andy Gross <andy.gross@ti.com>, David Airlie <airlied@linux.ie>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
	Rob Clark <rob.clark@linaro.org>,
	linaro-mm-sig@lists.linaro.org, Leo Li <sunpeng.li@amd.com>,
	Huang Rui <ray.huang@amd.com>,
	dri-devel@lists.freedesktop.org,
	Keith Whitwell <keith@tungstengraphics.com>,
	Jeff Hartmann <jhartmann@valinux.com>, Rob Clark <rob@ti.com>,
	Gareth Hughes <gareth@valinux.com>,
	Christian Koenig <christian.koenig@amd.com>,
	by <jhartmann@precisioninsight.com>,
	linux-media@vger.kernel.org
Subject: Re: [PATCH 00/19] [Set 2] Rid W=1 warnings from GPU
Date: Fri, 6 Nov 2020 23:26:30 +0100	[thread overview]
Message-ID: <20201106222630.GD826715@ravnborg.org> (raw)
In-Reply-To: <20201106214949.2042120-1-lee.jones@linaro.org>

Hi Lee and DRM folks.

On Fri, Nov 06, 2020 at 09:49:30PM +0000, Lee Jones wrote:
> 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.
> 
> There are 5000 warnings to work through.  It will take a couple more
> sets.  Although, ("drm/amd/display/dc/basics/fixpt31_32: Move
> variables to where they're used") does take care of 2000 of them!
> 
> Lee Jones (19):
>   drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header
>   drm/r128/ati_pcigart: Source file headers are not good candidates for
>     kernel-doc
Applied

>   drm/selftests/test-drm_dp_mst_helper: Move
>     'sideband_msg_req_encode_decode' onto the heap
>   drm/mga/mga_dma: Demote kernel-doc abusers to standard comment blocks
>   drm/mga/mga_state: Remove unused variable 'buf_priv'
Applied x2

>   drm/radeon/atom: Move prototype into shared location
>   drm/radeon/radeon_kms: Include header containing our own prototypes
>   drm/omapdrm/omap_gem: Fix misnamed and missing parameter descriptions
>   drm/omapdrm/omap_dmm_tiler: Demote abusive use of kernel-doc format
>   drm/radeon/radeon: Move prototype into shared header
>   drm/radeon/radeon_drv: Source file headers are not good candidates for
>     kernel-doc
>   drm/amd/display/dc/basics/fixpt31_32: Move variables to where they're
>     used
>   drm/radeon/radeon_drv: Move prototypes to a shared headerfile
>   drm/amd/amdgpu/amdgpu_device: Provide documentation for 'reg_addr'
>     params
>   drm/radeon: Move prototypes to shared header
>   drm/amd/amdgpu/amdgpu_kms: Remove 'struct drm_amdgpu_info_device
>     dev_info' from the stack
>   drm/radeon/radeon_kms: Fix misnaming of 'radeon_info_ioctl's dev param
>   drm/radeon/atombios_crtc: Remove description of non-existent function
>     param 'encoder'
>   drm/v3d/v3d_drv: Remove unused static variable 'v3d_v3d_pm_ops'

I have applied the three patches that has no obvious maintainer as indicated
above. I assume the respective maintaines to pick radeon, omapdrm, ttm,
amd, v3d and selftest patches.

	Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

WARNING: multiple messages have this Message-ID (diff)
From: Sam Ravnborg <sam@ravnborg.org>
To: Lee Jones <lee.jones@linaro.org>
Cc: David Airlie <airlied@linux.ie>,
	dri-devel@lists.freedesktop.org, Huang Rui <ray.huang@amd.com>,
	Rob Clark <rob@ti.com>, Gareth Hughes <gareth@valinux.com>,
	amd-gfx@lists.freedesktop.org, Rob Clark <rob.clark@linaro.org>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Keith Whitwell <keith@tungstengraphics.com>,
	Andy Gross <andy.gross@ti.com>,
	linux-media@vger.kernel.org, Leo Li <sunpeng.li@amd.com>,
	linaro-mm-sig@lists.linaro.org,
	by <jhartmann@precisioninsight.com>,
	Jeff Hartmann <jhartmann@valinux.com>,
	linux-kernel@vger.kernel.org, Faith <faith@valinux.com>,
	Alex Deucher <alexander.deucher@amd.com>,
	Christian Koenig <christian.koenig@amd.com>
Subject: Re: [PATCH 00/19] [Set 2] Rid W=1 warnings from GPU
Date: Fri, 6 Nov 2020 23:26:30 +0100	[thread overview]
Message-ID: <20201106222630.GD826715@ravnborg.org> (raw)
In-Reply-To: <20201106214949.2042120-1-lee.jones@linaro.org>

Hi Lee and DRM folks.

On Fri, Nov 06, 2020 at 09:49:30PM +0000, Lee Jones wrote:
> 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.
> 
> There are 5000 warnings to work through.  It will take a couple more
> sets.  Although, ("drm/amd/display/dc/basics/fixpt31_32: Move
> variables to where they're used") does take care of 2000 of them!
> 
> Lee Jones (19):
>   drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header
>   drm/r128/ati_pcigart: Source file headers are not good candidates for
>     kernel-doc
Applied

>   drm/selftests/test-drm_dp_mst_helper: Move
>     'sideband_msg_req_encode_decode' onto the heap
>   drm/mga/mga_dma: Demote kernel-doc abusers to standard comment blocks
>   drm/mga/mga_state: Remove unused variable 'buf_priv'
Applied x2

>   drm/radeon/atom: Move prototype into shared location
>   drm/radeon/radeon_kms: Include header containing our own prototypes
>   drm/omapdrm/omap_gem: Fix misnamed and missing parameter descriptions
>   drm/omapdrm/omap_dmm_tiler: Demote abusive use of kernel-doc format
>   drm/radeon/radeon: Move prototype into shared header
>   drm/radeon/radeon_drv: Source file headers are not good candidates for
>     kernel-doc
>   drm/amd/display/dc/basics/fixpt31_32: Move variables to where they're
>     used
>   drm/radeon/radeon_drv: Move prototypes to a shared headerfile
>   drm/amd/amdgpu/amdgpu_device: Provide documentation for 'reg_addr'
>     params
>   drm/radeon: Move prototypes to shared header
>   drm/amd/amdgpu/amdgpu_kms: Remove 'struct drm_amdgpu_info_device
>     dev_info' from the stack
>   drm/radeon/radeon_kms: Fix misnaming of 'radeon_info_ioctl's dev param
>   drm/radeon/atombios_crtc: Remove description of non-existent function
>     param 'encoder'
>   drm/v3d/v3d_drv: Remove unused static variable 'v3d_v3d_pm_ops'

I have applied the three patches that has no obvious maintainer as indicated
above. I assume the respective maintaines to pick radeon, omapdrm, ttm,
amd, v3d and selftest patches.

	Sam

  parent reply	other threads:[~2020-11-06 22:26 UTC|newest]

Thread overview: 158+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-06 21:49 [PATCH 00/19] [Set 2] Rid W=1 warnings from GPU Lee Jones
2020-11-06 21:49 ` Lee Jones
2020-11-06 21:49 ` Lee Jones
2020-11-06 21:49 ` [PATCH 01/19] drm/ttm/ttm_range_manager: Demote non-conformant kernel-doc header Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-07 12:12   ` Christian König
2020-11-07 12:12     ` Christian König
2020-11-06 21:49 ` [PATCH 02/19] drm/r128/ati_pcigart: Source file headers are not good candidates for kernel-doc Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 22:22   ` Sam Ravnborg
2020-11-06 22:22     ` Sam Ravnborg
2020-11-06 21:49 ` [PATCH 03/19] drm/selftests/test-drm_dp_mst_helper: Move 'sideband_msg_req_encode_decode' onto the heap Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49 ` [PATCH 04/19] drm/mga/mga_dma: Demote kernel-doc abusers to standard comment blocks Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 22:23   ` Sam Ravnborg
2020-11-06 22:23     ` Sam Ravnborg
2020-11-06 21:49 ` [PATCH 05/19] drm/mga/mga_state: Remove unused variable 'buf_priv' Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 22:23   ` Sam Ravnborg
2020-11-06 22:23     ` Sam Ravnborg
2020-11-06 21:49 ` [PATCH 06/19] drm/radeon/atom: Move prototype into shared location Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 21:54   ` Alex Deucher
2020-11-10 21:54     ` Alex Deucher
2020-11-10 21:54     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 07/19] drm/radeon/radeon_kms: Include header containing our own prototypes Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 21:55   ` Alex Deucher
2020-11-10 21:55     ` Alex Deucher
2020-11-10 21:55     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 08/19] drm/omapdrm/omap_gem: Fix misnamed and missing parameter descriptions Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 12:01   ` Tomi Valkeinen
2020-11-10 12:01     ` Tomi Valkeinen
2020-11-06 21:49 ` [PATCH 09/19] drm/omapdrm/omap_dmm_tiler: Demote abusive use of kernel-doc format Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49 ` [PATCH 10/19] drm/radeon/radeon: Move prototype into shared header Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-07 14:26   ` Sam Ravnborg
2020-11-07 14:26     ` Sam Ravnborg
2020-11-07 14:26     ` Sam Ravnborg
2020-11-07 15:14     ` Lee Jones
2020-11-07 15:14       ` Lee Jones
2020-11-07 15:14       ` Lee Jones
2020-11-09 11:06     ` Lee Jones
2020-11-09 11:06       ` Lee Jones
2020-11-09 11:06       ` Lee Jones
2020-11-09 15:25       ` Sam Ravnborg
2020-11-09 15:25         ` Sam Ravnborg
2020-11-09 15:25         ` Sam Ravnborg
2020-11-09 16:10         ` Lee Jones
2020-11-09 16:10           ` Lee Jones
2020-11-09 16:10           ` Lee Jones
2020-11-06 21:49 ` [PATCH 11/19] drm/radeon/radeon_drv: Source file headers are not good candidates for kernel-doc Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 21:56   ` Alex Deucher
2020-11-10 21:56     ` Alex Deucher
2020-11-10 21:56     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 12/19] drm/amd/display/dc/basics/fixpt31_32: Move variables to where they're used Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-09 19:58   ` Alex Deucher
2020-11-09 19:58     ` Alex Deucher
2020-11-09 19:58     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 13/19] drm/radeon/radeon_drv: Move prototypes to a shared headerfile Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 21:58   ` Alex Deucher
2020-11-10 21:58     ` Alex Deucher
2020-11-10 21:58     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 14/19] drm/amd/amdgpu/amdgpu_device: Provide documentation for 'reg_addr' params Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-09 19:54   ` Alex Deucher
2020-11-09 19:54     ` Alex Deucher
2020-11-09 19:54     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 15/19] drm/radeon: Move prototypes to shared header Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 21:59   ` Alex Deucher
2020-11-10 21:59     ` Alex Deucher
2020-11-10 21:59     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 16/19] drm/amd/amdgpu/amdgpu_kms: Remove 'struct drm_amdgpu_info_device dev_info' from the stack Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-09 20:05   ` Alex Deucher
2020-11-09 20:05     ` Alex Deucher
2020-11-09 20:05     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 17/19] drm/radeon/radeon_kms: Fix misnaming of 'radeon_info_ioctl's dev param Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-07 14:29   ` Sam Ravnborg
2020-11-07 14:29     ` Sam Ravnborg
2020-11-07 14:29     ` Sam Ravnborg
2020-11-07 15:13     ` Lee Jones
2020-11-07 15:13       ` Lee Jones
2020-11-07 15:13       ` Lee Jones
2020-11-09 19:50   ` Alex Deucher
2020-11-09 19:50     ` Alex Deucher
2020-11-09 19:50     ` Alex Deucher
2020-11-09 19:55     ` Sam Ravnborg
2020-11-09 19:55       ` Sam Ravnborg
2020-11-09 19:55       ` Sam Ravnborg
2020-11-09 20:06       ` Alex Deucher
2020-11-09 20:06         ` Alex Deucher
2020-11-09 20:06         ` Alex Deucher
2020-11-09 20:10       ` Lee Jones
2020-11-09 20:10         ` Lee Jones
2020-11-09 20:10         ` Lee Jones
2020-11-09 20:52         ` Sam Ravnborg
2020-11-09 20:52           ` Sam Ravnborg
2020-11-09 20:52           ` Sam Ravnborg
2020-11-09 21:15           ` Lee Jones
2020-11-09 21:15             ` Lee Jones
2020-11-09 21:15             ` Lee Jones
2020-11-09 19:59     ` Lee Jones
2020-11-09 19:59       ` Lee Jones
2020-11-09 19:59       ` Lee Jones
2020-11-06 21:49 ` [PATCH 18/19] drm/radeon/atombios_crtc: Remove description of non-existent function param 'encoder' Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-10 22:01   ` Alex Deucher
2020-11-10 22:01     ` Alex Deucher
2020-11-10 22:01     ` Alex Deucher
2020-11-06 21:49 ` [PATCH 19/19] drm/v3d/v3d_drv: Remove unused static variable 'v3d_v3d_pm_ops' Lee Jones
2020-11-06 21:49   ` Lee Jones
2020-11-06 22:26 ` Sam Ravnborg [this message]
2020-11-06 22:26   ` [PATCH 00/19] [Set 2] Rid W=1 warnings from GPU Sam Ravnborg
2020-11-06 22:26   ` Sam Ravnborg
2020-11-07 12:18 ` Christian König
2020-11-07 12:18   ` Christian König
2020-11-07 12:18   ` Christian König
2020-11-07 15:07   ` Lee Jones
2020-11-07 15:07     ` Lee Jones
2020-11-07 15:07     ` Lee Jones
2020-11-07 17:34   ` Sam Ravnborg
2020-11-07 17:34     ` Sam Ravnborg
2020-11-07 17:34     ` Sam Ravnborg
2020-11-07 18:41     ` Lee Jones
2020-11-07 18:41       ` Lee Jones
2020-11-07 18:41       ` Lee Jones
2020-11-07 19:29       ` Sam Ravnborg
2020-11-07 19:29         ` Sam Ravnborg
2020-11-07 19:29         ` Sam Ravnborg
2020-11-13  7:19 ` Lee Jones
2020-11-13  7:19   ` Lee Jones
2020-11-13  7:19   ` Lee Jones
2020-11-13  7:28   ` Alex Deucher
2020-11-13  7:28     ` Alex Deucher
2020-11-13  7:28     ` Alex Deucher
2020-11-13  7:36     ` Lee Jones
2020-11-13  7:36       ` Lee Jones
2020-11-13  7:36       ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201106222630.GD826715@ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=airlied@linux.ie \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=andy.gross@ti.com \
    --cc=christian.koenig@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=faith@valinux.com \
    --cc=gareth@valinux.com \
    --cc=jhartmann@precisioninsight.com \
    --cc=jhartmann@valinux.com \
    --cc=keith@tungstengraphics.com \
    --cc=lee.jones@linaro.org \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=ray.huang@amd.com \
    --cc=rob.clark@linaro.org \
    --cc=rob@ti.com \
    --cc=sunpeng.li@amd.com \
    --cc=tomi.valkeinen@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.