Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751730AbbFNKQF (ORCPT ); Sun, 14 Jun 2015 06:16:05 -0400 Received: from mail-oi0-f54.google.com ([209.85.218.54]:34288 "EHLO mail-oi0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750908AbbFNKP6 (ORCPT ); Sun, 14 Jun 2015 06:15:58 -0400 MIME-Version: 1.0 In-Reply-To: <557A58A5.4010208@codeaurora.org> References: <=fu.wei@linaro.org> <1433958452-23721-1-git-send-email-fu.wei@linaro.org> <1433958452-23721-5-git-send-email-fu.wei@linaro.org> <557A58A5.4010208@codeaurora.org> Date: Sun, 14 Jun 2015 18:15:57 +0800 Message-ID: Subject: Re: [PATCH non-pretimeout 4/7] Watchdog: introduce ARM SBSA watchdog driver From: Fu Wei To: Timur Tabi Cc: Suravee Suthikulpanit , Linaro ACPI Mailman List , linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Wei Fu , G Gregory , Al Stone , Hanjun Guo , Ashwin Chaugule , Arnd Bergmann , Guenter Roeck , Vipul Gandhi , Wim Van Sebroeck , Jon Masters , Leo Duran , Jon Corbet , Mark Rutland , Catalin Marinas , Will Deacon , rjw@rjwysocki.net Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2066 Lines: 61 Hi Timur On 12 June 2015 at 11:57, Timur Tabi wrote: > fu.wei@linaro.org wrote: >> >> + if (timeout <= gwdt->max_wor_timeout) >> + writel_relaxed(timeout * gwdt->clk, >> + gwdt->control_base + SBSA_GWDT_WOR); >> + else >> + writel_relaxed(gwdt->max_wor_timeout * gwdt->clk, >> + gwdt->control_base + SBSA_GWDT_WOR); > > > You pre-calculate the maximum timeout possible already, so why do you need > the if-statement? Have took Guenter's suggestion on this. > > Frankly, your non-pretimeout driver is almost identical to mine, which was > posted weeks ago. At this point, you're really just copying my driver but > putting your name on it. Everyone can see how this driver become to this one gradually. For non-pretimeout, if there is not pretimeout variable, I can only use timeout to config both of them. This is definitely not the copy of yours(check yours again, you never programme WCV, and from the comment from you , you didn't believe driver can do that. you use panic because of my patch), and I posted my patchset(with pretimeout) to linaro-acpi list before you posted yours to linux mailing list. And I always focus on mine. Let people judge from all these patchset, I don't want to argue with you on this any more. > > > -- > 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-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/