Re: [PATCH v4 09/15] target/loongarch: Add LoongArch32 cpu la132

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

On 22/8/23 05:27, Song Gao wrote:
> From: Jiajie Chen <c@jia.je>
> 
> Add LoongArch32 cpu la132.
> 
> Due to lack of public documentation of la132, it is currently a
> synthetic LoongArch32 cpu model. Details need to be added in the future.
> 
> Signed-off-by: Jiajie Chen <c@jia.je>
> Signed-off-by: Song Gao <gaosong@loongson.cn>
> ---
>   target/loongarch/cpu.c | 30 ++++++++++++++++++++++++++++++

Thanks for splitting the hw/ patch out, ...

>   1 file changed, 30 insertions(+)
> 
> diff --git a/target/loongarch/cpu.c b/target/loongarch/cpu.c
> index 67eb6c3135..d3c3e0d8a1 100644
> --- a/target/loongarch/cpu.c
> +++ b/target/loongarch/cpu.c
> @@ -440,6 +440,35 @@ static void loongarch_la464_initfn(Object *obj)
>       env->CSR_ASID = FIELD_DP64(0, CSR_ASID, ASIDBITS, 0xa);
>   }
>   
> +static void loongarch_la132_initfn(Object *obj)
> +{
> +    LoongArchCPU *cpu = LOONGARCH_CPU(obj);
> +    CPULoongArchState *env = &cpu->env;
> +
> +    int i;
> +
> +    for (i = 0; i < 21; i++) {
> +        env->cpucfg[i] = 0x0;
> +    }
> +
> +    cpu->dtb_compatible = "loongarch,Loongson-1C103";
> +    env->cpucfg[0] = 0x148042;  /* PRID */

... and filling the PRid register.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>