> On 16 Dec 2019, at 11:42 pm, Sebastian Huber 
> <sebastian.hu...@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
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to