Hi Sean.
--- a/include/drm/drm_connector.h +++ b/include/drm/drm_connector.h @@ -543,8 +543,8 @@ struct drm_connector_state { * * This is also used in the atomic helpers to map encoders to their * current and previous connectors, see
* &drm_atomic_get_old_connector_for_encoder() and* &drm_atomic_get_new_connector_for_encoder().
* &drm_atomic_get_old_connector_for_encoder and* &drm_atomic_get_new_connector_for_encoder.Please fix this to use () for the functions and drop the "&". This is what we use in drm kernel-doc for functions. See for example function references in doc of writeback_job in the same file.
Doing this won't get a hyperlink in the docs. It seems like these hooks aren't recognized as functions by sphinx (not sure didn't look into it too deeply). The other "_funcs.*" hooks are also handled with '&' (there are lots of examples in drm_connector.h).
I think preserving the hyperlinks probably outweighs the missing (), thoughts?
Hmm, I actually tested it here - with sphinx_1.4.9. The links was preserved, the only difference was that they had the () prefixed to their name.
Do you happen to use an older sphinx version?
Sam