Re: [PATCH v2] clarify usage expectations for cnt32_to_63()

Russell King
2008-11-09T13:43:03Z

On Sun, Nov 09, 2008 at 08:34:23AM -0500, Nicolas Pitre wrote:
> Do you really have such instances where multiple call sites are needed?
> That sounds even more confusing to me than the current model. Better
> encapsulate the usage of this macro within some function which has a
> stronger meaning, such as sched_clock(), and call _that_ from multiple
> sites instead.

What if sched_clock() is inline and uses cnt32_to_63()? I think that's
where the problem lies. Better add a comment that it shouldn't be used
inside another inline function.

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/