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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5FF36CA101F for ; Wed, 10 Sep 2025 14:07:53 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id D48A98318E; Wed, 10 Sep 2025 16:07:51 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=dolcini.it Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=dolcini.it header.i=@dolcini.it header.b="jM6Lo5wy"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 3173D83236; Wed, 10 Sep 2025 16:07:51 +0200 (CEST) Received: from mail11.truemail.it (mail11.truemail.it [217.194.8.81]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 20B4D830B4 for ; Wed, 10 Sep 2025 16:07:49 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=dolcini.it Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=francesco@dolcini.it Received: from francesco-nb (93-49-2-63.ip317.fastwebnet.it [93.49.2.63]) by mail11.truemail.it (Postfix) with ESMTPA id ED5BF1FB71; Wed, 10 Sep 2025 16:07:47 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dolcini.it; s=default; t=1757513268; bh=6dHvfUBn64koVhK9IdMXnHV1ftYWjKQ9YWPDbYNI+Pw=; h=From:To:Subject; b=jM6Lo5wywmx43blK9yezAueVkdEdKoogypLjtGIvlyPiLghf+BFcfR1bBc4RUZ8Vd EyLYrB8+CcIvAjn+AlATWfuHwlM4+ZWbVnTWP+biz2nlFbEXvnxQbPKXptQb9cxHdd WdQ3CZDuHGU7P5vyBy0Z2cUlSRgseLS/ZNrcts8pkA5iYI7wvmvClWLLMGyqcO5Kcy NRSaQzp5V6AOO66Ki0hrTQUgj4ZvdtcvXo7VAv31v9ubcNiCDf2/BWJ7+GE46Qlaqk 3ItPaCwwRSG0mh+2a/HFlz/Asr/WGPeg+suu1YhEffRzmjGtLEF3XJN+3zFx/U6dge 6348WrfYXBlEQ== Date: Wed, 10 Sep 2025 16:07:43 +0200 From: Francesco Dolcini To: =?utf-8?Q?=C5=81ukasz?= Majewski Cc: Francesco Dolcini , Svyatoslav Ryhel , Tom Rini , Francesco Dolcini , u-boot@lists.denx.de, Stephen Warren Subject: Re: [PATCH 1/2] timer: tegra: Bind watchdog driver when timer node is processed Message-ID: <20250910140743.GA41850@francesco-nb> References: <20250908144107.10bcb8a1@wsk> <20250909121537.1c2a95b1@wsk> <20250909153948.GY124814@bill-the-cat> <20250909154710.GZ124814@bill-the-cat> <20250910085148.GA9815@francesco-nb> <20250910142821.7aff2ea1@wsk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20250910142821.7aff2ea1@wsk> X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean On Wed, Sep 10, 2025 at 02:28:21PM +0200, Łukasz Majewski wrote: > Hi Francesco, > > > Hello, > > > > On Tue, Sep 09, 2025 at 07:08:34PM +0300, Svyatoslav Ryhel wrote: > > > вт, 9 вер. 2025 р. о 18:47 Tom Rini пише: > > > > > > > ... > > > > It's a devkit where I'm pretty sure the watchdog exists, so we can > > > > enable it there and be done. Just because it doesn't use it today > > > > doesn't mean it can't use it in the future if we enable it, the > > > > "use watchdog to make sure A/B system configures are stable" is a > > > > common use case. > > ... > > > Not sure if changes into colibri_t30_defconfig are needed, but if > > > Francesco would like them, then ok. > > > > I would prefer the defconfig not to be touched. > > > > I'm still puzzled why this cannot be changed? In general I like the idea of the watchdog driver to be enabled by default on the defconfig. This is the sane thing to do in a lot of situation. Enabling it however complicates some use cases, attaching a debugger, or maybe we have a situation in which the OS is currently not keeping the WD alive, therefore you would be potentially in trouble if you would update the bootloader. I am not familiar with the general colibri t30 user base to be enough confident this change will be fine, so this is the reason I am a little hesitant to change for such an old product. With that said, I am ok to change it, no strong arguments against. Francesco