diff options
author | Ohad Ben-Cohen <ohad@wizery.com> | 2011-12-06 08:22:10 -0500 |
---|---|---|
committer | Joerg Roedel <joerg.roedel@amd.com> | 2011-12-06 08:56:08 -0500 |
commit | 1a36ea815a3557c03819ec7c90a6b2fb128385ca (patch) | |
tree | c215095a0161370a71d8c09ec01b1125346c250e | |
parent | 608205c082eb4c1326b56fa1525a8106fa7a38c9 (diff) |
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>
-rw-r--r-- | drivers/iommu/omap-iommu.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/iommu/omap-iommu.c b/drivers/iommu/omap-iommu.c index b7f863d72c08..cbcbf31ec3be 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, | |||
1177 | else if (iopte_is_large(*pte)) | 1177 | else if (iopte_is_large(*pte)) |
1178 | ret = omap_iommu_translate(*pte, da, IOLARGE_MASK); | 1178 | ret = omap_iommu_translate(*pte, da, IOLARGE_MASK); |
1179 | else | 1179 | else |
1180 | dev_err(dev, "bogus pte 0x%x", *pte); | 1180 | dev_err(dev, "bogus pte 0x%x, da 0x%lx", *pte, da); |
1181 | } else { | 1181 | } else { |
1182 | if (iopgd_is_section(*pgd)) | 1182 | if (iopgd_is_section(*pgd)) |
1183 | ret = omap_iommu_translate(*pgd, da, IOSECTION_MASK); | 1183 | ret = omap_iommu_translate(*pgd, da, IOSECTION_MASK); |
1184 | else if (iopgd_is_super(*pgd)) | 1184 | else if (iopgd_is_super(*pgd)) |
1185 | ret = omap_iommu_translate(*pgd, da, IOSUPER_MASK); | 1185 | ret = omap_iommu_translate(*pgd, da, IOSUPER_MASK); |
1186 | else | 1186 | else |
1187 | dev_err(dev, "bogus pgd 0x%x", *pgd); | 1187 | dev_err(dev, "bogus pgd 0x%x, da 0x%lx", *pgd, da); |
1188 | } | 1188 | } |
1189 | 1189 | ||
1190 | return ret; | 1190 | return ret; |