Re: [PATCH v3 1/2] irqchip: Kconfig: module build support for the TI interrupt router driver

From: Thomas Gleixner
Date: Wed Oct 16 2024 - 12:38:29 EST


On Wed, Oct 16 2024 at 11:41, Guillaume La Roque wrote:
> From: Nicolas Frayer <nfrayer@xxxxxxxxxxxx>

irqchip: Kconfig: is not a valid prefix.

This is about the TI SCI router, so this wants to use the
irqchip/ti-whatever prefix.

>
> Added module build support in Kconfig for the TI SCI interrupt router
> driver

Added?

This wants to be 'Add ...'

You fail to explain why it is valid to build this as a module, i.e. you
did the analysis that there is no dependency on this before modules can
be loaded.

> MODULE_AUTHOR("Lokesh Vutla <lokeshvutla@ticom>");
> MODULE_DESCRIPTION("K3 Interrupt Router driver over TI SCI protocol");
> +MODULE_LICENSE("GPL");

This change is not mentioned in the change log.

Thanks,

tglx