On 2020-02-11 at 22:55:30 +0530, Anshuman Gupta wrote: > intel_plane_fb_max_stride should return the max stride of > primary plane for first available pipe in intel device info > pipe_mask. > Similarly glk_force_audio_cdclk() should also use the first > available CRTC instead of pipe 'A' crtc to force the cdclk > changes. > > changes since RFC: > - Introduced a helper to get first intel_crtc intel_get_first_crtc. [Ville] > v1: > - Used intel_get_first_crtc() instead of PIPE_A crtc in > glk_force_audio_cdclk(). [Ville] Hi Ville, You had provided your RB on earlier revision https://patchwork.freedesktop.org/patch/351873/?series=72468&rev=2 later u have provided the above review comment, as current patch fixes the above review comment. can i use your RB for this patch, if current patch is ok. Thanks, Anshuman Gupta. > > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > Signed-off-by: Anshuman Gupta <anshuman.gupta@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_audio.c | 2 +- > drivers/gpu/drm/i915/display/intel_display.c | 5 +++-- > drivers/gpu/drm/i915/display/intel_display_types.h | 6 ++++++ > 3 files changed, 10 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_audio.c b/drivers/gpu/drm/i915/display/intel_audio.c > index 30fb7c887ff0..19bf206037c2 100644 > --- a/drivers/gpu/drm/i915/display/intel_audio.c > +++ b/drivers/gpu/drm/i915/display/intel_audio.c > @@ -844,7 +844,7 @@ static void glk_force_audio_cdclk(struct drm_i915_private *dev_priv, > struct intel_crtc *crtc; > int ret; > > - crtc = intel_get_crtc_for_pipe(dev_priv, PIPE_A); > + crtc = intel_get_first_crtc(dev_priv); > if (!crtc) > return; > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c > index 5ba0b40fbfde..6fdaeb019fef 100644 > --- a/drivers/gpu/drm/i915/display/intel_display.c > +++ b/drivers/gpu/drm/i915/display/intel_display.c > @@ -2715,9 +2715,10 @@ u32 intel_plane_fb_max_stride(struct drm_i915_private *dev_priv, > > /* > * We assume the primary plane for pipe A has > - * the highest stride limits of them all. > + * the highest stride limits of them all, > + * if in case pipe A is disabled, use the first pipe from pipe_mask. > */ > - crtc = intel_get_crtc_for_pipe(dev_priv, PIPE_A); > + crtc = intel_get_first_crtc(dev_priv); > if (!crtc) > return 0; > > diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h > index d359f1636ba8..359eaa6703a8 100644 > --- a/drivers/gpu/drm/i915/display/intel_display_types.h > +++ b/drivers/gpu/drm/i915/display/intel_display_types.h > @@ -1403,6 +1403,12 @@ vlv_pipe_to_channel(enum pipe pipe) > } > } > > +static inline struct intel_crtc * > +intel_get_first_crtc(struct drm_i915_private *dev_priv) > +{ > + return to_intel_crtc(drm_crtc_from_index(&dev_priv->drm, 0)); > +} > + > static inline struct intel_crtc * > intel_get_crtc_for_pipe(struct drm_i915_private *dev_priv, enum pipe pipe) > { > -- > 2.24.0 > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
