Em Fri, 19 Aug 2016 10:37:50 -0300 Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> escreveu: > Em Fri, 19 Aug 2016 15:24:16 +0200 > Hans Verkuil <hansverk@xxxxxxxxx> escreveu: > > > On 08/19/2016 03:05 PM, Mauro Carvalho Chehab wrote: > > > There are some things there that aren't ok for ReST format. > > > > > > Fix them. > > > > > > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> > > > > OK, so I posted a similar patch for this: > > > > https://patchwork.linuxtv.org/patch/36376/ > > > > It's part of one of my pull requests, but I guess you can skip that patch. > > Sorry, I didn't notice. I'm actually waiting for Jon to merge the > remaining patches for the Sphinx rst core before starting handling > and merging our patches. I want to avoid having a separate branch for > documentation this time. > > IMHO, my approach is better than you for functions, as it declares > functions and struct using the proper macros that are also used > by kernel-doc script. > > Yet, I didn't touch at MAINTAINERS nor I used ``constant`` or > :c:type:/:c:func: for references of the functions/struct. > Would you care to rebase your patch on the top of mine doing such > changes? Hmm... actually we should do something else... As those functions are declared (or should be declared) using the kernel documentation macros, we should have references for them already. So, we need to actually change the text there to point to the references created by kernel-doc script. like: https://linuxtv.org/downloads/v4l-dvb-apis-new/_sources/media/kapi/v4l2-dev.txt Thanks, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
