Message ID | 1456480266-7904-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Fri, Feb 26, 2016 at 11:51:06AM +0200, Laurent Pinchart wrote: > Instead of forcing bridges to implement empty callbacks make them all > optional. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > drivers/gpu/drm/drm_bridge.c | 12 ++++++++---- > include/drm/drm_crtc.h | 8 ++++++++ > 2 files changed, 16 insertions(+), 4 deletions(-) Applied, thanks. Thierry
On Fri, Feb 26, 2016 at 11:51:06AM +0200, Laurent Pinchart wrote: > Instead of forcing bridges to implement empty callbacks make them all > optional. > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Care to go around the tree a nuke them all? On a quick grep I spotted at least some in msm. -Daniel > --- > drivers/gpu/drm/drm_bridge.c | 12 ++++++++---- > include/drm/drm_crtc.h | 8 ++++++++ > 2 files changed, 16 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c > index bd93453afa61..b3654404abd0 100644 > --- a/drivers/gpu/drm/drm_bridge.c > +++ b/drivers/gpu/drm/drm_bridge.c > @@ -186,7 +186,8 @@ void drm_bridge_disable(struct drm_bridge *bridge) > > drm_bridge_disable(bridge->next); > > - bridge->funcs->disable(bridge); > + if (bridge->funcs->disable) > + bridge->funcs->disable(bridge); > } > EXPORT_SYMBOL(drm_bridge_disable); > > @@ -206,7 +207,8 @@ void drm_bridge_post_disable(struct drm_bridge *bridge) > if (!bridge) > return; > > - bridge->funcs->post_disable(bridge); > + if (bridge->funcs->post_disable) > + bridge->funcs->post_disable(bridge); > > drm_bridge_post_disable(bridge->next); > } > @@ -256,7 +258,8 @@ void drm_bridge_pre_enable(struct drm_bridge *bridge) > > drm_bridge_pre_enable(bridge->next); > > - bridge->funcs->pre_enable(bridge); > + if (bridge->funcs->pre_enable) > + bridge->funcs->pre_enable(bridge); > } > EXPORT_SYMBOL(drm_bridge_pre_enable); > > @@ -276,7 +279,8 @@ void drm_bridge_enable(struct drm_bridge *bridge) > if (!bridge) > return; > > - bridge->funcs->enable(bridge); > + if (bridge->funcs->enable) > + bridge->funcs->enable(bridge); > > drm_bridge_enable(bridge->next); > } > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > index 7fad193dc645..fbc225414f01 100644 > --- a/include/drm/drm_crtc.h > +++ b/include/drm/drm_crtc.h > @@ -1584,6 +1584,8 @@ struct drm_bridge_funcs { > * > * The bridge can assume that the display pipe (i.e. clocks and timing > * signals) feeding it is still running when this callback is called. > + * > + * The disable callback is optional. > */ > void (*disable)(struct drm_bridge *bridge); > > @@ -1600,6 +1602,8 @@ struct drm_bridge_funcs { > * The bridge must assume that the display pipe (i.e. clocks and timing > * singals) feeding it is no longer running when this callback is > * called. > + * > + * The post_disable callback is optional. > */ > void (*post_disable)(struct drm_bridge *bridge); > > @@ -1628,6 +1632,8 @@ struct drm_bridge_funcs { > * will not yet be running when this callback is called. The bridge must > * not enable the display link feeding the next bridge in the chain (if > * there is one) when this callback is called. > + * > + * The pre_enable callback is optional. > */ > void (*pre_enable)(struct drm_bridge *bridge); > > @@ -1645,6 +1651,8 @@ struct drm_bridge_funcs { > * signals) feeding it is running when this callback is called. This > * callback must enable the display link feeding the next bridge in the > * chain if there is one. > + * > + * The enable callback is optional. > */ > void (*enable)(struct drm_bridge *bridge); > }; > -- > Regards, > > Laurent Pinchart > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
On 02/26/2016 03:21 PM, Laurent Pinchart wrote: > Instead of forcing bridges to implement empty callbacks make them all > optional. Acked-by: Archit Taneja <architt@codeaurora.org> > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > --- > drivers/gpu/drm/drm_bridge.c | 12 ++++++++---- > include/drm/drm_crtc.h | 8 ++++++++ > 2 files changed, 16 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c > index bd93453afa61..b3654404abd0 100644 > --- a/drivers/gpu/drm/drm_bridge.c > +++ b/drivers/gpu/drm/drm_bridge.c > @@ -186,7 +186,8 @@ void drm_bridge_disable(struct drm_bridge *bridge) > > drm_bridge_disable(bridge->next); > > - bridge->funcs->disable(bridge); > + if (bridge->funcs->disable) > + bridge->funcs->disable(bridge); > } > EXPORT_SYMBOL(drm_bridge_disable); > > @@ -206,7 +207,8 @@ void drm_bridge_post_disable(struct drm_bridge *bridge) > if (!bridge) > return; > > - bridge->funcs->post_disable(bridge); > + if (bridge->funcs->post_disable) > + bridge->funcs->post_disable(bridge); > > drm_bridge_post_disable(bridge->next); > } > @@ -256,7 +258,8 @@ void drm_bridge_pre_enable(struct drm_bridge *bridge) > > drm_bridge_pre_enable(bridge->next); > > - bridge->funcs->pre_enable(bridge); > + if (bridge->funcs->pre_enable) > + bridge->funcs->pre_enable(bridge); > } > EXPORT_SYMBOL(drm_bridge_pre_enable); > > @@ -276,7 +279,8 @@ void drm_bridge_enable(struct drm_bridge *bridge) > if (!bridge) > return; > > - bridge->funcs->enable(bridge); > + if (bridge->funcs->enable) > + bridge->funcs->enable(bridge); > > drm_bridge_enable(bridge->next); > } > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > index 7fad193dc645..fbc225414f01 100644 > --- a/include/drm/drm_crtc.h > +++ b/include/drm/drm_crtc.h > @@ -1584,6 +1584,8 @@ struct drm_bridge_funcs { > * > * The bridge can assume that the display pipe (i.e. clocks and timing > * signals) feeding it is still running when this callback is called. > + * > + * The disable callback is optional. > */ > void (*disable)(struct drm_bridge *bridge); > > @@ -1600,6 +1602,8 @@ struct drm_bridge_funcs { > * The bridge must assume that the display pipe (i.e. clocks and timing > * singals) feeding it is no longer running when this callback is > * called. > + * > + * The post_disable callback is optional. > */ > void (*post_disable)(struct drm_bridge *bridge); > > @@ -1628,6 +1632,8 @@ struct drm_bridge_funcs { > * will not yet be running when this callback is called. The bridge must > * not enable the display link feeding the next bridge in the chain (if > * there is one) when this callback is called. > + * > + * The pre_enable callback is optional. > */ > void (*pre_enable)(struct drm_bridge *bridge); > > @@ -1645,6 +1651,8 @@ struct drm_bridge_funcs { > * signals) feeding it is running when this callback is called. This > * callback must enable the display link feeding the next bridge in the > * chain if there is one. > + * > + * The enable callback is optional. > */ > void (*enable)(struct drm_bridge *bridge); > }; >
On Thu, Mar 24, 2016 at 12:19:32PM +0530, Archit Taneja wrote: > > > On 02/26/2016 03:21 PM, Laurent Pinchart wrote: > >Instead of forcing bridges to implement empty callbacks make them all > >optional. > > Acked-by: Archit Taneja <architt@codeaurora.org> Applied to drm-misc. Still hoping for follow-ups to nuke the now redundant dummy funcs. Thanks, Daniel > > > > >Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > >--- > > drivers/gpu/drm/drm_bridge.c | 12 ++++++++---- > > include/drm/drm_crtc.h | 8 ++++++++ > > 2 files changed, 16 insertions(+), 4 deletions(-) > > > >diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c > >index bd93453afa61..b3654404abd0 100644 > >--- a/drivers/gpu/drm/drm_bridge.c > >+++ b/drivers/gpu/drm/drm_bridge.c > >@@ -186,7 +186,8 @@ void drm_bridge_disable(struct drm_bridge *bridge) > > > > drm_bridge_disable(bridge->next); > > > >- bridge->funcs->disable(bridge); > >+ if (bridge->funcs->disable) > >+ bridge->funcs->disable(bridge); > > } > > EXPORT_SYMBOL(drm_bridge_disable); > > > >@@ -206,7 +207,8 @@ void drm_bridge_post_disable(struct drm_bridge *bridge) > > if (!bridge) > > return; > > > >- bridge->funcs->post_disable(bridge); > >+ if (bridge->funcs->post_disable) > >+ bridge->funcs->post_disable(bridge); > > > > drm_bridge_post_disable(bridge->next); > > } > >@@ -256,7 +258,8 @@ void drm_bridge_pre_enable(struct drm_bridge *bridge) > > > > drm_bridge_pre_enable(bridge->next); > > > >- bridge->funcs->pre_enable(bridge); > >+ if (bridge->funcs->pre_enable) > >+ bridge->funcs->pre_enable(bridge); > > } > > EXPORT_SYMBOL(drm_bridge_pre_enable); > > > >@@ -276,7 +279,8 @@ void drm_bridge_enable(struct drm_bridge *bridge) > > if (!bridge) > > return; > > > >- bridge->funcs->enable(bridge); > >+ if (bridge->funcs->enable) > >+ bridge->funcs->enable(bridge); > > > > drm_bridge_enable(bridge->next); > > } > >diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > >index 7fad193dc645..fbc225414f01 100644 > >--- a/include/drm/drm_crtc.h > >+++ b/include/drm/drm_crtc.h > >@@ -1584,6 +1584,8 @@ struct drm_bridge_funcs { > > * > > * The bridge can assume that the display pipe (i.e. clocks and timing > > * signals) feeding it is still running when this callback is called. > >+ * > >+ * The disable callback is optional. > > */ > > void (*disable)(struct drm_bridge *bridge); > > > >@@ -1600,6 +1602,8 @@ struct drm_bridge_funcs { > > * The bridge must assume that the display pipe (i.e. clocks and timing > > * singals) feeding it is no longer running when this callback is > > * called. > >+ * > >+ * The post_disable callback is optional. > > */ > > void (*post_disable)(struct drm_bridge *bridge); > > > >@@ -1628,6 +1632,8 @@ struct drm_bridge_funcs { > > * will not yet be running when this callback is called. The bridge must > > * not enable the display link feeding the next bridge in the chain (if > > * there is one) when this callback is called. > >+ * > >+ * The pre_enable callback is optional. > > */ > > void (*pre_enable)(struct drm_bridge *bridge); > > > >@@ -1645,6 +1651,8 @@ struct drm_bridge_funcs { > > * signals) feeding it is running when this callback is called. This > > * callback must enable the display link feeding the next bridge in the > > * chain if there is one. > >+ * > >+ * The enable callback is optional. > > */ > > void (*enable)(struct drm_bridge *bridge); > > }; > > > > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > hosted by The Linux Foundation > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
On 3/29/2016 12:04 PM, Daniel Vetter wrote: > On Thu, Mar 24, 2016 at 12:19:32PM +0530, Archit Taneja wrote: >> >> >> On 02/26/2016 03:21 PM, Laurent Pinchart wrote: >>> Instead of forcing bridges to implement empty callbacks make them all >>> optional. >> >> Acked-by: Archit Taneja <architt@codeaurora.org> > > Applied to drm-misc. Still hoping for follow-ups to nuke the now > redundant dummy funcs. Only the dw-hdmi bridge driver used such nops. I posted a patch for that. Archit > > Thanks, Daniel > >> >>> >>> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> >>> --- >>> drivers/gpu/drm/drm_bridge.c | 12 ++++++++---- >>> include/drm/drm_crtc.h | 8 ++++++++ >>> 2 files changed, 16 insertions(+), 4 deletions(-) >>> >>> diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c >>> index bd93453afa61..b3654404abd0 100644 >>> --- a/drivers/gpu/drm/drm_bridge.c >>> +++ b/drivers/gpu/drm/drm_bridge.c >>> @@ -186,7 +186,8 @@ void drm_bridge_disable(struct drm_bridge *bridge) >>> >>> drm_bridge_disable(bridge->next); >>> >>> - bridge->funcs->disable(bridge); >>> + if (bridge->funcs->disable) >>> + bridge->funcs->disable(bridge); >>> } >>> EXPORT_SYMBOL(drm_bridge_disable); >>> >>> @@ -206,7 +207,8 @@ void drm_bridge_post_disable(struct drm_bridge *bridge) >>> if (!bridge) >>> return; >>> >>> - bridge->funcs->post_disable(bridge); >>> + if (bridge->funcs->post_disable) >>> + bridge->funcs->post_disable(bridge); >>> >>> drm_bridge_post_disable(bridge->next); >>> } >>> @@ -256,7 +258,8 @@ void drm_bridge_pre_enable(struct drm_bridge *bridge) >>> >>> drm_bridge_pre_enable(bridge->next); >>> >>> - bridge->funcs->pre_enable(bridge); >>> + if (bridge->funcs->pre_enable) >>> + bridge->funcs->pre_enable(bridge); >>> } >>> EXPORT_SYMBOL(drm_bridge_pre_enable); >>> >>> @@ -276,7 +279,8 @@ void drm_bridge_enable(struct drm_bridge *bridge) >>> if (!bridge) >>> return; >>> >>> - bridge->funcs->enable(bridge); >>> + if (bridge->funcs->enable) >>> + bridge->funcs->enable(bridge); >>> >>> drm_bridge_enable(bridge->next); >>> } >>> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h >>> index 7fad193dc645..fbc225414f01 100644 >>> --- a/include/drm/drm_crtc.h >>> +++ b/include/drm/drm_crtc.h >>> @@ -1584,6 +1584,8 @@ struct drm_bridge_funcs { >>> * >>> * The bridge can assume that the display pipe (i.e. clocks and timing >>> * signals) feeding it is still running when this callback is called. >>> + * >>> + * The disable callback is optional. >>> */ >>> void (*disable)(struct drm_bridge *bridge); >>> >>> @@ -1600,6 +1602,8 @@ struct drm_bridge_funcs { >>> * The bridge must assume that the display pipe (i.e. clocks and timing >>> * singals) feeding it is no longer running when this callback is >>> * called. >>> + * >>> + * The post_disable callback is optional. >>> */ >>> void (*post_disable)(struct drm_bridge *bridge); >>> >>> @@ -1628,6 +1632,8 @@ struct drm_bridge_funcs { >>> * will not yet be running when this callback is called. The bridge must >>> * not enable the display link feeding the next bridge in the chain (if >>> * there is one) when this callback is called. >>> + * >>> + * The pre_enable callback is optional. >>> */ >>> void (*pre_enable)(struct drm_bridge *bridge); >>> >>> @@ -1645,6 +1651,8 @@ struct drm_bridge_funcs { >>> * signals) feeding it is running when this callback is called. This >>> * callback must enable the display link feeding the next bridge in the >>> * chain if there is one. >>> + * >>> + * The enable callback is optional. >>> */ >>> void (*enable)(struct drm_bridge *bridge); >>> }; >>> >> >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> hosted by The Linux Foundation >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel >
diff --git a/drivers/gpu/drm/drm_bridge.c b/drivers/gpu/drm/drm_bridge.c index bd93453afa61..b3654404abd0 100644 --- a/drivers/gpu/drm/drm_bridge.c +++ b/drivers/gpu/drm/drm_bridge.c @@ -186,7 +186,8 @@ void drm_bridge_disable(struct drm_bridge *bridge) drm_bridge_disable(bridge->next); - bridge->funcs->disable(bridge); + if (bridge->funcs->disable) + bridge->funcs->disable(bridge); } EXPORT_SYMBOL(drm_bridge_disable); @@ -206,7 +207,8 @@ void drm_bridge_post_disable(struct drm_bridge *bridge) if (!bridge) return; - bridge->funcs->post_disable(bridge); + if (bridge->funcs->post_disable) + bridge->funcs->post_disable(bridge); drm_bridge_post_disable(bridge->next); } @@ -256,7 +258,8 @@ void drm_bridge_pre_enable(struct drm_bridge *bridge) drm_bridge_pre_enable(bridge->next); - bridge->funcs->pre_enable(bridge); + if (bridge->funcs->pre_enable) + bridge->funcs->pre_enable(bridge); } EXPORT_SYMBOL(drm_bridge_pre_enable); @@ -276,7 +279,8 @@ void drm_bridge_enable(struct drm_bridge *bridge) if (!bridge) return; - bridge->funcs->enable(bridge); + if (bridge->funcs->enable) + bridge->funcs->enable(bridge); drm_bridge_enable(bridge->next); } diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index 7fad193dc645..fbc225414f01 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -1584,6 +1584,8 @@ struct drm_bridge_funcs { * * The bridge can assume that the display pipe (i.e. clocks and timing * signals) feeding it is still running when this callback is called. + * + * The disable callback is optional. */ void (*disable)(struct drm_bridge *bridge); @@ -1600,6 +1602,8 @@ struct drm_bridge_funcs { * The bridge must assume that the display pipe (i.e. clocks and timing * singals) feeding it is no longer running when this callback is * called. + * + * The post_disable callback is optional. */ void (*post_disable)(struct drm_bridge *bridge); @@ -1628,6 +1632,8 @@ struct drm_bridge_funcs { * will not yet be running when this callback is called. The bridge must * not enable the display link feeding the next bridge in the chain (if * there is one) when this callback is called. + * + * The pre_enable callback is optional. */ void (*pre_enable)(struct drm_bridge *bridge); @@ -1645,6 +1651,8 @@ struct drm_bridge_funcs { * signals) feeding it is running when this callback is called. This * callback must enable the display link feeding the next bridge in the * chain if there is one. + * + * The enable callback is optional. */ void (*enable)(struct drm_bridge *bridge); };
Instead of forcing bridges to implement empty callbacks make them all optional. Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> --- drivers/gpu/drm/drm_bridge.c | 12 ++++++++---- include/drm/drm_crtc.h | 8 ++++++++ 2 files changed, 16 insertions(+), 4 deletions(-)