From: Ohad Ben-Cohen <ohad@wizery.com>
Date: Tue, 6 Dec 2011 13:22:10 +0000 (+0200)
Subject: iommu/omap: be verbose when omap_iommu_iova_to_phys fails
X-Git-Tag: upstream/snapshot3+hdmi~8306^2~2^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1a36ea815a3557c03819ec7c90a6b2fb128385ca;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

iommu/omap: be verbose when omap_iommu_iova_to_phys fails

An omap_iommu_iova_to_phys failure usually means that iova wasn't mapped.

When that happens, it's helpful to know the value of iova, so add it
to the error message.

Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com>
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
---

diff --git a/drivers/iommu/omap-iommu.c b/drivers/iommu/omap-iommu.c
index b7f863d..cbcbf31 100644
--- a/drivers/iommu/omap-iommu.c
+++ b/drivers/iommu/omap-iommu.c
@@ -1177,14 +1177,14 @@ static phys_addr_t omap_iommu_iova_to_phys(struct iommu_domain *domain,
 		else if (iopte_is_large(*pte))
 			ret = omap_iommu_translate(*pte, da, IOLARGE_MASK);
 		else
-			dev_err(dev, "bogus pte 0x%x", *pte);
+			dev_err(dev, "bogus pte 0x%x, da 0x%lx", *pte, da);
 	} else {
 		if (iopgd_is_section(*pgd))
 			ret = omap_iommu_translate(*pgd, da, IOSECTION_MASK);
 		else if (iopgd_is_super(*pgd))
 			ret = omap_iommu_translate(*pgd, da, IOSUPER_MASK);
 		else
-			dev_err(dev, "bogus pgd 0x%x", *pgd);
+			dev_err(dev, "bogus pgd 0x%x, da 0x%lx", *pgd, da);
 	}
 
 	return ret;