diff options
author | Simon Ser <simon.ser@intel.com> | 2019-05-06 17:30:33 +0300 |
---|---|---|
committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-05-09 08:43:27 +0300 |
commit | 23200bc1b008542bd4d744755540b07881d41fae (patch) | |
tree | 582d8a7d08c113e06ceaaffd5793e2ea855301a7 /lib/igt_edid.c | |
parent | 3e3ff0e48989abd25fce4916e85e8fef20a3c63a (diff) |
lib/igt_edid: fix detailed pixel timing analog/digital
The generated EDIDs were wrongly indicating that they support analog sync.
Fixup the detailed timings flags to advertise digital sync instead.
Currently the Linux kernel seems to ignore this completely. However I'd prefer
to fix this anyway to make sure we don't run into issues if an EDID consumer
actually cares about it.
The header definitions for EDID_PT_* values has been re-organized to make it
clearer in which situations the flags are relevant.
Changes from v1 to v2:
- Fix misleading commit message
- Revert the "misc" → "features" rename
- Re-organize EDID_PT_* definitions to make them clearer
Changes from v2 to v3:
- Include review changelog in commit message
- Fix "Fixes:" tag to conform to the kernel style
- Re-order EDID_PT_* definitions to sort by descending bitshift
Signed-off-by: Simon Ser <simon.ser@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Fixes: a2fd0489c87a ("lib/igt_edid: new library for generating EDIDs")
Diffstat (limited to 'lib/igt_edid.c')
-rw-r--r-- | lib/igt_edid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/igt_edid.c b/lib/igt_edid.c index 52e66ab2..9d604b13 100644 --- a/lib/igt_edid.c +++ b/lib/igt_edid.c @@ -110,7 +110,7 @@ void detailed_timing_set_mode(struct detailed_timing *dt, drmModeModeInfo *mode, pt->width_height_mm_hi = (width_mm & 0xF00) >> 4 | (height_mm & 0xF00) >> 8; - pt->misc = 0; + pt->misc = EDID_PT_SYNC_DIGITAL_SEPARATE; if (mode->flags & DRM_MODE_FLAG_PHSYNC) pt->misc |= EDID_PT_HSYNC_POSITIVE; if (mode->flags & DRM_MODE_FLAG_PVSYNC) |