Hello,
(...)
@@ -232,7 +232,7 @@ void __kunit_test_suites_exit(struct kunit_suite **suites);
- kunit_test_suites() - used to register one or more &struct kunit_suite
with KUnit.
- @suites: a statically allocated list of &struct kunit_suite.
- suites - a statically allocated list of &struct kunit_suite.
So, I am pretty sure you can name the variadic arguments and then that gives you a valid parameter to use with kernel doc. Can you try that out?
You mean the warning "Excess function parameter 'suites' description in 'kunit_test_suites'"?
Yep, I just tried it out locally and it should work.
Something like '@...:' should be possible, to list the variadic under "Parameters", and display the corresponding description.
For curiosity I went through the kunit docs and fixed some formatting issues. Great piece of work, thanks for realizing kunit!
Last but not least, going through the document, there is a chapter "API" consisting of a single page with one link to "API". Is this on purpose? I would suggest to take out this level of indirection and remove the .rst file, linking API directly.
I'll send my proposals in a PATCH v2. Let me know then what you think after that.
Best, L