Re: [patch 03/30] genirq: Move irq_set_lockdep_class() to core

From: Andy Shevchenko
Date: Fri Dec 11 2020 - 12:55:28 EST


On Thu, Dec 10, 2020 at 10:14 PM Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
>
> irq_set_lockdep_class() is used from modules and requires irq_to_desc() to
> be exported. Move it into the core code which lifts another requirement for
> the export.

...

> + if (IS_ENABLED(CONFIG_LOCKDEP))
> + __irq_set_lockdep_class(irq, lock_class, request_class);

Maybe I missed something, but even if the compiler does not warn the
use of if IS_ENABLED() with complimentary #ifdef seems inconsistent.

> +#ifdef CONFIG_LOCKDEP
...
> +EXPORT_SYMBOL_GPL(irq_set_lockdep_class);
> +#endif


--
With Best Regards,
Andy Shevchenko