On Wed, 02 Sep 2020 18:46:15 +0100,
Krzysztof Kozlowski <[email protected]> wrote:
>
> Fix kerneldoc W=1 warnings:
>
> drivers/irqchip/irq-ti-sci-inta.c:144: warning: Function parameter or
> member 'vint_id' not described in 'ti_sci_inta_xlate_irq'
> drivers/irqchip/irq-ti-sci-inta.c:144: warning: Excess function parameter
> 'irq' description in 'ti_sci_inta_xlate_irq'
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> drivers/irqchip/irq-ti-sci-inta.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/irqchip/irq-ti-sci-inta.c
> b/drivers/irqchip/irq-ti-sci-inta.c
> index bc863ef7998d..84929f3e2003 100644
> --- a/drivers/irqchip/irq-ti-sci-inta.c
> +++ b/drivers/irqchip/irq-ti-sci-inta.c
> @@ -134,7 +134,7 @@ static void ti_sci_inta_irq_handler(struct irq_desc *desc)
> /**
> * ti_sci_inta_xlate_irq() - Translate hwirq to parent's hwirq.
> * @inta: IRQ domain corresponding to Interrupt Aggregator
> - * @irq: Hardware irq corresponding to the above irq domain
> + * @vint_id: TISCI vint ID
I'm not sure that replacing a flaky variable name with something that
is essentially unintelligible is a huge improvement.
M.
--
Without deviation from the norm, progress is not possible.
