On 9/5/24 02:52, zhangjiao2 wrote:
From: zhang jiao zhangjiao2@cmss.chinamobile.com
This macro NSEC_PER_SEC is never referenced in the code. Just remove it.
Is this duplicate patch?
I think I commented on your patch on futex - include how you found the problem in change logs. Also you have to include subsystem prefix in the subject line:
selftests/timers: Remove unused NSEC_PER_SEC macro
I see another patch with similar problems from you. Refer to submitting patches document in the kernel repo.
Signed-off-by: zhang jiao zhangjiao2@cmss.chinamobile.com
tools/testing/selftests/timers/change_skew.c | 3 --- 1 file changed, 3 deletions(-)
diff --git a/tools/testing/selftests/timers/change_skew.c b/tools/testing/selftests/timers/change_skew.c index 4421cd562c24..18e794a46c23 100644 --- a/tools/testing/selftests/timers/change_skew.c +++ b/tools/testing/selftests/timers/change_skew.c @@ -30,9 +30,6 @@ #include <time.h> #include "../kselftest.h" -#define NSEC_PER_SEC 1000000000LL
- int change_skew_test(int ppm) { struct timex tx;
thanks, -- Shuah