Messages in this thread | | | Date | Fri, 13 Feb 2015 08:29:13 +0800 | Subject | Re: [PATCH 2/6] timekeeping: Pass readout base to update_fast_timekeeper() | From | John Stultz <> |
| |
On Wed, Feb 11, 2015 at 12:01 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Modify update_fast_timekeeper() to take a struct tk_read_base > pointer as its argument (instead of a struct timekeeper pointer) > and update its kerneldoc comment to reflect that. > > That will allow a struct tk_read_base that is not part of a > struct timekeeper to be passed to it in the next patch. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Thomas was really behind the update_fast_timekeeper design so I'm not sure I can critique if this fits in the larger vision here, but I don't see anything wrong and its not breaking in any of my tests, so...
Acked-by: John Stultz <john.stultz@linaro.org>
thanks -john
| |