Re: [PATCH V13 02/13] ACPI: irq: Allow acpi_gsi_to_irq() to have an arch-specific fallback

[Hanjun Guo]

On 2022/6/27 19:39, Jianmin Lv wrote:
> From: Marc Zyngier <maz@xxxxxxxxxx>
>
> It appears that the generic version of acpi_gsi_to_irq() doesn't
> fallback to establishing a mapping if there is no pre-existing
> one while the x86 version does.
>
> While arm64 seems unaffected by it, LoongArch is relying on the x86
> behaviour. In an effort to prevent new architectures from reinventing
> the proverbial wheel, provide an optional callback that the arch code
> can set to restore the x86 behaviour.
>
> Hopefully we can eventually get rid of this in the future once
> the expected behaviour has been clarified.
>
> Reported-by: Jianmin Lv <lvjianmin@xxxxxxxxxxx>

Why do we need the Reported-by tag here?

> Signed-off-by: Marc Zyngier <maz@xxxxxxxxxx>
> Signed-off-by: Jianmin Lv <lvjianmin@xxxxxxxxxxx>
> ---
>   drivers/acpi/irq.c   | 18 ++++++++++++++++--
>   include/linux/acpi.h |  1 +
>   2 files changed, 17 insertions(+), 2 deletions(-)

The rest is looking good to me,

Reviewed-by: Hanjun Guo <guohanjun@xxxxxxxxxx>
Tested-by: Hanjun Guo <guohanjun@xxxxxxxxxx>

Thanks
Hanjun