Re: [PATCH v2 3/9] arm/mps: Remove 32bit define for MPS3_DDR_SIZE

[Richard Henderson <richard.henderson@linaro.org>]

On 8/18/26 10:20, Peter Xu wrote:
> QEMU has switched to 64bit-only hosts for all system emulations.
> 
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
>   hw/arm/mps2-tz.c | 9 ---------
>   hw/arm/mps3r.c   | 8 --------
>   2 files changed, 17 deletions(-)

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

r~

> 
> diff --git a/hw/arm/mps2-tz.c b/hw/arm/mps2-tz.c
> index d6d178642b..f47b0cacb8 100644
> --- a/hw/arm/mps2-tz.c
> +++ b/hw/arm/mps2-tz.c
> @@ -178,16 +178,7 @@ OBJECT_DECLARE_TYPE(MPS2TZMachineState, MPS2TZMachineClass, MPS2TZ_MACHINE)
>   
>   /* Slow 32Khz S32KCLK frequency in Hz */
>   #define S32KCLK_FRQ (32 * 1000)
> -
> -/*
> - * The MPS3 DDR is 2GiB, but on a 32-bit host QEMU doesn't permit
> - * emulation of that much guest RAM, so artificially make it smaller.
> - */
> -#if HOST_LONG_BITS == 32
> -#define MPS3_DDR_SIZE (1 * GiB)
> -#else
>   #define MPS3_DDR_SIZE (2 * GiB)
> -#endif
>   
>   /* For cpu{0,1}_mpu_{ns,s}, means "leave at SSE's default value" */
>   #define MPU_REGION_DEFAULT UINT32_MAX
> diff --git a/hw/arm/mps3r.c b/hw/arm/mps3r.c
> index caad820d3b..82a354e958 100644
> --- a/hw/arm/mps3r.c
> +++ b/hw/arm/mps3r.c
> @@ -59,15 +59,7 @@ typedef struct RAMInfo {
>       int flags;
>   } RAMInfo;
>   
> -/*
> - * The MPS3 DDR is 3GiB, but on a 32-bit host QEMU doesn't permit
> - * emulation of that much guest RAM, so artificially make it smaller.
> - */
> -#if HOST_LONG_BITS == 32
> -#define MPS3_DDR_SIZE (1 * GiB)
> -#else
>   #define MPS3_DDR_SIZE (3 * GiB)
> -#endif
>   
>   /*
>    * Flag values: