]> git.baikalelectronics.ru Git - arm-tf.git/commitdiff
fix(debug): decouple "get_el_str()" from backtrace
authorManish Pandey <manish.pandey2@arm.com>
Tue, 1 Nov 2022 16:16:55 +0000 (16:16 +0000)
committerManish Pandey <manish.pandey2@arm.com>
Tue, 8 Nov 2022 10:10:19 +0000 (10:10 +0000)
get_el_str() was implemented under ENABLE_BACKTRACE macro but being
used at generic places too, this causes multiple definition of this
function.
Remove duplicate definition of this function and move it out of
backtrace scope. Also, this patch fixes a small bug where in default
case S-EL1 is returned which ideally should be EL1, as there is no
notion of security state in EL string.

Signed-off-by: Manish Pandey <manish.pandey2@arm.com>
Change-Id: Ib186ea03b776e2478eff556065449ebd478c3538

common/backtrace/backtrace.c
include/common/debug.h
plat/common/aarch64/plat_common.c
plat/marvell/armada/a3k/common/a3700_ea.c

index 89380b3e44b8b7ab37a48535c78d4f7a6fb5914e..f994ae5b6dda747bcbdf90cc2503f3622a3195d1 100644 (file)
@@ -54,17 +54,6 @@ static inline uintptr_t extract_address(uintptr_t address)
        return ret;
 }
 
-const char *get_el_str(unsigned int el)
-{
-       if (el == 3U) {
-               return "EL3";
-       } else if (el == 2U) {
-               return "EL2";
-       } else {
-               return "S-EL1";
-       }
-}
-
 /*
  * Returns true if the address points to a virtual address that can be read at
  * the current EL, false otherwise.
index a7ca0d7889e12cf9cbd511a24e9b70e67a5bf412..af47999c3e48e3ed36e1a52794d528066c95b0f1 100644 (file)
 # define VERBOSE(...)  no_tf_log(LOG_MARKER_VERBOSE __VA_ARGS__)
 #endif
 
+const char *get_el_str(unsigned int el);
+
 #if ENABLE_BACKTRACE
 void backtrace(const char *cookie);
-const char *get_el_str(unsigned int el);
 #else
 #define backtrace(x)
 #endif
index 8f998af3f4f0777cfc55c16e0b060ab381f8cf3b..851ed24fb8cd51594c0af1eeb6cd0add638f9cbe 100644 (file)
@@ -67,17 +67,15 @@ int plat_sdei_validate_entry_point(uintptr_t ep, unsigned int client_mode)
 }
 #endif
 
-#if !ENABLE_BACKTRACE
-static const char *get_el_str(unsigned int el)
+const char *get_el_str(unsigned int el)
 {
        if (el == MODE_EL3) {
                return "EL3";
        } else if (el == MODE_EL2) {
                return "EL2";
        }
-       return "S-EL1";
+       return "EL1";
 }
-#endif /* !ENABLE_BACKTRACE */
 
 /* RAS functions common to AArch64 ARM platforms */
 void plat_default_ea_handler(unsigned int ea_reason, uint64_t syndrome, void *cookie,
index bc12845a55b1c6a6033be15171f2088cb147bf0a..fd4e3b2471617df6faaf6a5aa94161c1ac43a250 100644 (file)
 
 #define A53_SERR_INT_AXI_SLVERR_ON_EXTERNAL_ACCESS 0xbf000002
 
-#if !ENABLE_BACKTRACE
-static const char *get_el_str(unsigned int el)
-{
-       if (el == MODE_EL3) {
-               return "EL3";
-       } else if (el == MODE_EL2) {
-               return "EL2";
-       }
-       return "S-EL1";
-}
-#endif /* !ENABLE_BACKTRACE */
-
 /*
  * This source file with custom plat_ea_handler function is compiled only when
  * building TF-A with compile option HANDLE_EA_EL3_FIRST=1