On Wed, 19 Oct 2016 20:38:39 +0200 Silvio Fricke <silvio.fricke@xxxxxxxxx> wrote: > This patch add a "misc" documentation section and add the workqueue Thanks for doing this; can I add a few comments? - Ccing the docs maintainer is considered good form with documentation patches :) That's especially true of anything touching index.rst, which has become a bit of a conflict point. - On a quick read, the conversion of workqueue.txt seems good. - I don't like the "misc" thing, though. As we convert the documentation over, we want to impose a bit more organization on it, both in the documentation structure and in the filesystem structure. That means moving a lot of the stuff out of the top-level Documentation/ directory, which is an unmanageable mess at the moment. So one thing I would really rather not see is people creating index.rst files with "../" in them. The file hierarchy should match the document hierarchy. For 4.9 we created the driver-api manual for stuff that's clearly driver-specific. My thinking had been to make a core-api equivalent for stuff that's relevant outside of the driver code; workqueue.txt clearly qualifies there. So I guess what I'd like to see is something very similar to what you've done, but can we start the core-api manual, and put the RST file as core-api/workqueue.rst? The core-api manual itself doesn't really need to be any more than what you did for misc/ at this point. That done, if it's OK with you, Tejun, I'd prefer to take it through the docs tree with your ack so I don't have to write yet another note to Linus explaining the index.rst conflicts... Sound OK? Thanks, jon -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
