From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hanjun Guo Subject: Re: [RFC part3 PATCH 2/2] ARM64 / clocksource: Use arch_timer_acpi_init() Date: Wed, 04 Dec 2013 00:30:35 +0800 Message-ID: <529E072B.3060901@linaro.org> References: <1386069328-22502-1-git-send-email-hanjun.guo@linaro.org> <1386069328-22502-3-git-send-email-hanjun.guo@linaro.org> <529DE21E.4080707@linaro.org> <20131203144309.GC7552@e106331-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20131203144309.GC7552@e106331-lin.cambridge.arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Mark Rutland Cc: Linus Walleij , Daniel Lezcano , "Rafael J. Wysocki" , Catalin Marinas , Will Deacon , Russell King - ARM Linux , "grant.likely@linaro.org" , Matthew Garrett , Olof Johansson , Bjorn Helgaas , "rob.herring@calxeda.com" , Jon Masters , ACPI Devel Maling List , "linux-arm-kernel@lists.infradead.org" , Patch Tracking , "linux-kernel@vger.kernel.org" , linaro-kernel , "linaro-acpi@lists.linaro.org" List-Id: linux-acpi@vger.kernel.org On 2013=E5=B9=B412=E6=9C=8803=E6=97=A5 22:43, Mark Rutland wrote: > On Tue, Dec 03, 2013 at 02:13:49PM +0000, Linus Walleij wrote: >> On Tue, Dec 3, 2013 at 2:52 PM, Hanjun Guo w= rote: >> >>> I will introduce has_arch_timer_node() as you said and use >>> it as follows: >>> >>> if (has_arch_timer_node()) >>> clocksource_of_init(); >>> esle >>> arch_timer_acpi_init(); /* try ACPI way */ >>> >>> Is this make sense to you? > What does arch_timer_acpi_init() do? Is it just detecting the presenc= e > of the timer, or grabbing the rate from a property in an ACPI table? It seems that you didn't get the PATCH 1/2, and my part1/part2 patch se= t is also missing, I will resend all the patch set, sorry for the noise. Thanks Hanjun