[PATCH] drm: Ignore drm_gem_vram_mm_funcs in generated documentation

Gerd Hoffmann kraxel at redhat.com
Thu Jun 6 07:45:15 UTC 2019


On Thu, Jun 06, 2019 at 08:39:12AM +0200, Thomas Zimmermann wrote:
> Hi
> 
> Am 05.06.19 um 17:58 schrieb Gerd Hoffmann:
> > On Wed, Jun 05, 2019 at 11:59:04AM +0200, Thomas Zimmermann wrote:
> >> Hi
> >>
> >> Am 05.06.19 um 11:03 schrieb Gerd Hoffmann:
> >>> On Tue, Jun 04, 2019 at 01:13:30PM +0200, Thomas Zimmermann wrote:
> >>>> The documentation tools interpret drm_gem_vram_mm_funcs as function and
> >>>> there appears to be no way of inline-documenting constants.
> >>>
> >>>> -/**
> >>>> +/*
> >>>>   * drm_gem_vram_mm_funcs - Functions for &struct drm_vram_mm
> >>>
> >>> "struct drm_gem_vram_mm_funcs" ?
> >>>
> >>> (see include/drm/drm_gem.h where struct drm_gem_object_funcs is
> >>> documented that way).
> >>
> >> The documented source line is
> >>
> >>   const struct drm_vram_mm_funcs drm_gem_vram_mm_funcs = {
> >>
> >> and I tried to document the purpose of the constant 'drm_gem_vram_mm_funcs'.
> > 
> > Ah.  Missed that detail.
> > 
> >> Documenting global constants is not described in the kernel guide and I
> >> could not find a source-code example either. Function and struct style
> >> with '-' didn't work; member-field style with '@' neither. The only way
> >> seems to be adding it to an .rst file in Documentation. But then it's
> >> not part of the API documentation, but somewhere in the text.
> > 
> > Ok, patch is fine then.
> > 
> > Acked-by: Gerd Hoffmann <kraxel at redhat.com>
> 
> Thanks.
> 
> > (can you commit & push yourself meanwhile?)
> 
> Not yet. The respective bug report is still open.

Ok, I'll push it then.

cheers,
  Gerd



More information about the dri-devel mailing list