On Tue, Sep 02, 2014 at 05:03:35PM -0700, clinton.a.tay...@intel.com wrote:
> From: Clint Taylor <clinton.a.tay...@intel.com>
> 
> Pixel replicated modes should be non-2x horizontal timings and pixel
> replicated by the HW across the HDMI cable at 2X pixel clock. Current
> horizontal resolution of 1440 does not allow pixel duplication to
> occur and scaling artifacts occur on the TV. HDMI certification
> 7-26 currently fails for all pixel replicated modes. This change will
> allow HDMI certification with 480i/576i modes once pixel replication
> is turned on.
> 
> Signed-off-by: Clint Taylor <clinton.a.tay...@intel.com>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com>
> Reviewed-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

Patch is missing the per-patch revision log and the cover letter also
doesn't have anything. Please remember to add that the next time around.

Anyway, patches merged to topic/core-stuff to make sure they don't get
lost.

Thanks, Daniel
> ---
>  drivers/gpu/drm/drm_edid.c |   96 
> ++++++++++++++++++++++----------------------
>  1 file changed, 48 insertions(+), 48 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index daf3cd8..1bdbfd0 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -632,27 +632,27 @@ static const struct drm_display_mode edid_cea_modes[] = 
> {
>                  DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC |
>                       DRM_MODE_FLAG_INTERLACE),
>         .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 6 - 1440x480i@60Hz */
> -     { DRM_MODE("1440x480i", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1478,
> -                1602, 1716, 0, 480, 488, 494, 525, 0,
> +     /* 6 - 720(1440)x480i@60Hz */
> +     { DRM_MODE("720x480i", DRM_MODE_TYPE_DRIVER, 13500, 720, 739,
> +                801, 858, 0, 480, 488, 494, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 7 - 1440x480i@60Hz */
> -     { DRM_MODE("1440x480i", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1478,
> -                1602, 1716, 0, 480, 488, 494, 525, 0,
> +     /* 7 - 720(1440)x480i@60Hz */
> +     { DRM_MODE("720x480i", DRM_MODE_TYPE_DRIVER, 13500, 720, 739,
> +                801, 858, 0, 480, 488, 494, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 8 - 1440x240@60Hz */
> -     { DRM_MODE("1440x240", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1478,
> -                1602, 1716, 0, 240, 244, 247, 262, 0,
> +     /* 8 - 720(1440)x240@60Hz */
> +     { DRM_MODE("720x240", DRM_MODE_TYPE_DRIVER, 13500, 720, 739,
> +                801, 858, 0, 240, 244, 247, 262, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 9 - 1440x240@60Hz */
> -     { DRM_MODE("1440x240", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1478,
> -                1602, 1716, 0, 240, 244, 247, 262, 0,
> +     /* 9 - 720(1440)x240@60Hz */
> +     { DRM_MODE("720x240", DRM_MODE_TYPE_DRIVER, 13500, 720, 739,
> +                801, 858, 0, 240, 244, 247, 262, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 60, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> @@ -714,27 +714,27 @@ static const struct drm_display_mode edid_cea_modes[] = 
> {
>                  DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC |
>                       DRM_MODE_FLAG_INTERLACE),
>         .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 21 - 1440x576i@50Hz */
> -     { DRM_MODE("1440x576i", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1464,
> -                1590, 1728, 0, 576, 580, 586, 625, 0,
> +     /* 21 - 720(1440)x576i@50Hz */
> +     { DRM_MODE("720x576i", DRM_MODE_TYPE_DRIVER, 13500, 720, 732,
> +                795, 864, 0, 576, 580, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 22 - 1440x576i@50Hz */
> -     { DRM_MODE("1440x576i", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1464,
> -                1590, 1728, 0, 576, 580, 586, 625, 0,
> +     /* 22 - 720(1440)x576i@50Hz */
> +     { DRM_MODE("720x576i", DRM_MODE_TYPE_DRIVER, 13500, 720, 732,
> +                795, 864, 0, 576, 580, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 23 - 1440x288@50Hz */
> -     { DRM_MODE("1440x288", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1464,
> -                1590, 1728, 0, 288, 290, 293, 312, 0,
> +     /* 23 - 720(1440)x288@50Hz */
> +     { DRM_MODE("720x288", DRM_MODE_TYPE_DRIVER, 13500, 720, 732,
> +                795, 864, 0, 288, 290, 293, 312, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 24 - 1440x288@50Hz */
> -     { DRM_MODE("1440x288", DRM_MODE_TYPE_DRIVER, 27000, 1440, 1464,
> -                1590, 1728, 0, 288, 290, 293, 312, 0,
> +     /* 24 - 720(1440)x288@50Hz */
> +     { DRM_MODE("720x288", DRM_MODE_TYPE_DRIVER, 13500, 720, 732,
> +                795, 864, 0, 288, 290, 293, 312, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 50, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> @@ -837,15 +837,15 @@ static const struct drm_display_mode edid_cea_modes[] = 
> {
>                  796, 864, 0, 576, 581, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC),
>         .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 44 - 1440x576i@100Hz */
> -     { DRM_MODE("1440x576", DRM_MODE_TYPE_DRIVER, 54000, 1440, 1464,
> -                1590, 1728, 0, 576, 580, 586, 625, 0,
> +     /* 44 - 720(1440)x576i@100Hz */
> +     { DRM_MODE("720x576i", DRM_MODE_TYPE_DRIVER, 27000, 720, 732,
> +                795, 864, 0, 576, 580, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 45 - 1440x576i@100Hz */
> -     { DRM_MODE("1440x576", DRM_MODE_TYPE_DRIVER, 54000, 1440, 1464,
> -                1590, 1728, 0, 576, 580, 586, 625, 0,
> +     /* 45 - 720(1440)x576i@100Hz */
> +     { DRM_MODE("720x576i", DRM_MODE_TYPE_DRIVER, 27000, 720, 732,
> +                795, 864, 0, 576, 580, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 100, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> @@ -870,15 +870,15 @@ static const struct drm_display_mode edid_cea_modes[] = 
> {
>                  798, 858, 0, 480, 489, 495, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC),
>         .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 50 - 1440x480i@120Hz */
> -     { DRM_MODE("1440x480i", DRM_MODE_TYPE_DRIVER, 54000, 1440, 1478,
> -                1602, 1716, 0, 480, 488, 494, 525, 0,
> +     /* 50 - 720(1440)x480i@120Hz */
> +     { DRM_MODE("720x480i", DRM_MODE_TYPE_DRIVER, 27000, 720, 739,
> +                801, 858, 0, 480, 488, 494, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 51 - 1440x480i@120Hz */
> -     { DRM_MODE("1440x480i", DRM_MODE_TYPE_DRIVER, 54000, 1440, 1478,
> -                1602, 1716, 0, 480, 488, 494, 525, 0,
> +     /* 51 - 720(1440)x480i@120Hz */
> +     { DRM_MODE("720x480i", DRM_MODE_TYPE_DRIVER, 27000, 720, 739,
> +                801, 858, 0, 480, 488, 494, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 120, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> @@ -892,15 +892,15 @@ static const struct drm_display_mode edid_cea_modes[] = 
> {
>                  796, 864, 0, 576, 581, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC),
>         .vrefresh = 200, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 54 - 1440x576i@200Hz */
> -     { DRM_MODE("1440x576i", DRM_MODE_TYPE_DRIVER, 108000, 1440, 1464,
> -                1590, 1728, 0, 576, 580, 586, 625, 0,
> +     /* 54 - 720(1440)x576i@200Hz */
> +     { DRM_MODE("720x576i", DRM_MODE_TYPE_DRIVER, 54000, 720, 732,
> +                795, 864, 0, 576, 580, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 200, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 55 - 1440x576i@200Hz */
> -     { DRM_MODE("1440x576i", DRM_MODE_TYPE_DRIVER, 108000, 1440, 1464,
> -                1590, 1728, 0, 576, 580, 586, 625, 0,
> +     /* 55 - 720(1440)x576i@200Hz */
> +     { DRM_MODE("720x576i", DRM_MODE_TYPE_DRIVER, 54000, 720, 732,
> +                795, 864, 0, 576, 580, 586, 625, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 200, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> @@ -914,15 +914,15 @@ static const struct drm_display_mode edid_cea_modes[] = 
> {
>                  798, 858, 0, 480, 489, 495, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC),
>         .vrefresh = 240, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -     /* 58 - 1440x480i@240 */
> -     { DRM_MODE("1440x480i", DRM_MODE_TYPE_DRIVER, 108000, 1440, 1478,
> -                1602, 1716, 0, 480, 488, 494, 525, 0,
> +     /* 58 - 720(1440)x480i@240 */
> +     { DRM_MODE("720x480i", DRM_MODE_TYPE_DRIVER, 54000, 720, 739,
> +                801, 858, 0, 480, 488, 494, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 240, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_4_3, },
> -     /* 59 - 1440x480i@240 */
> -     { DRM_MODE("1440x480i", DRM_MODE_TYPE_DRIVER, 108000, 1440, 1478,
> -                1602, 1716, 0, 480, 488, 494, 525, 0,
> +     /* 59 - 720(1440)x480i@240 */
> +     { DRM_MODE("720x480i", DRM_MODE_TYPE_DRIVER, 54000, 720, 739,
> +                801, 858, 0, 480, 488, 494, 525, 0,
>                  DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC |
>                       DRM_MODE_FLAG_INTERLACE | DRM_MODE_FLAG_DBLCLK),
>         .vrefresh = 240, .picture_aspect_ratio = HDMI_PICTURE_ASPECT_16_9, },
> -- 
> 1.7.9.5
> 
> _______________________________________________
> dri-devel mailing list
> dri-de...@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to