tree 416f40090d20f20c4871d8bee5f79bb33de4c03e
parent 6ff6e184f1f4d4993d45ca3f934c8288890965fe
author Jason Gunthorpe <jgg@nvidia.com> 1701972190 -0400
committer Joerg Roedel <jroedel@suse.de> 1702372728 +0100

iommu/of: Use -ENODEV consistently in of_iommu_configure()

Instead of returning 1 and trying to handle positive error codes just
stick to the convention of returning -ENODEV. Remove references to ops
from of_iommu_configure(), a NULL ops will already generate an error code.

There is no reason to check dev->bus, if err=0 at this point then the
called configure functions thought there was an iommu and we should try to
probe it. Remove it.

Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com>
Reviewed-by: Moritz Fischer <moritzf@google.com>
Tested-by: Hector Martin <marcan@marcan.st>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
Link: https://lore.kernel.org/r/3-v2-16e4def25ebb+820-iommu_fwspec_p1_jgg@nvidia.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>