Quoting Michal Wajdeczko (2017-10-03 17:35:59) > We're trying to resolve inter-header dependencies. > > Signed-off-by: Michal Wajdeczko <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Cc: Chris Wilson <[email protected]> > --- > 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 <[email protected]> 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
