diff options
author | Daniel Mentz <danielmentz@google.com> | 2023-11-08 07:22:26 +0100 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2023-12-01 10:13:49 +0100 |
commit | c2183b3dcc9dd41b768569ea88bededa58cceebb (patch) | |
tree | 0c6f7ac2c6584cff1301e7386454eea3c728582b /drivers/iommu/of_iommu.c | |
parent | iommu/vt-d: Set variable intel_dirty_ops to static (diff) | |
download | linux-c2183b3dcc9dd41b768569ea88bededa58cceebb.tar.xz linux-c2183b3dcc9dd41b768569ea88bededa58cceebb.zip |
iommu: Fix printk arg in of_iommu_get_resv_regions()
The variable phys is defined as (struct resource *) which aligns with
the printk format specifier %pr. Taking the address of it results in a
value of type (struct resource **) which is incompatible with the format
specifier %pr. Therefore, remove the address of operator (&).
Fixes: a5bf3cfce8cb ("iommu: Implement of_iommu_get_resv_regions()")
Signed-off-by: Daniel Mentz <danielmentz@google.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Link: https://lore.kernel.org/r/20231108062226.928985-1-danielmentz@google.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu/of_iommu.c')
-rw-r--r-- | drivers/iommu/of_iommu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iommu/of_iommu.c b/drivers/iommu/of_iommu.c index c25b4ae6aeee..35ba090f3b5e 100644 --- a/drivers/iommu/of_iommu.c +++ b/drivers/iommu/of_iommu.c @@ -197,7 +197,7 @@ iommu_resv_region_get_type(struct device *dev, if (start == phys->start && end == phys->end) return IOMMU_RESV_DIRECT; - dev_warn(dev, "treating non-direct mapping [%pr] -> [%pap-%pap] as reservation\n", &phys, + dev_warn(dev, "treating non-direct mapping [%pr] -> [%pap-%pap] as reservation\n", phys, &start, &end); return IOMMU_RESV_RESERVED; } |