From: Hanjun Guo <hanjun.guo@linaro.org> To: "Suthikulpanit, Suravee" <Suravee.Suthikulpanit@amd.com>, Catalin Marinas <catalin.marinas@arm.com>, "Rafael J. Wysocki" <rjw@rjwysocki.net>, Olof Johansson <olof@lixom.net>, Arnd Bergmann <arnd@arndb.de>, Mark Rutland <mark.rutland@arm.com>, Grant Likely <grant.likely@linaro.org>, Will Deacon <will.deacon@arm.com> Cc: Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>, Graeme Gregory <graeme.gregory@linaro.org>, Sudeep Holla <Sudeep.Holla@arm.com>, Jon Masters <jcm@redhat.com>, Jason Cooper <jason@lakedaemon.net>, Marc Zyngier <marc.zyngier@arm.com>, Bjorn Helgaas <bhelgaas@google.com>, Mark Brown <broonie@kernel.org>, Rob Herring <robh@kernel.org>, Robert Richter <rric@kernel.org>, Randy Dunlap <rdunlap@infradead.org>, "Charles.Garcia-Tobin@arm.com" <Charles.Garcia-Tobin@arm.com>, "phoenix.liyi@huawei.com" <phoenix.liyi@huawei.com>, Timur Tabi <timur@codeaurora.org>, "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linaro-acpi@lists.linaro.org" <linaro-acpi@lists.linaro.org> Subject: Re: [PATCH v6 14/17] ARM64 / ACPI: Parse GTDT to initialize arch timer Date: Mon, 05 Jan 2015 16:59:44 +0800 [thread overview] Message-ID: <54AA5280.3050802@linaro.org> (raw) In-Reply-To: <D0CF9C94.CD69%suravee.suthikulpanit@amd.com> On 2015年01月05日 15:55, Suthikulpanit, Suravee wrote: > On 1/4/15, 04:55, "Hanjun Guo" <hanjun.guo@linaro.org> wrote: > >> Using the information presented by GTDT to initialize the arch >> timer (not memory-mapped). >> >> Originally-by: Amit Daniel Kachhap <amit.daniel@samsung.com> >> Tested-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> >> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> >> --- >> arch/arm64/kernel/time.c | 7 ++ >> drivers/clocksource/arm_arch_timer.c | 132 >> ++++++++++++++++++++++++++++------- >> include/linux/clocksource.h | 6 ++ >> 3 files changed, 118 insertions(+), 27 deletions(-) [...] >> +static void __init arch_timer_of_init(struct device_node *np) >> +{ >> + int i; >> + >> + if (arch_timers_present & ARCH_CP15_TIMER) { >> + pr_warn("arch_timer: multiple nodes in dt, skipping\n"); >> + return; >> + } >> + >> + arch_timers_present |= ARCH_CP15_TIMER; >> + for (i = PHYS_SECURE_PPI; i < MAX_TIMER_PPI; i++) >> + arch_timer_ppi[i] = irq_of_parse_and_map(np, i); >> + >> + arch_timer_detect_rate(NULL, np); >> + >> + arch_timer_c3stop = !of_property_read_bool(np, "always-on"); >> + >> + /* >> + * If we cannot rely on firmware initializing the timer registers then >> + * we should use the physical timers instead. >> + */ >> + if (IS_ENABLED(CONFIG_ARM) && >> + of_property_read_bool(np, "arm,cpu-registers-not-fw-configured")) >> + arch_timer_use_virtual = false; >> + >> + arch_timer_init(); >> +} >> +CLOCKSOURCE_OF_DECLARE(armv7_arch_timer, "arm,armv7-timer", >> arch_timer_of_init); >> +CLOCKSOURCE_OF_DECLARE(armv8_arch_timer, "arm,armv8-timer", >> arch_timer_of_init); > > Hanjun, > > FYI, it seems that the tree that you have rebased the patch series has an > issue determining clocksource in ARM64 introduced by this commit. > > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=0 > b46b8a718c6e90910a1b1b0fe797be3c167e186 > > Here is the fix from Catalin that already went upstream: > > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drive > rs/clocksource/arm_arch_timer.c?id=d6ad36913083d683aad4e02e53580c995f1a6ede This bug fix patch was merged in Dec 30 by Linus, which is the date after 3.19-rc2 released :) Thanks Hanjun
WARNING: multiple messages have this Message-ID (diff)
From: hanjun.guo@linaro.org (Hanjun Guo) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v6 14/17] ARM64 / ACPI: Parse GTDT to initialize arch timer Date: Mon, 05 Jan 2015 16:59:44 +0800 [thread overview] Message-ID: <54AA5280.3050802@linaro.org> (raw) In-Reply-To: <D0CF9C94.CD69%suravee.suthikulpanit@amd.com> On 2015?01?05? 15:55, Suthikulpanit, Suravee wrote: > On 1/4/15, 04:55, "Hanjun Guo" <hanjun.guo@linaro.org> wrote: > >> Using the information presented by GTDT to initialize the arch >> timer (not memory-mapped). >> >> Originally-by: Amit Daniel Kachhap <amit.daniel@samsung.com> >> Tested-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> >> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> >> --- >> arch/arm64/kernel/time.c | 7 ++ >> drivers/clocksource/arm_arch_timer.c | 132 >> ++++++++++++++++++++++++++++------- >> include/linux/clocksource.h | 6 ++ >> 3 files changed, 118 insertions(+), 27 deletions(-) [...] >> +static void __init arch_timer_of_init(struct device_node *np) >> +{ >> + int i; >> + >> + if (arch_timers_present & ARCH_CP15_TIMER) { >> + pr_warn("arch_timer: multiple nodes in dt, skipping\n"); >> + return; >> + } >> + >> + arch_timers_present |= ARCH_CP15_TIMER; >> + for (i = PHYS_SECURE_PPI; i < MAX_TIMER_PPI; i++) >> + arch_timer_ppi[i] = irq_of_parse_and_map(np, i); >> + >> + arch_timer_detect_rate(NULL, np); >> + >> + arch_timer_c3stop = !of_property_read_bool(np, "always-on"); >> + >> + /* >> + * If we cannot rely on firmware initializing the timer registers then >> + * we should use the physical timers instead. >> + */ >> + if (IS_ENABLED(CONFIG_ARM) && >> + of_property_read_bool(np, "arm,cpu-registers-not-fw-configured")) >> + arch_timer_use_virtual = false; >> + >> + arch_timer_init(); >> +} >> +CLOCKSOURCE_OF_DECLARE(armv7_arch_timer, "arm,armv7-timer", >> arch_timer_of_init); >> +CLOCKSOURCE_OF_DECLARE(armv8_arch_timer, "arm,armv8-timer", >> arch_timer_of_init); > > Hanjun, > > FYI, it seems that the tree that you have rebased the patch series has an > issue determining clocksource in ARM64 introduced by this commit. > > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=0 > b46b8a718c6e90910a1b1b0fe797be3c167e186 > > Here is the fix from Catalin that already went upstream: > > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drive > rs/clocksource/arm_arch_timer.c?id=d6ad36913083d683aad4e02e53580c995f1a6ede This bug fix patch was merged in Dec 30 by Linus, which is the date after 3.19-rc2 released :) Thanks Hanjun
next prev parent reply other threads:[~2015-01-05 8:59 UTC|newest] Thread overview: 87+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-01-04 10:55 [PATCH v6 00/17] Introduce ACPI for ARM64 based on ACPI 5.1 Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 01/17] ACPI / processor: Convert apic_id to phys_id to make it arch agnostic Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-07 1:50 ` Rafael J. Wysocki 2015-01-07 1:50 ` Rafael J. Wysocki 2015-01-07 1:50 ` Rafael J. Wysocki 2015-01-07 9:49 ` Hanjun Guo 2015-01-07 9:49 ` Hanjun Guo 2015-01-07 9:49 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 02/17] ACPI / processor: Rename acpi_(un)map_lsapic() to acpi_(un)map_cpu() Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 03/17] ARM64 / ACPI: Get RSDP and ACPI boot-time tables Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 04/17] ARM64 / ACPI: Introduce sleep-arm.c Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 05/17] ARM64 / ACPI: Introduce early_param for "acpi" and pass acpi=force to enable ACPI Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 06/17] ARM64 / ACPI: If we chose to boot from acpi then disable FDT Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 07/17] ARM64 / ACPI: Make PCI optional for ACPI on ARM64 Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 08/17] ARM64 / ACPI: Parse FADT table to get PSCI flags for PSCI init Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-09 19:04 ` Lorenzo Pieralisi 2015-01-09 19:04 ` Lorenzo Pieralisi 2015-01-09 19:04 ` Lorenzo Pieralisi 2015-01-12 4:26 ` Hanjun Guo 2015-01-12 4:26 ` Hanjun Guo 2015-01-12 4:26 ` Hanjun Guo 2015-01-12 11:41 ` Lorenzo Pieralisi 2015-01-12 11:41 ` Lorenzo Pieralisi 2015-01-12 11:41 ` Lorenzo Pieralisi 2015-01-13 6:54 ` Hanjun Guo 2015-01-13 6:54 ` Hanjun Guo 2015-01-13 6:54 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 09/17] ACPI / table: Print GIC information when MADT is parsed Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 10/17] ARM64 / ACPI: Parse MADT for SMP initialization Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 11/17] ACPI / processor: Make it possible to get CPU hardware ID via GICC Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 12/17] ARM64 / ACPI: Introduce ACPI_IRQ_MODEL_GIC and register device's gsi Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 13/17] ARM64 / ACPI: Add GICv2 specific ACPI boot support Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 14/17] ARM64 / ACPI: Parse GTDT to initialize arch timer Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-05 7:55 ` Suthikulpanit, Suravee 2015-01-05 7:55 ` Suthikulpanit, Suravee 2015-01-05 7:55 ` Suthikulpanit, Suravee 2015-01-05 8:59 ` Hanjun Guo [this message] 2015-01-05 8:59 ` Hanjun Guo 2015-01-05 8:59 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 15/17] ARM64 / ACPI: Select ACPI_REDUCED_HARDWARE_ONLY if ACPI is enabled on ARM64 Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 16/17] ARM64 / ACPI: Enable ARM64 in Kconfig Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-04 10:55 ` [PATCH v6 17/17] Documentation: ACPI for ARM64 Hanjun Guo 2015-01-04 10:55 ` Hanjun Guo 2015-01-06 9:52 ` 答复: " liyi 00215672 2015-01-06 9:52 ` liyi 00215672 2015-01-06 11:21 ` Hanjun Guo 2015-01-06 11:21 ` Hanjun Guo 2015-01-06 11:21 ` Hanjun Guo 2015-01-06 20:49 ` Arnd Bergmann 2015-01-06 20:49 ` Arnd Bergmann 2015-01-06 20:49 ` Arnd Bergmann 2015-01-19 20:33 ` Christoffer Dall 2015-01-19 20:33 ` Christoffer Dall 2015-01-19 20:33 ` Christoffer Dall 2015-01-21 12:37 ` Hanjun Guo 2015-01-21 12:37 ` Hanjun Guo 2015-01-21 12:37 ` Hanjun Guo 2015-01-21 13:03 ` Stefano Stabellini 2015-01-21 13:03 ` Stefano Stabellini 2015-01-21 13:03 ` Stefano Stabellini 2015-01-21 16:04 ` Christoffer Dall 2015-01-21 16:04 ` Christoffer Dall 2015-01-21 16:04 ` Christoffer Dall 2015-01-06 7:05 ` [PATCH v6 00/17] Introduce ACPI for ARM64 based on ACPI 5.1 Yijing Wang 2015-01-06 7:05 ` Yijing Wang 2015-01-06 7:05 ` Yijing Wang 2015-01-06 11:17 ` Hanjun Guo 2015-01-06 11:17 ` Hanjun Guo 2015-01-06 11:17 ` Hanjun Guo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=54AA5280.3050802@linaro.org \ --to=hanjun.guo@linaro.org \ --cc=Charles.Garcia-Tobin@arm.com \ --cc=Lorenzo.Pieralisi@arm.com \ --cc=Sudeep.Holla@arm.com \ --cc=Suravee.Suthikulpanit@amd.com \ --cc=arnd@arndb.de \ --cc=bhelgaas@google.com \ --cc=broonie@kernel.org \ --cc=catalin.marinas@arm.com \ --cc=graeme.gregory@linaro.org \ --cc=grant.likely@linaro.org \ --cc=jason@lakedaemon.net \ --cc=jcm@redhat.com \ --cc=linaro-acpi@lists.linaro.org \ --cc=linux-acpi@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=marc.zyngier@arm.com \ --cc=mark.rutland@arm.com \ --cc=olof@lixom.net \ --cc=phoenix.liyi@huawei.com \ --cc=rdunlap@infradead.org \ --cc=rjw@rjwysocki.net \ --cc=robh@kernel.org \ --cc=rric@kernel.org \ --cc=timur@codeaurora.org \ --cc=will.deacon@arm.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.