From 6cfcd715898198cbd8edd955ed24cf57f6e8d9ee Mon Sep 17 00:00:00 2001 From: Daniel Vetter Date: Sat, 22 Mar 2014 20:07:35 +0100 Subject: lib: remove uncessary #includes from headers Only include what the header itself needs. The big fish here is intel-gpu-tools.h. More will follow. One ugly thing removed here is the duplicated GEN6_TD_CTL #define, one of which was broken. Signed-off-by: Daniel Vetter --- lib/drmtest.c | 1 + lib/gen6_render.h | 8 -------- lib/gen7_media.h | 2 ++ lib/gen8_media.h | 2 ++ lib/igt_core.h | 2 ++ lib/igt_debugfs.h | 3 +-- lib/igt_kms.h | 4 ++-- lib/intel_batchbuffer.h | 1 + lib/intel_chipset.h | 2 +- lib/intel_gpu_tools.h | 5 ----- lib/intel_iosf.c | 1 + lib/intel_mmio.c | 1 + lib/intel_os.c | 1 + lib/intel_reg_map.c | 1 + lib/media_fill_gen7.c | 1 + lib/media_fill_gen8.c | 1 + lib/rendercopy_gen6.c | 1 + lib/rendercopy_gen7.c | 1 + lib/rendercopy_gen8.c | 1 + 19 files changed, 21 insertions(+), 18 deletions(-) (limited to 'lib') diff --git a/lib/drmtest.c b/lib/drmtest.c index c2751e75..112f0497 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -55,6 +55,7 @@ #include "igt_debugfs.h" #include "../version.h" #include "config.h" +#include "intel_reg.h" /* This file contains a bunch of wrapper functions to directly use gem ioctls. * Mostly useful to write kernel tests. */ diff --git a/lib/gen6_render.h b/lib/gen6_render.h index 90205148..60dc93e6 100644 --- a/lib/gen6_render.h +++ b/lib/gen6_render.h @@ -261,10 +261,6 @@ #define GEN6_TS_STRG_VAL 0x7e04 #define GEN6_TS_RDATA 0x7e08 -/* TD_CTL on gen6 is 0x7000, to not break stuff which depends on this... */ -#ifndef GEN6_TD_CTL -#define GEN6_TD_CTL 0x8000 -#endif #define GEN6_TD_CTL_MUX_SHIFT 8 #define GEN6_TD_CTL_EXTERNAL_HALT_R0_DEBUG_MATCH (1 << 7) #define GEN6_TD_CTL_FORCE_EXTERNAL_HALT (1 << 6) @@ -570,10 +566,6 @@ #define GEN6_TS_STRG_VAL 0x7e04 #define GEN6_TS_RDATA 0x7e08 -/* TD_CTL on gen6 is 0x7000, to not break stuff which depends on this... */ -#ifndef GEN6_TD_CTL -#define GEN6_TD_CTL 0x8000 -#endif #define GEN6_TD_CTL_MUX_SHIFT 8 #define GEN6_TD_CTL_EXTERNAL_HALT_R0_DEBUG_MATCH (1 << 7) #define GEN6_TD_CTL_FORCE_EXTERNAL_HALT (1 << 6) diff --git a/lib/gen7_media.h b/lib/gen7_media.h index d75ee1be..d5f9921a 100644 --- a/lib/gen7_media.h +++ b/lib/gen7_media.h @@ -1,6 +1,8 @@ #ifndef GEN7_MEDIA_H #define GEN7_MEDIA_H +#include + #define GEN7_SURFACEFORMAT_R32G32B32A32_FLOAT 0x000 #define GEN7_SURFACEFORMAT_R32G32B32A32_SINT 0x001 #define GEN7_SURFACEFORMAT_R32G32B32A32_UINT 0x002 diff --git a/lib/gen8_media.h b/lib/gen8_media.h index b890df4e..1214cd15 100644 --- a/lib/gen8_media.h +++ b/lib/gen8_media.h @@ -1,6 +1,8 @@ #ifndef GEN8_MEDIA_H #define GEN8_MEDIA_H +#include + #define GEN8_SURFACEFORMAT_R32G32B32A32_FLOAT 0x000 #define GEN8_SURFACEFORMAT_R32G32B32A32_SINT 0x001 #define GEN8_SURFACEFORMAT_R32G32B32A32_UINT 0x002 diff --git a/lib/igt_core.h b/lib/igt_core.h index a8bb28e1..c21448e7 100644 --- a/lib/igt_core.h +++ b/lib/igt_core.h @@ -30,6 +30,8 @@ #ifndef IGT_CORE_H #define IGT_CORE_H +#include + bool __igt_fixture(void); void __igt_fixture_complete(void); void __igt_fixture_end(void) __attribute__((noreturn)); diff --git a/lib/igt_debugfs.h b/lib/igt_debugfs.h index d3d23896..3312a8bd 100644 --- a/lib/igt_debugfs.h +++ b/lib/igt_debugfs.h @@ -27,9 +27,8 @@ #include #include -#include -#include "igt_display.h" +enum pipe; int igt_debugfs_open(const char *filename, int mode); FILE *igt_debugfs_fopen(const char *filename, diff --git a/lib/igt_kms.h b/lib/igt_kms.h index df658b0b..6590fd58 100644 --- a/lib/igt_kms.h +++ b/lib/igt_kms.h @@ -26,11 +26,11 @@ #define __IGT_KMS_H__ #include - +#include #include #include -#include +#include "igt_display.h" struct kmstest_connector_config { drmModeCrtc *crtc; diff --git a/lib/intel_batchbuffer.h b/lib/intel_batchbuffer.h index 77640642..244bdbda 100644 --- a/lib/intel_batchbuffer.h +++ b/lib/intel_batchbuffer.h @@ -4,6 +4,7 @@ #include #include "intel_bufmgr.h" #include "igt_core.h" +#include "intel_reg.h" #define BATCH_SZ 4096 #define BATCH_RESERVED 16 diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h index 2f307788..f6757d4a 100644 --- a/lib/intel_chipset.h +++ b/lib/intel_chipset.h @@ -28,7 +28,6 @@ #ifndef _INTEL_CHIPSET_H #define _INTEL_CHIPSET_H -#include #include struct pci_device *intel_get_pci_device(void); @@ -36,6 +35,7 @@ uint32_t intel_get_drm_devid(int fd); int intel_gen(uint32_t devid); extern enum pch_type intel_pch; + enum pch_type { PCH_NONE, PCH_IBX, diff --git a/lib/intel_gpu_tools.h b/lib/intel_gpu_tools.h index aab07290..591fd3f4 100644 --- a/lib/intel_gpu_tools.h +++ b/lib/intel_gpu_tools.h @@ -29,13 +29,8 @@ #define INTEL_GPU_TOOLS_H #include -#include #include -#include "intel_chipset.h" -#include "intel_reg.h" -#include "drmtest.h" - /* register access helpers from intel_mmio.c */ extern void *mmio; void intel_get_mmio(struct pci_device *pci_dev); diff --git a/lib/intel_iosf.c b/lib/intel_iosf.c index f57212f6..b7b4ce23 100644 --- a/lib/intel_iosf.c +++ b/lib/intel_iosf.c @@ -4,6 +4,7 @@ #include #include #include "intel_gpu_tools.h" +#include "intel_reg.h" #define TIMEOUT_US 500000 diff --git a/lib/intel_mmio.c b/lib/intel_mmio.c index 22480098..3e1aac74 100644 --- a/lib/intel_mmio.c +++ b/lib/intel_mmio.c @@ -43,6 +43,7 @@ #include "intel_gpu_tools.h" #include "igt_debugfs.h" +#include "intel_chipset.h" #define FAKEKEY 0x2468ace0 diff --git a/lib/intel_os.c b/lib/intel_os.c index ce4dcbca..4b72f517 100644 --- a/lib/intel_os.c +++ b/lib/intel_os.c @@ -49,6 +49,7 @@ #include "intel_gpu_tools.h" #include "i915_drm.h" +#include "drmtest.h" uint64_t intel_get_total_ram_mb(void) diff --git a/lib/intel_reg_map.c b/lib/intel_reg_map.c index 26ddd065..ec63d79e 100644 --- a/lib/intel_reg_map.c +++ b/lib/intel_reg_map.c @@ -30,6 +30,7 @@ #include #include #include "intel_gpu_tools.h" +#include "intel_chipset.h" static struct intel_register_range gen_bwcl_register_map[] = { {0x00000000, 0x00000fff, INTEL_RANGE_RW}, diff --git a/lib/media_fill_gen7.c b/lib/media_fill_gen7.c index 520c4a22..7494d44d 100644 --- a/lib/media_fill_gen7.c +++ b/lib/media_fill_gen7.c @@ -1,5 +1,6 @@ #include "media_fill.h" #include "gen7_media.h" +#include "intel_reg.h" #include diff --git a/lib/media_fill_gen8.c b/lib/media_fill_gen8.c index 7e0a5677..d65d2608 100644 --- a/lib/media_fill_gen8.c +++ b/lib/media_fill_gen8.c @@ -1,5 +1,6 @@ #include "media_fill.h" #include "gen8_media.h" +#include "intel_reg.h" #include diff --git a/lib/rendercopy_gen6.c b/lib/rendercopy_gen6.c index 4f5b5790..a908ae6b 100644 --- a/lib/rendercopy_gen6.c +++ b/lib/rendercopy_gen6.c @@ -18,6 +18,7 @@ #include "intel_gpu_tools.h" #include "rendercopy.h" #include "gen6_render.h" +#include "intel_reg.h" #define ALIGN(x, y) (((x) + (y)-1) & ~((y)-1)) #define VERTEX_SIZE (3*4) diff --git a/lib/rendercopy_gen7.c b/lib/rendercopy_gen7.c index ddd6b97c..05e68293 100644 --- a/lib/rendercopy_gen7.c +++ b/lib/rendercopy_gen7.c @@ -18,6 +18,7 @@ #include "intel_gpu_tools.h" #include "rendercopy.h" #include "gen7_render.h" +#include "intel_reg.h" #define ALIGN(x, y) (((x) + (y)-1) & ~((y)-1)) diff --git a/lib/rendercopy_gen8.c b/lib/rendercopy_gen8.c index f6dd2b5b..ef19ebf4 100644 --- a/lib/rendercopy_gen8.c +++ b/lib/rendercopy_gen8.c @@ -18,6 +18,7 @@ #include "intel_gpu_tools.h" #include "rendercopy.h" #include "gen8_render.h" +#include "intel_reg.h" #include -- cgit v1.2.3