On Thu, Nov 28, 2013 at 03:29:17PM +0000, Damien Lespiau wrote: > To make it clear what exactly mode_valid() should return. > > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > --- > include/drm/drm_crtc_helper.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/drm/drm_crtc_helper.h b/include/drm/drm_crtc_helper.h > index ef6ad3a..b1388b5 100644 > --- a/include/drm/drm_crtc_helper.h > +++ b/include/drm/drm_crtc_helper.h > @@ -120,8 +120,8 @@ struct drm_encoder_helper_funcs { > */ > struct drm_connector_helper_funcs { > int (*get_modes)(struct drm_connector *connector); > - int (*mode_valid)(struct drm_connector *connector, > - struct drm_display_mode *mode); > + enum drm_mode_status (*mode_valid)(struct drm_connector *connector, > + struct drm_display_mode *mode); > struct drm_encoder *(*best_encoder)(struct drm_connector *connector); > }; > > -- > 1.8.3.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx