From: Imre Deak <imre.deak@intel.com>
To: ville.syrjala@linux.intel.com
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH v2 0/4] drm/i915: Re-enable HDMI 12bpc
Date: Wed, 01 Jul 2015 21:08:15 +0300 [thread overview]
Message-ID: <1435774095.9389.8.camel@intel.com> (raw)
In-Reply-To: <1435667634-4574-1-git-send-email-ville.syrjala@linux.intel.com>
On Tue, 2015-06-30 at 15:33 +0300, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Here's my second attempt at flipping HDMI 12bpc back on. In my last attempt [1]
> Imre found that lots of standard CEA modes (1080p60 etc.) no longer work on
> BXT due to the 12bpc port clock landing in a range that the DPLL can't generate.
> CHV has the same limitation but it doesn't do 12bpc so the situation there
> isn't as bad.
>
> This series attempts to work around this problem by falling back to 8bpc when
> the 1.5x port clock frequency turns out to be bad. Additionally we will from
> now on filter out any mode where both 8bpc and 12bpc port clock is bad.
>
> [1] http://lists.freedesktop.org/archives/intel-gfx/2015-June/068988.html
>
> Ville Syrjälä (4):
> drm/i915: Fix HDMI 12bpc and pixel repeat clock readout for DDI
> platforms
> drm/i915: Bump HDMI min port clock to 25 MHz
> drm/i915: Account for CHV/BXT DPLL clock limittions
> Revert "drm/i915: Disable 12bpc hdmi for now"
>
> drivers/gpu/drm/i915/intel_ddi.c | 49 +++++++++++++++++-----------------
> drivers/gpu/drm/i915/intel_hdmi.c | 55 +++++++++++++++++++++++++++------------
> 2 files changed, 63 insertions(+), 41 deletions(-)
Looks ok to me. 12bpc modesets and the fall-back logic seems to work
fine on BXT. On the series:
Reviewed-and-tested-by: Imre Deak <imre.deak@intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2015-07-01 18:08 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-30 12:33 [PATCH v2 0/4] drm/i915: Re-enable HDMI 12bpc ville.syrjala
2015-06-30 12:33 ` [PATCH 1/4] drm/i915: Fix HDMI 12bpc and pixel repeat clock readout for DDI platforms ville.syrjala
2015-06-30 12:33 ` [PATCH 2/4] drm/i915: Bump HDMI min port clock to 25 MHz ville.syrjala
2015-06-30 12:33 ` [PATCH 3/4] drm/i915: Account for CHV/BXT DPLL clock limitations ville.syrjala
2015-06-30 16:23 ` [PATCH v2 " ville.syrjala
2015-06-30 12:33 ` [PATCH 4/4] Revert "drm/i915: Disable 12bpc hdmi for now" ville.syrjala
2015-07-01 18:08 ` Imre Deak [this message]
2015-07-06 7:45 ` [PATCH v2 0/4] drm/i915: Re-enable HDMI 12bpc Daniel Vetter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1435774095.9389.8.camel@intel.com \
--to=imre.deak@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=ville.syrjala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.