From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paulo Zanoni Subject: Re: [PATCH 16/16] drm/i915: Rename ilk_check_wm to ilk_validate_wm_level Date: Fri, 11 Oct 2013 14:08:07 -0300 Message-ID: References: <1381335490-4906-1-git-send-email-ville.syrjala@linux.intel.com> <1381335490-4906-17-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-ob0-f181.google.com (mail-ob0-f181.google.com [209.85.214.181]) by gabe.freedesktop.org (Postfix) with ESMTP id 33C9BE6BFE for ; Fri, 11 Oct 2013 10:08:08 -0700 (PDT) Received: by mail-ob0-f181.google.com with SMTP id va2so2963680obc.40 for ; Fri, 11 Oct 2013 10:08:07 -0700 (PDT) In-Reply-To: <1381335490-4906-17-git-send-email-ville.syrjala@linux.intel.com> 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: =?ISO-8859-1?Q?Ville_Syrj=E4l=E4?= Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org 2013/10/9 : > From: Ville Syrj=E4l=E4 > > Makes the behaviour of the function more clear. > > Signed-off-by: Ville Syrj=E4l=E4 Thanks :) Reviewed-by: Paulo Zanoni > --- > drivers/gpu/drm/i915/intel_pm.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel= _pm.c > index 7b52e39..211a946 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -2383,9 +2383,9 @@ static void ilk_compute_wm_maximums(struct drm_devi= ce *dev, > max->fbc =3D ilk_fbc_wm_max(); > } > > -static bool ilk_check_wm(int level, > - const struct hsw_wm_maximums *max, > - struct intel_wm_level *result) > +static bool ilk_validate_wm_level(int level, > + const struct hsw_wm_maximums *max, > + struct intel_wm_level *result) > { > bool ret; > > @@ -2635,7 +2635,7 @@ static bool intel_compute_pipe_wm(struct drm_crtc *= crtc, > pipe_wm->linetime =3D hsw_compute_linetime_wm(dev, crtc); > > /* At least LP0 must be valid */ > - return ilk_check_wm(0, &max, &pipe_wm->wm[0]); > + return ilk_validate_wm_level(0, &max, &pipe_wm->wm[0]); > } > > /* > @@ -2680,7 +2680,7 @@ static void ilk_wm_merge(struct drm_device *dev, > > ilk_merge_wm_level(dev, level, wm); > > - if (!ilk_check_wm(level, max, wm)) > + if (!ilk_validate_wm_level(level, max, wm)) > break; > > /* > -- > 1.8.1.5 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- = Paulo Zanoni