Message ID | 20200316211551.30042-1-rasmus.villemoes@prevas.dk |
---|---|
State | Superseded |
Headers | show
Return-Path: <u-boot-bounces@lists.denx.de> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=<UNKNOWN>) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=prevas.dk Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=prevas.dk header.i=@prevas.dk header.a=rsa-sha256 header.s=selector1 header.b=iobsIYGx; dkim-atps=neutral Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 48h8HV5zD6z9sP7 for <incoming@patchwork.ozlabs.org>; Tue, 17 Mar 2020 08:16:22 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id C75EB81896; Mon, 16 Mar 2020 22:16:10 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=prevas.dk Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=prevas.dk header.i=@prevas.dk header.b="iobsIYGx"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 46702818A9; Mon, 16 Mar 2020 22:16:09 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-0.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FORGED_SPF_HELO,MSGID_FROM_MTA_HEADER, SPF_HELO_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2 Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-he1eur04on0726.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe0d::726]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 9922E8188C for <u-boot@lists.denx.de>; Mon, 16 Mar 2020 22:16:05 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=prevas.dk Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=rasmus.villemoes@prevas.dk ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GzAUk8Eu3pkSm6SXPqnJ7zCPvk4LLEDRImImM3pYlWZzUwe8WAyw6DrXBFhnGJH1UvAjKRHjuoiRrrqnNCdujZJHWpcnMvEGqqebaE9pGi7eJE/LLTnyA4uNYnY0DqICl8DcqlLZD6afH+LXBhPkBm4O1Y96RUYW4iLbF9GG66wgh+epONwpsTaKmRReKSUE9LQXwsuGrFcDtHlnfen1TG/Q35Q/YZ55bnlCZNBxAUB6Ani9Z03h6jajkqmYirRhkN3gKtSSiJve7ihEjntehvaotV4v/U9zjYDR07uGBUEEUlFE9+K4mcBAsIMh7amz+zHgBaL9BwWiG8dUG+saMw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=V6IUGf0vW1xwshvrZJAea9rw4Qiu+jKCjV8KWNUBSwQ=; b=GE4PtHWGGUAvv/4VKGMYDLaF2p40QefyY0bDH8z4Gzvv6qJF9TniDZtXyP2XqIqW+bjkvAfxrkZFT9cYzFAnY4VWe2sICd4uH0BsXHNCghsG/3068VK7rjNwgm+VPvZc0olLg9s+7zyVjILdUGRO2wrOenmggSJ9bLwxbC7m8/qHeldJnbPc9+97iXERD9guCKfzGR5hmhDkD7b1tjDstSDshPlXn1s2OWCKjsqdnz9QNcdbGPkc8Zh+3kB9SyHmlnOYLH8ZFt2KshNTMSd4pSGgvZTe5EC589nrSYxybqyx9CzSSHo2mfVZuiubuCJHqM7VcVgk67uNd2VrO1oh2g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=prevas.dk; dmarc=pass action=none header.from=prevas.dk; dkim=pass header.d=prevas.dk; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=prevas.dk; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=V6IUGf0vW1xwshvrZJAea9rw4Qiu+jKCjV8KWNUBSwQ=; b=iobsIYGxikrrwQoLKTDDGYjDLCcejesebRPj8MkCmXGRfhCzucLxRTAnu9X9KIohFlWByVIS3/gGAzxjfKKGgdWO0Bl6pFlcF15uLCaDNRHmCBK+T+13dAUllDxSCjMz+3JdJhVHkJVHyE/6POQhlBc6TxhsHKP+Qcw9NXuwqLQ= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=rasmus.villemoes@prevas.dk; Received: from VI1PR10MB2765.EURPRD10.PROD.OUTLOOK.COM (20.178.126.85) by VI1PR10MB2639.EURPRD10.PROD.OUTLOOK.COM (20.178.204.27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2814.14; Mon, 16 Mar 2020 21:16:02 +0000 Received: from VI1PR10MB2765.EURPRD10.PROD.OUTLOOK.COM ([fe80::e58f:1097:b71d:32c7]) by VI1PR10MB2765.EURPRD10.PROD.OUTLOOK.COM ([fe80::e58f:1097:b71d:32c7%5]) with mapi id 15.20.2814.021; Mon, 16 Mar 2020 21:16:02 +0000 From: Rasmus Villemoes <rasmus.villemoes@prevas.dk> To: u-boot@lists.denx.de Cc: Wolfgang Denk <wd@denx.de>, Simon Glass <sjg@chromium.org>, Christophe Leroy <christophe.leroy@c-s.fr>, Rasmus Villemoes <rasmus.villemoes@prevas.dk> Subject: [PATCH] powerpc: allow opting out of WATCHDOG_RESET() from timer interrupt Date: Mon, 16 Mar 2020 22:15:51 +0100 Message-Id: <20200316211551.30042-1-rasmus.villemoes@prevas.dk> X-Mailer: git-send-email 2.23.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: AM7PR03CA0002.eurprd03.prod.outlook.com (2603:10a6:20b:130::12) To VI1PR10MB2765.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:803:e1::21) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from prevas-ravi.prevas.se (5.186.114.43) by AM7PR03CA0002.eurprd03.prod.outlook.com (2603:10a6:20b:130::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2814.16 via Frontend Transport; Mon, 16 Mar 2020 21:16:01 +0000 X-Mailer: git-send-email 2.23.0 X-Originating-IP: [5.186.114.43] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b00a1d79-2f33-408a-d895-08d7c9ef3af2 X-MS-TrafficTypeDiagnostic: VI1PR10MB2639: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: <VI1PR10MB2639C671A842357F6B76FCB393F90@VI1PR10MB2639.EURPRD10.PROD.OUTLOOK.COM> X-MS-Oob-TLC-OOBClassifiers: OLM:7219; X-Forefront-PRVS: 03449D5DD1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(366004)(396003)(376002)(136003)(346002)(39850400004)(199004)(26005)(316002)(478600001)(956004)(81166006)(81156014)(6506007)(8676002)(44832011)(2616005)(52116002)(8936002)(1076003)(8976002)(86362001)(6666004)(186003)(16526019)(36756003)(6916009)(107886003)(54906003)(6486002)(4326008)(5660300002)(66946007)(6512007)(66556008)(66476007)(2906002); DIR:OUT; SFP:1102; SCL:1; SRVR:VI1PR10MB2639; H:VI1PR10MB2765.EURPRD10.PROD.OUTLOOK.COM; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; Received-SPF: None (protection.outlook.com: prevas.dk does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: me9exg629uLdhhnLdVWsQJp0Rqk1hUZqUN7yCyVB0NMi7dJ+NpzZIXYe/UPlr1Ef2DK5rARKaVGW+K1ycx1NX0obhPBhxaVDr0EscbPBcrUoEpcSAborxlRadY5DhJdMDh5QPCo0xriqXndoLxxi2tTeXg+iHqL1GO1Mu2CI0rJ/uLJ5xbtFvd42gAHkZxSw/I8KfUjP98RM+TlkycTgSbcnRKRCvt2EmM0rcNsoLpjke10RXhHqRicP4XzID9BEMEBMW/6u6dofGuIwuc99fuASsZHf7QV7EraYeuuqdOajtHlnPWmjwFV/ab9c4MgpYtfujADLmlsfy1osAii/BoUyeabTV0VI+nqXF+lsJfKLhZrp1nOq1TNIozXfxSB0luw7EPCOe3N2xkhhlrCqXL3MbYLmJMdc5VsdQ8HxJr0723FwIE+vLHCHdLPgHcwt X-MS-Exchange-AntiSpam-MessageData: d7ky/v7NBtRNQ5RTm/QZIDVzB/KuxsrfXxFrALexg5iu9fiEUUj/T33WosbE4rvYpJLtGcz4pVdyvDDr4twwr+PmXPXjcVbuSESQmskduqgjnYT0nO3lzf85BpeJbiuthj2b37JwxK7xXyaer7993g== X-OriginatorOrg: prevas.dk X-MS-Exchange-CrossTenant-Network-Message-Id: b00a1d79-2f33-408a-d895-08d7c9ef3af2 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2020 21:16:02.1144 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: d350cf71-778d-4780-88f5-071a4cb1ed61 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: EFMxfPnBs/OIe8YCtxFlX04zewpWgnGA3JXIBwa5kGupSUs2rR+HWOHTR58ts+PJNk/kyKhQ0uxGQBrl3qmzs8gHY6UdI05GoxDQK5yEqGE= X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR10MB2639 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.30rc1 Precedence: list List-Id: U-Boot discussion <u-boot.lists.denx.de> List-Unsubscribe: <https://lists.denx.de/options/u-boot>, <mailto:u-boot-request@lists.denx.de?subject=unsubscribe> List-Archive: <https://lists.denx.de/pipermail/u-boot/> List-Post: <mailto:u-boot@lists.denx.de> List-Help: <mailto:u-boot-request@lists.denx.de?subject=help> List-Subscribe: <https://lists.denx.de/listinfo/u-boot>, <mailto:u-boot-request@lists.denx.de?subject=subscribe> Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de> X-Virus-Scanned: clamav-milter 0.102.2 at phobos.denx.de X-Virus-Status: Clean |
Series |
powerpc: allow opting out of WATCHDOG_RESET() from timer interrupt
|
expand
|
diff --git a/arch/powerpc/lib/interrupts.c b/arch/powerpc/lib/interrupts.c index 64ee0cc210..23ac5bca1e 100644 --- a/arch/powerpc/lib/interrupts.c +++ b/arch/powerpc/lib/interrupts.c @@ -79,7 +79,7 @@ void timer_interrupt(struct pt_regs *regs) timestamp++; #if defined(CONFIG_WATCHDOG) || defined (CONFIG_HW_WATCHDOG) - if ((timestamp % (CONFIG_SYS_WATCHDOG_FREQ)) == 0) + if (CONFIG_SYS_WATCHDOG_FREQ && (timestamp % (CONFIG_SYS_WATCHDOG_FREQ)) == 0) WATCHDOG_RESET (); #endif /* CONFIG_WATCHDOG || CONFIG_HW_WATCHDOG */
When using CONFIG_(SPL_)WDT, the watchdog_reset function is a lot more complicated than just poking a few SOC-specific registers - it involves accessing all kinds of global data, and if the interrupt happens at the wrong time (say, in the middle of an WATCHDOG_RESET() call from ordinary code), that can end up corrupting said global data. Also, having WATCHDOG_RESET() called automatically from the timer interrupt runs counter to the idea of a watchdog device - if the board runs into an infinite loops with interrupts still enabled, the watchdog will never fire. Allow the board to opt out of this behaviour by setting CONFIG_SYS_WATCHDOG_FREQ to 0 - as that setting is currently nonsensical, it cannot affect any existing boards. Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk> --- arch/powerpc/lib/interrupts.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)