target: Fix compiler error when define DEBUG_PROTO_IS_NOOP

[iysheng]

Detailed description

Fix compile error when define DEBUG_PROTO_IS_NOOP like below

Your checklist for this pull request

• [x] I've read the Code of Conduct
• [ x I've read the guidelines for contributing to this repository
• [ ] It builds for hardware native (see Building the firmware)
• [x] It builds as BMDA (see Building the BMDA)
• [ ] I've tested it to the best of my ability
• [x] My commit messages provide a useful short description of what the commits do

Closing issues

[dragonmux]

We're unsure what your build environment is doing there.. it is doing something fantastically wrong with BMDA.. but the guards this removes are not wrong. Removing them is very much wrong.

The way this mechanism is designed to work and indeed does work in-tree, is that the _IS_NOOP macros determine if that log level is a no-operation via the DEBUG_<level> macro being defined to do nothing and discard the logged information. See: https://github.com/blackmagic-debug/blackmagic/blob/e4da644bb8c0c81498a0f3819bbd99fd6627cd8e/src/include/general.h#L99-L120

The idea being that the _IS_NOOP macros can then be used to guard data used only for logging statements (like these data pointers) and not define them when the statements do nothing, resulting in unused variable warnings that are then errors via -Werror.

Your build environment is mistakenly defining DEBUG_PROTO_IS_NOOP here when building BMDA, and all of the logging levels are never no-ops for BMDA - all of them always resolve to debug_* calls.

[iysheng]

ok，I just force design DEBUG_PROTO_IS_NOOP in my Makefile to triger this error.😅