diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2019-08-24 15:28:45 +0200 |
|---|---|---|
| committer | Rob Herring <robh@kernel.org> | 2019-09-13 16:54:23 -0500 |
| commit | c680e9abaade9145ef195f9b28afa1a3b975438b (patch) | |
| tree | 6c68dfb9e46ca1c6e19c5c6633c0db20d1ab1ef2 /drivers/iommu/mtk_iommu_v1.c | |
| parent | 693af5f3eeaa46115c44f1d639dea4a8d8342a4b (diff) | |
| download | linux-stable-c680e9abaade9145ef195f9b28afa1a3b975438b.tar.gz linux-stable-c680e9abaade9145ef195f9b28afa1a3b975438b.tar.bz2 linux-stable-c680e9abaade9145ef195f9b28afa1a3b975438b.zip | |
iommu: pass cell_count = -1 to of_for_each_phandle with cells_name
Currently of_for_each_phandle ignores the cell_count parameter when a
cells_name is given. I intend to change that and let the iterator fall
back to a non-negative cell_count if the cells_name property is missing
in the referenced node.
To not change how existing of_for_each_phandle's users iterate, fix them
to pass cell_count = -1 when also cells_name is given which yields the
expected behaviour with and without my change.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Joerg Roedel <jroedel@suse.de>
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'drivers/iommu/mtk_iommu_v1.c')
| -rw-r--r-- | drivers/iommu/mtk_iommu_v1.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iommu/mtk_iommu_v1.c b/drivers/iommu/mtk_iommu_v1.c index abeeac488372..68d1de70de0c 100644 --- a/drivers/iommu/mtk_iommu_v1.c +++ b/drivers/iommu/mtk_iommu_v1.c @@ -426,7 +426,7 @@ static int mtk_iommu_add_device(struct device *dev) int err; of_for_each_phandle(&it, err, dev->of_node, "iommus", - "#iommu-cells", 0) { + "#iommu-cells", -1) { int count = of_phandle_iterator_args(&it, iommu_spec.args, MAX_PHANDLE_ARGS); iommu_spec.np = of_node_get(it.node); |