On Tue, Nov 15, 2016 at 10:37:26AM +0000, Chris Wilson wrote: > On Mon, Nov 14, 2016 at 12:58:22PM +0100, Daniel Vetter wrote: > > kerneldoc expects the comment next to definitions, otherwise it can't > > pick up exported vs. internal stuff. > > > > This fixes a warning from the doc build done with: > > > > $ make DOCBOOKS="" htmldocs > > > > Fixes: d8187177b0b1 ("drm: add helper for printing to log or seq_file") > > Cc: Rob Clark <robdclark@xxxxxxxxx> > > Cc: Sean Paul <seanpaul@xxxxxxxxxxxx> > > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > > Oh well, I liked the practice of having interface descriptions in the > headers. If they are in the body, I may as well just read the code. I'm divided. On one hand it makes it more easily readable for users of the code&functions. Otoh having it closer might increase the odds that it's not forgotten when the semantics change. Personally I just have lots of windows open, with both code and rendered docs ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx