On Tue, Nov 16, 2021 at 07:14:30PM +0200, Jani Nikula wrote: > Use forward declarations instead. > > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> Patchset looks ok, on the series: Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_ddi.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_ddi.h b/drivers/gpu/drm/i915/display/intel_ddi.h > index 6f4551c9d5b7..c2fea6562917 100644 > --- a/drivers/gpu/drm/i915/display/intel_ddi.h > +++ b/drivers/gpu/drm/i915/display/intel_ddi.h > @@ -6,11 +6,11 @@ > #ifndef __INTEL_DDI_H__ > #define __INTEL_DDI_H__ > > -#include "intel_display.h" > #include "i915_reg.h" > > struct drm_connector_state; > struct drm_i915_private; > +struct intel_atomic_state; > struct intel_connector; > struct intel_crtc; > struct intel_crtc_state; > @@ -18,6 +18,8 @@ struct intel_dp; > struct intel_dpll_hw_state; > struct intel_encoder; > struct intel_shared_dpll; > +enum pipe; > +enum port; > enum transcoder; > > i915_reg_t dp_tp_ctl_reg(struct intel_encoder *encoder, > -- > 2.30.2 >