[PATCH v2 0/7] drm: make headers self-contained and drop drmP.h

* [PATCH v2 0/7] drm: make headers self-contained and drop drmP.h
@ 2019-05-26 17:35 Sam Ravnborg
  2019-05-26 17:35 ` [PATCH v2 1/7] drm: make drm/drm_auth.h self contained Sam Ravnborg
                   ` (11 more replies)
  0 siblings, 12 replies; 17+ messages in thread
From: Sam Ravnborg @ 2019-05-26 17:35 UTC (permalink / raw)
  To: dri-devel; +Cc: Maxime Ripard, intel-gfx, David Airlie, Sean Paul

While removing use of drmP.h from files in drm/* I
noticed that I had to add the same include files due to
dependencies in the header files.

It is better to let the header files be self-contained and
let the users pull in only the additional headers files required.
So I went ahead and made the relevant header files self-contained.
(I did not check if this made any includes redundant in some files,
I do not have tooling in place to do so).

Daniel suggested to add support for testing that they stay
self contained.
Jani Nikula has sent a patch to kbuild to make this part of the
kbuild machinery. I have used it locally and as soon as it
lands in kbuild I will start using it for drm.
We could have duplicated the infrastructure now but that seemed
too much code chrunch.

This patchset include the actual removal of drmP.h as one big patch.
This is build tested on alpha (always interesting), arm, arm64, x86 etc.

For all files touched the following was done:
- include files divided up in blocks in following order:
    	linux/*
	video/*
	drm/*
	""
- within each block the include files are sorted alphabetically

v2:
- use same ordering af blocks
- move includes down below license text
- added patch with actual drmP.h removal
- reworded some subjects to make them more descriptive
- fixed a few spelling erros in changelogs (but a few may remain)

        Sam

Sam Ravnborg (7):
      drm: make drm/drm_auth.h self contained
      drm: make drm/drm_legacy.h self-contained
      drm: make drm_crtc_internal.h self-contained
      drm: make drm_internal.h self-contained
      drm: make drm_legacy.h self-contained
      drm: make drm_trace.h self-contained
      drm: drop use of drmP.h in drm/*

 drivers/gpu/drm/ati_pcigart.c                |  5 ++++-
 drivers/gpu/drm/drm_agpsupport.c             | 11 +++++++++--
 drivers/gpu/drm/drm_atomic.c                 |  9 +++++++--
 drivers/gpu/drm/drm_atomic_helper.c          | 11 +++++++----
 drivers/gpu/drm/drm_atomic_state_helper.c    |  7 ++++---
 drivers/gpu/drm/drm_auth.c                   | 10 ++++++++--
 drivers/gpu/drm/drm_blend.c                  |  9 ++++++---
 drivers/gpu/drm/drm_bufs.c                   | 21 ++++++++++++++++-----
 drivers/gpu/drm/drm_client.c                 |  2 +-
 drivers/gpu/drm/drm_color_mgmt.c             |  8 ++++++--
 drivers/gpu/drm/drm_context.c                |  8 +++++++-
 drivers/gpu/drm/drm_crtc_helper.c            | 14 ++++++++------
 drivers/gpu/drm/drm_crtc_internal.h          | 24 ++++++++++++++++++++----
 drivers/gpu/drm/drm_debugfs.c                | 13 ++++++++-----
 drivers/gpu/drm/drm_debugfs_crc.c            |  9 ++++++++-
 drivers/gpu/drm/drm_dma.c                    |  6 +++++-
 drivers/gpu/drm/drm_drv.c                    |  9 ++++++---
 drivers/gpu/drm/drm_dumb_buffers.c           |  4 +++-
 drivers/gpu/drm/drm_encoder.c                |  4 +++-
 drivers/gpu/drm/drm_fb_helper.c              | 19 ++++++++++++-------
 drivers/gpu/drm/drm_file.c                   | 11 +++++++----
 drivers/gpu/drm/drm_flip_work.c              |  6 ++++--
 drivers/gpu/drm/drm_fourcc.c                 |  2 +-
 drivers/gpu/drm/drm_framebuffer.c            | 13 +++++++++----
 drivers/gpu/drm/drm_gem.c                    |  8 ++++++--
 drivers/gpu/drm/drm_gem_cma_helper.c         | 11 ++++++-----
 drivers/gpu/drm/drm_gem_framebuffer_helper.c |  1 -
 drivers/gpu/drm/drm_hashtab.c                | 10 +++++++---
 drivers/gpu/drm/drm_internal.h               | 10 +++++++++-
 drivers/gpu/drm/drm_ioc32.c                  |  9 ++++++---
 drivers/gpu/drm/drm_ioctl.c                  | 22 ++++++++++++++--------
 drivers/gpu/drm/drm_irq.c                    | 13 +++++++++----
 drivers/gpu/drm/drm_kms_helper_common.c      |  3 ++-
 drivers/gpu/drm/drm_lease.c                  | 15 ++++++++++-----
 drivers/gpu/drm/drm_legacy.h                 |  4 ++++
 drivers/gpu/drm/drm_legacy_misc.c            |  7 ++++++-
 drivers/gpu/drm/drm_lock.c                   |  8 ++++++--
 drivers/gpu/drm/drm_memory.c                 |  9 +++++++--
 drivers/gpu/drm/drm_mm.c                     |  9 +++++----
 drivers/gpu/drm/drm_mode_config.c            |  6 +++++-
 drivers/gpu/drm/drm_mode_object.c            |  9 +++++++--
 drivers/gpu/drm/drm_modes.c                  |  7 +++++--
 drivers/gpu/drm/drm_modeset_lock.c           |  2 +-
 drivers/gpu/drm/drm_of.c                     |  5 +++--
 drivers/gpu/drm/drm_pci.c                    | 11 ++++++++---
 drivers/gpu/drm/drm_plane_helper.c           |  9 +++++----
 drivers/gpu/drm/drm_prime.c                  |  7 +++++--
 drivers/gpu/drm/drm_print.c                  |  7 ++++++-
 drivers/gpu/drm/drm_probe_helper.c           |  7 ++++---
 drivers/gpu/drm/drm_property.c               |  7 ++++++-
 drivers/gpu/drm/drm_rect.c                   |  4 +++-
 drivers/gpu/drm/drm_scatter.c                |  9 +++++++--
 drivers/gpu/drm/drm_scdc_helper.c            |  2 +-
 drivers/gpu/drm/drm_simple_kms_helper.c      |  5 +++--
 drivers/gpu/drm/drm_syncobj.c                | 13 +++++++++----
 drivers/gpu/drm/drm_sysfs.c                  | 15 +++++++++++----
 drivers/gpu/drm/drm_trace.h                  |  2 ++
 drivers/gpu/drm/drm_trace_points.c           |  3 ++-
 drivers/gpu/drm/drm_vblank.c                 | 12 +++++++++---
 drivers/gpu/drm/drm_vm.c                     | 14 +++++++++++++-
 drivers/gpu/drm/drm_vma_manager.c            |  6 +++---
 drivers/gpu/drm/drm_vram_mm_helper.c         |  4 +++-
 drivers/gpu/drm/drm_writeback.c              |  6 ++++--
 include/drm/drm_auth.h                       | 11 +++++++++--
 include/drm/drm_legacy.h                     | 12 ++++++------
 65 files changed, 406 insertions(+), 163 deletions(-)

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

^ permalink raw reply	[flat|nested] 17+ messages in thread