Re: [PATCH net-next v8 00/15] dpll: add SiTime SiT9531x DPLL clock driver

[Ali Rouhi]

> Clashiko had several comments, and some of them definitely look real.
> Please note that you are supposed to handle such comments proactively,
> see commit c82ff94592fb68f529afe63ca7f5ddb7dae4ba83

Thanks for the pointer.  We have gone through all fourteen per-patch reviews
and replied to each finding individually in the corresponding sub-threads:
the majority are fixed in the upcoming v9, a few are answered with the design
rationale, and one review suggestion (renaming a vendor property to a "-hz"
suffix) turned out to be rejected by dt_binding_check itself, which the reply
on patch 02 documents.  v9 with all of the fixes and a full changelog follows.

On the second half of that commit -- running LLM reviews in advance of the
post -- you are right that we had not been doing it, and a 15-patch series
drawing 86 findings is exactly the case the paragraph warns about.  We have
since set up Sashiko locally and now run it before posting rather than
after.  v9 has been through it twice over the whole series, and what it
found is folded into the patches rather than left for the bots to raise
again.  We intend to keep doing this for any further revision.