On 2023-12-11 19:41, Caleb Connolly wrote:
The dev_pci_iommu_enable() function is only available when CONFIG_PCI is
enabled, replace the runtime check with a preprocessor one to fix
compilation with pci disabled.

Signed-off-by: Caleb Connolly <caleb.conno...@linaro.org>
---
 drivers/iommu/iommu-uclass.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/iommu/iommu-uclass.c b/drivers/iommu/iommu-uclass.c
index 72f123df55a5..98731d5e2c44 100644
--- a/drivers/iommu/iommu-uclass.c
+++ b/drivers/iommu/iommu-uclass.c
@@ -100,9 +100,10 @@ int dev_iommu_enable(struct udevice *dev)
                dev->iommu = dev_iommu;
        }

-       if (CONFIG_IS_ENABLED(PCI) && count < 0 &&
-           device_is_on_pci_bus(dev))
+#if CONFIG_IS_ENABLED(PCI)
+       if (count < 0 && device_is_on_pci_bus(dev))
                return dev_pci_iommu_enable(dev);
+#endif

        return 0;
 }

Perhaps there's no need to introduce an ifdef here.

Reply via email to