Re: [PATCH] x86/acpi: Fix improper handling of SCI INT for platforms supporting only IOAPIC mode

From: Rafael J. Wysocki
Date: Wed Nov 15 2017 - 12:22:48 EST


On Fri, Nov 10, 2017 at 10:38 AM, Vikas C Sajjan
<vikas.cha.sajjan@xxxxxxx> wrote:
> The platforms which support only IOAPIC mode and whose SCI INT is
> greater than 16, passes SCI INT via FADT and not via MADT int src override
> structure. In such cases current logic fails to handle it and throws error
> "Invalid bus_irq %u for legacy override". This patch handles the above
> mentioned case. While at it, also modify function mp_override_legacy_irq()
> to use the newly introduced function mp_register_ioapic_irq().

Actually, is it necessary to make this extra change here?

How complicated would it be to separate it out?

Thanks,
Rafael