On 07/10/2015 05:19 PM, Lorenzo Pieralisi wrote:
On Thu, Jul 09, 2015 at 06:44:28AM +0100, Suravee Suthikulpanit wrote:
This patch declares acpi_register_gsi() as weak and allows architecture to specify its own acpi_register_gsi().
In ARM64 case, we need to specify GIC_INT_TYPE_XXX when calling irq_domain_ops.init_alloc_info(), which is specific to ARM64.
I think it is better to define an ACPI hook that allows you to retrieve what interrupt controller type you want to alloc from instead of adding an acpi_register_gsi function that is 99% generic to arm64 code.
I will review this code on the kernel mailing lists, but I am not happy with this patch approach.
Lorenzo
Noted, and I have refactor some code and implement the hook. I'll send the new revision soon.
Suravee