Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932270AbaAXAVH (ORCPT ); Thu, 23 Jan 2014 19:21:07 -0500 Received: from mail-pa0-f45.google.com ([209.85.220.45]:35164 "EHLO mail-pa0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751957AbaAXAVF (ORCPT ); Thu, 23 Jan 2014 19:21:05 -0500 Message-ID: <52E1B1DE.5050507@linaro.org> Date: Fri, 24 Jan 2014 08:20:46 +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: Mark Rutland CC: Linus Walleij , "Rafael J. Wysocki" , Catalin Marinas , Will Deacon , Russell King - ARM Linux , ACPI Devel Maling List , "linux-arm-kernel@lists.infradead.org" , "grant.likely@linaro.org" , Matthew Garrett , Olof Johansson , Bjorn Helgaas , Rob Herring , Arnd Bergmann , Patch Tracking , "linux-kernel@vger.kernel.org" , linaro-kernel , "linaro-acpi@lists.linaro.org" , Charles Garcia-Tobin , Amit Daniel Kachhap Subject: Re: [PATCH 18/20] clocksource / acpi: Add macro CLOCKSOURCE_ACPI_DECLARE References: <1389961514-13562-1-git-send-email-hanjun.guo@linaro.org> <1389961514-13562-19-git-send-email-hanjun.guo@linaro.org> <20140122114537.GA15591@e106331-lin.cambridge.arm.com> In-Reply-To: <20140122114537.GA15591@e106331-lin.cambridge.arm.com> 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 2014年01月22日 19:45, Mark Rutland wrote: > On Wed, Jan 22, 2014 at 08:26:50AM +0000, Linus Walleij wrote: >> On Fri, Jan 17, 2014 at 1:25 PM, Hanjun Guo wrote: >> >>> From: Amit Daniel Kachhap >>> >>> This macro does the same job as CLOCKSOURCE_OF_DECLARE. The device >>> name from the ACPI timer table is matched with all the registered >>> timer controllers and matching initialisation routine is invoked. >>> >>> Signed-off-by: Amit Daniel Kachhap >>> Signed-off-by: Hanjun Guo >> Actually I have a fat patch renaming CLOCKSOURCE_OF_DECLARE() >> to TIMER_OF_DECLARE() and I think this macro, if needed, should >> be named TIMER_ACPI_DECLARE(). >> >> The reason is that "clocksource" is a Linux-internal name and this >> macro pertains to the hardware name in respective system >> description type. >> >>> +#ifdef CONFIG_ACPI >>> +#define CLOCKSOURCE_ACPI_DECLARE(name, compat, fn) \ >>> + static const struct acpi_device_id __clksrc_acpi_table_##name \ >>> + __used __section(__clksrc_acpi_table) \ >>> + = { .id = compat, \ >>> + .driver_data = (kernel_ulong_t)fn } >>> +#else >>> +#define CLOCKSOURCE_ACPI_DECLARE(name, compat, fn) >>> +#endif >> This hammers down the world to compile one binary for ACPI >> and one binary for device tree. Maybe that's fine, I don't know. > How does it do that? > > As far as I could tell CONFIG_ACPI and CONFIG_OF are not mutually > exclusive, and this just means that we only build the datastructures for > matching from ACPI when CONFIG_ACPI is enabled. > > Have I missed something? > > I definitely don't want to see mutually exclusive ACPI and DT support. ACPI and DT did the same job so I think they should mutually exclusive. if we enable both DT and ACPI in one system, this will leading confusions. Further more, firmware guys will be happy to present all the devices in one way, not both of them. Thanks Hanjun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/