[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