On Tue, Dec 15, 2015 at 12:22:40PM +0200, Mika Kahola wrote:
> On Tue, 2015-11-24 at 11:29 +0100, Maarten Lankhorst wrote:
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index 3c46037b6e55..178a042f917e 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -5991,22 +5991,31 @@ static int broxton_calc_cdclk(struct drm_i915_private *dev_priv,
> > static int intel_mode_max_pixclk(struct drm_device *dev,
> > struct drm_atomic_state *state)
> > {
> > - struct intel_crtc *intel_crtc;
> > - struct intel_crtc_state *crtc_state;
> > - int max_pixclk = 0;
> > + struct intel_atomic_state *intel_state = to_intel_atomic_state(state);
> > + struct drm_i915_private *dev_priv = dev->dev_private;
> This is a nitpick but we should use nowadays to_i915()
If you're going to bring that up, we should be passing in the right
pointer to begin with!
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
