[PATCH 6/7] bsps: Remove uses of BSP_GET_WORK_AREA_DEBUG

Chris Johns chrisj at rtems.org
Mon Dec 16 23:34:55 UTC 2019


> On 16 Dec 2019, at 11:42 pm, Sebastian Huber <Sebastian.Huber at embedded-brains.de> wrote:
> 
> 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).  BSP developers should use a debugger instead.

You will need to establish a base level requirement of a debugger across the whole project before this statement makes sense and doing that does not make sense to me.

> If this output via printk() turns out to be still wanted by users, it
> can be added as an application configuration option, e.g.
> CONFIGURE_ENABLE_VERBOSE_MEMORY_INITIALIZATION.

Why not use that config define that other part you want to removed?

Also  please consider at this time of year I am really busy and patches need to sit longer before being considered as OK. :)

Chris


More information about the devel mailing list