From: Manasi Navare <manasi.d.navare@intel.com>
To: intel-gfx@lists.freedesktop.org
Subject: DP compliance failure due to dithering for 18bpp video pattern
Date: Mon, 9 Jan 2017 17:43:30 -0800 [thread overview]
Message-ID: <20170110014329.GA12751@intel.com> (raw)
Hi All,
We are seeing CRC check failures in some of the 18bpp video pattern
DP Compliance tests causing the tests to fail. On further investigation, it is
rootcaused to dithering that the i915 driver enables in case of 18bpp pipe
configuration that messes up the CRC and causes the test to fail.
Some of the approaches that can solve this problem are:
1. Add a new method in intel_dp.c to request the compliance test state.
Call this new method in intel_display.c to not enable dithering during a
compliance test. Issue with this is it makes the general portion of the driver
compliance aware.
2. Move the dithering enable to compute_config methods in all encoder source
files. Issue: Lot of duplicate code and DP is the only encoder that uses 18bpc.
3. Disable dithering at all times in the driver. However this can cause image
quality issue with 8bpc plane and 6 bit pipe.
Any suggestions on which approach can be implemented in order to pass
compliance?
Regards
Manasi
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next reply other threads:[~2017-01-10 1:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-10 1:43 Manasi Navare [this message]
2017-01-11 15:09 ` DP compliance failure due to dithering for 18bpp video pattern Jani Nikula
2017-01-11 15:35 ` Ville Syrjälä
2017-01-12 10:32 ` Jani Nikula
2017-01-12 23:30 ` Manasi Navare
2017-01-12 23:41 ` Rodrigo Vivi
2017-01-12 23:53 ` Manasi Navare
2017-01-13 7:11 ` Jani Nikula
2017-01-17 6:42 ` Manasi Navare
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=20170110014329.GA12751@intel.com \
--to=manasi.d.navare@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).