On Thu, 17 Nov 2016, Jani Nikula <jani.nik...@intel.com> wrote: > On Thu, 17 Nov 2016, Daniel Vetter <daniel.vet...@ffwll.ch> wrote: >> We don't just need better doc toolchains, we also need better docs for >> our doc toolchain! > > Mea culpa. Thanks, LGTM.
Oh, the example struct now has too "foo" fields, documented twice. That's not good. BR, Jani. > > BR, > Jani. > > >> Cc: Daniel Vetter <dan...@ffwll.ch> >> Cc: Jani Nikula <jani.nik...@intel.com> >> Cc: Jonathan Corbet <cor...@lwn.net> >> Cc: linux-doc@vger.kernel.org >> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com> >> --- >> Documentation/kernel-documentation.rst | 7 ++++++- >> 1 file changed, 6 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/kernel-documentation.rst >> b/Documentation/kernel-documentation.rst >> index 10cc7ddb6235..a5bd12d8bd4e 100644 >> --- a/Documentation/kernel-documentation.rst >> +++ b/Documentation/kernel-documentation.rst >> @@ -484,7 +484,10 @@ span multiple lines. The continuation lines may contain >> indentation. >> In-line member documentation comments >> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >> >> -The structure members may also be documented in-line within the definition:: >> +The structure members may also be documented in-line within the definition. >> +There are two styles, single-line comments where both the opening ``/**`` >> and >> +closing ``*/`` are on the same line, and multi-line comments where they are >> each >> +on a line of their own, like all other kernel-doc comments:: >> >> /** >> * struct foo - Brief description. >> @@ -502,6 +505,8 @@ The structure members may also be documented in-line >> within the definition:: >> * Here, the member description may contain several paragraphs. >> */ >> int baz; >> + /** @foo: Single line description. */ >> + int foo; >> } >> >> Private members -- Jani Nikula, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
