On Mon, 22 Feb 2016, Christopher S. Hall wrote:
> +{
> +     struct timekeeper *tk = &tk_core.timekeeper;
> +     bool interp_forward;
> +     u64 corr_raw, corr_real;
> +     int ret;

Once more:

        struct timekeeper *tk = &tk_core.timekeeper;
        u64 corr_raw, corr_real;
        bool interp_forward;
        int ret;

Is way simpler to parse fast.

>  {
>       struct timekeeper *tk = &tk_core.timekeeper;
> @@ -929,6 +1046,12 @@ int get_device_system_crosststamp(int (*get_time_fn)
>       ktime_t base_real;
>       s64 nsec_raw;
>       s64 nsec_real;
> +     cycles_t cycles;
> +     cycle_t now;
> +     cycle_t interval_start;
> +     unsigned int clock_was_set_seq;
> +     u8 cs_was_changed_seq;
> +     bool do_interp;

Single lines for same types .....

Other than that:

Reviewed-by: Thomas Gleixner <t...@linutronix.de>

Reply via email to