From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5EA4BC433E1 for ; Fri, 14 Aug 2020 15:11:24 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2746220768 for ; Fri, 14 Aug 2020 15:11:24 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="E5wGGecr"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="YbAHF1rr" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2746220768 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=roeck-us.net Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=MMEMZhJYUF1FQDrPciuAhZ47gZOqQSq40VVCwXeYw4Y=; b=E5wGGecrTph8l2XcKzZaAeaUt 98EbwsI4FhAjruFCBytYO6/ja2I7hu+v53xXdugZbh5D0Rh3lRmDLZbmNOd87F4v1gWP5byekVLhE vwWxvvNZ/ZiRiGJE6YXoBV0T++IqFvtRKEJKlOaGlao9CVa54any3Q2FbdXz8ivehYf9JzFPuJ6YM s8QYCq4FZwZv+HCzbGmP+zXbNmVQrwwC3Tx5QRBJyXonVKpBvvBF0VfW/I8X0wYgQXUvrPsaNv6bD PUzDogQZ+pWo9pC1/Ufg5PQMEUS1AC9o8IGUFwqU+9WETUqCXsKu35678DWnX2Ss85mZFJU+J7qRu KwmASXXQw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k6bKl-0004QB-LA; Fri, 14 Aug 2020 15:09:31 +0000 Received: from mail-pg1-x541.google.com ([2607:f8b0:4864:20::541]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k6bKi-0004Ps-Mw for linux-arm-kernel@lists.infradead.org; Fri, 14 Aug 2020 15:09:29 +0000 Received: by mail-pg1-x541.google.com with SMTP id o5so4683937pgb.2 for ; Fri, 14 Aug 2020 08:09:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=O74o2CeF9BmakoAygKNVUbF4kexNvSnVutgrgt6zSA8=; b=YbAHF1rrWqufIK2/c1cW1QSxEPyRIRyDc0fmEmL/2o9JfW9T7Fs/UUVR5E2vZmNt66 M0o2xC88G/u4zQizSg9RXsYwvPiAq+wQfsNFfIM05SrJ1yRp0ujkSFtsyNyrAn2aAZME vAPb/g1hNLtrcygBLu96Ew92+69mtppirYzOKO0AoDNxG3rf9GozbXVz8qPYzpA9YRUZ IF3/UMJJ24LEUnw2pck+qFfFcGkX+KSb78QASCN72FjkeyBDoXTojIhL6ID9+WPRF6WC RJshRyK1iN/9u3CCuGbA63oQi2BonfFaZkagtDd/eFcqG2qtyyOudB5pN2e/aqZDhukQ eOzQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to:user-agent; bh=O74o2CeF9BmakoAygKNVUbF4kexNvSnVutgrgt6zSA8=; b=sbXER2ND0MJhQvACX8v8ftM6bHXxiuDfm/Cn/vY93fue2Vv7IV7jMlXXJJMReDfe3X fwgosLSeO//LJKF9szJbg85+PlwYXfoXFjJT9v5iaepckdSxO/qsvMG/+5MoNKKdMJTB AJ5h2ZvQH7T4aU14CNqtf8Iw5S4g/66Fvr2GD/Gy401YNcExTx6LAdwg5Y4YBkLVd2R/ tnlQlrvDdGE706+TJJkpGUoOAOBTuZIlstYt3UA0ovpQayfqosCk7jytUGCohJLuV7Wu 1DHgjaQko+b4DxJGkr9tvrEeazy8UBq697XFri0yWEe317p1vhz03LOYorTw3gr52zlP mZSw== X-Gm-Message-State: AOAM531IctncWhu9WMqsSDfs3haRICFOOTKE2yarlYg/2eQw5U21AZH9 leRc/SYUTc43PeWRlmfX+nU= X-Google-Smtp-Source: ABdhPJxtlAfkZ1IM04n+CudW6LpO308CAYJkMzUoIjE1G1o/LkaM+di+OamfGIMNs59QYkvBAX8vag== X-Received: by 2002:a65:6108:: with SMTP id z8mr2067642pgu.266.1597417766635; Fri, 14 Aug 2020 08:09:26 -0700 (PDT) Received: from localhost ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id i11sm8272957pjv.30.2020.08.14.08.09.25 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 14 Aug 2020 08:09:25 -0700 (PDT) Date: Fri, 14 Aug 2020 08:09:24 -0700 From: Guenter Roeck To: Anson Huang Subject: Re: [PATCH V4 1/2] watchdog: imx7ulp: Strictly follow the sequence for wdog operations Message-ID: <20200814150924.GA255699@roeck-us.net> References: <1596150213-31638-1-git-send-email-Anson.Huang@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1596150213-31638-1-git-send-email-Anson.Huang@nxp.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200814_110928_821575_DD8BCC51 X-CRM114-Status: GOOD ( 25.19 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-watchdog@vger.kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, linux-kernel@vger.kernel.org, Linux-imx@nxp.com, kernel@pengutronix.de, wim@linux-watchdog.org, festevam@gmail.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Jul 31, 2020 at 07:03:32AM +0800, Anson Huang wrote: > According to reference manual, the i.MX7ULP WDOG's operations except > refresh should follow below sequence: > > 1. disable global interrupts; > 2. unlock the wdog and wait unlock bit set; > 3. reconfigure the wdog and wait for reconfiguration bit set; > 4. enabel global interrupts. > > Strictly follow the recommended sequence can make it more robust. > > Signed-off-by: Anson Huang Reviewed-by: Guenter Roeck > --- > changes since V3: > - wdog timeout should NOT update when setting timeout register failed. > --- > drivers/watchdog/imx7ulp_wdt.c | 74 ++++++++++++++++++++++++++++++++++-------- > 1 file changed, 61 insertions(+), 13 deletions(-) > > diff --git a/drivers/watchdog/imx7ulp_wdt.c b/drivers/watchdog/imx7ulp_wdt.c > index 7993c8c..badfc0b 100644 > --- a/drivers/watchdog/imx7ulp_wdt.c > +++ b/drivers/watchdog/imx7ulp_wdt.c > @@ -5,6 +5,7 @@ > > #include > #include > +#include > #include > #include > #include > @@ -36,6 +37,7 @@ > #define DEFAULT_TIMEOUT 60 > #define MAX_TIMEOUT 128 > #define WDOG_CLOCK_RATE 1000 > +#define WDOG_WAIT_TIMEOUT 20 > > static bool nowayout = WATCHDOG_NOWAYOUT; > module_param(nowayout, bool, 0000); > @@ -48,17 +50,40 @@ struct imx7ulp_wdt_device { > struct clk *clk; > }; > > -static void imx7ulp_wdt_enable(struct watchdog_device *wdog, bool enable) > +static int imx7ulp_wdt_wait(void __iomem *base, u32 mask) > +{ > + u32 val = readl(base + WDOG_CS); > + > + if (!(val & mask) && readl_poll_timeout_atomic(base + WDOG_CS, val, > + val & mask, 0, > + WDOG_WAIT_TIMEOUT)) > + return -ETIMEDOUT; > + > + return 0; > +} > + > +static int imx7ulp_wdt_enable(struct watchdog_device *wdog, bool enable) > { > struct imx7ulp_wdt_device *wdt = watchdog_get_drvdata(wdog); > > u32 val = readl(wdt->base + WDOG_CS); > + int ret; > > + local_irq_disable(); > writel(UNLOCK, wdt->base + WDOG_CNT); > + ret = imx7ulp_wdt_wait(wdt->base, WDOG_CS_ULK); > + if (ret) > + goto enable_out; > if (enable) > writel(val | WDOG_CS_EN, wdt->base + WDOG_CS); > else > writel(val & ~WDOG_CS_EN, wdt->base + WDOG_CS); > + imx7ulp_wdt_wait(wdt->base, WDOG_CS_RCS); > + > +enable_out: > + local_irq_enable(); > + > + return ret; > } > > static bool imx7ulp_wdt_is_enabled(void __iomem *base) > @@ -79,17 +104,12 @@ static int imx7ulp_wdt_ping(struct watchdog_device *wdog) > > static int imx7ulp_wdt_start(struct watchdog_device *wdog) > { > - > - imx7ulp_wdt_enable(wdog, true); > - > - return 0; > + return imx7ulp_wdt_enable(wdog, true); > } > > static int imx7ulp_wdt_stop(struct watchdog_device *wdog) > { > - imx7ulp_wdt_enable(wdog, false); > - > - return 0; > + return imx7ulp_wdt_enable(wdog, false); > } > > static int imx7ulp_wdt_set_timeout(struct watchdog_device *wdog, > @@ -97,22 +117,37 @@ static int imx7ulp_wdt_set_timeout(struct watchdog_device *wdog, > { > struct imx7ulp_wdt_device *wdt = watchdog_get_drvdata(wdog); > u32 val = WDOG_CLOCK_RATE * timeout; > + int ret; > > + local_irq_disable(); > writel(UNLOCK, wdt->base + WDOG_CNT); > + ret = imx7ulp_wdt_wait(wdt->base, WDOG_CS_ULK); > + if (ret) > + goto timeout_out; > writel(val, wdt->base + WDOG_TOVAL); > + imx7ulp_wdt_wait(wdt->base, WDOG_CS_RCS); > > wdog->timeout = timeout; > > - return 0; > +timeout_out: > + local_irq_enable(); > + > + return ret; > } > > static int imx7ulp_wdt_restart(struct watchdog_device *wdog, > unsigned long action, void *data) > { > struct imx7ulp_wdt_device *wdt = watchdog_get_drvdata(wdog); > + int ret; > + > + ret = imx7ulp_wdt_enable(wdog, true); > + if (ret) > + return ret; > > - imx7ulp_wdt_enable(wdog, true); > - imx7ulp_wdt_set_timeout(&wdt->wdd, 1); > + ret = imx7ulp_wdt_set_timeout(&wdt->wdd, 1); > + if (ret) > + return ret; > > /* wait for wdog to fire */ > while (true) > @@ -136,19 +171,30 @@ static const struct watchdog_info imx7ulp_wdt_info = { > WDIOF_MAGICCLOSE, > }; > > -static void imx7ulp_wdt_init(void __iomem *base, unsigned int timeout) > +static int imx7ulp_wdt_init(void __iomem *base, unsigned int timeout) > { > u32 val; > + int ret; > > + local_irq_disable(); > /* unlock the wdog for reconfiguration */ > writel_relaxed(UNLOCK_SEQ0, base + WDOG_CNT); > writel_relaxed(UNLOCK_SEQ1, base + WDOG_CNT); > + ret = imx7ulp_wdt_wait(base, WDOG_CS_ULK); > + if (ret) > + goto init_out; > > /* set an initial timeout value in TOVAL */ > writel(timeout, base + WDOG_TOVAL); > /* enable 32bit command sequence and reconfigure */ > val = WDOG_CS_CMD32EN | WDOG_CS_CLK | WDOG_CS_UPDATE; > writel(val, base + WDOG_CS); > + imx7ulp_wdt_wait(base, WDOG_CS_RCS); > + > +init_out: > + local_irq_enable(); > + > + return ret; > } > > static void imx7ulp_wdt_action(void *data) > @@ -199,7 +245,9 @@ static int imx7ulp_wdt_probe(struct platform_device *pdev) > watchdog_stop_on_reboot(wdog); > watchdog_stop_on_unregister(wdog); > watchdog_set_drvdata(wdog, imx7ulp_wdt); > - imx7ulp_wdt_init(imx7ulp_wdt->base, wdog->timeout * WDOG_CLOCK_RATE); > + ret = imx7ulp_wdt_init(imx7ulp_wdt->base, wdog->timeout * WDOG_CLOCK_RATE); > + if (ret) > + return ret; > > return devm_watchdog_register_device(dev, wdog); > } _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel