[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-arm/cpu64.c: Actually register Cortex-A5
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH] target-arm/cpu64.c: Actually register Cortex-A57 impdef registers |
Date: |
Wed, 28 May 2014 00:06:07 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, May 27, 2014 at 10:30:59PM +0100, Peter Maydell wrote:
> cpu64.c contains a reginfo list for the impdef registers on
> the Cortex-A57; however we forgot to actually call define_arm_cp_regs(),
> so it was sitting there doing nothing. Remedy this omission.
>
> Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Edgar E. Iglesias <address@hidden>
> ---
> target-arm/cpu64.c | 1 +
> 1 file changed, 1 insertion(+)
>
> Oops. clang 3.4 warns about the unused variable (along with
> a lot of similar issues in other parts of the codebase).
> Linux doesn't actually look at any of the A57 impdef regs,
> or we'd have noticed this earlier.
>
> diff --git a/target-arm/cpu64.c b/target-arm/cpu64.c
> index 8daa622..ff4c2b4 100644
> --- a/target-arm/cpu64.c
> +++ b/target-arm/cpu64.c
> @@ -128,6 +128,7 @@ static void aarch64_a57_initfn(Object *obj)
> cpu->ccsidr[1] = 0x201fe012; /* 48KB L1 icache */
> cpu->ccsidr[2] = 0x70ffe07a; /* 2048KB L2 cache */
> cpu->dcz_blocksize = 4; /* 64 bytes */
> + define_arm_cp_regs(cpu, cortexa57_cp_reginfo);
> }
>
> #ifdef CONFIG_USER_ONLY
> --
> 1.9.2
>
>