Re: [PATCH 2/2] wdog: add i.MX system controller watchdog support

[Fabio Estevam]

Hi Stefano,

[Added the new U-Boot mailing list address]

On Wed, Sep 30, 2026 at 12:53 PM Stefano Manni <stefano.ma...@gmail.com> wrote:
>
> i.MX8QXP is an ARMv8 SoC which has a Cortex-M4 system controller
> inside, the system controller is in charge of controlling power,
> clock and watchdog etc..
>
> This patch adds i.MX system controller watchdog driver support,

Nit: Please avoid writing "This patch adds...".

Use imperative mood instead: "Add i.MX system controller watchdog...".

Also, it would be better to squash the two patches into a single one.

Please also add a user for this driver; otherwise, it's just dead code.

Thanks