All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [Intel-gfx] [PATCH 19/21] drm/i915: axe lots of unnecessary includes from i915_drv.h
Date: Wed,  5 Jan 2022 21:56:35 +0200	[thread overview]
Message-ID: <c22172f880630f96573ecafd0cf26aa725b28d8b.1641411696.git.jani.nikula@intel.com> (raw)
In-Reply-To: <cover.1641411696.git.jani.nikula@intel.com>

It's fairly difficult to ensure these are actually not needed due to
indirect includes via other files. However, it's easier to add them back
as needed and, most importantly, where needed instead of exhaustively
proving they're unnecessary.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 4421bfd3fe2f..7ceb4b596c1c 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -34,25 +34,12 @@
 
 #include <asm/hypervisor.h>
 
-#include <linux/io-mapping.h>
 #include <linux/i2c.h>
 #include <linux/i2c-algo-bit.h>
-#include <linux/backlight.h>
-#include <linux/hash.h>
 #include <linux/intel-iommu.h>
-#include <linux/kref.h>
-#include <linux/mm_types.h>
-#include <linux/perf_event.h>
 #include <linux/pm_qos.h>
-#include <linux/dma-resv.h>
-#include <linux/stackdepot.h>
 
-#include <drm/drm_gem.h>
-#include <drm/drm_auth.h>
-#include <drm/drm_util.h>
-#include <drm/drm_dsc.h>
 #include <drm/drm_connector.h>
-#include <drm/i915_mei_hdcp_interface.h>
 #include <drm/ttm/ttm_device.h>
 
 #include "i915_params.h"
@@ -90,17 +77,12 @@
 #include "intel_runtime_pm.h"
 #include "intel_step.h"
 #include "intel_uncore.h"
-#include "intel_wakeref.h"
 #include "intel_wopcm.h"
 
 #include "i915_gem.h"
-#include "i915_gem_gtt.h"
 #include "i915_gpu_error.h"
 #include "i915_perf_types.h"
-#include "i915_request.h"
 #include "i915_scheduler.h"
-#include "gt/intel_timeline.h"
-#include "i915_vma.h"
 
 
 /* General customization:
-- 
2.30.2


  parent reply	other threads:[~2022-01-05 19:58 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-05 19:56 [Intel-gfx] [PATCH 00/21] drm/i915: clean up i915_drv.h Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 01/21] drm/i915: split out i915_getparam.h from i915_drv.h Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 02/21] drm/i915: split out i915_cmd_parser.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 03/21] drm/i915: split out i915_gem_evict.h " Jani Nikula
2022-01-06  8:55   ` Tvrtko Ursulin
2022-01-07 13:22     ` Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 04/21] drm/i915: split out gem/i915_gem_userptr.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 05/21] drm/i915: split out gem/i915_gem_tiling.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 06/21] drm/i915: split out i915_gem.h declarations " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 07/21] drm/i915: split out i915_gem_internal.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 08/21] drm/i915: remove leftover i915_gem_pm.h declarations " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 09/21] drm/i915: split out gem/i915_gem_dmabuf.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 10/21] drm/i915: split out gem/i915_gem_create.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 11/21] drm/i915: split out gem/i915_gem_domain.h " Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 12/21] drm/i915: move i915_cache_level_str() static in i915_debugfs.c Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 13/21] drm/i915: move i915_gem_vm_lookup() where it's used Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 14/21] drm/i915: move i915_reset_count()/i915_reset_engine_count() out of i915_drv.h Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 15/21] drm/i915: split out i915_file_private.h from i915_drv.h Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 16/21] drm/i915: don't include drm_cache.h in i915_drv.h Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 17/21] drm/i915: include shmem_fs.h only where needed Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 18/21] drm/i915: include some drm headers " Jani Nikula
2022-01-05 19:56 ` Jani Nikula [this message]
2022-01-05 19:56 ` [Intel-gfx] [PATCH 20/21] drm/i915: fix drm_i915.h include grouping and sorting Jani Nikula
2022-01-05 19:56 ` [Intel-gfx] [PATCH 21/21] drm/i915: group drm_i915.h forward declarations together Jani Nikula
2022-01-05 20:10 ` [Intel-gfx] ✗ Fi.CI.BUILD: failure for drm/i915: clean up i915_drv.h Patchwork
2022-01-05 20:28   ` Jani Nikula

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=c22172f880630f96573ecafd0cf26aa725b28d8b.1641411696.git.jani.nikula@intel.com \
    --to=jani.nikula@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    /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.