Please use "selftests: harness:" as subject prefix for the patches. Also mention the specific typo to make the subject more unique.
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 >
