Commit fe9e557d authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Andrzej Hajda

drm/bridge: Fix references to drm_bridge_funcs in documentation

The documentation for most of the bridge atomic operations incorrectly
reference the non-atomic version when stating the operations are
optional. Fix them, and make sure that all references to bridge
operations are correctly marked with @.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: default avatarAndrzej Hajda <a.hajda@samsung.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190821235502.15475-1-laurent.pinchart+renesas@ideasonboard.com
parent 3250cdf9
...@@ -42,7 +42,7 @@ struct drm_bridge_funcs { ...@@ -42,7 +42,7 @@ struct drm_bridge_funcs {
* This callback is invoked whenever our bridge is being attached to a * This callback is invoked whenever our bridge is being attached to a
* &drm_encoder. * &drm_encoder.
* *
* The attach callback is optional. * The @attach callback is optional.
* *
* RETURNS: * RETURNS:
* *
...@@ -56,7 +56,7 @@ struct drm_bridge_funcs { ...@@ -56,7 +56,7 @@ struct drm_bridge_funcs {
* This callback is invoked whenever our bridge is being detached from a * This callback is invoked whenever our bridge is being detached from a
* &drm_encoder. * &drm_encoder.
* *
* The detach callback is optional. * The @detach callback is optional.
*/ */
void (*detach)(struct drm_bridge *bridge); void (*detach)(struct drm_bridge *bridge);
...@@ -76,7 +76,7 @@ struct drm_bridge_funcs { ...@@ -76,7 +76,7 @@ struct drm_bridge_funcs {
* atomic helpers to validate modes supplied by userspace in * atomic helpers to validate modes supplied by userspace in
* drm_atomic_helper_check_modeset(). * drm_atomic_helper_check_modeset().
* *
* This function is optional. * The @mode_valid callback is optional.
* *
* NOTE: * NOTE:
* *
...@@ -108,7 +108,7 @@ struct drm_bridge_funcs { ...@@ -108,7 +108,7 @@ struct drm_bridge_funcs {
* this function passes all other callbacks must succeed for this * this function passes all other callbacks must succeed for this
* configuration. * configuration.
* *
* The mode_fixup callback is optional. * The @mode_fixup callback is optional.
* *
* NOTE: * NOTE:
* *
...@@ -146,7 +146,7 @@ struct drm_bridge_funcs { ...@@ -146,7 +146,7 @@ struct drm_bridge_funcs {
* The bridge can assume that the display pipe (i.e. clocks and timing * The bridge can assume that the display pipe (i.e. clocks and timing
* signals) feeding it is still running when this callback is called. * signals) feeding it is still running when this callback is called.
* *
* The disable callback is optional. * The @disable callback is optional.
*/ */
void (*disable)(struct drm_bridge *bridge); void (*disable)(struct drm_bridge *bridge);
...@@ -165,7 +165,7 @@ struct drm_bridge_funcs { ...@@ -165,7 +165,7 @@ struct drm_bridge_funcs {
* singals) feeding it is no longer running when this callback is * singals) feeding it is no longer running when this callback is
* called. * called.
* *
* The post_disable callback is optional. * The @post_disable callback is optional.
*/ */
void (*post_disable)(struct drm_bridge *bridge); void (*post_disable)(struct drm_bridge *bridge);
...@@ -214,7 +214,7 @@ struct drm_bridge_funcs { ...@@ -214,7 +214,7 @@ struct drm_bridge_funcs {
* not enable the display link feeding the next bridge in the chain (if * not enable the display link feeding the next bridge in the chain (if
* there is one) when this callback is called. * there is one) when this callback is called.
* *
* The pre_enable callback is optional. * The @pre_enable callback is optional.
*/ */
void (*pre_enable)(struct drm_bridge *bridge); void (*pre_enable)(struct drm_bridge *bridge);
...@@ -234,7 +234,7 @@ struct drm_bridge_funcs { ...@@ -234,7 +234,7 @@ struct drm_bridge_funcs {
* callback must enable the display link feeding the next bridge in the * callback must enable the display link feeding the next bridge in the
* chain if there is one. * chain if there is one.
* *
* The enable callback is optional. * The @enable callback is optional.
*/ */
void (*enable)(struct drm_bridge *bridge); void (*enable)(struct drm_bridge *bridge);
...@@ -283,7 +283,7 @@ struct drm_bridge_funcs { ...@@ -283,7 +283,7 @@ struct drm_bridge_funcs {
* would be prudent to also provide an implementation of @enable if * would be prudent to also provide an implementation of @enable if
* you are expecting driver calls into &drm_bridge_enable. * you are expecting driver calls into &drm_bridge_enable.
* *
* The enable callback is optional. * The @atomic_enable callback is optional.
*/ */
void (*atomic_enable)(struct drm_bridge *bridge, void (*atomic_enable)(struct drm_bridge *bridge,
struct drm_atomic_state *state); struct drm_atomic_state *state);
...@@ -305,7 +305,7 @@ struct drm_bridge_funcs { ...@@ -305,7 +305,7 @@ struct drm_bridge_funcs {
* would be prudent to also provide an implementation of @disable if * would be prudent to also provide an implementation of @disable if
* you are expecting driver calls into &drm_bridge_disable. * you are expecting driver calls into &drm_bridge_disable.
* *
* The disable callback is optional. * The @atomic_disable callback is optional.
*/ */
void (*atomic_disable)(struct drm_bridge *bridge, void (*atomic_disable)(struct drm_bridge *bridge,
struct drm_atomic_state *state); struct drm_atomic_state *state);
...@@ -330,7 +330,7 @@ struct drm_bridge_funcs { ...@@ -330,7 +330,7 @@ struct drm_bridge_funcs {
* @post_disable if you are expecting driver calls into * @post_disable if you are expecting driver calls into
* &drm_bridge_post_disable. * &drm_bridge_post_disable.
* *
* The post_disable callback is optional. * The @atomic_post_disable callback is optional.
*/ */
void (*atomic_post_disable)(struct drm_bridge *bridge, void (*atomic_post_disable)(struct drm_bridge *bridge,
struct drm_atomic_state *state); struct drm_atomic_state *state);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment