Re: [PATCH 0/6] drm/i915: drmP.h include removal w/ drm prep work

From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc: Sam Ravnborg <sam@ravnborg.org>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH 0/6] drm/i915: drmP.h include removal w/ drm prep work
Date: Thu, 27 Dec 2018 14:59:44 +0200	[thread overview]
Message-ID: <877efvunun.fsf@intel.com> (raw)
In-Reply-To: <cover.1545915059.git.jani.nikula@intel.com>

On Thu, 27 Dec 2018, Jani Nikula <jani.nikula@intel.com> wrote:
> First make some drm headers self-contained, removing the implicit
> dependency on a previous drmP.h include. Then remove all drmP.h includes
> from drm/i915.
>
> Inspired by Sam's series [1]. Theres a one line trivial conflict between
> that one and this series in drm_file.h (patch 3), but I'm keeping this
> series self-contained. Should be easy enough to resolve.

[1] https://patchwork.freedesktop.org/series/54464/

>
> I'm fine with merging the first 5 through either drm-misc or drm-intel,
> but I'd rather merge the last one through drm-intel.
>
> BR,
> Jani.
>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
>
> Jani Nikula (6):
>   drm: include drm_device.h from drm_legacy.h
>   drm: include kernel.h and agp_backend.h from intel-gtt.h
>   drm: include idr.h from drm_file.h
>   drm: include types.h from drm_hdcp.h
>   drm: include drm_file.h from drm_syncobj.h
>   drm/i915: drop all drmP.h includes
>
>  drivers/gpu/drm/i915/dvo.h                 | 1 -
>  drivers/gpu/drm/i915/i915_drv.c            | 1 -
>  drivers/gpu/drm/i915/i915_drv.h            | 2 +-
>  drivers/gpu/drm/i915/i915_gem.c            | 1 -
>  drivers/gpu/drm/i915/i915_gem_context.c    | 1 -
>  drivers/gpu/drm/i915/i915_gem_dmabuf.c     | 1 -
>  drivers/gpu/drm/i915/i915_gem_evict.c      | 1 -
>  drivers/gpu/drm/i915/i915_gem_execbuffer.c | 1 -
>  drivers/gpu/drm/i915/i915_gem_fence_reg.c  | 1 -
>  drivers/gpu/drm/i915/i915_gem_gtt.c        | 1 -
>  drivers/gpu/drm/i915/i915_gem_internal.c   | 1 -
>  drivers/gpu/drm/i915/i915_gem_object.h     | 3 ++-
>  drivers/gpu/drm/i915/i915_gem_shrinker.c   | 1 -
>  drivers/gpu/drm/i915/i915_gem_stolen.c     | 1 -
>  drivers/gpu/drm/i915/i915_gem_tiling.c     | 1 -
>  drivers/gpu/drm/i915/i915_gem_userptr.c    | 1 -
>  drivers/gpu/drm/i915/i915_ioc32.c          | 1 -
>  drivers/gpu/drm/i915/i915_irq.c            | 1 -
>  drivers/gpu/drm/i915/i915_suspend.c        | 1 -
>  drivers/gpu/drm/i915/i915_trace.h          | 1 -
>  drivers/gpu/drm/i915/intel_acpi.c          | 1 -
>  drivers/gpu/drm/i915/intel_atomic.c        | 1 -
>  drivers/gpu/drm/i915/intel_atomic_plane.c  | 1 -
>  drivers/gpu/drm/i915/intel_audio.c         | 1 -
>  drivers/gpu/drm/i915/intel_bios.c          | 1 -
>  drivers/gpu/drm/i915/intel_connector.c     | 1 -
>  drivers/gpu/drm/i915/intel_crt.c           | 1 -
>  drivers/gpu/drm/i915/intel_display.c       | 1 -
>  drivers/gpu/drm/i915/intel_dp.c            | 1 -
>  drivers/gpu/drm/i915/intel_dp_mst.c        | 1 -
>  drivers/gpu/drm/i915/intel_dsi.h           | 1 -
>  drivers/gpu/drm/i915/intel_dsi_vbt.c       | 1 -
>  drivers/gpu/drm/i915/intel_dvo.c           | 1 -
>  drivers/gpu/drm/i915/intel_fbdev.c         | 1 -
>  drivers/gpu/drm/i915/intel_frontbuffer.c   | 1 -
>  drivers/gpu/drm/i915/intel_hdcp.c          | 1 -
>  drivers/gpu/drm/i915/intel_hdmi.c          | 1 -
>  drivers/gpu/drm/i915/intel_hotplug.c       | 1 -
>  drivers/gpu/drm/i915/intel_i2c.c           | 1 -
>  drivers/gpu/drm/i915/intel_lrc.c           | 1 -
>  drivers/gpu/drm/i915/intel_lvds.c          | 1 -
>  drivers/gpu/drm/i915/intel_mocs.h          | 1 -
>  drivers/gpu/drm/i915/intel_opregion.c      | 1 -
>  drivers/gpu/drm/i915/intel_overlay.c       | 1 -
>  drivers/gpu/drm/i915/intel_psr.c           | 1 -
>  drivers/gpu/drm/i915/intel_ringbuffer.c    | 1 -
>  drivers/gpu/drm/i915/intel_sdvo.c          | 1 -
>  drivers/gpu/drm/i915/intel_sprite.c        | 1 -
>  drivers/gpu/drm/i915/intel_tv.c            | 1 -
>  drivers/gpu/drm/i915/intel_vdsc.c          | 1 -
>  drivers/gpu/drm/i915/vlv_dsi.c             | 1 -
>  include/drm/drm_file.h                     | 1 +
>  include/drm/drm_hdcp.h                     | 2 ++
>  include/drm/drm_legacy.h                   | 1 +
>  include/drm/drm_syncobj.h                  | 4 +++-
>  include/drm/intel-gtt.h                    | 3 +++
>  56 files changed, 13 insertions(+), 52 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx