armv7-r/mpu: remove mpu_show_regioninfo()

mpu_show_regioninfo() shows MPU configurations for debugging purpose.
Though it might be useful and provide meaningful information to BSP
developers, there are already another tools for them to examine MPU
settings. To normal users, it only increases binary footprint almost
half kilobyte.

Here is footprint comparison with this commit:

                text   data    bss      dec filename
    Before      2568      0      5     2573 arm_assert.c
     After      2148      0      5     2153 arm_assert.c

Let's drop it.

Change-Id: Ia20e92f7380c2d56516f1b229836e08354d9c5ea
Signed-off-by: Heesub Shin <heesub.shin@samsung.com>

diff --git a/os/arch/arm/src/armv7-r/arm_assert.c b/os/arch/arm/src/armv7-r/arm_assert.c
index 8f5fcde..6e42419 100644 (file)
--- a/os/arch/arm/src/armv7-r/arm_assert.c
+++ b/os/arch/arm/src/armv7-r/arm_assert.c
@@ -685,10 +685,6 @@ static void up_dumpstate(void)
 
        dump_stack();
 
-#ifdef CONFIG_ARMV7M_MPU
-       mpu_show_regioninfo();
-#endif
-
        /* Dump the state of all tasks (if available) */
 
        up_showtasks();

diff --git a/os/arch/arm/src/armv7-r/mpu.h b/os/arch/arm/src/armv7-r/mpu.h
index 2b6d276..dc3ce5c 100644 (file)
--- a/os/arch/arm/src/armv7-r/mpu.h
+++ b/os/arch/arm/src/armv7-r/mpu.h
@@ -447,46 +447,6 @@ static inline inline uint32_t get_mpu_region_num(void)
 }
 
 /****************************************************************************
- * Name: mpu_show_regioninfo
- *
- * Description:
- *   Show the current mpu region setting
- *
- ****************************************************************************/
-
-static inline void mpu_show_regioninfo(void)
-{
-#if defined(CONFIG_DEBUG) && defined(CONFIG_DEBUG_ERROR)
-       int idx, temp;
-       uint32_t regval, base, size, attr;
-
-       /* save the current region before printing the information */
-       temp = get_mpu_region_num();
-
-       lldbg("*****************************************************************************\n");
-       lldbg("*REGION_NO.\tBASE_ADDRESS\t    SIZE\t    STATUS\tATTRIBUTES*\n");
-       lldbg("*****************************************************************************\n");
-       for (idx = 0; idx < 8; idx++) {
-               mpu_set_rgnr(idx);
-               regval = get_mpu_region_num();
-               base = get_mpu_region_base();
-               size = get_mpu_region_size();
-               attr = get_mpu_region_access_ctrl();
-               lldbg("%8d\t\t%8X\t%8X\t%8d\t%8X\n",
-                       regval & MPU_RGNR_MASK,
-                       base & MPU_RBAR_ADDR_MASK,
-                       size ? (1 << (((size & MPU_RASR_RSIZE_MASK) >> MPU_RASR_RSIZE_SHIFT) + 1)) : 0,
-                       size & MPU_RASR_ENABLE ? 1 : 0,
-                       attr);
-       }
-       lldbg("*****************************************************************************\n");
-
-       /* restore the previous region */
-       mpu_set_rgnr(temp);
-#endif
-}
-
-/****************************************************************************
  * Name: mpu_control
  *
  * Description: