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

Philippe Mathieu-Daudé <[email protected]>
Newsgroups gmane.comp.emulators.qemu
Message-ID <[email protected]>
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 <[email protected]>
> ---
>   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é <[email protected]>

> -#else
> -#define ASPEED_RAM_SIZE(sz) (sz)
> -#endif
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.