On Sat, 18 Mar 2017, Deepa Dinamani wrote: > - int (*clock_get) (const clockid_t which_clock, struct timespec * tp); > + int (*clock_get) (const clockid_t which_clock, struct timespec64 * tp);
If you change things, then please fixup the white space noise as well '*tp' is the proper syntax. > @@ -1033,12 +1033,14 @@ SYSCALL_DEFINE2(clock_gettime, const clockid_t, > which_clock, > { > struct k_clock *kc = clockid_to_kclock(which_clock); > struct timespec kernel_tp; > + struct timespec64 kernel_tp64; Ordering please Other than the nitpicks, this looks good.