From: Marc Zyngier <marc.zyngier@arm.com> To: Vitaly Kuzmichev <vkuzmichev@mvista.com> Cc: "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-watchdog@vger.kernel.org" <linux-watchdog@vger.kernel.org>, Russell King <linux@arm.linux.org.uk>, Wim Van Sebroeck <wim@iguana.be>, "arm@kernel.org" <arm@kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Arnd Bergmann <arnd@arndb.de>, John Stultz <johnstul@us.ibm.com>, Nicolas Pitre <nico@fluxnic.net>, Thomas Gleixner <tglx@linutronix.de> Subject: Re: [PATCH V2 0/6] arm_smp_twd: mpcore_wdt: Fix MPCORE watchdog setup Date: Wed, 06 Jul 2011 11:05:51 +0100 [thread overview] Message-ID: <4E14337F.1050101@arm.com> (raw) In-Reply-To: <1309892440-3260-1-git-send-email-vkuzmichev@mvista.com> Hi Vitaly, On 05/07/11 20:00, Vitaly Kuzmichev wrote: > The series of patches fixes various bugs in ARM MPCORE watchdog setup. > They also introduce some changes in common ARM_SMP_TWD timer setup code. > The PATCH 2/6 adds an exported function to obtain twd timer rate in > mpcore_wdt. However, this solution might be unacceptable, and the > better is to copy calibration loop and clk_get_rate calling to mpcore_wdt. My take on this is that the calibration loop should simply be killed once Colin's patch using the "smp_twd" clock is merged and relevant platforms updated. Once this is done, the watchdog driver should use the clock directly, without exporting a symbol. > The series of patches is based on arm-platforms.git/local_timers_as_devices > branch since it looks like that it is going to be merged into the mainline. Be careful here. This branch is a work in progress, likely to change very quickly, contains code that has not been posted to the ML yet, and will probably eat your pet for breakfast. As far as mainline merging is concerned, there is still a long way to go (see the GIC consolidation patches, on which the local_timers_as_devices branch relies). > V1 series was not discussed widely because I forgot to add all necessary > maintainers in Cc. Thanks for that. M. -- Jazz is not dead. It just smells funny...
WARNING: multiple messages have this Message-ID (diff)
From: marc.zyngier@arm.com (Marc Zyngier) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH V2 0/6] arm_smp_twd: mpcore_wdt: Fix MPCORE watchdog setup Date: Wed, 06 Jul 2011 11:05:51 +0100 [thread overview] Message-ID: <4E14337F.1050101@arm.com> (raw) In-Reply-To: <1309892440-3260-1-git-send-email-vkuzmichev@mvista.com> Hi Vitaly, On 05/07/11 20:00, Vitaly Kuzmichev wrote: > The series of patches fixes various bugs in ARM MPCORE watchdog setup. > They also introduce some changes in common ARM_SMP_TWD timer setup code. > The PATCH 2/6 adds an exported function to obtain twd timer rate in > mpcore_wdt. However, this solution might be unacceptable, and the > better is to copy calibration loop and clk_get_rate calling to mpcore_wdt. My take on this is that the calibration loop should simply be killed once Colin's patch using the "smp_twd" clock is merged and relevant platforms updated. Once this is done, the watchdog driver should use the clock directly, without exporting a symbol. > The series of patches is based on arm-platforms.git/local_timers_as_devices > branch since it looks like that it is going to be merged into the mainline. Be careful here. This branch is a work in progress, likely to change very quickly, contains code that has not been posted to the ML yet, and will probably eat your pet for breakfast. As far as mainline merging is concerned, there is still a long way to go (see the GIC consolidation patches, on which the local_timers_as_devices branch relies). > V1 series was not discussed widely because I forgot to add all necessary > maintainers in Cc. Thanks for that. M. -- Jazz is not dead. It just smells funny...
next prev parent reply other threads:[~2011-07-06 10:06 UTC|newest] Thread overview: 117+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-10-04 9:45 [PATCH 0/4] ARM: smp_twd: mpcore_wdt: Fix MPCORE watchdog setup vkuzmichev 2010-10-04 9:45 ` vkuzmichev at mvista.com 2010-10-04 9:45 ` [PATCH 1/4] ARM: smp_twd: mpcore_wdt: Fix watchdog counter loading vkuzmichev 2010-10-04 9:45 ` vkuzmichev at mvista.com 2010-10-04 9:45 ` [PATCH 2/4] ARM: smp_twd: Fix typo in twd_timer_rate printout vkuzmichev 2010-10-04 9:45 ` vkuzmichev at mvista.com 2010-10-04 9:45 ` [PATCH 3/4] ARM: mpcore_wdt: Fix WDIOC_SETOPTIONS handling vkuzmichev 2010-10-04 9:45 ` vkuzmichev at mvista.com 2010-10-04 9:45 ` [PATCH 4/4] ARM: mpcore_wdt: Fix timer mode setup vkuzmichev 2010-10-04 9:45 ` vkuzmichev at mvista.com 2011-05-27 8:26 ` [PATCH 0/4] ARM: smp_twd: mpcore_wdt: Fix MPCORE watchdog setup Wim Van Sebroeck 2011-05-27 8:26 ` Wim Van Sebroeck 2011-05-27 10:44 ` Marc Zyngier 2011-05-27 10:44 ` Marc Zyngier 2011-05-27 10:44 ` Marc Zyngier 2011-07-05 19:00 ` [PATCH V2 0/6] arm_smp_twd: " Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-06 10:05 ` Marc Zyngier [this message] 2011-07-06 10:05 ` Marc Zyngier 2011-07-06 10:14 ` Russell King - ARM Linux 2011-07-06 10:14 ` Russell King - ARM Linux 2011-07-06 11:05 ` Catalin Marinas 2011-07-06 11:05 ` Catalin Marinas 2011-07-06 11:16 ` Russell King - ARM Linux 2011-07-06 11:16 ` Russell King - ARM Linux 2011-07-10 10:39 ` Russell King - ARM Linux 2011-07-10 10:39 ` Russell King - ARM Linux 2011-07-10 10:39 ` Russell King - ARM Linux 2011-07-11 9:43 ` Catalin Marinas 2011-07-11 9:43 ` Catalin Marinas 2011-07-11 9:43 ` Catalin Marinas 2011-07-11 9:54 ` Russell King - ARM Linux 2011-07-11 9:54 ` Russell King - ARM Linux 2011-07-11 9:54 ` Russell King - ARM Linux 2011-07-11 11:06 ` Catalin Marinas 2011-07-11 11:06 ` Catalin Marinas 2011-07-11 11:06 ` Catalin Marinas 2011-07-06 12:27 ` Vitaly Kuzmichev 2011-07-06 12:27 ` Vitaly Kuzmichev 2011-07-06 12:39 ` Marc Zyngier 2011-07-06 12:39 ` Marc Zyngier 2011-07-06 15:06 ` Vitaly Kuzmichev 2011-07-06 15:06 ` Vitaly Kuzmichev 2011-07-05 19:00 ` [PATCH V2 1/6] arm_smp_twd: Fix typo in 'twd_timer_rate' printing Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-05 19:00 ` [PATCH V2 2/6] arm_smp_twd: mpcore_wdt: Fix watchdog counter loading Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-06 12:05 ` Arnd Bergmann 2011-07-06 12:05 ` Arnd Bergmann 2011-07-05 19:00 ` [PATCH V2 3/6] mpcore_wdt: Fix WDIOC_SETOPTIONS handling Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-06 12:05 ` Sergei Shtylyov 2011-07-06 12:05 ` Sergei Shtylyov 2011-07-06 12:22 ` Arnd Bergmann 2011-07-06 12:22 ` Arnd Bergmann 2011-07-06 13:13 ` Vitaly Kuzmichev 2011-07-06 13:13 ` Vitaly Kuzmichev 2011-07-06 13:48 ` Russell King - ARM Linux 2011-07-06 13:48 ` Russell King - ARM Linux 2011-07-06 13:52 ` Russell King - ARM Linux 2011-07-06 13:52 ` Russell King - ARM Linux 2011-07-06 14:19 ` Arnd Bergmann 2011-07-06 14:19 ` Arnd Bergmann 2011-07-06 15:27 ` Arnd Bergmann 2011-07-06 15:27 ` Arnd Bergmann 2011-07-06 17:28 ` Wim Van Sebroeck 2011-07-06 17:28 ` Wim Van Sebroeck 2011-07-05 19:00 ` [PATCH V2 4/6] mpcore_wdt: Fix timer mode setup Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-05 19:00 ` [PATCH V2 5/6] mpcore_wdt: Add cpufreq notifier to reload counter Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-06 12:09 ` Sergei Shtylyov 2011-07-06 12:09 ` Sergei Shtylyov 2011-07-05 19:00 ` [PATCH V2 6/6] mpcore_wdt: Move declarations in a separate header Vitaly Kuzmichev 2011-07-05 19:00 ` Vitaly Kuzmichev 2011-07-06 11:58 ` Arnd Bergmann 2011-07-06 11:58 ` Arnd Bergmann 2011-07-06 12:36 ` Vitaly Kuzmichev 2011-07-06 12:36 ` Vitaly Kuzmichev 2011-07-06 12:48 ` Arnd Bergmann 2011-07-06 12:48 ` Arnd Bergmann 2011-07-10 10:42 ` Russell King - ARM Linux 2011-07-10 10:42 ` Russell King - ARM Linux 2011-07-06 12:11 ` Sergei Shtylyov 2011-07-06 12:11 ` Sergei Shtylyov 2011-07-07 12:22 ` [PATCH V3 0/4] ARM: smp_twd: mpcore_wdt: Fix MPCORE watchdog setup Vitaly Kuzmichev 2011-07-07 12:22 ` Vitaly Kuzmichev 2011-07-07 12:23 ` [PATCH V3 1/4] ARM: smp_twd: Fix typo in 'twd_timer_rate' printing Vitaly Kuzmichev 2011-07-07 12:23 ` Vitaly Kuzmichev 2011-07-07 12:39 ` Russell King - ARM Linux 2011-07-07 12:39 ` Russell King - ARM Linux 2011-07-07 12:50 ` Vitaly Kuzmichev 2011-07-07 12:50 ` Vitaly Kuzmichev 2011-07-07 13:31 ` Vitaly Kuzmichev 2011-07-07 13:31 ` Vitaly Kuzmichev 2011-07-07 16:20 ` Vitaly Kuzmichev 2011-07-07 16:20 ` Vitaly Kuzmichev 2011-07-10 10:47 ` Russell King - ARM Linux 2011-07-10 10:47 ` Russell King - ARM Linux 2011-07-11 11:50 ` Vitaly Kuzmichev 2011-07-11 11:50 ` Vitaly Kuzmichev 2011-07-07 12:23 ` [PATCH V3 2/4] ARM: smp_twd: mpcore_wdt: Fix watchdog counter loading Vitaly Kuzmichev 2011-07-07 12:23 ` Vitaly Kuzmichev 2011-07-07 12:44 ` Arnd Bergmann 2011-07-07 12:44 ` Arnd Bergmann 2011-07-07 14:09 ` Vitaly Kuzmichev 2011-07-07 14:09 ` Vitaly Kuzmichev 2011-07-07 12:23 ` [PATCH V3 3/4] ARM: mpcore_wdt: Fix WDIOC_SETOPTIONS handling Vitaly Kuzmichev 2011-07-07 12:23 ` Vitaly Kuzmichev 2011-07-07 12:45 ` Arnd Bergmann 2011-07-07 12:45 ` Arnd Bergmann 2011-07-07 12:23 ` [PATCH V3 4/4] ARM: mpcore_wdt: Fix timer mode setup Vitaly Kuzmichev 2011-07-07 12:23 ` Vitaly Kuzmichev 2011-07-07 12:46 ` [PATCH V3 0/4] ARM: smp_twd: mpcore_wdt: Fix MPCORE watchdog setup Wim Van Sebroeck 2011-07-07 12:46 ` Wim Van Sebroeck 2011-07-07 14:06 ` [PATCH V4 2/4] ARM: smp_twd: mpcore_wdt: Fix watchdog counter loading Vitaly Kuzmichev 2011-07-07 14:06 ` Vitaly Kuzmichev
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=4E14337F.1050101@arm.com \ --to=marc.zyngier@arm.com \ --cc=arm@kernel.org \ --cc=arnd@arndb.de \ --cc=johnstul@us.ibm.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-watchdog@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=nico@fluxnic.net \ --cc=tglx@linutronix.de \ --cc=vkuzmichev@mvista.com \ --cc=wim@iguana.be \ /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.