On Thu, 25 Sep 2014 18:54:40 +0100 Rob Jones <rob.jones@xxxxxxxxxxxxxxx> wrote: > On 25/09/14 18:50, Andrew Morton wrote: > > On Thu, 25 Sep 2014 10:10:05 +0100 Rob Jones <rob.jones@xxxxxxxxxxxxxxx> wrote: > > > >>> A global exported-to-modules interface should be documented, please. > >>> Especially when it has a void* argument. seq_file.c is patchy - some > >>> of it is documented, some of it uses the read-programmers-mind > >>> approach. > >> > >> I have included documentation as the second patch. Would it have been > >> better to include them in a single patch? I didn't do that because > >> seq_file and Documentation have different maintainers. I'm still > >> learning the protocols here. > > > > A single patch would be OK. > > > > Documentation/ is nice, but I don't think people think to look there. > > Some kerneldoc within the .c would be a good addition. > > Now is a good time, can you point me at an instance of good practice of > this? Search fs/seq_file.c for "^/**"? -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
