If an IOMMU node is present in the DT but marked as disabled, we should
avoid trying to do anything with it. We currently sort-of get away with
this by virtue of a disabled device probably not having called
of_iommu_set_ops(), but that is hardly safe to rely upon in general, and
either way we don't want to treat it as an error condition with the
resulting "Failed to initialise IOMMU" message.
Signed-off-by: Robin Murphy <robin.murphy@xxxxxxx>
---
v2: New.
drivers/iommu/of_iommu.c | 29 +++++++++++++++--------------
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/drivers/iommu/of_iommu.c b/drivers/iommu/of_iommu.c
index af499aea0a1a..662f9a600f4f 100644
--- a/drivers/iommu/of_iommu.c
+++ b/drivers/iommu/of_iommu.c
@@ -140,7 +140,7 @@ const struct iommu_ops *of_iommu_configure(struct device *dev,
struct of_phandle_args iommu_spec;
struct device_node *np;
const struct iommu_ops *ops = NULL;
- int idx = 0;
+ int idx;
/*
* We can't do much for PCI devices without knowing how
@@ -154,24 +154,25 @@ const struct iommu_ops *of_iommu_configure(struct device *dev,
* See the `Notes:' section of
* Documentation/devicetree/bindings/iommu/iommu.txt
*/
- while (!of_parse_phandle_with_args(master_np, "iommus",
- "#iommu-cells", idx,
- &iommu_spec)) {
+ for (idx = 0;
+ !of_parse_phandle_with_args(master_np, "iommus", "#iommu-cells",
+ idx, &iommu_spec);
+ of_node_put(np), idx++) {
np = iommu_spec.np;
+ if (!of_device_is_available(np))
+ continue;
+
ops = of_iommu_get_ops(np);
+ if (!ops || !ops->of_xlate)
+ continue;
- if (!ops || !ops->of_xlate || ops->of_xlate(dev, &iommu_spec))
- goto err_put_node;
-
- of_node_put(np);
- idx++;
+ if (ops->of_xlate(dev, &iommu_spec)) {
+ of_node_put(np);
+ return NULL;
+ }
}
return ops;
-
-err_put_node:
- of_node_put(np);
- return NULL;
}
void __init of_iommu_init(void)
@@ -182,7 +183,7 @@ void __init of_iommu_init(void)
for_each_matching_node_and_match(np, matches, &match) {
const of_iommu_init_fn init_fn = match->data;
- if (init_fn(np))
+ if (of_device_is_available(np) && init_fn(np))
pr_err("Failed to initialise IOMMU %s\n",
of_node_full_name(np));
}
--
2.8.1.dirty
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
