Quoting Michal Wajdeczko (2017-10-03 17:35:59) > We're trying to resolve inter-header dependencies. > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_uncore.h | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_uncore.h b/drivers/gpu/drm/i915/intel_uncore.h > index 03786f9..66eae2c 100644 > --- a/drivers/gpu/drm/i915/intel_uncore.h > +++ b/drivers/gpu/drm/i915/intel_uncore.h > @@ -25,6 +25,12 @@ > #ifndef __INTEL_UNCORE_H__ > #define __INTEL_UNCORE_H__ > > +#include <linux/spinlock.h> > +#include <linux/notifier.h> > +#include <linux/hrtimer.h> > + > +#include "i915_reg.h" Hmm, what do we pull in directly? Something like i915_mmio_offset()? In my head I have i915_reg.h == unwieldy list of register defines and nothing more. Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Looks sensible but if it's only one small thing for i915_reg.h can you add a comment and we can try and split it up. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
