Re: [Intel-gfx] [PATCH] drm/i915/cx0: prefer forward declarations over includes
On Thu, 21 Sep 2023, Rodrigo Vivi wrote: > On Thu, Sep 21, 2023 at 06:53:25PM +0300, Jani Nikula wrote: >> Avoid including the world from headers when forward declarations >> suffice. >> >> Cc: Radhakrishna Sripada >> Signed-off-by: Jani Nikula > > Trusting more your compiler then my eyes, but > > Reviewed-by: Rodrigo Vivi Thanks, pushed to drm-intel-next. BR, Jani. > >> --- >> drivers/gpu/drm/i915/display/intel_cx0_phy.h | 14 -- >> 1 file changed, 8 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.h >> b/drivers/gpu/drm/i915/display/intel_cx0_phy.h >> index 4c4db5cdcbd0..912e0eeb0be3 100644 >> --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.h >> +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.h >> @@ -10,14 +10,15 @@ >> #include >> #include >> >> -#include "i915_drv.h" >> -#include "intel_display_types.h" >> - >> -struct drm_i915_private; >> -struct intel_encoder; >> -struct intel_crtc_state; >> enum icl_port_dpll_id; >> enum phy; >> +struct drm_i915_private; >> +struct intel_atomic_state; >> +struct intel_c10pll_state; >> +struct intel_c20pll_state; >> +struct intel_crtc_state; >> +struct intel_encoder; >> +struct intel_hdmi; >> >> bool intel_is_c10phy(struct drm_i915_private *dev_priv, enum phy phy); >> void intel_mtl_pll_enable(struct intel_encoder *encoder, >> @@ -44,4 +45,5 @@ void intel_cx0_phy_set_signal_levels(struct intel_encoder >> *encoder, >> const struct intel_crtc_state *crtc_state); >> int intel_cx0_phy_check_hdmi_link_rate(struct intel_hdmi *hdmi, int clock); >> int intel_mtl_tbt_calc_port_clock(struct intel_encoder *encoder); >> + >> #endif /* __INTEL_CX0_PHY_H__ */ >> -- >> 2.39.2 >> -- Jani Nikula, Intel
Re: [Intel-gfx] [PATCH] drm/i915/cx0: prefer forward declarations over includes
> -Original Message- > From: Nikula, Jani > Sent: Thursday, September 21, 2023 9:23 PM > To: intel-gfx@lists.freedesktop.org > Cc: Nikula, Jani ; Sripada, Radhakrishna > > Subject: [PATCH] drm/i915/cx0: prefer forward declarations over includes > > Avoid including the world from headers when forward declarations > suffice. > > Cc: Radhakrishna Sripada LGTM, Reviewed-by: Radhakrishna Sripada > Signed-off-by: Jani Nikula > --- > drivers/gpu/drm/i915/display/intel_cx0_phy.h | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.h > b/drivers/gpu/drm/i915/display/intel_cx0_phy.h > index 4c4db5cdcbd0..912e0eeb0be3 100644 > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.h > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.h > @@ -10,14 +10,15 @@ > #include > #include > > -#include "i915_drv.h" > -#include "intel_display_types.h" > - > -struct drm_i915_private; > -struct intel_encoder; > -struct intel_crtc_state; > enum icl_port_dpll_id; > enum phy; > +struct drm_i915_private; > +struct intel_atomic_state; > +struct intel_c10pll_state; > +struct intel_c20pll_state; > +struct intel_crtc_state; > +struct intel_encoder; > +struct intel_hdmi; > > bool intel_is_c10phy(struct drm_i915_private *dev_priv, enum phy phy); > void intel_mtl_pll_enable(struct intel_encoder *encoder, > @@ -44,4 +45,5 @@ void intel_cx0_phy_set_signal_levels(struct intel_encoder > *encoder, >const struct intel_crtc_state *crtc_state); > int intel_cx0_phy_check_hdmi_link_rate(struct intel_hdmi *hdmi, int clock); > int intel_mtl_tbt_calc_port_clock(struct intel_encoder *encoder); > + > #endif /* __INTEL_CX0_PHY_H__ */ > -- > 2.39.2
Re: [Intel-gfx] [PATCH] drm/i915/cx0: prefer forward declarations over includes
On Thu, Sep 21, 2023 at 06:53:25PM +0300, Jani Nikula wrote: > Avoid including the world from headers when forward declarations > suffice. > > Cc: Radhakrishna Sripada > Signed-off-by: Jani Nikula Trusting more your compiler then my eyes, but Reviewed-by: Rodrigo Vivi > --- > drivers/gpu/drm/i915/display/intel_cx0_phy.h | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.h > b/drivers/gpu/drm/i915/display/intel_cx0_phy.h > index 4c4db5cdcbd0..912e0eeb0be3 100644 > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.h > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.h > @@ -10,14 +10,15 @@ > #include > #include > > -#include "i915_drv.h" > -#include "intel_display_types.h" > - > -struct drm_i915_private; > -struct intel_encoder; > -struct intel_crtc_state; > enum icl_port_dpll_id; > enum phy; > +struct drm_i915_private; > +struct intel_atomic_state; > +struct intel_c10pll_state; > +struct intel_c20pll_state; > +struct intel_crtc_state; > +struct intel_encoder; > +struct intel_hdmi; > > bool intel_is_c10phy(struct drm_i915_private *dev_priv, enum phy phy); > void intel_mtl_pll_enable(struct intel_encoder *encoder, > @@ -44,4 +45,5 @@ void intel_cx0_phy_set_signal_levels(struct intel_encoder > *encoder, >const struct intel_crtc_state *crtc_state); > int intel_cx0_phy_check_hdmi_link_rate(struct intel_hdmi *hdmi, int clock); > int intel_mtl_tbt_calc_port_clock(struct intel_encoder *encoder); > + > #endif /* __INTEL_CX0_PHY_H__ */ > -- > 2.39.2 >
[Intel-gfx] [PATCH] drm/i915/cx0: prefer forward declarations over includes
Avoid including the world from headers when forward declarations suffice. Cc: Radhakrishna Sripada Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/display/intel_cx0_phy.h | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.h b/drivers/gpu/drm/i915/display/intel_cx0_phy.h index 4c4db5cdcbd0..912e0eeb0be3 100644 --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.h +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.h @@ -10,14 +10,15 @@ #include #include -#include "i915_drv.h" -#include "intel_display_types.h" - -struct drm_i915_private; -struct intel_encoder; -struct intel_crtc_state; enum icl_port_dpll_id; enum phy; +struct drm_i915_private; +struct intel_atomic_state; +struct intel_c10pll_state; +struct intel_c20pll_state; +struct intel_crtc_state; +struct intel_encoder; +struct intel_hdmi; bool intel_is_c10phy(struct drm_i915_private *dev_priv, enum phy phy); void intel_mtl_pll_enable(struct intel_encoder *encoder, @@ -44,4 +45,5 @@ void intel_cx0_phy_set_signal_levels(struct intel_encoder *encoder, const struct intel_crtc_state *crtc_state); int intel_cx0_phy_check_hdmi_link_rate(struct intel_hdmi *hdmi, int clock); int intel_mtl_tbt_calc_port_clock(struct intel_encoder *encoder); + #endif /* __INTEL_CX0_PHY_H__ */ -- 2.39.2