From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DD2652505DD for ; Thu, 3 Apr 2025 15:10:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743693015; cv=none; b=h3Mqdx7HMafOtQYWmnAv/icMTo4jZfW+hrDxuoMzI1G0zCM2OxE6F72zmkSB2L4hGAzZUzFz7Bq+/cUvDfc1FtGtbbre/uZTEE0VdukDIcaJIbFuMFjXr/Iqu5XpMNZwLK1UTnAXE2U2IoLqliyvbYLS9kTq6fTsITMvz8uXKlA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743693015; c=relaxed/simple; bh=3xsuXjoivOlqLYm45J8XhER5c8e8iHHhlk5GjLRhsfg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=b9ggsNGirNFhHHLsib0SuQeW37obG28+3ik9hEW82+P9m0CTc2flUn219WeHbtUa+3iHHGj2Gg/mOw+5zRDQkcX74+Vgaxw5mBDLBjO78bBMrHrR2YUUxc74NVIwYZ0b/gAUcA827Od10LjF4OWA2cbyZ5e8vFp932E4Xd0F0wA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=zXkmIKKp; arc=none smtp.client-ip=209.85.128.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="zXkmIKKp" Received: by mail-wm1-f42.google.com with SMTP id 5b1f17b1804b1-43cf3192f3bso10350865e9.1 for ; Thu, 03 Apr 2025 08:10:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1743693012; x=1744297812; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=zhJ5g8kBFwNl+XQ2nWiu5GACX6MCWJ6Smtvjb3q8Bto=; b=zXkmIKKpoWKGbFoYbwdGvthQe7yCka+T6ZN97cA73Bvd4VJqm4JxzdBhILlH9wQPiC FA3DQQcR5Nghzb4xW7vcLEZvyWzwLxCJRv1I+NqFjTy27b6eYdBx8+YvuwHHZ+Vc5cDg nK/TL584fOgLuBal/PBangJattCVGn5MW5FlwhMoYBdwkmOdvlFdomBqGOppI/39EbPA k2EeHRQaoRjnC8/YbwfksUafBQs4wZ+wcz71bExzQKOkTCtIJBX12vxPEbpVgv940PC/ AureMuHS199yjTQXLbV5LFLIdmN/e4c21QpWtKfOzs9Bf5arg2mN2yFLJ2n5laAv247g wCYQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1743693012; x=1744297812; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=zhJ5g8kBFwNl+XQ2nWiu5GACX6MCWJ6Smtvjb3q8Bto=; b=gyeZ59QruQ7Kzr801+UiacWIxCc6ICj5iy6nMsDQGrWgszwSBDgI2X4po2gJt1A7f5 xuxWbInHZhwi+rvssAcJGTGoqa1srm9z0pH8equdNp4g5W4UF31qNG593giYkDmXCo8m +okGeEGdQRyBUjc8Ni3oW+VYqUBShGfOr4DnbePvqsK9yBLpNp1gbbK1QevsZa3entJ3 u+VOg8J22NPtlqLVpUdhjbpW1m69GLxznoWUSkki0Q+95hXesFhYxMe/1HkoRRC9RoCY zgbNnqPZPoGjN36MZAuhrpYZnqwIkDCPd3p2JIHpoykTuiVLRPr8xDgHlQvSw6LSh3Tn 76rQ== X-Forwarded-Encrypted: i=1; AJvYcCVL8wDpFG9MxT72CWWfM/ancPXAc9n30KKIw/XyUGOS6JoqfTwwzcCg3mGk5g4ym/N+8MgWI9EL6jmT8HkLEA==@vger.kernel.org X-Gm-Message-State: AOJu0YymHmt0k7eIBbDPT0LbsjJZt+JWV07ILO8Ke7Vx4GRjtuP9+EO4 L2uS80xHyQa0sGuZ4jTvODKxc3Scw/+TYYK/ltzxuYRivZX/oJBILfxDq7X3WPo= X-Gm-Gg: ASbGnctmCIGBZFpixnnIXpOwDrIuB3KVX3iWsYsH+HW2SecqBwFiufl0R3jjrqXx3LX OcjMsQLgaFEw1qXIs8TlFKFaS8wqk1FS7Y8z38/X3/TFCYeZ6gJju6ZTYnJit7++xyhP0o1B6DS H8AROn7y03AbW6odNUSIrxHdzR/4HIkhhm5zvbMXwofFsFd/mVu943zZsPL1//S3QFMZ4neEQq6 bBX9AwBbooA321zRm8r0nRwHjGgU2OyxU4V9rOri6Ovb4FQVdohYkrSznZUbVK9zZBdELfDg5oE 2AH5nBLfLaRAYAk+zo9tQ8GHa4WjWD3S7P/jz40YXkTspPz8iEqjr15w6i6k6dYnjqKHgHhHg+8 c6IBFYMqC X-Google-Smtp-Source: AGHT+IHDhBcexcgG0SxAh6e6AVDE03TCQZ+YSRRIFhr8aPiKmaRvD2AOaeBMWRz175FPNNgUcLCquQ== X-Received: by 2002:a05:600c:5486:b0:43c:f895:cb4e with SMTP id 5b1f17b1804b1-43db6249aefmr197046525e9.17.1743693012137; Thu, 03 Apr 2025 08:10:12 -0700 (PDT) Received: from [192.168.10.46] (146725694.box.freepro.com. [130.180.211.218]) by smtp.googlemail.com with ESMTPSA id 5b1f17b1804b1-43ec16a5776sm24935105e9.22.2025.04.03.08.10.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 03 Apr 2025 08:10:11 -0700 (PDT) Message-ID: Date: Thu, 3 Apr 2025 17:10:10 +0200 Precedence: bulk X-Mailing-List: linux-watchdog@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/2] dt-bindings: watchdog: Add NXP Software Watchdog Timer To: Ghennadi Procopciuc , wim@linux-watchdog.org Cc: linux@roeck-us.net, linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org, S32@nxp.com, ghennadi.procopciuc@nxp.com, thomas.fossati@linaro.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, devicetree@vger.kernel.org, Vincent Guittot References: <20250402154942.3645283-1-daniel.lezcano@linaro.org> <20250402154942.3645283-2-daniel.lezcano@linaro.org> <64b6d599-fe67-586a-e4b0-73d9b73499de@oss.nxp.com> Content-Language: en-US From: Daniel Lezcano In-Reply-To: <64b6d599-fe67-586a-e4b0-73d9b73499de@oss.nxp.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 03/04/2025 08:19, Ghennadi Procopciuc wrote: > On 4/2/2025 6:49 PM, Daniel Lezcano wrote: > [ ... ] >> +examples: >> + - | >> + watchdog@0x40100000 { >> + compatible = "nxp,s32g2-swt"; >> + reg = <0x40100000 0x1000>; >> + clocks = <&clks 0x3a>; >> + timeout-sec = <10>; >> + }; > > The S32G reference manual specifies two clocks for the SWT module: one > for the registers and another for the counter itself. Shouldn't both > clocks be represented in the bindings? AFAICS, there are two clocks as described in the documentation for the s32g2 page 846, section 23.7.3.3 SWT clocking. The module and the register clock are fed by the XBAR_DIV3_CLK which is an system clock always-on. The counter is fed by the FIRC_CLK which described as "FIRC_CLK is the default clock for the entire system at power-up." From my understanding, we should not describe the XBAR_DIV3_CLK as it is a system clock. And the FIRC_CLK is only there to get the clock rate in the driver. -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog