Re: [PATCH v5 3/6] rust: irq: add support for non-threaded IRQs and handlers
From: Alice Ryhl
Date: Fri Jul 04 2025 - 03:32:25 EST
On Thu, Jul 03, 2025 at 08:49:07PM +0200, Danilo Krummrich wrote:
> On 7/3/25 7:12 PM, Daniel Almeida wrote:
> > > +/// Callbacks for an IRQ handler.
> > > +pub trait Handler: Sync {
> >
> > I wonder if we should require ’static here too?
> >
> > Same for the Threaded trait.
>
> You already have
>
> impl<T: Handler + 'static> Registration<T>
>
> which I think this is good enough.
If we're not going to support non-static handlers, then I think it's
simpler to place the 'static bound on the trait.
Alice