Re: [PATCH v2 1/2] dt-bindings: watchdog: Add NXP Software Watchdog Timer

[Daniel Lezcano]

On 03/04/2025 17:35, Krzysztof Kozlowski wrote:
> On 03/04/2025 17:33, Daniel Lezcano wrote:
> > On 03/04/2025 16:16, Krzysztof Kozlowski wrote:
> > > On 03/04/2025 01:01, Daniel Lezcano wrote:
> > > > On 02/04/2025 17:56, Krzysztof Kozlowski wrote:
> > > > > On 02/04/2025 17:49, Daniel Lezcano wrote:
> > > > > > Describe the Software Watchdog Timer available on the S32G platforms.
> > > > > >
> > > > > > Cc: Ghennadi Procopciuc <ghennadi.procopciuc@xxxxxxx>
> > > > > > Cc: Thomas Fossati <thomas.fossati@xxxxxxxxxx>
> > > > > > Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> > > > >
> > > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> > > > >
> > > > > Best regards,
> > > > > Krzysztof
> > > >
> > > > I ran the make dt_binding_check but inadvertently removed the "oneOf:"
> > > > after :/
> > > Other errors were also not fixed even though you got report on them on
> > > 29th of March.
> >
> > Which ones ?
> >
> > All errors were coming from the oneOf removal, after re-adding it they
> > are all gone
>
>
> The same as from previous patchset - wrong unit address, maybe more.

Indeed, my parser was not up-to-date and did not reported the error.

Thanks for pointing this out

  -- Daniel


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog