Re: [PATCH] drm: Fix kerneldoc warns in connector-related docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Aug 09, 2019 at 10:15:51PM +0200, Sam Ravnborg wrote:
> 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?

I'm on 1.7.9. I just rechecked and was a bit confused in my last mail. The
drm_atomic_get_*_connector_for_encoder links do work with (), it's the ones
drm_connector_helper_funcs in the paragraph above that need the '&'. So I'll
switch up these 2 and leave the others as-is. Cool?

Sean

> 
> 	Sam

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux