On Wed, Dec 06, 2023, Andrew Jones wrote: > @@ -162,7 +162,7 @@ static void check_clocksource(void) > goto out; > } > > - TEST_ASSERT(!strncmp(clk_name, "tsc\n", st.st_size), > + TEST_ASSERT(!strncmp(clk_name, "tsc", st.st_size), This newline is functionally necessary. It's in the strncmp() (*#$@ sysfs appends newlines to everything), not the TEST_ASSERT message. I'll give you a pass and fixup when applying since I'm guessing you don't have x86 hardware ;-) I double checked the other arch patches and didn't see anything sneaky like this.
