Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-10-13 Thread Fu Wei
Hi Dave, On 16 September 2015 at 09:57, Dave Young wrote: >> >> diff --git a/drivers/watchdog/sbsa_gwdt.c b/drivers/watchdog/sbsa_gwdt.c >> >> new file mode 100644 >> >> index 000..7ae45cc >> >> --- /dev/null >> >> +++ b/drivers/watchdog/sbsa_gwdt.c >> >> @@ -0,0 +1,459 @@ >> >> +/* >> >> +

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-10-13 Thread Fu Wei
Hi Dave, On 16 September 2015 at 09:57, Dave Young wrote: >> >> diff --git a/drivers/watchdog/sbsa_gwdt.c b/drivers/watchdog/sbsa_gwdt.c >> >> new file mode 100644 >> >> index 000..7ae45cc >> >> --- /dev/null >> >> +++ b/drivers/watchdog/sbsa_gwdt.c >> >> @@ -0,0 +1,459 @@

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-15 Thread Dave Young
> >> diff --git a/drivers/watchdog/sbsa_gwdt.c b/drivers/watchdog/sbsa_gwdt.c > >> new file mode 100644 > >> index 000..7ae45cc > >> --- /dev/null > >> +++ b/drivers/watchdog/sbsa_gwdt.c > >> @@ -0,0 +1,459 @@ > >> +/* > >> + * SBSA(Server Base System Architecture) Generic Watchdog driver > >>

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-15 Thread Fu Wei
Hi Dave, On 15 September 2015 at 16:38, Dave Young wrote: > On 08/25/15 at 01:01am, fu@linaro.org wrote: >> From: Fu Wei >> >> This driver bases on linux kernel watchdog framework, and >> use "pretimeout" in the framework. It supports getting timeout and >> pretimeout from parameter and FDT

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-15 Thread Dave Young
On 08/25/15 at 01:01am, fu@linaro.org wrote: > From: Fu Wei > > This driver bases on linux kernel watchdog framework, and > use "pretimeout" in the framework. It supports getting timeout and > pretimeout from parameter and FDT at the driver init stage. > In first timeout, the interrupt

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-15 Thread Dave Young
> >> diff --git a/drivers/watchdog/sbsa_gwdt.c b/drivers/watchdog/sbsa_gwdt.c > >> new file mode 100644 > >> index 000..7ae45cc > >> --- /dev/null > >> +++ b/drivers/watchdog/sbsa_gwdt.c > >> @@ -0,0 +1,459 @@ > >> +/* > >> + * SBSA(Server Base System Architecture) Generic Watchdog driver > >>

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-15 Thread Fu Wei
Hi Dave, On 15 September 2015 at 16:38, Dave Young wrote: > On 08/25/15 at 01:01am, fu@linaro.org wrote: >> From: Fu Wei >> >> This driver bases on linux kernel watchdog framework, and >> use "pretimeout" in the framework. It supports getting timeout

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-15 Thread Dave Young
On 08/25/15 at 01:01am, fu@linaro.org wrote: > From: Fu Wei > > This driver bases on linux kernel watchdog framework, and > use "pretimeout" in the framework. It supports getting timeout and > pretimeout from parameter and FDT at the driver init stage. > In first timeout,

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-14 Thread Fu Wei
Hi Guenter, Jon, On 11 September 2015 at 10:50, Guenter Roeck wrote: > On 09/10/2015 03:29 PM, Jon Masters wrote: >> >> On 08/24/2015 01:01 PM, fu@linaro.org wrote: >> >>> + /* >>> +* Get the frequency of system counter from the cp15 interface of >>> ARM >>> +* Generic

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-14 Thread Fu Wei
Hi Guenter, Jon, On 11 September 2015 at 10:50, Guenter Roeck wrote: > On 09/10/2015 03:29 PM, Jon Masters wrote: >> >> On 08/24/2015 01:01 PM, fu@linaro.org wrote: >> >>> + /* >>> +* Get the frequency of system counter from the cp15 interface of >>> ARM >>>

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-10 Thread Guenter Roeck
On 09/10/2015 03:29 PM, Jon Masters wrote: On 08/24/2015 01:01 PM, fu@linaro.org wrote: + /* +* Get the frequency of system counter from the cp15 interface of ARM +* Generic timer. We don't need to check it, because if it returns "0", +* system would panic in

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-10 Thread Guenter Roeck
On Thu, Sep 10, 2015 at 06:29:53PM -0400, Jon Masters wrote: > On 08/24/2015 01:01 PM, fu@linaro.org wrote: > > > + /* > > +* Get the frequency of system counter from the cp15 interface of ARM > > +* Generic timer. We don't need to check it, because if it returns "0", > > +*

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-10 Thread Jon Masters
On 08/24/2015 01:01 PM, fu@linaro.org wrote: > + /* > + * Get the frequency of system counter from the cp15 interface of ARM > + * Generic timer. We don't need to check it, because if it returns "0", > + * system would panic in very early stage. > + */ > +

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-10 Thread Jon Masters
On 08/24/2015 01:01 PM, fu@linaro.org wrote: > + /* > + * Get the frequency of system counter from the cp15 interface of ARM > + * Generic timer. We don't need to check it, because if it returns "0", > + * system would panic in very early stage. > + */ > +

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-10 Thread Guenter Roeck
On Thu, Sep 10, 2015 at 06:29:53PM -0400, Jon Masters wrote: > On 08/24/2015 01:01 PM, fu@linaro.org wrote: > > > + /* > > +* Get the frequency of system counter from the cp15 interface of ARM > > +* Generic timer. We don't need to check it, because if it returns "0", > > +*

Re: [PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-09-10 Thread Guenter Roeck
On 09/10/2015 03:29 PM, Jon Masters wrote: On 08/24/2015 01:01 PM, fu@linaro.org wrote: + /* +* Get the frequency of system counter from the cp15 interface of ARM +* Generic timer. We don't need to check it, because if it returns "0", +* system would panic in

[PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-08-24 Thread fu . wei
From: Fu Wei This driver bases on linux kernel watchdog framework, and use "pretimeout" in the framework. It supports getting timeout and pretimeout from parameter and FDT at the driver init stage. In first timeout, the interrupt routine run panic to save system context. Signed-off-by: Fu Wei

[PATCH v7 5/8] Watchdog: introduce ARM SBSA watchdog driver

2015-08-24 Thread fu . wei
From: Fu Wei fu@linaro.org This driver bases on linux kernel watchdog framework, and use pretimeout in the framework. It supports getting timeout and pretimeout from parameter and FDT at the driver init stage. In first timeout, the interrupt routine run panic to save system context.