On Fri, Feb 28, 2014 at 03:37:10PM +0000, Laurent Pinchart wrote: > The IOMMU core expects the unmap operation to return the number of bytes > that have been unmapped or 0 on failure, a negative return value being > treated like a number of bytes.
Makes sense, thanks. It's a pity we can't propagate the error code. Will > Signed-off-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com> > --- > drivers/iommu/arm-smmu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c > index c33a310..e7528f0 100644 > --- a/drivers/iommu/arm-smmu.c > +++ b/drivers/iommu/arm-smmu.c > @@ -1464,7 +1464,7 @@ static size_t arm_smmu_unmap(struct iommu_domain > *domain, unsigned long iova, > > ret = arm_smmu_handle_mapping(smmu_domain, iova, 0, size, 0); > arm_smmu_tlb_inv_context(&smmu_domain->root_cfg); > - return ret ? ret : size; > + return ret ? 0 : size; > } > > static phys_addr_t arm_smmu_iova_to_phys(struct iommu_domain *domain, > -- > 1.8.3.2 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-ker...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
