Re: [PATCH v2 1/3] dt-bindings: watchdog: Add MA35D1 Watchdog binding

zychen <[email protected]> Wed, 29 Jul 2026 08:50:42 +0800

Newsgroups	org.kernel.vger.linux-watchdog,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID	<[email protected]>


Krzysztof Kozlowski 於 2026/7/28 下午 07:47 寫道:
> On Fri, Jul 24, 2026 at 05:14:25PM +0800, Zi-Yu Chen wrote:
>> Add device tree binding documentation for the watchdog timer (WDT)
>> controller found on Nuvoton MA35D1 SoC.
>>
>> Signed-off-by: Zi-Yu Chen <[email protected]>
> 
> A nit, subject: drop second/last, redundant "binding". The
> "dt-bindings" prefix is already stating that these are bindings.
> See also:
> https://elixir.bootlin.com/linux/v7.1-rc7/source/Documentation/devicetree/bindings/submitting-patches.rst#L23
> 
> Reviewed-by: Krzysztof Kozlowski <[email protected]>
> 
> Best regards,
> Krzysztof
> 

Hi Krzysztof, 

Thanks for the review and the Reviewed-by tag.
I'll drop the redundant "binding" from the subject in the next revision.

Thanks,
Zi-Yu Chen