On 2013?12?05? 11:38, Arnd Bergmann wrote:

On Tuesday 03 December 2013, Hanjun Guo wrote:

quoted

+static unsigned int gsi_to_irq(unsigned int gsi)
+{
+       int irq = irq_create_mapping(NULL, gsi);
+
+       return irq;
+}

I think this could use a comment regarding your plans for IRQ domains.

Do you expect that all ACPI systems would have only a single GIC IRQ
controller and a single domain, or do you plan to add irqdomain code
later?

we added irqdomain code in the part2 patch set, is that the code ok
with you?

Thanks
Hanjun