On Mon, Oct 27, 2025 at 12:55 PM Andy Shevchenko andriy.shevchenko@intel.com wrote:
On Mon, Oct 27, 2025 at 11:18:17AM +0100, Linus Walleij wrote:
Hmm... Isn't this already being discussed somewhere? I have some déjà-vu.
I brought it up on the PostmarkeOS IRC, otherwise I don't know.
/* We do not always have an IRQ */if (!data->has_irq)Wouldn't check for 0 be enough?
if (!data->irq)
But this driver does not store the IRQ number in the instance struct because it isn't used outside of the probe() function.
The information that needs to be stored is bool so that's why I stored a bool.
Yours, Linus Walleij