The subject reads odd, how about: dma-mapping: Provide an interface to allocate IOVA > +/** > + * dma_iova_free - Free an IOVA space > + * @dev: Device to free the IOVA space for > + * @state: IOVA state > + * > + * Undoes a successful dma_try_iova_alloc(). > + * > + * Note that all dma_iova_link() calls need to be undone first. For callers > + * that never call dma_iova_unlink(), dma_iova_destroy() can be used instead > + * which unlinks all ranges and frees the IOVA space in a single efficient > + * operation. > + */ Probably does't matter but dma_iova_destroy() doesn't exist yet here. Other than that: Reviewed-by: Luis Chamberlain <mcgrof@xxxxxxxxxx> Luis