With this in mind, I think readl_relaxed() should be better here.
Stanley,
Actually we never thought of endianness while coding it, and
followed the most generic approach taken by other archs.
Yes, i do accept *_relaxed() should be used instead.
I will do that in V4.
--
viresh