I still disagree, but it's not important. IMHO most of the uses of __raw_readl should be converted to readl or readl_relaxed if you are worried about efficiency.
The main difference between __raw_readl and readl_relaxed is that the endianess is well-defined on readl_relaxed.
Arnd
I agree with you about that. I will keep that in mind and also notice my colleages this in our future development.