[PATCH] Consistently name interlaced modes

Alex Deucher alexdeucher at gmail.com
Mon Aug 13 08:10:51 PDT 2012


On Fri, Aug 10, 2012 at 6:52 PM, Russell King - ARM Linux
<linux at arm.linux.org.uk> wrote:
> At the moment, there is an inconsistency in the way modes are named.
> Modes with timings parsed from the EDID information will call
> drm_mode_set_name(), which will name the mode using this form:
>
>         <horizontal-res>x<vertical-res><interlace-char>
>
> eg, 1920x1080i for an interlaced mode, or 1920x1080 for a progressive
> mode.
>
> However, timings parsed using the tables in drm_edid_modes.h do not
> have the 'i' suffix.  You are left to deduce that they're interlaced
> from xrandr's output by the lower vertical refresh frequencies.
>
> This patch changes the interlaced mode names in drm_edid_modes.h to
> follow the style set by drm_mode_set_name(), which makes it clear
> in xrandr which modes are interlaced and which are not (as xrandr
> groups the refresh rates on a line according to the name field.)
>
> Signed-off-by: Russell King <rmk+kernel at arm.linux.org.uk>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>


More information about the dri-devel mailing list