On 21.08.25 15:03, Mykola Kvach wrote: > Hi Mykyta, > > On Thu, Aug 21, 2025 at 1:51 PM Mykyta Poturai <mykyta_potu...@epam.com> > wrote: >> >> With PCI disabled the build fails due to undefined struct >> pci_host_bridge. >> >> Add ifdef guard around pci-host-rcar4.h to not include it when PCI >> support is disabled. >> >> Signed-off-by: Mykyta Poturai <mykyta_potu...@epam.com> >> --- >> This patch can be squashed with iommu/ipmmu-vmsa: Implement basic PCIE-IPMMU >> OSID support >> --- >> xen/drivers/passthrough/arm/ipmmu-vmsa.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/xen/drivers/passthrough/arm/ipmmu-vmsa.c >> b/xen/drivers/passthrough/arm/ipmmu-vmsa.c >> index ea9fa9ddf3..49f149e222 100644 >> --- a/xen/drivers/passthrough/arm/ipmmu-vmsa.c >> +++ b/xen/drivers/passthrough/arm/ipmmu-vmsa.c >> @@ -51,7 +51,9 @@ >> #include <asm/device.h> >> #include <asm/io.h> >> #include <asm/iommu_fwspec.h> >> +#ifdef CONFIG_HAS_PCI >> #include "../arch/arm/pci/pci-host-rcar4.h" >> +#endif > > As far as I know, the IPMMU VMSA driver can also be used on other > R-Car platforms. Is there a chance we could have HAS_PCI enabled > together with IPMMU VMSA, but not be on an R-Car Gen4 platform, > for example, on R-Car3 or possibly some future R-Car variants? > > Wouldn't it be better to use: > #ifdef CONFIG_RCAR4 > ?
Gen5 boards will use SMMU, so this will not be applicable to them. In any case, checking for RCAR4 will not help here, as PCI can be disabled regardless of the platform revision. > > Best regards, > Mykola -- Mykyta
