On Fri, 2025-03-28 at 17:55 +0530, Roopni Devanathan wrote: > > +TRACE_EVENT(drv_set_rts_threshold, > + TP_PROTO(struct ieee80211_local *local, s8 radio_id, u32 value), > + > + TP_ARGS(local, radio_id, value), > + > + TP_STRUCT__entry( > + LOCAL_ENTRY > + __field(s8, radio_id) > + __field(u32, value) > + ), > + > + TP_fast_assign( > + LOCAL_ASSIGN; > + __entry->radio_id = radio_id; > + __entry->value = value; > + ), > + > + TP_printk( > + LOCAL_PR_FMT " value:%d", > + LOCAL_PR_ARG, __entry->value > + ) > I think it'd make sense (and be trivial) to have the radio ID here. Same in the first patch, but I missed it because the print was hidden outside the context :) johannes