Re: [PATCH 2/2] irqchip/ti-sci-inta: Fix kerneldoc

[Marc Zyngier]

On Wed, 02 Sep 2020 18:46:15 +0100,
Krzysztof Kozlowski <krzk@xxxxxxxxxx> 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 <krzk@xxxxxxxxxx>
> ---
>  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.