On Sat, Jun 21, 2014 at 7:18 AM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote: >> + >> + 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. > >> + mem = devm_ioremap_resource(dev, res); >> + if (!mem) >> + return -ENOMEM; -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html