On Mon, Aug 10, 2015 at 01:57:21PM +0200, Thierry Reding wrote: > On Mon, Aug 10, 2015 at 11:55:37AM +0200, Daniel Vetter wrote: > > Spotted while reading code for random reasons. > > > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > > --- > > drivers/gpu/drm/drm_edid.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c > > index 4a403eb90ded..4780b1924bef 100644 > > --- a/drivers/gpu/drm/drm_edid.c > > +++ b/drivers/gpu/drm/drm_edid.c > > @@ -3810,7 +3810,7 @@ int drm_add_modes_noedid(struct drm_connector *connector, > > struct drm_display_mode *mode; > > struct drm_device *dev = connector->dev; > > > > - count = sizeof(drm_dmt_modes) / sizeof(struct drm_display_mode); > > + count = ARRAY_SIZE(drm_dmt_modes); > > if (hdisplay < 0) > > hdisplay = 0; > > if (vdisplay < 0) > > > Reviewed-by: Thierry Reding <treding@xxxxxxxxxx> Thanks for the review, applied to drm-misc. -Danie -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel