[PATCH v2 7/7] amd_iommu: Remove duplicated definitions

Alejandro Jimenez posted 7 patches 5 months, 2 weeks ago
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>
There is a newer version of this series
[PATCH v2 7/7] amd_iommu: Remove duplicated definitions
Posted by Alejandro Jimenez 5 months, 2 weeks ago
No functional change.

Signed-off-by: Alejandro Jimenez <alejandro.j.jimenez@oracle.com>
---
 hw/i386/amd_iommu.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/hw/i386/amd_iommu.h b/hw/i386/amd_iommu.h
index 1836b7061d848..31e473924fa20 100644
--- a/hw/i386/amd_iommu.h
+++ b/hw/i386/amd_iommu.h
@@ -206,10 +206,6 @@
 /* Completion Wait data size */
 #define AMDVI_COMPLETION_DATA_SIZE    8
 
-#define AMDVI_COMMAND_SIZE   16
-/* Completion Wait data size */
-#define AMDVI_COMPLETION_DATA_SIZE    8
-
 #define AMDVI_COMMAND_SIZE   16
 
 #define AMDVI_INT_ADDR_FIRST    0xfee00000
-- 
2.43.5
Re: [PATCH v2 7/7] amd_iommu: Remove duplicated definitions
Posted by Vasant Hegde 5 months, 2 weeks ago

On 5/29/2025 3:47 AM, Alejandro Jimenez wrote:
> No functional change.
> 
> Signed-off-by: Alejandro Jimenez <alejandro.j.jimenez@oracle.com>

I had similar fix locally :-) Thanks for fixing it.

Reviewed-by: Vasant Hegde <vasant.hegde@amd.com>

-Vasant