On Thu 2024-12-05 17:17:55, Siddharth Menon wrote: > Update documentation to explain the TEST_CONFIG_DEPS flag in lib.mk. > TEST_CONFIG_DEPS is used to validate the presence of required config flags > specified in the selftest makefile before compiling or running a test. > > Signed-off-by: Siddharth Menon <simeddon@xxxxxxxxx> > --- > Documentation/dev-tools/kselftest.rst | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/dev-tools/kselftest.rst b/Documentation/dev-tools/kselftest.rst > index fdb1df86783a..e816b282363f 100644 > --- a/Documentation/dev-tools/kselftest.rst > +++ b/Documentation/dev-tools/kselftest.rst > @@ -301,6 +301,9 @@ Contributing new tests (details) > > e.g: tools/testing/selftests/android/config > > + * Use TEST_CONFIG_DEPS to specify required config options to be enabled > + before a test is allowed to run or compile. > + > * Create a .gitignore file inside test directory and add all generated objects > in it. It might be a matter of taste. It is a chicken & egg problem. I personally find it weird to document something which does not exist yet. Please, either update the documentation together with the code or later :-) Best Regards, Petr PS: I haven't got this mail. I have got only 2nd and 3rd patch. I prefer to see the entire patchset. I suggest to always send all patches to the same list of people and mailing lists.