From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 9/9] drm/i915: Assert dpll running in intel_crtc_load_lut() on pre-PCH platforms Date: Wed, 5 Jun 2013 22:40:53 +0200 Message-ID: <20130605204053.GN15743@phenom.ffwll.local> References: <1370342947-20757-1-git-send-email-ville.syrjala@linux.intel.com> <1370342947-20757-10-git-send-email-ville.syrjala@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-ea0-f174.google.com (mail-ea0-f174.google.com [209.85.215.174]) by gabe.freedesktop.org (Postfix) with ESMTP id BA648E5CE7 for ; Wed, 5 Jun 2013 13:40:58 -0700 (PDT) Received: by mail-ea0-f174.google.com with SMTP id z7so1506039eaf.19 for ; Wed, 05 Jun 2013 13:40:57 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Rodrigo Vivi Cc: intel-gfx List-Id: intel-gfx@lists.freedesktop.org On Wed, Jun 05, 2013 at 04:41:54PM -0300, Rodrigo Vivi wrote: > why is this needed? > anyways: Reviewed-by: Rodrigo Vivi An r-b is a formal statement like a s-o-b. From Documentaion/SubmittingPatches: Reviewer's statement of oversight By offering my Reviewed-by: tag, I state that: (a) I have carried out a technical review of this patch to evaluate its appropriateness and readiness for inclusion into the mainline kernel. (b) Any problems, concerns, or questions relating to the patch have been communicated back to the submitter. I am satisfied with the submitter's response to my comments. (c) While there may be things that could be improved with this submission, I believe that it is, at this time, (1) a worthwhile modification to the kernel, and (2) free of known issues which would argue against its inclusion. (d) While I have reviewed the patch and believe it to be sound, I do not (unless explicitly stated elsewhere) make any warranties or guarantees that it will achieve its stated purpose or function properly in any given situation. If you're asking "why is this needed?" it does _not_ deserve an r-b on a pretty fundamental level since it clearly violates c) 1) above. Please don't just go through the motions when doing review. Thanks, Daniel > = > On Tue, Jun 4, 2013 at 7:49 AM, wrote: > > From: Ville Syrj=E4l=E4 > > > > Signed-off-by: Ville Syrj=E4l=E4 > > --- > > drivers/gpu/drm/i915/intel_display.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i91= 5/intel_display.c > > index 90d02c7..3be69bc 100644 > > --- a/drivers/gpu/drm/i915/intel_display.c > > +++ b/drivers/gpu/drm/i915/intel_display.c > > @@ -6340,6 +6340,9 @@ void intel_crtc_load_lut(struct drm_crtc *crtc) > > if (!crtc->enabled || !intel_crtc->active) > > return; > > > > + if (!HAS_PCH_SPLIT(dev_priv->dev)) > > + assert_pll_enabled(dev_priv, pipe); > > + > > /* use legacy palette for Ironlake */ > > if (HAS_PCH_SPLIT(dev)) > > palreg =3D LGC_PALETTE(pipe); > > -- > > 1.8.1.5 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > = > = > = > -- = > Rodrigo Vivi > Blog: http://blog.vivi.eng.br > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- = Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch