On Tue, 25 Nov 2025 07:01:43 -0600
Tom Zanussi <[email protected]> wrote:
> > -static const struct event_trigger_ops event_hist_trigger_ops = {
> > - .trigger = event_hist_trigger,
> > - .print = event_hist_trigger_print,
> > - .init = event_hist_trigger_init,
> > - .free = event_hist_trigger_free,
> > -};
> > +static struct event_command trigger_hist_cmd;
> >
>
> I don't think this is needed, since the same declaration already
> appears just above the event_hist_trigger_parse() declaration.
No it doesn't. Nice catch. I'll send a v2.
Thanks for the review.
-- Steve