From patchwork Tue Oct 27 13:23:38 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?VmlsbGUgU3lyasOkbMOk?= X-Patchwork-Id: 7496971 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 6EB6ABEEA4 for ; Tue, 27 Oct 2015 13:23:56 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 759952095A for ; Tue, 27 Oct 2015 13:23:55 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 573B320957 for ; Tue, 27 Oct 2015 13:23:54 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 352FE6E593; Tue, 27 Oct 2015 06:23:53 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id 209116EAD4 for ; Tue, 27 Oct 2015 06:23:47 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga102.fm.intel.com with ESMTP; 27 Oct 2015 06:23:48 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,205,1444719600"; d="scan'208";a="804314622" Received: from stinkbox.fi.intel.com (HELO stinkbox) ([10.237.72.174]) by orsmga001.jf.intel.com with SMTP; 27 Oct 2015 06:23:45 -0700 Received: by stinkbox (sSMTP sendmail emulation); Tue, 27 Oct 2015 15:23:43 +0200 From: ville.syrjala@linux.intel.com To: intel-gfx@lists.freedesktop.org Date: Tue, 27 Oct 2015 15:23:38 +0200 Message-Id: <1445952218-25830-3-git-send-email-ville.syrjala@linux.intel.com> X-Mailer: git-send-email 2.4.10 In-Reply-To: <1445952218-25830-1-git-send-email-ville.syrjala@linux.intel.com> References: <1445952218-25830-1-git-send-email-ville.syrjala@linux.intel.com> MIME-Version: 1.0 Subject: [Intel-gfx] [PATCH 3/3] drm/i915: Share cdclk code for BDW and BXT X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Ville Syrjälä The difference betwen the BXT and BDW cdclk code boils down to two things: claming the cdclk to the maximum supported, and panel fitter downscaling ratio Unifying the the max cdclk clamping is easy, just do it. And as far as the panel fitter is concerned, SKL+ already (ab)use the pch pfit state for its pipe scaler information, so it will compute the adjusted pixel rate correctly. So we can just use the BDW code for BXT, as long as we lift the BXT pixel rate -> cdclk selection into the correct place. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 87 +++++++++++++----------------------- 1 file changed, 30 insertions(+), 57 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 978f543..0c782c7 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -5932,25 +5932,6 @@ static int valleyview_calc_cdclk(struct drm_i915_private *dev_priv, return 200000; } -static int broxton_calc_cdclk(struct drm_i915_private *dev_priv, - int max_pixclk) -{ - /* - * FIXME: - * - set 19.2MHz bypass frequency if there are no active pipes - */ - if (max_pixclk > 576000) - return 624000; - else if (max_pixclk > 384000) - return 576000; - else if (max_pixclk > 288000) - return 384000; - else if (max_pixclk > 144000) - return 288000; - else - return 144000; -} - /* Compute the max pixel clock for new configuration. Uses atomic state if * that's non-NULL, look at current state otherwise. */ static int intel_mode_max_pixclk(struct drm_device *dev, @@ -5990,21 +5971,6 @@ static int valleyview_modeset_calc_cdclk(struct drm_atomic_state *state) return 0; } -static int broxton_modeset_calc_cdclk(struct drm_atomic_state *state) -{ - struct drm_device *dev = state->dev; - struct drm_i915_private *dev_priv = dev->dev_private; - int max_pixclk = intel_mode_max_pixclk(dev, state); - - if (max_pixclk < 0) - return max_pixclk; - - to_intel_atomic_state(state)->cdclk = - broxton_calc_cdclk(dev_priv, max_pixclk); - - return 0; -} - static void vlv_program_pfi_credits(struct drm_i915_private *dev_priv) { unsigned int credits, default_credits; @@ -9497,14 +9463,6 @@ void hsw_disable_pc8(struct drm_i915_private *dev_priv) intel_prepare_ddi(dev); } -static void broxton_modeset_commit_cdclk(struct drm_atomic_state *old_state) -{ - struct drm_device *dev = old_state->dev; - unsigned int req_cdclk = to_intel_atomic_state(old_state)->cdclk; - - broxton_set_cdclk(dev, req_cdclk); -} - /* compute the max rate for new configuration */ static int ilk_max_pixel_rate(struct drm_atomic_state *state) { @@ -9621,14 +9579,31 @@ static int broadwell_modeset_calc_cdclk(struct drm_atomic_state *state) * FIXME should also account for plane ratio * once 64bpp pixel formats are supported. */ - if (max_pixclk > 540000) - cdclk = 675000; - else if (max_pixclk > 450000) - cdclk = 540000; - else if (max_pixclk > 337500) - cdclk = 450000; - else - cdclk = 337500; + if (IS_BROXTON(dev_priv)) { + /* + * FIXME: + * - set 19.2MHz bypass frequency if there are no active pipes + */ + if (max_pixclk > 576000) + cdclk = 624000; + else if (max_pixclk > 384000) + cdclk = 576000; + else if (max_pixclk > 288000) + cdclk = 384000; + else if (max_pixclk > 144000) + cdclk = 288000; + else + cdclk = 144000; + } else { + if (max_pixclk > 540000) + cdclk = 675000; + else if (max_pixclk > 450000) + cdclk = 540000; + else if (max_pixclk > 337500) + cdclk = 450000; + else + cdclk = 337500; + } /* * FIXME move the cdclk caclulation to @@ -9650,7 +9625,10 @@ static void broadwell_modeset_commit_cdclk(struct drm_atomic_state *old_state) struct drm_device *dev = old_state->dev; unsigned int req_cdclk = to_intel_atomic_state(old_state)->cdclk; - broadwell_set_cdclk(dev, req_cdclk); + if (IS_BROXTON(dev)) + broxton_set_cdclk(dev, req_cdclk); + else + broadwell_set_cdclk(dev, req_cdclk); } static int haswell_crtc_compute_clock(struct intel_crtc *crtc, @@ -14619,7 +14597,7 @@ static void intel_init_display(struct drm_device *dev) dev_priv->display.fdi_link_train = hsw_fdi_link_train; } - if (IS_BROADWELL(dev)) { + if (IS_BROADWELL(dev) || IS_BROXTON(dev)) { dev_priv->display.modeset_commit_cdclk = broadwell_modeset_commit_cdclk; dev_priv->display.modeset_calc_cdclk = @@ -14629,11 +14607,6 @@ static void intel_init_display(struct drm_device *dev) valleyview_modeset_commit_cdclk; dev_priv->display.modeset_calc_cdclk = valleyview_modeset_calc_cdclk; - } else if (IS_BROXTON(dev)) { - dev_priv->display.modeset_commit_cdclk = - broxton_modeset_commit_cdclk; - dev_priv->display.modeset_calc_cdclk = - broxton_modeset_calc_cdclk; } switch (INTEL_INFO(dev)->gen) {