On 12/14/2016 10:39 AM, arvind Yadav wrote:
Hi David,Your patch subject should also be changed to insert spaces between
I have gave my comment.
Thanks
Arvind
On Wednesday 14 December 2016 11:44 PM, David Daney wrote:
On 12/14/2016 10:06 AM, arvind Yadav wrote:-I just gave as example where i have seen ioremap issue.
Yes, I have seen this error. We have a device with very less memory.This makes absolutely no sense to me. OCTEON is a mips64 SoC with a
Basically it's OMAP2 board. We have to port Android L on this.
It's has 3.10 kernel version. In this device, we were getting Page
allocation failure.
ton of memory where ioremap can never fail, and it doesn't run
Android, and you are talking about OMAP2.
Please don't relate. I know, Now it will not fail. ioremap will through
NULL on failure. We should catch this error. Even other driver of MIPS
soc is having same check. It's just check which will not impact any
functionality or performance of this driver. It will avoid NULL pointer
error. We know, if function is returning any error. we should catch.
semicolon, so this would be:
net: ethernet: cavium: octeon: octeon_mgmt: