From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 03/10] drm/i915: add pipe_config->timings_set Date: Sun, 3 Mar 2013 19:01:12 +0100 Message-ID: <20130303180112.GM9021@phenom.ffwll.local> References: <1361491014-13888-1-git-send-email-daniel.vetter@ffwll.ch> <1361491014-13888-4-git-send-email-daniel.vetter@ffwll.ch> <20130222135115.GN4469@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail-we0-f182.google.com (mail-we0-f182.google.com [74.125.82.182]) by gabe.freedesktop.org (Postfix) with ESMTP id 0A71CE5F23 for ; Sun, 3 Mar 2013 09:58:40 -0800 (PST) Received: by mail-we0-f182.google.com with SMTP id t57so3917054wey.27 for ; Sun, 03 Mar 2013 09:58:40 -0800 (PST) Content-Disposition: inline In-Reply-To: <20130222135115.GN4469@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: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Cc: Daniel Vetter , Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Fri, Feb 22, 2013 at 03:51:15PM +0200, Ville Syrj=E4l=E4 wrote: > On Fri, Feb 22, 2013 at 12:56:47AM +0100, Daniel Vetter wrote: > > Only used by the lvds encoder. Note that we shouldn't do the same > > simple conversion with the FORCE_6BPC flag, since that's much better > > handled by moving all the pipe_bpc computation around. > > = > > Signed-off-by: Daniel Vetter > > --- > > drivers/gpu/drm/i915/intel_display.c | 12 +++++++++++- > > drivers/gpu/drm/i915/intel_drv.h | 10 ++++++---- > > drivers/gpu/drm/i915/intel_lvds.c | 19 +++++++++---------- > > 3 files changed, 26 insertions(+), 15 deletions(-) > > = > > > @@ -359,6 +357,7 @@ static bool intel_lvds_mode_fixup(struct drm_encode= r *encoder, > > I915_WRITE(BCLRPAT(pipe), 0); > > = > > drm_mode_set_crtcinfo(adjusted_mode, 0); > > + pipe_config->timings_set =3D true; > = > This changes the behaviour a bit. Previously the flag was only set from > the centering funcs, but now it's set always. Is that intentional? The crtc code also calls drm_mode_set_crtcinfo(adjusted_mode, 0) if ->timings_set is false, so doesn't result in any behaviour change. Hence I've figure that fewer lines of code should be better. Want me to change it back or just add a bit of text to the commit message? E.g. "Note that since the lvds code unconditionally sets the crtc timings, we can also unconditionally set the respective flag and not just when we set special timings like the old code did." Cheers, Daniel > = > > = > > switch (intel_connector->panel.fitting_mode) { > > case DRM_MODE_SCALE_CENTER: > > @@ -661,7 +660,6 @@ static int intel_lvds_set_property(struct drm_conne= ctor *connector, > > } > > = > > static const struct drm_encoder_helper_funcs intel_lvds_helper_funcs = =3D { > > - .mode_fixup =3D intel_lvds_mode_fixup, > > .mode_set =3D intel_lvds_mode_set, > > }; > > = > > @@ -1102,6 +1100,7 @@ bool intel_lvds_init(struct drm_device *dev) > > intel_encoder->enable =3D intel_enable_lvds; > > intel_encoder->pre_enable =3D intel_pre_enable_lvds; > > intel_encoder->pre_pll_enable =3D intel_pre_pll_enable_lvds; > > + intel_encoder->compute_config =3D intel_lvds_compute_config; > > intel_encoder->disable =3D intel_disable_lvds; > > intel_encoder->get_hw_state =3D intel_lvds_get_hw_state; > > intel_connector->get_hw_state =3D intel_connector_get_hw_state; > > -- = > > 1.7.11.4 > > = > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > = > -- = > Ville Syrj=E4l=E4 > Intel OTC -- = Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch