Commit 10006112 authored by Mathew Maidment's avatar Mathew Maidment Committed by Peter Maydell
Browse files

target/arm: Correct MPUIR privilege level in register_cp_regs_for_features() conditional case 



The duplication of id_tlbtr_reginfo was unintentionally added within
3281af81 which should have been
id_mpuir_reginfo.

The effect was that for OMAP and StrongARM CPUs we would
incorrectly UNDEF writes to MPUIR rather than NOPing them.

Signed-off-by: default avatarMathew Maidment <mathew1800@gmail.com>
Message-id: 20180501184933.37609-2-mathew1800@gmail.com
[PMM: tweak commit message]
Reviewed-by: default avatarPeter Maydell <peter.maydell@linaro.org>
Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
parent 6d9c1b8d

target/arm/helper.c

+1 −1
Original line number Diff line number Diff line
@@ -5347,7 +5347,7 @@ void register_cp_regs_for_features(ARMCPU *cpu) 
            for (r = id_cp_reginfo; r->type != ARM_CP_SENTINEL; r++) {

                r->access = PL1_RW;

            }

            id_tlbtr_reginfo.access = PL1_RW;

            id_mpuir_reginfo.access = PL1_RW;

            id_tlbtr_reginfo.access = PL1_RW;

        }

        if (arm_feature(env, ARM_FEATURE_V8)) {