From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 3/3] drm/i915: explicitly set up PIPECONF (and gamma table) on haswell Date: Thu, 13 Jun 2013 10:57:52 +0300 Message-ID: <20130613075752.GS5004@intel.com> References: <1371077699-30702-1-git-send-email-daniel.vetter@ffwll.ch> <1371077699-30702-3-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 5C0E7E5C7C for ; Thu, 13 Jun 2013 00:57:57 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1371077699-30702-3-git-send-email-daniel.vetter@ffwll.ch> 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: Daniel Vetter Cc: Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Thu, Jun 13, 2013 at 12:54:59AM +0200, Daniel Vetter wrote: > Again we don't really support different settings, so don't let the > BIOS sneak stuff through. > = > Since the motivation for this patch series is to ensure we have the > correct gamma table mode selected also add the required write to the > GAMMA_MODE register to select the 8bit legacy table. > = > And since I find lowercase letters in #defines offensive, also > bikeshed those. > = > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/i915/i915_reg.h | 6 +++--- > drivers/gpu/drm/i915/intel_display.c | 7 ++++--- > 2 files changed, 7 insertions(+), 6 deletions(-) > = > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_= reg.h > index 01e8783..8136b00 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -3670,9 +3670,9 @@ > #define _GAMMA_MODE_B 0x4ac80 > #define GAMMA_MODE(pipe) _PIPE(pipe, _GAMMA_MODE_A, _GAMMA_MODE_B) > #define GAMMA_MODE_MODE_MASK (3 << 0) > -#define GAMMA_MODE_MODE_8bit (0 << 0) > -#define GAMMA_MODE_MODE_10bit (1 << 0) > -#define GAMMA_MODE_MODE_12bit (2 << 0) > +#define GAMMA_MODE_MODE_8BIT (0 << 0) > +#define GAMMA_MODE_MODE_10BIT (1 << 0) > +#define GAMMA_MODE_MODE_12BIT (2 << 0) > #define GAMMA_MODE_MODE_SPLIT (3 << 0) > = > /* interrupts */ > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/= intel_display.c > index 4ca0273..e1184eb 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -5393,13 +5393,11 @@ static void haswell_set_pipeconf(struct drm_crtc = *crtc) > enum transcoder cpu_transcoder =3D intel_crtc->config.cpu_transcoder; > uint32_t val; > = > - val =3D I915_READ(PIPECONF(cpu_transcoder)); > + val =3D 0; > = > - val &=3D ~(PIPECONF_DITHER_EN | PIPECONF_DITHER_TYPE_MASK); > if (intel_crtc->config.dither) > val |=3D (PIPECONF_DITHER_EN | PIPECONF_DITHER_TYPE_SP); > = > - val &=3D ~PIPECONF_INTERLACE_MASK_HSW; > if (intel_crtc->config.adjusted_mode.flags & DRM_MODE_FLAG_INTERLACE) > val |=3D PIPECONF_INTERLACED_ILK; > else > @@ -5407,6 +5405,9 @@ static void haswell_set_pipeconf(struct drm_crtc *c= rtc) > = > I915_WRITE(PIPECONF(cpu_transcoder), val); > POSTING_READ(PIPECONF(cpu_transcoder)); > + > + I915_WRITE(GAMMA_MODE(intel_crtc->pipe), GAMMA_MODE_MODE_8BIT); > + POSTING_READ(GAMMA_MODE(intel_crtc->pipe)); Why the POSTING_READ()? In fact, why do we have any posting reads in xxx_set_pipeconf()? Otherwise, for the series: Reviewed-by: Ville Syrj=E4l=E4 > } > = > static bool ironlake_compute_clocks(struct drm_crtc *crtc, > -- = > 1.7.11.7 > = > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- = Ville Syrj=E4l=E4 Intel OTC