[PATCH 1/2] drm/edid: Add the missing "Hz" to VIC 58,59 comment

[Daniel Vetter]

On Fri, Nov 04, 2016 at 08:29:11AM +0100, Andrzej Hajda wrote:
> On 03.11.2016 13:53, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> >
> > All the VICs apart from 58 and 59 have the word "Hz" included in the
> > comment. Include it for 59 and 59 as well.
> >
> > Cc: Shashank Sharma <shashank.sharma at intel.com>
> > Cc: Andrzej Hajda <a.hajda at samsung.com>
> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> Reviewed-by: Andrzej Hajda <a.hajda at samsung.com>

Applied to drm-misc. Can you pls also review patch 2/2?

Thanks, Daniel

> --
> Regards
> Andrzej
> 
> > ---
> >  drivers/gpu/drm/drm_edid.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> > index 9506933b41cd..7eec18925b70 100644
> > --- a/drivers/gpu/drm/drm_edid.c
> > +++ b/drivers/gpu/drm/drm_edid.c
> > @@ -957,13 +957,13 @@ 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 - 720(1440)x480i at 240 */
> > +	/* 58 - 720(1440)x480i at 240Hz */
> >  	{ 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 - 720(1440)x480i at 240 */
> > +	/* 59 - 720(1440)x480i at 240Hz */
> >  	{ 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 |
> 
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch