From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1946053AbbEOOGw (ORCPT ); Fri, 15 May 2015 10:06:52 -0400 Received: from mout.kundenserver.de ([212.227.126.187]:54500 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1422736AbbEOOGs (ORCPT ); Fri, 15 May 2015 10:06:48 -0400 From: Arnd Bergmann To: fu.wei@linaro.org Cc: Suravee.Suthikulpanit@amd.com, linaro-acpi@lists.linaro.org, linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, tekkamanninja@gmail.com, graeme.gregory@linaro.org, al.stone@linaro.org, hanjun.guo@linaro.org, timur@codeaurora.org, ashwin.chaugule@linaro.org, linux@roeck-us.net, vgandhi@codeaurora.org, wim@iguana.be, jcm@redhat.com, leo.duran@amd.com, corbet@lwn.net Subject: Re: [PATCH 1/6] Documentation: add sbsa-gwdt.txt documentation Date: Fri, 15 May 2015 16:06:29 +0200 Message-ID: <8715806.mTJfA5WAcP@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1431688090-2145-2-git-send-email-fu.wei@linaro.org> References: <=fu.wei@linaro.org> <1431688090-2145-1-git-send-email-fu.wei@linaro.org> <1431688090-2145-2-git-send-email-fu.wei@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:xe7r5Qk3GurLcywq/0GgjqfQ40L2MICVKhE4zkMTPvfEpfKSjq8 cJ3iC90yRtuLXa+CexMgWTfVeNIHzKEX7S86k6FaZzBu8MUI3RTs1kTNLA0JkrFkzgQ8Xzl zgSYnaEQE+bdoeD9HiPMH4ORxj7NtbFRhlAxjMrFxmvjk4C6KT3BPNC7zKEPojPLc/B1WuL qNGRcFzGNw38xKQtbyrxA== X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 15 May 2015 19:08:05 fu.wei@linaro.org wrote: > From: Fu Wei > > The sbsa-gwdt.txt documentation in devicetree/bindings/watchdog is for > introducing SBSA(Server Base System Architecture) Generic Watchdog > device node info into FDT > > Signed-off-by: Fu Wei Looks good, just one minor comment: > +Optional properties > +- timeout-sec : Watchdog pre-timeout and timeout values (in seconds). > + The first is timeout values, then pre-timeout. Please update Documentation/watchdog/watchdog-kernel-api.txt as well, to mention the second number in the timeout-sec property. Arnd