Re: [PATCH v2] ARM: tlb: ASID macro should give 32bit result for BE correct operation

On 7 October 2013 08:52, Russell King - ARM Linux linux@arm.linux.org.uk wrote:

On Mon, Oct 07, 2013 at 08:48:23AM -0700, Victor Kamensky wrote:

...

-#define ASID(mm) ((mm)->context.id.counter & ~ASID_MASK) +#define ASID(mm) ((unsigned int)((mm)->context.id.counter & ~ASID_MASK))

Not a big problem, but "unsigned" is sufficient as per my suggestion. See:

    http://en.wikipedia.org/wiki/C_data_types

or look it up in the C standards. :) I'll take it either way.

Understood, I just tried to be consistent with the rest of file - 'unsigned int' was used just few lines above place that got changed.

Thanks, Victor