Commit 92b7ed7f authored by Alexandru Ardelean's avatar Alexandru Ardelean Committed by Jonathan Cameron
Browse files

iio: core: drop devm_iio_trigger_free() API call 



It's unused so far, so it can be removed. Also makes sense to remove it
to discourage weird uses of this call during review.

Signed-off-by: default avatarAlexandru Ardelean <alexandru.ardelean@analog.com>
Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 83381c98

Documentation/driver-api/driver-model/devres.rst

+0 −1
Original line number Diff line number Diff line
@@ -289,7 +289,6 @@ IIO 
  devm_iio_kfifo_free()

  devm_iio_triggered_buffer_setup()

  devm_iio_trigger_alloc()

  devm_iio_trigger_free()

  devm_iio_trigger_register()

  devm_iio_channel_get()

  devm_iio_channel_release()

Documentation/driver-api/iio/triggers.rst

+0 −1
Original line number Diff line number Diff line
@@ -4,7 +4,6 @@ Triggers 


* struct :c:type:`iio_trigger` — industrial I/O trigger device

* :c:func:`devm_iio_trigger_alloc` — Resource-managed iio_trigger_alloc

* :c:func:`devm_iio_trigger_free` — Resource-managed iio_trigger_free

* :c:func:`devm_iio_trigger_register` — Resource-managed iio_trigger_register

  iio_trigger_unregister

* :c:func:`iio_trigger_validate_own_device` — Check if a trigger and IIO

drivers/iio/industrialio-trigger.c

+0 −32
Original line number Diff line number Diff line
@@ -585,18 +585,6 @@ static void devm_iio_trigger_release(struct device *dev, void *res) 
	iio_trigger_free(*(struct iio_trigger **)res);

}



static int devm_iio_trigger_match(struct device *dev, void *res, void *data)

{

	struct iio_trigger **r = res;



	if (!r || !*r) {

		WARN_ON(!r || !*r);

		return 0;

	}



	return *r == data;

}



/**

 * devm_iio_trigger_alloc - Resource-managed iio_trigger_alloc()

 * @dev:		Device to allocate iio_trigger for
@@ -608,9 +596,6 @@ static int devm_iio_trigger_match(struct device *dev, void *res, void *data) 
 * Managed iio_trigger_alloc.  iio_trigger allocated with this function is

 * automatically freed on driver detach.

 *

 * If an iio_trigger allocated with this function needs to be freed separately,

 * devm_iio_trigger_free() must be used.

 *

 * RETURNS:

 * Pointer to allocated iio_trigger on success, NULL on failure.

 */
@@ -640,23 +625,6 @@ struct iio_trigger *devm_iio_trigger_alloc(struct device *dev, 
}

EXPORT_SYMBOL_GPL(devm_iio_trigger_alloc);



/**

 * devm_iio_trigger_free - Resource-managed iio_trigger_free()

 * @dev:		Device this iio_dev belongs to

 * @iio_trig:		the iio_trigger associated with the device

 *

 * Free iio_trigger allocated with devm_iio_trigger_alloc().

 */

void devm_iio_trigger_free(struct device *dev, struct iio_trigger *iio_trig)

{

	int rc;



	rc = devres_release(dev, devm_iio_trigger_release,

			    devm_iio_trigger_match, iio_trig);

	WARN_ON(rc);

}

EXPORT_SYMBOL_GPL(devm_iio_trigger_free);



static void devm_iio_trigger_unreg(struct device *dev, void *res)

{

	iio_trigger_unregister(*(struct iio_trigger **)res);

include/linux/iio/iio.h

+0 −2
Original line number Diff line number Diff line
@@ -691,8 +691,6 @@ void iio_device_free(struct iio_dev *indio_dev); 
struct iio_dev *devm_iio_device_alloc(struct device *dev, int sizeof_priv);

struct iio_trigger *devm_iio_trigger_alloc(struct device *dev,

						const char *fmt, ...);

void devm_iio_trigger_free(struct device *dev, struct iio_trigger *iio_trig);



/**

 * iio_buffer_enabled() - helper function to test if the buffer is enabled

 * @indio_dev:		IIO device structure for device