diff mbox series

[RFC,1/3] target/arm: Replace target_ulong -> hwaddr in ARMMMUFaultInfo

Message ID 20250415172246.79470-2-philmd@linaro.org
State New
Headers show
Series target/arm/internals: Replace some target_ulong uses | expand

Commit Message

Philippe Mathieu-Daudé April 15, 2025, 5:22 p.m. UTC 
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/arm/internals.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Pierrick Bouvier April 15, 2025, 5:31 p.m. UTC | #1 
On 4/15/25 10:22, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   target/arm/internals.h | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/target/arm/internals.h b/target/arm/internals.h
> index 28585c07555..175fb792375 100644
> --- a/target/arm/internals.h
> +++ b/target/arm/internals.h
> @@ -25,6 +25,7 @@
>   #ifndef TARGET_ARM_INTERNALS_H
>   #define TARGET_ARM_INTERNALS_H
>   
> +#include "exec/hwaddr.h"
>   #include "exec/breakpoint.h"
>   #include "hw/registerfields.h"
>   #include "tcg/tcg-gvec-desc.h"
> @@ -724,8 +725,8 @@ typedef struct ARMMMUFaultInfo ARMMMUFaultInfo;
>   struct ARMMMUFaultInfo {
>       ARMFaultType type;
>       ARMGPCF gpcf;
> -    target_ulong s2addr;
> -    target_ulong paddr;
> +    hwaddr s2addr;
> +    hwaddr paddr;
>       ARMSecuritySpace paddr_space;
>       int level;
>       int domain;

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Richard Henderson April 16, 2025, 5:06 p.m. UTC | #2 
On 4/15/25 10:22, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   target/arm/internals.h | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

> 
> diff --git a/target/arm/internals.h b/target/arm/internals.h
> index 28585c07555..175fb792375 100644
> --- a/target/arm/internals.h
> +++ b/target/arm/internals.h
> @@ -25,6 +25,7 @@
>   #ifndef TARGET_ARM_INTERNALS_H
>   #define TARGET_ARM_INTERNALS_H
>   
> +#include "exec/hwaddr.h"
>   #include "exec/breakpoint.h"
>   #include "hw/registerfields.h"
>   #include "tcg/tcg-gvec-desc.h"
> @@ -724,8 +725,8 @@ typedef struct ARMMMUFaultInfo ARMMMUFaultInfo;
>   struct ARMMMUFaultInfo {
>       ARMFaultType type;
>       ARMGPCF gpcf;
> -    target_ulong s2addr;
> -    target_ulong paddr;
> +    hwaddr s2addr;
> +    hwaddr paddr;
>       ARMSecuritySpace paddr_space;
>       int level;
>       int domain;
diff mbox series

Patch

diff --git a/target/arm/internals.h b/target/arm/internals.h
index 28585c07555..175fb792375 100644
--- a/target/arm/internals.h
+++ b/target/arm/internals.h
@@ -25,6 +25,7 @@ 
 #ifndef TARGET_ARM_INTERNALS_H
 #define TARGET_ARM_INTERNALS_H
 
+#include "exec/hwaddr.h"
 #include "exec/breakpoint.h"
 #include "hw/registerfields.h"
 #include "tcg/tcg-gvec-desc.h"
@@ -724,8 +725,8 @@  typedef struct ARMMMUFaultInfo ARMMMUFaultInfo;
 struct ARMMMUFaultInfo {
     ARMFaultType type;
     ARMGPCF gpcf;
-    target_ulong s2addr;
-    target_ulong paddr;
+    hwaddr s2addr;
+    hwaddr paddr;
     ARMSecuritySpace paddr_space;
     int level;
     int domain;