On Thu, 2014-06-19 at 16:46 +0100, Rob Jones wrote: [...] > +int devm_gpio_request_array(struct device *dev, > + const struct gpio *array, > + size_t num) > +{ > + int i, err = 0; > + > + for (i = 0; i < num; i++, array++) { > + err = devm_gpio_request_one(dev, > + array->gpio, > + array->flags, > + array->label); > + if (err) { > + while (i--) > + devm_gpio_free(dev, (--array)->gpio); Missing break here. > + } > + } > + > + return err; > +} > +EXPORT_SYMBOL(devm_gpio_request_array); [...] -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html