Re: [PATCH 1/2] dt-bindings: watchdog: samsung-wdt: Add exynos5515-wdt compatible

Aiden Isik <[email protected]>
Newsgroups org.kernel.vger.linux-watchdog,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-samsung-soc
Message-ID <xYkqqAlkGO9BORDl0gr_RUf7wX_yGFV0umuQjGA6F4GEOpAhKqpe6AuwMy60a6DKPQoCwzD9BljivI8jwFvfvK2U2JCpkVSJ5nQU8pkblhs=@protonmail.com>
> DCO mismatch.

> Please run scripts/checkpatch.pl on the patches and fix reported
> warnings. After that, run also 'scripts/checkpatch.pl --strict' on the
> patches and (probably) fix more warnings. Some warnings can be ignored,
> especially from --strict run, but the code here looks like it needs a
> fix. Feel free to get in touch if the warning is not clear.

I did run checkpatch (promise) but then b4 failed to send with
that address so I changed it.
Will fix in v2 thanks.

> No need for the comment, old code is poor example here.

I'll remove it.

> samsung,cluster-index: false

> Just like other cases.

Will fix. Thanks for the review.

> Best regards,
> Krzysztof

Kind regards,
- Aiden Isik
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.