Re: [PATCH v2 7/9] arm/aspeed: Drop ASPEED_RAM_SIZE

["Philippe Mathieu-Daudé" <philmd@oss.qualcomm.com>]

On 18/8/26 19:20, Peter Xu wrote:
> QEMU has switched to 64bit-only hosts for all system/user emulations.  It's
> not needed anymore.
> 
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
>   include/hw/arm/aspeed.h           | 7 -------
>   hw/arm/aspeed_ast2600_anacapa.c   | 2 +-
>   hw/arm/aspeed_ast2600_bletchley.c | 2 +-
>   hw/arm/aspeed_ast2600_catalina.c  | 2 +-
>   hw/arm/aspeed_ast2600_fby35.c     | 2 +-
>   hw/arm/aspeed_ast2600_fuji.c      | 2 +-
>   hw/arm/aspeed_ast2600_gb200nvl.c  | 2 +-
>   7 files changed, 6 insertions(+), 13 deletions(-)
> 
> diff --git a/include/hw/arm/aspeed.h b/include/hw/arm/aspeed.h
> index a00238ed74..7a012d87de 100644
> --- a/include/hw/arm/aspeed.h
> +++ b/include/hw/arm/aspeed.h
> @@ -25,13 +25,6 @@ DECLARE_OBJ_CHECKERS(AspeedMachineState, AspeedMachineClass,
>   #define ASPEED_MAC2_ON   (1 << 2)
>   #define ASPEED_MAC3_ON   (1 << 3)
>   
> -/* On 32-bit hosts, lower RAM to 1G because of the 2047 MB limit */
> -#if HOST_LONG_BITS == 32
> -#define ASPEED_RAM_SIZE(sz) MIN((sz), 1 * GiB)

Indeed now dead code.

Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>

> -#else
> -#define ASPEED_RAM_SIZE(sz) (sz)
> -#endif