From: Fu Wei <fu.wei@linaro.org> To: Timur Tabi <timur@codeaurora.org> Cc: Guenter Roeck <linux@roeck-us.net>, Linaro ACPI Mailman List <linaro-acpi@lists.linaro.org>, linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, lkml <linux-kernel@vger.kernel.org>, linux-doc@vger.kernel.org, "Rafael J. Wysocki" <rjw@rjwysocki.net>, Arnd Bergmann <arnd@arndb.de>, Jonathan Corbet <corbet@lwn.net>, Jon Masters <jcm@redhat.com>, Pratyush Anand <panand@redhat.com>, Will Deacon <will.deacon@arm.com>, Wim Van Sebroeck <wim@iguana.be>, Catalin Marinas <catalin.marinas@arm.com>, Wei Fu <tekkamanninja@gmail.com>, Rob Herring <robherring2@gmail.com>, Vipul Gandhi <vgandhi@codeaurora.org>, Dave Young <dyoung@redhat.com> Subject: Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver Date: Thu, 5 Nov 2015 22:03:37 +0800 [thread overview] Message-ID: <CADyBb7sD2Y2kGj-suOs1UeUwXPjH39mvX_Sobn0Y7WfsJ9hF=Q@mail.gmail.com> (raw) In-Reply-To: <563B5DF9.6080102@codeaurora.org> Hi Timur On 5 November 2015 at 21:47, Timur Tabi <timur@codeaurora.org> wrote: > Guenter Roeck wrote: >> >> I would feel much more comfortable if the driver would just use the >> standard >> watchdog timeout and live with (worst case) 20 seconds timeout for now. > > > Actually, I'm wondering where the 20 seconds comes from. When I load my > driver on our hardware, it calculates a maximum timeout of 214 seconds, and > that's just to WS0. SBSA 2.3 Page 23 : Note: the watchdog offset register is 32 bits wide. This gives a maximum watch period of around 10s at a system counter frequency of 400MHz. If a larger watch period is required then the compare value can be programmed directly into the compare value register. 214s means your system counter is approximately at 20MHz which is in the range of (10MHz ~ 400MHz) SBSA 2.3 Page 13 : The System Counter (of the Generic Timer) shall run at a minimum frequency of 10MHz and maximum of 400MHz. > > -- > Sent by an employee of the Qualcomm Innovation Center, Inc. > The Qualcomm Innovation Center, Inc. is a member of the > Code Aurora Forum, hosted by The Linux Foundation. -- Best regards, Fu Wei Software Engineer Red Hat Software (Beijing) Co.,Ltd.Shanghai Branch Ph: +86 21 61221326(direct) Ph: +86 186 2020 4684 (mobile) Room 1512, Regus One Corporate Avenue,Level 15, One Corporate Avenue,222 Hubin Road,Huangpu District, Shanghai,China 200021
WARNING: multiple messages have this Message-ID (diff)
From: Fu Wei <fu.wei-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> To: Timur Tabi <timur-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> Cc: Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>, Linaro ACPI Mailman List <linaro-acpi-cunTk1MwBs8s++Sfvej+rw@public.gmane.org>, linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, lkml <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "Rafael J. Wysocki" <rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org>, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>, Jonathan Corbet <corbet-T1hC0tSOHrs@public.gmane.org>, Jon Masters <jcm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Pratyush Anand <panand-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>, Wim Van Sebroeck <wim-IQzOog9fTRqzQB+pC5nmwQ@public.gmane.org>, Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>, Wei Fu <tekkamanninja-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, Vipul Gandhi <vgandhi-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>, Dave Young <dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Subject: Re: [Linaro-acpi] [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver Date: Thu, 5 Nov 2015 22:03:37 +0800 [thread overview] Message-ID: <CADyBb7sD2Y2kGj-suOs1UeUwXPjH39mvX_Sobn0Y7WfsJ9hF=Q@mail.gmail.com> (raw) In-Reply-To: <563B5DF9.6080102-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> Hi Timur On 5 November 2015 at 21:47, Timur Tabi <timur-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> wrote: > Guenter Roeck wrote: >> >> I would feel much more comfortable if the driver would just use the >> standard >> watchdog timeout and live with (worst case) 20 seconds timeout for now. > > > Actually, I'm wondering where the 20 seconds comes from. When I load my > driver on our hardware, it calculates a maximum timeout of 214 seconds, and > that's just to WS0. SBSA 2.3 Page 23 : Note: the watchdog offset register is 32 bits wide. This gives a maximum watch period of around 10s at a system counter frequency of 400MHz. If a larger watch period is required then the compare value can be programmed directly into the compare value register. 214s means your system counter is approximately at 20MHz which is in the range of (10MHz ~ 400MHz) SBSA 2.3 Page 13 : The System Counter (of the Generic Timer) shall run at a minimum frequency of 10MHz and maximum of 400MHz. > > -- > Sent by an employee of the Qualcomm Innovation Center, Inc. > The Qualcomm Innovation Center, Inc. is a member of the > Code Aurora Forum, hosted by The Linux Foundation. -- Best regards, Fu Wei Software Engineer Red Hat Software (Beijing) Co.,Ltd.Shanghai Branch Ph: +86 21 61221326(direct) Ph: +86 186 2020 4684 (mobile) Room 1512, Regus One Corporate Avenue,Level 15, One Corporate Avenue,222 Hubin Road,Huangpu District, Shanghai,China 200021 -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-11-05 14:03 UTC|newest] Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-10-27 16:06 [PATCH v8 0/5] Watchdog: introduce ARM SBSA watchdog driver fu.wei 2015-10-27 16:06 ` [PATCH v8 1/5] Documentation: add sbsa-gwdt driver documentation fu.wei 2015-10-27 16:22 ` Mark Rutland 2015-10-27 16:22 ` Mark Rutland 2015-10-28 4:10 ` Fu Wei 2015-10-28 4:10 ` Fu Wei 2015-10-30 17:46 ` Timur Tabi 2015-10-30 17:46 ` Timur Tabi 2015-10-30 18:35 ` Fu Wei 2015-10-30 18:53 ` Timur Tabi 2015-10-30 18:53 ` Timur Tabi 2015-10-30 19:05 ` Mark Rutland 2015-10-30 20:37 ` Timur Tabi 2015-11-02 4:10 ` Fu Wei 2015-11-02 4:10 ` Fu Wei 2015-11-02 4:03 ` Fu Wei 2015-11-02 4:03 ` Fu Wei 2015-11-02 4:06 ` Timur Tabi 2015-11-02 4:06 ` Timur Tabi 2015-11-02 4:23 ` Jon Masters 2015-10-27 16:06 ` [PATCH v8 2/5] ARM64: add SBSA Generic Watchdog device node in foundation-v8.dts fu.wei 2015-10-27 16:06 ` [PATCH v8 3/5] ARM64: add SBSA Generic Watchdog device node in amd-seattle-soc.dtsi fu.wei 2015-10-27 16:06 ` [PATCH v8 4/5] Watchdog: introdouce "pretimeout" into framework fu.wei 2015-10-27 16:06 ` [PATCH v8 5/5] Watchdog: introduce ARM SBSA watchdog driver fu.wei 2015-11-05 1:59 ` [Linaro-acpi] " Timur Tabi 2015-11-05 5:13 ` Guenter Roeck 2015-11-05 11:58 ` Fu Wei 2015-11-05 13:47 ` Timur Tabi 2015-11-05 13:47 ` Timur Tabi 2015-11-05 13:47 ` Timur Tabi 2015-11-05 14:03 ` Fu Wei [this message] 2015-11-05 14:03 ` Fu Wei 2015-11-05 14:08 ` Timur Tabi 2015-11-05 14:35 ` Fu Wei 2015-11-05 14:40 ` Timur Tabi 2015-11-05 15:00 ` Fu Wei 2015-11-05 16:41 ` Guenter Roeck 2015-11-05 17:58 ` Fu Wei 2015-11-05 17:59 ` Timur Tabi 2015-11-05 18:04 ` Fu Wei 2015-11-13 0:06 ` Al Stone 2015-11-13 0:23 ` Timur Tabi 2015-11-19 23:50 ` Al Stone 2015-11-13 0:25 ` Guenter Roeck 2015-11-13 0:25 ` Guenter Roeck 2015-11-20 0:11 ` Al Stone 2015-11-20 0:11 ` Al Stone 2015-11-20 0:26 ` Timur Tabi
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='CADyBb7sD2Y2kGj-suOs1UeUwXPjH39mvX_Sobn0Y7WfsJ9hF=Q@mail.gmail.com' \ --to=fu.wei@linaro.org \ --cc=arnd@arndb.de \ --cc=catalin.marinas@arm.com \ --cc=corbet@lwn.net \ --cc=devicetree@vger.kernel.org \ --cc=dyoung@redhat.com \ --cc=jcm@redhat.com \ --cc=linaro-acpi@lists.linaro.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-watchdog@vger.kernel.org \ --cc=linux@roeck-us.net \ --cc=panand@redhat.com \ --cc=rjw@rjwysocki.net \ --cc=robherring2@gmail.com \ --cc=tekkamanninja@gmail.com \ --cc=timur@codeaurora.org \ --cc=vgandhi@codeaurora.org \ --cc=will.deacon@arm.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.