Hi Linus, Thank you for the patch. On Monday, 29 January 2018 11:55:31 EET Linus Walleij wrote: > Why is the argument to ->mode_valid() called "crtc"? > I guess a copy/paste error. Fix it. I confirm it's a mistake, so you can probably replace "guess" with a word carrying a bit more confidence :) > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > include/drm/drm_bridge.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h > index bb7b97dfb93e..3270fec46979 100644 > --- a/include/drm/drm_bridge.h > +++ b/include/drm/drm_bridge.h > @@ -91,7 +91,7 @@ struct drm_bridge_funcs { > * > * drm_mode_status Enum > */ > - enum drm_mode_status (*mode_valid)(struct drm_bridge *crtc, > + enum drm_mode_status (*mode_valid)(struct drm_bridge *bridge, > const struct drm_display_mode *mode); > > /** -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel