[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <176bd9cf-688a-480e-93b7-8f1710563c56@kernel.org>
Date: Sat, 13 Dec 2025 04:05:39 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Guenter Roeck <linux@...ck-us.net>,
"Thomas Perrot (Schneider Electric)" <thomas.perrot@...tlin.com>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Linus Walleij <linusw@...nel.org>,
Bartosz Golaszewski <brgl@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Fabio Estevam <festevam@...il.com>,
Jérémie Dautheribes <jeremie.dautheribes@...tlin.com>,
Wim Van Sebroeck <wim@...ux-watchdog.org>, Lee Jones <lee@...nel.org>
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-gpio@...r.kernel.org, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org, linux-watchdog@...r.kernel.org,
Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH 3/8] dt-bindings: watchdog: Add AAEON embedded controller
watchdog binding
On 12/12/2025 12:46, Guenter Roeck wrote:
> On 12/12/25 00:20, Krzysztof Kozlowski wrote:
>> On 12/12/2025 08:41, Thomas Perrot (Schneider Electric) wrote:
>>> +
>>> +additionalProperties: false
>>> +
>>> +examples:
>>> + - |
>>> + watchdog {
>>> + compatible = "aaeon,srg-imx8pl-wdt";
>>
>> No, that was discussed many times on the mailing list already. Fold the
>> child into the parent. Your driver model really do not matter for DT.
>>
>
> Theoretically there could be a 'timeout' property since this is
> a watchdog driver, but that isn't supported by the driver, so
> I agree.
>
Timeout can be added to the parent node. The most differentiating factor
are dedicated resources and none are here.
Best regards,
Krzysztof
Powered by blists - more mailing lists