On Wed, 13 Jan 2016, Ville Syrjälä <ville.syrj...@linux.intel.com> wrote: > On Wed, Jan 13, 2016 at 04:35:20PM +0200, Jani Nikula wrote: >> Per DP spec, the source device should fall back to 18 bpp, VESA range >> RGB when the sink capability is unknown. Fix the color depth >> clamping. 18 bpp color depth should ensure full color range in automatic >> mode. >> >> The clamping has been HDMI specific since its introduction in >> >> commit 996a2239f93b03c5972923f04b097f65565c5bed >> Author: Daniel Vetter <daniel.vet...@ffwll.ch> >> Date: Fri Apr 19 11:24:34 2013 +0200 >> >> drm/i915: Disable high-bpc on pre-1.4 EDID screens >> >> Cc: sta...@vger.kernel.org >> Reported-by: Dihan Wickremasuriya <nayo...@gmail.com> >> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=105331 >> Signed-off-by: Jani Nikula <jani.nik...@intel.com> > > Makes sense to me as far as the spec is concerned. > Reviewed-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
Also got Tested-by in the bug. Pushed to drm-intel-next-queued, thanks for the review. BR, Jani. > >> --- >> drivers/gpu/drm/i915/intel_display.c | 20 +++++++++++++++----- >> 1 file changed, 15 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_display.c >> b/drivers/gpu/drm/i915/intel_display.c >> index 07ca19b0ec17..6eaecd9385ab 100644 >> --- a/drivers/gpu/drm/i915/intel_display.c >> +++ b/drivers/gpu/drm/i915/intel_display.c >> @@ -12171,11 +12171,21 @@ connected_sink_compute_bpp(struct intel_connector >> *connector, >> pipe_config->pipe_bpp = connector->base.display_info.bpc*3; >> } >> >> - /* Clamp bpp to 8 on screens without EDID 1.4 */ >> - if (connector->base.display_info.bpc == 0 && bpp > 24) { >> - DRM_DEBUG_KMS("clamping display bpp (was %d) to default limit >> of 24\n", >> - bpp); >> - pipe_config->pipe_bpp = 24; >> + /* Clamp bpp to default limit on screens without EDID 1.4 */ >> + if (connector->base.display_info.bpc == 0) { >> + int type = connector->base.connector_type; >> + int clamp_bpp = 24; >> + >> + /* Fall back to 18 bpp when DP sink capability is unknown. */ >> + if (type == DRM_MODE_CONNECTOR_DisplayPort || >> + type == DRM_MODE_CONNECTOR_eDP) >> + clamp_bpp = 18; >> + >> + if (bpp > clamp_bpp) { >> + DRM_DEBUG_KMS("clamping display bpp (was %d) to default >> limit of %d\n", >> + bpp, clamp_bpp); >> + pipe_config->pipe_bpp = clamp_bpp; >> + } >> } >> } >> >> -- >> 2.1.4 -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx