On Fri, 1 Dec 2017 18:40:10 +0100 Arnaud Pouliquen <arnaud.pouliquen@xxxxxx> wrote: > Add devm_iio_hw_consumer_alloc function that calls iio_hw_consumer_free > when the device is unbound from the bus. > > Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@xxxxxx> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > --- > drivers/iio/buffer/industrialio-hw-consumer.c | 66 +++++++++++++++++++++++++++ > include/linux/iio/hw-consumer.h | 2 + > 2 files changed, 68 insertions(+) > > diff --git a/drivers/iio/buffer/industrialio-hw-consumer.c b/drivers/iio/buffer/industrialio-hw-consumer.c > index 0253be1..e980a79 100644 > --- a/drivers/iio/buffer/industrialio-hw-consumer.c > +++ b/drivers/iio/buffer/industrialio-hw-consumer.c > @@ -138,6 +138,72 @@ void iio_hw_consumer_free(struct iio_hw_consumer *hwc) > } > EXPORT_SYMBOL_GPL(iio_hw_consumer_free); > > +static void devm_iio_hw_consumer_release(struct device *dev, void *res) > +{ > + iio_hw_consumer_free(*(struct iio_hw_consumer **)res); > +} > + > +static int devm_iio_hw_consumer_match(struct device *dev, void *res, void *data) > +{ > + struct iio_hw_consumer **r = res; > + > + if (!r || !*r) { > + WARN_ON(!r || !*r); > + return 0; > + } > + return *r == data; > +} > + > +/** > + * devm_iio_hw_consumer_alloc - Resource-managed iio_hw_consumer_alloc() > + * @dev: Pointer to consumer device. > + * > + * Managed iio_hw_consumer_alloc. iio_hw_consumer allocated with this function > + * is automatically freed on driver detach. > + * > + * If an iio_hw_consumer allocated with this function needs to be freed > + * separately, devm_iio_hw_consumer_free() must be used. > + * > + * returns pointer to allocated iio_hw_consumer on success, NULL on failure. > + */ > +struct iio_hw_consumer *devm_iio_hw_consumer_alloc(struct device *dev) > +{ > + struct iio_hw_consumer **ptr, *iio_hwc; > + > + ptr = devres_alloc(devm_iio_hw_consumer_release, sizeof(*ptr), > + GFP_KERNEL); > + if (!ptr) > + return NULL; > + > + iio_hwc = iio_hw_consumer_alloc(dev); > + if (IS_ERR(iio_hwc)) { > + devres_free(ptr); > + } else { > + *ptr = iio_hwc; > + devres_add(dev, ptr); > + } > + > + return iio_hwc; > +} > +EXPORT_SYMBOL_GPL(devm_iio_hw_consumer_alloc); > + > +/** > + * devm_iio_hw_consumer_free - Resource-managed iio_hw_consumer_free() > + * @dev: Pointer to consumer device. > + * @hwc: iio_hw_consumer to free. > + * > + * Free iio_hw_consumer allocated with devm_iio_hw_consumer_alloc(). > + */ > +void devm_iio_hw_consumer_free(struct device *dev, struct iio_hw_consumer *hwc) > +{ > + int rc; > + > + rc = devres_release(dev, devm_iio_hw_consumer_release, > + devm_iio_hw_consumer_match, hwc); > + WARN_ON(rc); > +} > +EXPORT_SYMBOL_GPL(devm_iio_hw_consumer_free); > + > /** > * iio_hw_consumer_enable() - Enable IIO hardware consumer > * @hwc: iio_hw_consumer to enable. > diff --git a/include/linux/iio/hw-consumer.h b/include/linux/iio/hw-consumer.h > index f16791b..90ecfce 100644 > --- a/include/linux/iio/hw-consumer.h > +++ b/include/linux/iio/hw-consumer.h > @@ -14,6 +14,8 @@ struct iio_hw_consumer; > > struct iio_hw_consumer *iio_hw_consumer_alloc(struct device *dev); > void iio_hw_consumer_free(struct iio_hw_consumer *hwc); > +struct iio_hw_consumer *devm_iio_hw_consumer_alloc(struct device *dev); > +void devm_iio_hw_consumer_free(struct device *dev, struct iio_hw_consumer *hwc); > int iio_hw_consumer_enable(struct iio_hw_consumer *hwc); > void iio_hw_consumer_disable(struct iio_hw_consumer *hwc); > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html