From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paulo Zanoni Subject: Re: [PATCH 2/6] drm/i915/crt: explicitly set up HOTPLUG_BITS on resume Date: Wed, 17 Oct 2012 18:42:26 -0300 Message-ID: References: <1349978908-7687-1-git-send-email-daniel.vetter@ffwll.ch> <1349978908-7687-3-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ie0-f177.google.com (mail-ie0-f177.google.com [209.85.223.177]) by gabe.freedesktop.org (Postfix) with ESMTP id 937C39E794 for ; Wed, 17 Oct 2012 14:42:27 -0700 (PDT) Received: by mail-ie0-f177.google.com with SMTP id e14so13511916iej.36 for ; Wed, 17 Oct 2012 14:42:27 -0700 (PDT) In-Reply-To: <1349978908-7687-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 Hi 2012/10/11 Daniel Vetter : > ... instead of relying on the register save/restore madness to do this. > > To extract a bit of code call drm_mode_config_reset both on resume > and boot-up and move the hw state frobbing from the crt_init to the > ->reset callback. The crt connector is the only one with a ->reset > callback, hence we can easily do this. The patch looks correct, but doesn't it make more sense to add some encoder-specific "intel_sanitize_encoder" callback instead of reusing drm's ->reset ? Maybe we're distorting the meaning of the ->reset callback? This is just a bikeshed, not a strong opinion, so feel free to merge it as it is: Reviewed-by: Paulo Zanoni > > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/i915/i915_drv.c | 1 - > drivers/gpu/drm/i915/intel_crt.c | 26 +++++++++++++------------- > drivers/gpu/drm/i915/intel_display.c | 2 ++ > 3 files changed, 15 insertions(+), 14 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index a7837e5..2aabce7 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -547,7 +547,6 @@ static int i915_drm_thaw(struct drm_device *dev) > > intel_modeset_init_hw(dev); > intel_modeset_setup_hw_state(dev); > - drm_mode_config_reset(dev); > drm_irq_install(dev); > } > > diff --git a/drivers/gpu/drm/i915/intel_crt.c b/drivers/gpu/drm/i915/intel_crt.c > index 46c90f5..53f3e87 100644 > --- a/drivers/gpu/drm/i915/intel_crt.c > +++ b/drivers/gpu/drm/i915/intel_crt.c > @@ -662,10 +662,22 @@ static int intel_crt_set_property(struct drm_connector *connector, > static void intel_crt_reset(struct drm_connector *connector) > { > struct drm_device *dev = connector->dev; > + struct drm_i915_private *dev_priv = dev->dev_private; > struct intel_crt *crt = intel_attached_crt(connector); > > - if (HAS_PCH_SPLIT(dev)) > + if (HAS_PCH_SPLIT(dev)) { > + u32 adpa; > + > + adpa = I915_READ(PCH_ADPA); > + adpa &= ~ADPA_CRT_HOTPLUG_MASK; > + adpa |= ADPA_HOTPLUG_BITS; > + I915_WRITE(PCH_ADPA, adpa); > + POSTING_READ(PCH_ADPA); > + > + DRM_DEBUG_KMS("pch crt adpa set to 0x%x\n", adpa); > crt->force_hotplug_required = 1; > + } > + > } > > /* > @@ -784,18 +796,6 @@ void intel_crt_init(struct drm_device *dev) > * Configure the automatic hotplug detection stuff > */ > crt->force_hotplug_required = 0; > - if (HAS_PCH_SPLIT(dev)) { > - u32 adpa; > - > - adpa = I915_READ(PCH_ADPA); > - adpa &= ~ADPA_CRT_HOTPLUG_MASK; > - adpa |= ADPA_HOTPLUG_BITS; > - I915_WRITE(PCH_ADPA, adpa); > - POSTING_READ(PCH_ADPA); > - > - DRM_DEBUG_KMS("pch crt adpa set to 0x%x\n", adpa); > - crt->force_hotplug_required = 1; > - } > > dev_priv->hotplug_supported_mask |= CRT_HOTPLUG_INT_STATUS; > } > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index d469b42..ffc3758 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -8334,6 +8334,8 @@ void intel_modeset_setup_hw_state(struct drm_device *dev) > intel_modeset_update_staged_output_state(dev); > > intel_modeset_check_state(dev); > + > + drm_mode_config_reset(dev); > } > > void intel_modeset_gem_init(struct drm_device *dev) > -- > 1.7.11.2 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Paulo Zanoni