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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BD9B6C433FE for ; Thu, 2 Dec 2021 08:34:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345283AbhLBIiB (ORCPT ); Thu, 2 Dec 2021 03:38:01 -0500 Received: from cpanel.siel.si ([46.19.9.99]:35472 "EHLO cpanel.siel.si" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241126AbhLBIh6 (ORCPT ); Thu, 2 Dec 2021 03:37:58 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=norik.com; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:MIME-Version: Date:Message-ID:From:References:Cc:To:Subject:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=50NrMl5trPApnR1HnsTIfr+L19Vcwp/lvKAchEeW0ic=; b=lKwa7F3hz19cFkBiO9cz7hlkdU L+a3109FO8lli4ZzOpmiipS3ilKlPoOn/YZPH6QHjFY+H4J3eABr88DY12B+xyYO3gXksHfmTmCVU waS/NjbI6QVJx9glLssED22cLQsNUpZ4x+3cPC/TDntcoU5+BMim+C3xTS5j/KC5V/e5r1TgWwh7l +OIYQnqDe7LxfcM6LLpaHjyOycY2mIZQeDQjByiCtnqCpENbcJYY22CpBJloLvYKrYie8Wm5xUKkz /qSFLH7EAX755kPpyugD1M3glzdQFHLoCKxi3G8OcBED/73bXbwXHPBNEBUvrQAa0q7b90TTOqJqc aRQ6pTsg==; Received: from [89.212.21.243] (port=50490 helo=[192.168.69.215]) by cpanel.siel.si with esmtpsa (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1mshXt-00DFh4-1K; Thu, 02 Dec 2021 09:34:32 +0100 Subject: Re: [PATCH v3 2/4] watchdog: da9062: reset board on watchdog timeout To: Guenter Roeck , support.opensource@diasemi.com, linux-watchdog@vger.kernel.org Cc: wim@linux-watchdog.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, devicetree@vger.kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, linux-arm-kernel@lists.infradead.org References: <20211201081512.3580837-1-andrej.picej@norik.com> <20211201081512.3580837-2-andrej.picej@norik.com> From: Andrej Picej Message-ID: <4414c18e-83dd-0d64-325b-f69807ba49f4@norik.com> Date: Thu, 2 Dec 2021 09:34:34 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - cpanel.siel.si X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - norik.com X-Get-Message-Sender-Via: cpanel.siel.si: authenticated_id: andrej.picej@norik.com X-Authenticated-Sender: cpanel.siel.si: andrej.picej@norik.com X-Source: X-Source-Args: X-Source-Dir: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1. 12. 21 22:26, Guenter Roeck wrote: > On 12/1/21 12:15 AM, Andrej Picej wrote: >> Implement a method to change watchdog timeout configuration based on DT >> binding ("dlg,wdt-sd"). There is a possibility to change the bahaviour >> of watchdog reset. Setting WATCHDOG_SD bit enables SHUTDOWN mode, and >> clearing it enables POWERDOWN mode on watchdog timeout. >> >> If no DT binding is specified the WATCHDOG_SD bit stays in default >> configuration, not breaking behaviour of devices which might depend on >> default fuse configuration. >> >> Note: This patch requires that the config register CONFIG_I is >> configured as writable in the da9062 multi function device. >> >> Signed-off-by: Andrej Picej >> --- >> Changes in v3: >>   - no changes >> >> Changes in v2: >>   - don't force the "reset" for all da9062-watchdog users, instead add DT >>     binding where the behavior can be selected >> --- >>   drivers/watchdog/da9062_wdt.c | 25 +++++++++++++++++++++++++ >>   1 file changed, 25 insertions(+) >> >> diff --git a/drivers/watchdog/da9062_wdt.c >> b/drivers/watchdog/da9062_wdt.c >> index f02cbd530538..e342e9e50cb1 100644 >> --- a/drivers/watchdog/da9062_wdt.c >> +++ b/drivers/watchdog/da9062_wdt.c >> @@ -85,8 +85,33 @@ static int da9062_wdt_start(struct watchdog_device >> *wdd) >>   { >>       struct da9062_watchdog *wdt = watchdog_get_drvdata(wdd); >>       unsigned int selector; >> +    unsigned int mask; >> +    u32 val; >>       int ret; >> +    /* Configure what happens on watchdog timeout. Can be specified with >> +     * "dlg,wdt-sd" dt-binding (0 -> POWERDOWN, 1 -> SHUTDOWN). >> +     * If "dlg,wdt-sd" dt-binding is NOT set use the default. >> +     */ > > Please use standard multi-line comments. This is not the networking > subsystem. > > Also, if you think this code should be here and not in the probe function, > as suggested by Adam, please provide a rationale. > I will fix the multi-line comment, move this code to probe and submit the changes in the next patch version. Thanks, Andrej 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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5D73CC433F5 for ; Thu, 2 Dec 2021 08:37:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=S0ZNAk1bEbzm9mT9Mag3Gj3Ohkxe2jmRhD4E1rdkz/c=; b=eimX5AsbLLPOWbe97SPy0fsi21 RCLiFUCDCJyhSOrcHJNJx6udl79VFNIJadSi+UmEy0t+9taxVjd1RoB0Jz0kgBAjkjlEon/XuTaM3 ZhibFg29NREzomWaI48R/eaifg/sfYEKljosgUh8J3SDvPt2M8bYOZB76RLu0TmNOQG5HOLEE+SJ2 3ctroDzTUjrFlTf+1jeYS34R+cXC/j5R1H44gzA5KLp7ewHZU1C8QdE939e2/i9r4qLhY6PeyDgUv XWdsuBYHVSFLCzRZEmzoWfpsA6LAn8OGcuNfFgCUxKqecpSLCrtXBxr8vwgYhk17ki8dFLjah3Qx/ 61Ro5GSA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mshZ9-00BPbY-Aa; Thu, 02 Dec 2021 08:35:43 +0000 Received: from cpanel.siel.si ([46.19.9.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mshY5-00BP5N-4f for linux-arm-kernel@lists.infradead.org; Thu, 02 Dec 2021 08:34:38 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=norik.com; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:MIME-Version: Date:Message-ID:From:References:Cc:To:Subject:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=50NrMl5trPApnR1HnsTIfr+L19Vcwp/lvKAchEeW0ic=; b=lKwa7F3hz19cFkBiO9cz7hlkdU L+a3109FO8lli4ZzOpmiipS3ilKlPoOn/YZPH6QHjFY+H4J3eABr88DY12B+xyYO3gXksHfmTmCVU waS/NjbI6QVJx9glLssED22cLQsNUpZ4x+3cPC/TDntcoU5+BMim+C3xTS5j/KC5V/e5r1TgWwh7l +OIYQnqDe7LxfcM6LLpaHjyOycY2mIZQeDQjByiCtnqCpENbcJYY22CpBJloLvYKrYie8Wm5xUKkz /qSFLH7EAX755kPpyugD1M3glzdQFHLoCKxi3G8OcBED/73bXbwXHPBNEBUvrQAa0q7b90TTOqJqc aRQ6pTsg==; Received: from [89.212.21.243] (port=50490 helo=[192.168.69.215]) by cpanel.siel.si with esmtpsa (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1mshXt-00DFh4-1K; Thu, 02 Dec 2021 09:34:32 +0100 Subject: Re: [PATCH v3 2/4] watchdog: da9062: reset board on watchdog timeout To: Guenter Roeck , support.opensource@diasemi.com, linux-watchdog@vger.kernel.org Cc: wim@linux-watchdog.org, linux-kernel@vger.kernel.org, robh+dt@kernel.org, devicetree@vger.kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, linux-arm-kernel@lists.infradead.org References: <20211201081512.3580837-1-andrej.picej@norik.com> <20211201081512.3580837-2-andrej.picej@norik.com> From: Andrej Picej Message-ID: <4414c18e-83dd-0d64-325b-f69807ba49f4@norik.com> Date: Thu, 2 Dec 2021 09:34:34 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - cpanel.siel.si X-AntiAbuse: Original Domain - lists.infradead.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - norik.com X-Get-Message-Sender-Via: cpanel.siel.si: authenticated_id: andrej.picej@norik.com X-Authenticated-Sender: cpanel.siel.si: andrej.picej@norik.com X-Source: X-Source-Args: X-Source-Dir: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211202_003437_392379_0E292778 X-CRM114-Status: GOOD ( 18.24 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: base64 Content-Type: text/plain; charset="utf-8"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org CgpPbiAxLiAxMi4gMjEgMjI6MjYsIEd1ZW50ZXIgUm9lY2sgd3JvdGU6Cj4gT24gMTIvMS8yMSAx MjoxNSBBTSwgQW5kcmVqIFBpY2VqIHdyb3RlOgo+PiBJbXBsZW1lbnQgYSBtZXRob2QgdG8gY2hh bmdlIHdhdGNoZG9nIHRpbWVvdXQgY29uZmlndXJhdGlvbiBiYXNlZCBvbiBEVAo+PiBiaW5kaW5n ICgiZGxnLHdkdC1zZCIpLiBUaGVyZSBpcyBhIHBvc3NpYmlsaXR5IHRvIGNoYW5nZSB0aGUgYmFo YXZpb3VyCj4+IG9mIHdhdGNoZG9nIHJlc2V0LiBTZXR0aW5nIFdBVENIRE9HX1NEIGJpdCBlbmFi bGVzIFNIVVRET1dOIG1vZGUsIGFuZAo+PiBjbGVhcmluZyBpdCBlbmFibGVzIFBPV0VSRE9XTiBt b2RlIG9uIHdhdGNoZG9nIHRpbWVvdXQuCj4+Cj4+IElmIG5vIERUIGJpbmRpbmcgaXMgc3BlY2lm aWVkIHRoZSBXQVRDSERPR19TRCBiaXQgc3RheXMgaW4gZGVmYXVsdAo+PiBjb25maWd1cmF0aW9u LCBub3QgYnJlYWtpbmcgYmVoYXZpb3VyIG9mIGRldmljZXMgd2hpY2ggbWlnaHQgZGVwZW5kIG9u Cj4+IGRlZmF1bHQgZnVzZSBjb25maWd1cmF0aW9uLgo+Pgo+PiBOb3RlOiBUaGlzIHBhdGNoIHJl cXVpcmVzIHRoYXQgdGhlIGNvbmZpZyByZWdpc3RlciBDT05GSUdfSSBpcwo+PiBjb25maWd1cmVk IGFzIHdyaXRhYmxlIGluIHRoZSBkYTkwNjIgbXVsdGkgZnVuY3Rpb24gZGV2aWNlLgo+Pgo+PiBT aWduZWQtb2ZmLWJ5OiBBbmRyZWogUGljZWogPGFuZHJlai5waWNlakBub3Jpay5jb20+Cj4+IC0t LQo+PiBDaGFuZ2VzIGluIHYzOgo+PiDCoCAtIG5vIGNoYW5nZXMKPj4KPj4gQ2hhbmdlcyBpbiB2 MjoKPj4gwqAgLSBkb24ndCBmb3JjZSB0aGUgInJlc2V0IiBmb3IgYWxsIGRhOTA2Mi13YXRjaGRv ZyB1c2VycywgaW5zdGVhZCBhZGQgRFQKPj4gwqDCoMKgIGJpbmRpbmcgd2hlcmUgdGhlIGJlaGF2 aW9yIGNhbiBiZSBzZWxlY3RlZAo+PiAtLS0KPj4gwqAgZHJpdmVycy93YXRjaGRvZy9kYTkwNjJf d2R0LmMgfCAyNSArKysrKysrKysrKysrKysrKysrKysrKysrCj4+IMKgIDEgZmlsZSBjaGFuZ2Vk LCAyNSBpbnNlcnRpb25zKCspCj4+Cj4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL3dhdGNoZG9nL2Rh OTA2Ml93ZHQuYyAKPj4gYi9kcml2ZXJzL3dhdGNoZG9nL2RhOTA2Ml93ZHQuYwo+PiBpbmRleCBm MDJjYmQ1MzA1MzguLmUzNDJlOWU1MGNiMSAxMDA2NDQKPj4gLS0tIGEvZHJpdmVycy93YXRjaGRv Zy9kYTkwNjJfd2R0LmMKPj4gKysrIGIvZHJpdmVycy93YXRjaGRvZy9kYTkwNjJfd2R0LmMKPj4g QEAgLTg1LDggKzg1LDMzIEBAIHN0YXRpYyBpbnQgZGE5MDYyX3dkdF9zdGFydChzdHJ1Y3Qgd2F0 Y2hkb2dfZGV2aWNlIAo+PiAqd2RkKQo+PiDCoCB7Cj4+IMKgwqDCoMKgwqAgc3RydWN0IGRhOTA2 Ml93YXRjaGRvZyAqd2R0ID0gd2F0Y2hkb2dfZ2V0X2RydmRhdGEod2RkKTsKPj4gwqDCoMKgwqDC oCB1bnNpZ25lZCBpbnQgc2VsZWN0b3I7Cj4+ICvCoMKgwqAgdW5zaWduZWQgaW50IG1hc2s7Cj4+ ICvCoMKgwqAgdTMyIHZhbDsKPj4gwqDCoMKgwqDCoCBpbnQgcmV0Owo+PiArwqDCoMKgIC8qIENv bmZpZ3VyZSB3aGF0IGhhcHBlbnMgb24gd2F0Y2hkb2cgdGltZW91dC4gQ2FuIGJlIHNwZWNpZmll ZCB3aXRoCj4+ICvCoMKgwqDCoCAqICJkbGcsd2R0LXNkIiBkdC1iaW5kaW5nICgwIC0+IFBPV0VS RE9XTiwgMSAtPiBTSFVURE9XTikuCj4+ICvCoMKgwqDCoCAqIElmICJkbGcsd2R0LXNkIiBkdC1i aW5kaW5nIGlzIE5PVCBzZXQgdXNlIHRoZSBkZWZhdWx0Lgo+PiArwqDCoMKgwqAgKi8KPiAKPiBQ bGVhc2UgdXNlIHN0YW5kYXJkIG11bHRpLWxpbmUgY29tbWVudHMuIFRoaXMgaXMgbm90IHRoZSBu ZXR3b3JraW5nCj4gc3Vic3lzdGVtLgo+IAo+IEFsc28sIGlmIHlvdSB0aGluayB0aGlzIGNvZGUg c2hvdWxkIGJlIGhlcmUgYW5kIG5vdCBpbiB0aGUgcHJvYmUgZnVuY3Rpb24sCj4gYXMgc3VnZ2Vz dGVkIGJ5IEFkYW0sIHBsZWFzZSBwcm92aWRlIGEgcmF0aW9uYWxlLgo+IAoKSSB3aWxsIGZpeCB0 aGUgbXVsdGktbGluZSBjb21tZW50LCBtb3ZlIHRoaXMgY29kZSB0byBwcm9iZSBhbmQKc3VibWl0 IHRoZSBjaGFuZ2VzIGluIHRoZSBuZXh0IHBhdGNoIHZlcnNpb24uCgpUaGFua3MsCkFuZHJlagoK X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgtYXJt LWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3Jn Cmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtl cm5lbAo=