On Tue, Jun 17, 2025 at 09:31:52AM +0200, Thomas Weißschuh wrote: >Please use "selftests: harness:" as subject prefix for the patches. >Also mention the specific typo to make the subject more unique. >
Thanks. Will adjust it. >On Mon, Jun 16, 2025 at 12:23:36PM +0000, Wei Yang wrote: >> The name is __constructor_order_forward. >> >> Just correct it. >> >> Signed-off-by: Wei Yang <richard.weiy...@gmail.com> > >With the above comments addressed: >Reviewed-by: Thomas Weißschuh <thomas.weisssc...@linutronix.de> > >> --- >> tools/testing/selftests/kselftest_harness.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tools/testing/selftests/kselftest_harness.h >> b/tools/testing/selftests/kselftest_harness.h >> index 2925e47db995..674a6112e6e1 100644 >> --- a/tools/testing/selftests/kselftest_harness.h >> +++ b/tools/testing/selftests/kselftest_harness.h >> @@ -936,7 +936,7 @@ static inline bool __test_passed(struct __test_metadata >> *metadata) >> * list so tests are run in source declaration order. >> * https://gcc.gnu.org/onlinedocs/gccint/Initialization.html >> * However, it seems not all toolchains do this correctly, so use >> - * __constructor_order_foward to detect which direction is called first >> + * __constructor_order_forward to detect which direction is called first >> * and adjust list building logic to get things running in the right >> * direction. >> */ >> -- >> 2.34.1 >> -- Wei Yang Help you, Help me
