diff options
| author | Rob Herring (Arm) <robh@kernel.org> | 2024-07-31 13:12:44 -0600 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2024-09-04 11:31:01 +0200 |
| commit | 04f4f33c941c221645d2a58b46f4d698b0f5aa39 (patch) | |
| tree | 9d91abf8b4105752bb506b4be9c62a5e6a2b1c84 /drivers/iommu/of_iommu.c | |
| parent | 84b2baf427968c1b2e3ae3b7afcb0118cdee0915 (diff) | |
| download | linux-04f4f33c941c221645d2a58b46f4d698b0f5aa39.tar.gz linux-04f4f33c941c221645d2a58b46f4d698b0f5aa39.tar.bz2 linux-04f4f33c941c221645d2a58b46f4d698b0f5aa39.zip | |
iommu: Use of_property_present()
Use of_property_present() to test for property presence rather than
of_(find|get)_property(). This is part of a larger effort to remove
callers of of_find_property() and similar functions. of_find_property()
leaks the DT struct property and data pointers which is a problem for
dynamically allocated nodes which may be freed.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Robin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/20240731191312.1710417-6-robh@kernel.org
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 78d61da75257..e7a6a1611d19 100644 --- a/drivers/iommu/of_iommu.c +++ b/drivers/iommu/of_iommu.c @@ -214,7 +214,7 @@ void of_iommu_get_resv_regions(struct device *dev, struct list_head *list) * that represent reservations in the IOVA space, which are regions that should * not be mapped. */ - if (of_find_property(it.node, "reg", NULL)) { + if (of_property_present(it.node, "reg")) { err = of_address_to_resource(it.node, 0, &phys); if (err < 0) { dev_err(dev, "failed to parse memory region %pOF: %d\n", |