]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
arm: use correct argument size of special registers
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Wed, 27 May 2020 18:04:24 +0000 (20:04 +0200)
committerTom Rini <trini@konsulko.com>
Tue, 7 Jul 2020 22:23:48 +0000 (18:23 -0400)
Compiling with clang on ARMv8 shows errors like:

./arch/arm/include/asm/system.h:162:32: note: use constraint modifier "w"
                asm volatile("msr sctlr_el1, %0" : : "r" (val) : "cc");
                                             ^~
                                             %w0

These errors are due to using an incorrect size for the variables used
for writing to and reading from special registers which have 64 bits on
ARMv8.

Mask off reserved bits when reading the exception level.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
arch/arm/include/asm/system.h

index 7a40b56acdcae28101f8d05eb6159e8167cbf188..b8c1b4ea7414f812cc5d0a96423196f9a2931ae7 100644 (file)
@@ -133,14 +133,16 @@ enum dcache_option {
 
 static inline unsigned int current_el(void)
 {
-       unsigned int el;
+       unsigned long el;
+
        asm volatile("mrs %0, CurrentEL" : "=r" (el) : : "cc");
-       return el >> 2;
+       return 3 & (el >> 2);
 }
 
 static inline unsigned int get_sctlr(void)
 {
-       unsigned int el, val;
+       unsigned int el;
+       unsigned long val;
 
        el = current_el();
        if (el == 1)
@@ -153,7 +155,7 @@ static inline unsigned int get_sctlr(void)
        return val;
 }
 
-static inline void set_sctlr(unsigned int val)
+static inline void set_sctlr(unsigned long val)
 {
        unsigned int el;