From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932426Ab3LENv6 (ORCPT ); Thu, 5 Dec 2013 08:51:58 -0500 Received: from mail-pb0-f41.google.com ([209.85.160.41]:47639 "EHLO mail-pb0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932203Ab3LENvz (ORCPT ); Thu, 5 Dec 2013 08:51:55 -0500 Message-ID: <52A084EB.5010103@linaro.org> Date: Thu, 05 Dec 2013 21:51:39 +0800 From: Hanjun Guo User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Arnd Bergmann CC: linux-arm-kernel@lists.infradead.org, "Rafael J. Wysocki" , Catalin Marinas , Will Deacon , Russell King - ARM Linux , Daniel Lezcano , Mark Rutland , Matthew Garrett , linaro-kernel@lists.linaro.org, Graeme Gregory , Al Stone , patches@linaro.org, Linus Walleij , Olof Johansson , linux-kernel@vger.kernel.org, Rob Herring , linaro-acpi@lists.linaro.org, linux-acpi@vger.kernel.org, Jon Masters , Grant Likely , Bjorn Helgaas Subject: Re: [RFC part1 PATCH 5/7] ARM64 / ACPI: Introduce arm_core.c and its related head file References: <1386088611-2801-1-git-send-email-hanjun.guo@linaro.org> <1386088611-2801-6-git-send-email-hanjun.guo@linaro.org> <201312050438.05117.arnd@arndb.de> In-Reply-To: <201312050438.05117.arnd@arndb.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2013年12月05日 11:38, Arnd Bergmann wrote: > On Tuesday 03 December 2013, Hanjun Guo wrote: >> +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