mbox series

[v1,0/11] drm: header maintenance

Message ID 20190718161507.2047-1-sam@ravnborg.org (mailing list archive)
Headers show
Series drm: header maintenance | expand

Message

Sam Ravnborg July 18, 2019, 4:14 p.m. UTC
First patch from Jani fixes so drm_print.h is self-contained.
Next two patches are trivial removal of uapi dependencies.

ati_pcigart is fixed to drop use of drm_os_linux.h

drm_vblank is likewise fixed to drop use of drm_os_linux.h
This was a non-trivial conversion, *review requested!*

The remaining patches are preparation for and removal of
uapi/drm/drmh from drm_file.h.
There were a few files where we had to push include
of drm/drm.h out to to have a clean build.

CK Hu - please let me apply the mediatek patch to
drm-misc-next, as it is required for the final patch.
Or push it to drm-misc-next yourself.

	Sam

Jani Nikula (1):
      drm/panel: make drm_panel.h self-contained

Sam Ravnborg (10):
      drm: drop uapi dependency from drm_print.h
      drm: drop uapi dependency from drm_vblank.h
      drm/ati_pcigart: drop dependency on drm_os_linux.h
      drm/vblank: drop use of DRM_WAIT_ON()
      drm: direct include of drm.h in drm_gem.c
      drm: direct include of drm.h in drm_gem_shmem_helper.c
      drm: direct include of drm.h in drm_prime.c
      drm: direct include of drm.h in drm_syncobj.c
      drm/mediatek: direct include of drm.h in mtk_drm_gem.c
      drm: drop uapi dependency from drm_file.h

 drivers/gpu/drm/ati_pcigart.c          | 10 ++++++----
 drivers/gpu/drm/drm_gem.c              |  1 +
 drivers/gpu/drm/drm_gem_shmem_helper.c |  1 +
 drivers/gpu/drm/drm_prime.c            |  1 +
 drivers/gpu/drm/drm_syncobj.c          |  1 +
 drivers/gpu/drm/drm_vblank.c           | 29 ++++++++++++++++++++---------
 drivers/gpu/drm/mediatek/mtk_drm_gem.c |  1 +
 include/drm/drm_file.h                 |  4 +---
 include/drm/drm_panel.h                |  1 +
 include/drm/drm_print.h                |  4 +---
 include/drm/drm_vblank.h               |  1 -
 11 files changed, 34 insertions(+), 20 deletions(-)

Comments

Sam Ravnborg July 19, 2019, 9:26 p.m. UTC | #1
On Thu, Jul 18, 2019 at 06:14:56PM +0200, Sam Ravnborg wrote:
> First patch from Jani fixes so drm_print.h is self-contained.
> Next two patches are trivial removal of uapi dependencies.
> 
> ati_pcigart is fixed to drop use of drm_os_linux.h
> 
> drm_vblank is likewise fixed to drop use of drm_os_linux.h
> This was a non-trivial conversion, *review requested!*
> 
> The remaining patches are preparation for and removal of
> uapi/drm/drmh from drm_file.h.
> There were a few files where we had to push include
> of drm/drm.h out to to have a clean build.
> 
> CK Hu - please let me apply the mediatek patch to
> drm-misc-next, as it is required for the final patch.
> Or push it to drm-misc-next yourself.
> 
> 	Sam
> 
> Jani Nikula (1):
>       drm/panel: make drm_panel.h self-contained
> 
> Sam Ravnborg (10):
>       drm: drop uapi dependency from drm_print.h
>       drm: drop uapi dependency from drm_vblank.h
>       drm/ati_pcigart: drop dependency on drm_os_linux.h
>       drm/vblank: drop use of DRM_WAIT_ON()
>       drm: direct include of drm.h in drm_gem.c
>       drm: direct include of drm.h in drm_gem_shmem_helper.c
>       drm: direct include of drm.h in drm_prime.c
>       drm: direct include of drm.h in drm_syncobj.c
>       drm/mediatek: direct include of drm.h in mtk_drm_gem.c
>       drm: drop uapi dependency from drm_file.h

Added relevant acks and pushed following patches to drm-misc-next:
       drm/panel: make drm_panel.h self-contained
       drm: drop uapi dependency from drm_vblank.h
       drm/ati_pcigart: drop dependency on drm_os_linux.h
       drm: direct include of drm.h in drm_gem.c
       drm: direct include of drm.h in drm_gem_shmem_helper.c
       drm: direct include of drm.h in drm_prime.c
       drm: direct include of drm.h in drm_syncobj.c
       drm/mediatek: direct include of drm.h in mtk_drm_gem.c

Following patches was dropped:
       drm: drop uapi dependency from drm_print.h
       - There was not a clear consensus what to do here, and the patch
	 broke one way to use the DRM_(PRINT) macros.
       - I did not have the time/enegy to start a logging debate.
         There is too much that could be done and it is not the right
	 time for me to look into the possibilities.

       drm: drop uapi dependency from drm_file.h
       - There were relevant push back from loosing the information
         that a uapi type was used to represent 'magic'

Following patch is worked on:
       drm/vblank: drop use of DRM_WAIT_ON()
       - Got excellent feedback from Michel Dänzer and Daniel Vetter.
       - An updated version will be posted when I have tested it
	 at my local setup

	Sam