RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

bsps: Remove uses of BSP_GET_WORK_AREA_DEBUG

Browse Source 
The code covered by BSP_GET_WORK_AREA_DEBUG was basically dead code
since there was no normal way to activate it (e.g. via a BSP
configuration option).  A follow up patch will bring back this feature
through a CONFIGURE_VERBOSE_SYSTEM_INITIALIZATION configuration option.

Update #3838.
This commit is contained in:
Sebastian Huber
2019-12-13 14:22:16 +01:00
parent c184b0cf31
commit 07e2eacf89
3 changed files with 0 additions and 65 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
bsps/arm/gdbarmsim/include/bsp.h
View File

@@ -39,8 +39,6 @@
extern "C" { extern "C" {
#endif #endif
//#define BSP_GET_WORK_AREA_DEBUG 1
/** /**
* @brief Support for simulated clock tick * @brief Support for simulated clock tick
*/ */

35
bsps/include/bsp/bootcard.h
View File

@@ -115,42 +115,7 @@ static inline void bsp_work_area_initialize_default(
} }
#endif #endif
/*
* The following may be helpful in debugging what goes wrong when
* you are allocating the Work Area in a new BSP.
*/
#ifdef BSP_GET_WORK_AREA_DEBUG
{
void *sp = __builtin_frame_address(0);
void *end = (char *) area.begin + area.size;
printk(
"work_area_start = 0x%p\n"
"work_area_size = %lu 0x%08lx\n"
"end = 0x%p\n"
"current stack pointer = 0x%p%s\n",
area.begin,
(unsigned long) area.size, /* decimal */
(unsigned long) area.size, /* hexadecimal */
end,
sp,
(uintptr_t) sp >= (uintptr_t) area.begin
&& (uintptr_t) sp <= (uintptr_t) end ?
" OVERLAPS!" : ""
);
}
#endif
_Workspace_Handler_initialization(&area, 1, NULL); _Workspace_Handler_initialization(&area, 1, NULL);
#ifdef BSP_GET_WORK_AREA_DEBUG
printk(
"heap_start = 0x%p\n"
"heap_size = %lu\n",
area.begin,
(unsigned long) area.size
);
#endif
RTEMS_Malloc_Initialize(&area, 1, NULL); RTEMS_Malloc_Initialize(&area, 1, NULL);
} }

28
bsps/sparc/shared/start/bspgetworkarea.c
View File

@@ -10,12 +10,8 @@
* http://www.rtems.org/license/LICENSE. * http://www.rtems.org/license/LICENSE.
*/ */
/* #define BSP_GET_WORK_AREA_DEBUG */
#include <bsp.h> #include <bsp.h>
#include <bsp/bootcard.h> #include <bsp/bootcard.h>
#ifdef BSP_GET_WORK_AREA_DEBUG
#include <rtems/bspIo.h>
#endif
/* Tells us where to put the workspace in case remote debugger is present. */ /* Tells us where to put the workspace in case remote debugger is present. */
extern uint32_t rdb_start; extern uint32_t rdb_start;
@@ -31,29 +27,5 @@ void bsp_work_area_initialize(void)
void *work_area_start = (void *)&end; void *work_area_start = (void *)&end;
uintptr_t work_area_size = (uintptr_t)rdb_start - (uintptr_t)work_area_start; uintptr_t work_area_size = (uintptr_t)rdb_start - (uintptr_t)work_area_start;
/*
* The following may be helpful in debugging what goes wrong when
* you are allocating the Work Area in a new BSP.
*/
#ifdef BSP_GET_WORK_AREA_DEBUG
{
void *sp = __builtin_frame_address(0);
void *end = *work_area_start + *work_area_size;
printk(
"work_area_start = 0x%p\n"
"work_area_size = %d 0x%08x\n"
"end = 0x%p\n"
"current stack pointer = 0x%p%s\n",
work_area_start,
work_area_size, /* decimal */
work_area_size, /* hexadecimal */
end,
sp,
((sp >= *work_area_start && sp <= end) ? " OVERLAPS!" : "")
);
printk( "rdb_start = 0x%08x\n", rdb_start );
}
#endif
bsp_work_area_initialize_default(work_area_start, work_area_size); bsp_work_area_initialize_default(work_area_start, work_area_size);
} }