Hi Ashwin/Timur,
Unless Rafael thinks otherwise, I see no major issues in V2, so there is no need for a respin.
I think stripping away the __iomem is wrong. The whole point behind the 'sparse' tool is to catch invalid accesses to I/O memory. When you typecast it away, then prevent sparse from catching those problem.
Thanks for your inputs. I will wait for few more days for feedback from Rafael and others before another respin with the suggested changes.
Thanks, Prashanth