On 21/06/14 20:55, Fabio Estevam wrote:

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

quoted

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.

Nothing like counter intuitive behaviour.  Ah well.

Thanks for explaining this!

quoted

quoted

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