On Tue, 1 Jul 2025 19:59:39 -0400 Steven Rostedt <rost...@goodmis.org> wrote:
> > + * __ftrace_event_enable_disable - enable or disable a trace event > > + * @file: trace event file associated with the event. > > + * @enable: 0 or 1 respectively to disable/enable the event (any other > > value is > > + * invalid). > > Saying 0 or 1 should assume that those are the only values. Don't need the > content in the parenthesis. BTW, it should be "0 or !0"? (or we should make it boolean) This description means if it is "2", that is undefined. Thanks, -- Masami Hiramatsu (Google) <mhira...@kernel.org>