diff mbox

[arm-devs,v3,2/9] target-arm/helper.c: Allow cp15.c15 dummy override

Message ID fc02eeb03aecc60f378c316dfe6cee532969027a.1386053678.git.peter.crosthwaite@xilinx.com
State New
Headers show

Commit Message

Peter Crosthwaite Dec. 3, 2013, 7 a.m. UTC
The cp15.c15 space is implementation defined. Currently there is a
dummy placeholder register RAZing it. Allow overriding of this RAZ
so implementations of specific registers can take precedence.

Signed-off-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
---

 target-arm/helper.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Peter Maydell Dec. 6, 2013, 2:36 p.m. UTC | #1
On 3 December 2013 07:00, Peter Crosthwaite
<peter.crosthwaite@xilinx.com> wrote:
> The cp15.c15 space is implementation defined. Currently there is a
> dummy placeholder register RAZing it. Allow overriding of this RAZ
> so implementations of specific registers can take precedence.
>
> Signed-off-by: Peter Crosthwaite <peter.crosthwaite@xilinx.com>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Ideally we'd actually implement the specifics of the A15 rather than
leaning on the dummy RAZ/WI, but oh well.

-- PMM
diff mbox

Patch

diff --git a/target-arm/helper.c b/target-arm/helper.c
index 3445813..587ff49 100644
--- a/target-arm/helper.c
+++ b/target-arm/helper.c
@@ -1338,7 +1338,8 @@  static const ARMCPRegInfo dummy_c15_cp_reginfo[] = {
      */
     { .name = "C15_IMPDEF", .cp = 15, .crn = 15,
       .crm = CP_ANY, .opc1 = CP_ANY, .opc2 = CP_ANY,
-      .access = PL1_RW, .type = ARM_CP_CONST | ARM_CP_NO_MIGRATE,
+      .access = PL1_RW,
+      .type = ARM_CP_CONST | ARM_CP_NO_MIGRATE | ARM_CP_OVERRIDE,
       .resetvalue = 0 },
     REGINFO_SENTINEL
 };