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>

Reply via email to