On Sat, Jun 21, 2014 at 7:18 AM, Jonathan Cameron [off-list ref] wrote:

quoted

+
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);

Could this not in theory fail?

Yes, but devm_ioremap_resource will take care of it, so we don't need
to check for error here.

quoted

+       mem = devm_ioremap_resource(dev, res);
+       if (!mem)
+               return -ENOMEM;