On 2022/4/14 20:42, Robin Murphy wrote:
@@ -1883,27 +1900,12 @@ static int iommu_bus_init(struct bus_type *bus)
   */
  int bus_set_iommu(struct bus_type *bus, const struct iommu_ops *ops)
  {
-       int err;
-
-       if (ops == NULL) {
-               bus->iommu_ops = NULL;
-               return 0;
-       }
-
-       if (bus->iommu_ops != NULL)
+       if (bus->iommu_ops && ops && bus->iommu_ops != ops)
                return -EBUSY;
bus->iommu_ops = ops;

Do we still need to keep above lines in bus_set_iommu()?

Best regards,
baolu
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to