On Thu, 12 Aug 2021 11:59:28 +0300 "Yordan Karadzhov (VMware)" <y.karadz@xxxxxxxxx> wrote: > -void tep_print_field(struct trace_seq *s, void *data, > - struct tep_format_field *field) > +void static inline _tep_print_field(struct trace_seq *s, void *data, > + struct tep_format_field *field, > + struct tep_print_parse **parse_ptr) Same here. We should call this simply "print_field()". -- Steve > { > struct tep_event *event = field->event; > + struct tep_print_parse *start_parse; > struct tep_print_parse *parse; > bool has_0x;