From mboxrd@z Thu Jan 1 00:00:00 1970 From: joro@8bytes.org (Joerg Roedel) Date: Mon, 26 May 2014 11:23:11 +0200 Subject: [PATCH v2 02/10] iommu/ipmmu-vmsa: Cleanup failures of ARM mapping creation or attachment In-Reply-To: <1400150451-13469-3-git-send-email-laurent.pinchart+renesas@ideasonboard.com> References: <1400150451-13469-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <1400150451-13469-3-git-send-email-laurent.pinchart+renesas@ideasonboard.com> Message-ID: <20140526092310.GA11299@8bytes.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, May 15, 2014 at 12:40:43PM +0200, Laurent Pinchart wrote: > Signed-off-by: Laurent Pinchart > --- > drivers/iommu/ipmmu-vmsa.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/iommu/ipmmu-vmsa.c b/drivers/iommu/ipmmu-vmsa.c > index 49e00f7..49dbedd 100644 > --- a/drivers/iommu/ipmmu-vmsa.c > +++ b/drivers/iommu/ipmmu-vmsa.c > @@ -917,7 +917,8 @@ static int ipmmu_add_device(struct device *dev) > SZ_1G, SZ_2G); > if (IS_ERR(mapping)) { > dev_err(mmu->dev, "failed to create ARM IOMMU mapping\n"); > - return PTR_ERR(mapping); > + ret = PTR_ERR(mapping); > + goto error; > } > > mmu->mapping = mapping; > @@ -933,6 +934,7 @@ static int ipmmu_add_device(struct device *dev) > return 0; > > error: > + arm_iommu_release_mapping(mmu->mapping); > kfree(dev->archdata.iommu); > dev->archdata.iommu = NULL; > iommu_group_remove_device(dev); Skipped this one because it didn't apply. The others are applied.