There are two places where the @model member of _virDomainIOMMUDef struct is typecasted to virDomainIOMMUModel which is completely unnecessary because the struct already defines the member of that type. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> --- src/qemu/qemu_domain_address.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_domain_address.c b/src/qemu/qemu_domain_address.c index 753733d1b9..026be99ba9 100644 --- a/src/qemu/qemu_domain_address.c +++ b/src/qemu/qemu_domain_address.c @@ -1003,7 +1003,7 @@ qemuDomainDeviceCalculatePCIConnectFlags(virDomainDeviceDef *dev, break; case VIR_DOMAIN_DEVICE_IOMMU: - switch ((virDomainIOMMUModel) dev->data.iommu->model) { + switch (dev->data.iommu->model) { case VIR_DOMAIN_IOMMU_MODEL_VIRTIO: return virtioFlags | VIR_PCI_CONNECT_INTEGRATED; @@ -2384,7 +2384,7 @@ qemuDomainAssignDevicePCISlots(virDomainDef *def, if (def->iommu) { virDomainIOMMUDef *iommu = def->iommu; - switch ((virDomainIOMMUModel) iommu->model) { + switch (iommu->model) { case VIR_DOMAIN_IOMMU_MODEL_VIRTIO: if (virDeviceInfoPCIAddressIsWanted(&iommu->info) && qemuDomainPCIAddressReserveNextAddr(addrs, &iommu->info) < 0) { -- 2.35.1