Re: [RFC][patch 10/12] move NTP adjusted clock multiplier to struct timekeeper

From: Linus Walleij
Date: Thu Aug 13 2009 - 07:15:22 EST


2009/7/29 Martin Schwidefsky <schwidefsky@xxxxxxxxxx>:

> Index: linux-2.6/arch/arm/plat-omap/common.c
> ===================================================================
> --- linux-2.6.orig/arch/arm/plat-omap/common.c
> +++ linux-2.6/arch/arm/plat-omap/common.c
> @@ -256,7 +256,7 @@ unsigned long long sched_clock(void)
> unsigned long long ret;
>
> ret = (unsigned long long)clocksource_32k.read(&clocksource_32k);
> - ret = (ret * clocksource_32k.mult_orig) >> clocksource_32k.shift;
> + ret = (ret * clocksource_32k.mult) >> clocksource_32k.shift;

Doesn't the overall effect of your patch mean that the above can simply be
replaced by

ret = cyc2ns(&clocksource_32k, ret);

Or clocksource_cyc2ns() if that was renamed (I don't have the entire
picture here...)

> return ret;
> }

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/