On 2/10/26 06:10, Jamin Lin wrote:
> Add a new AST2700 A2 EVB machine to model the newer A2 silicon.
>
> The ast2700a2-evb machine is largely identical to ast2700a1-evb.
> The only difference is the default DRAM size, which is increased
> to 2 GB.
>
> This change adds a dedicated ast2700a2-evb machine by copying the
> existing ast2700a1-evb configuration and updating the DRAM size
> accordingly.
>
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
> hw/arm/aspeed_ast27x0_evb.c | 27 +++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
> diff --git a/hw/arm/aspeed_ast27x0_evb.c b/hw/arm/aspeed_ast27x0_evb.c
> index 31f7d61117..32d944848e 100644
> --- a/hw/arm/aspeed_ast27x0_evb.c
> +++ b/hw/arm/aspeed_ast27x0_evb.c
> @@ -50,12 +50,39 @@ static void aspeed_machine_ast2700a1_evb_class_init(ObjectClass *oc,
> aspeed_machine_class_init_cpus_defaults(mc);
> }
>
> +static void aspeed_machine_ast2700a2_evb_class_init(ObjectClass *oc,
> + const void *data)
> +{
> + MachineClass *mc = MACHINE_CLASS(oc);
> + AspeedMachineClass *amc = ASPEED_MACHINE_CLASS(oc);
> +
> + mc->desc = "Aspeed AST2700 A2 EVB (Cortex-A35)";
> + amc->soc_name = "ast2700-a2";
> + amc->hw_strap1 = AST2700_EVB_HW_STRAP1;
> + amc->hw_strap2 = AST2700_EVB_HW_STRAP2;
> + amc->fmc_model = "w25q01jvq";
> + amc->spi_model = "w25q512jv";
> + amc->num_cs = 2;
> + amc->macs_mask = ASPEED_MAC0_ON | ASPEED_MAC1_ON | ASPEED_MAC2_ON;
> + amc->uart_default = ASPEED_DEV_UART12;
> + amc->i2c_init = ast2700_evb_i2c_init;
> + amc->vbootrom = true;
> + mc->default_ram_size = 2 * GiB;
> + aspeed_machine_class_init_cpus_defaults(mc);
> +}
> +
> static const TypeInfo aspeed_ast27x0_evb_types[] = {
> {
> .name = MACHINE_TYPE_NAME("ast2700a1-evb"),
> .parent = TYPE_ASPEED_MACHINE,
> .class_init = aspeed_machine_ast2700a1_evb_class_init,
> .interfaces = aarch64_machine_interfaces,
> + },
> + {
> + .name = MACHINE_TYPE_NAME("ast2700a2-evb"),
> + .parent = TYPE_ASPEED_MACHINE,
> + .class_init = aspeed_machine_ast2700a2_evb_class_init,
> + .interfaces = aarch64_machine_interfaces,
> }
> };
>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Thanks,
C.