From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753164AbbEPKeF (ORCPT ); Sat, 16 May 2015 06:34:05 -0400 Received: from mail-pd0-f175.google.com ([209.85.192.175]:36049 "EHLO mail-pd0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752500AbbEPKeB (ORCPT ); Sat, 16 May 2015 06:34:01 -0400 Message-ID: <55571D10.7090208@linaro.org> Date: Sat, 16 May 2015 18:33:52 +0800 From: Fu Wei Reply-To: Fu Wei Organization: Linaro User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: Arnd Bergmann , linaro-acpi@lists.linaro.org CC: Suravee.Suthikulpanit@amd.com, linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, corbet@lwn.net, jcm@redhat.com, timur@codeaurora.org, wim@iguana.be, tekkamanninja@gmail.com, vgandhi@codeaurora.org, linux@roeck-us.net Subject: Re: [Linaro-acpi] [PATCH 0/6] Watchdog: introdouce ARM SBSA watchdog driver References: <=fu.wei@linaro.org> <1431688090-2145-1-git-send-email-fu.wei@linaro.org> <179589472.1OI8rAN8jV@wuerfel> In-Reply-To: <179589472.1OI8rAN8jV@wuerfel> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arnd, OK, Thanks, Will reply those comments soon. On 05/15/2015 10:07 PM, Arnd Bergmann wrote: > On Friday 15 May 2015 19:08:04 fu.wei@linaro.org wrote: >> From: Fu Wei >> >> This patchset: >> >> (1)Introdouces Documentation/devicetree/bindings/watchdog/sbsa-gwdt.txt >> for FDT info of SBSA Generic Watchdog, and give two examples of >> adding SBSA Generic Watchdog device node into the dts files: >> foundation-v8.dts and amd-seattle-soc.dtsi >> >> (2)Introdouce "pretimeout" into the watchdog framework >> >> (3)Introdouces ARM SBSA watchdog driver >> a.Use linux kernel watchdog framework >> b.Work with FDT on ARM64 >> c.Use "pretimeout" in watchdog framework >> d.In first timeout(WS0), do panic to save system context >> e.Support geting timeout and pretimeout from >> parameter and FDT at the driver init stage >> >> (4)Parse SBSA Generic Watchdog Structure in GTDT table of ACPI, >> and create a platform device with that information >> This platform device can be used by This Watchdog driver >> >> This patchset has been tested with watchdog daemon >> (ACPI/FDT, module/build-in) on the following platforms: >> (1)ARM Foundation v8 model >> (2)AMD Seattle B0 >> >> > > I've had a few very minor comments to the individual patches, otherwise > looks good. I have not reviewed the ACPI part of the series, as I have > no knowledge of that. > > Arnd > -- Best regards, Fu Wei Software Engineer From Red Hat LEG Team Linaro.org | Open source software for ARM SoCs Ph: +86 186 2020 4684 (mobile) IRC: fuwei Skype: tekkamanninja Room 1512, Regus One Corporate Avenue,Level 15, One Corporate Avenue,222 Hubin Road,Huangpu District, Shanghai,China 200021 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fu Wei Subject: Re: [Linaro-acpi] [PATCH 0/6] Watchdog: introdouce ARM SBSA watchdog driver Date: Sat, 16 May 2015 18:33:52 +0800 Message-ID: <55571D10.7090208@linaro.org> References: <=fu.wei@linaro.org> <1431688090-2145-1-git-send-email-fu.wei@linaro.org> <179589472.1OI8rAN8jV@wuerfel> Reply-To: Fu Wei Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <179589472.1OI8rAN8jV@wuerfel> Sender: linux-watchdog-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnd Bergmann , linaro-acpi-cunTk1MwBs8s++Sfvej+rw@public.gmane.org Cc: Suravee.Suthikulpanit-5C7GfCeVMHo@public.gmane.org, linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, corbet-T1hC0tSOHrs@public.gmane.org, jcm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, timur-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, wim-IQzOog9fTRqzQB+pC5nmwQ@public.gmane.org, tekkamanninja-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, vgandhi-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Arnd, OK, Thanks, Will reply those comments soon. On 05/15/2015 10:07 PM, Arnd Bergmann wrote: > On Friday 15 May 2015 19:08:04 fu.wei-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org wrote: >> From: Fu Wei >> >> This patchset: >> >> (1)Introdouces Documentation/devicetree/bindings/watchdog/sbsa-gwdt.txt >> for FDT info of SBSA Generic Watchdog, and give two examples of >> adding SBSA Generic Watchdog device node into the dts files: >> foundation-v8.dts and amd-seattle-soc.dtsi >> >> (2)Introdouce "pretimeout" into the watchdog framework >> >> (3)Introdouces ARM SBSA watchdog driver >> a.Use linux kernel watchdog framework >> b.Work with FDT on ARM64 >> c.Use "pretimeout" in watchdog framework >> d.In first timeout(WS0), do panic to save system context >> e.Support geting timeout and pretimeout from >> parameter and FDT at the driver init stage >> >> (4)Parse SBSA Generic Watchdog Structure in GTDT table of ACPI, >> and create a platform device with that information >> This platform device can be used by This Watchdog driver >> >> This patchset has been tested with watchdog daemon >> (ACPI/FDT, module/build-in) on the following platforms: >> (1)ARM Foundation v8 model >> (2)AMD Seattle B0 >> >> > > I've had a few very minor comments to the individual patches, otherwise > looks good. I have not reviewed the ACPI part of the series, as I have > no knowledge of that. > > Arnd > -- Best regards, Fu Wei Software Engineer From Red Hat LEG Team Linaro.org | Open source software for ARM SoCs Ph: +86 186 2020 4684 (mobile) IRC: fuwei Skype: tekkamanninja 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