On Tue, Feb 27, 2018 at 11:09 AM, Peter Maydell <peter.mayd...@linaro.org> wrote: > On 27 February 2018 at 07:41, Linus Walleij <linus.wall...@linaro.org> wrote: >> On Sat, Feb 17, 2018 at 7:32 PM, Philippe Mathieu-Daudé <f4...@amsat.org> >> wrote: >> >>> [Me] >>>> +#define DEBUG_SII9022 0 >>>> + >>>> +#define DPRINTF(fmt, ...) \ >>>> + do { \ >>>> + if (DEBUG_SII9022) { \ >>>> + printf("sii9022: " fmt, ## __VA_ARGS__); \ >>>> + } \ >>>> + } while (0) >>> >>> Can you replace DPRINTF() by trace events? >> >> Absolutely but which ones? >> >> I do not feel senior enough to also invent new trace events >> for displays or I2C devices... > > Just put a trace event where you've put DPRINTF debug statements.
Yeah, hm the question might be silly or something but I don't know how to do that. I guess I should include "trace.h". #include "trace.h" says it is autogenerated from tracetool. Is there some documentation I should be reading or a good example commit I can study to get the hang of it? Yours, Linus Walleij