On Fri, Apr 19, 2024 at 05:52:38PM +0300, Andy Shevchenko wrote: > On Fri, Apr 19, 2024 at 05:34:44PM +0300, Konstantin P. wrote: > > On Fri, Apr 19, 2024 at 5:24 PM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: > > > On 19/04/2024 16:17, Konstantin P. wrote: > > ... > > > > Commits must stand on their own. Cover letter is not merged. This is the > > > place where you add new hardware, so here you describe and explain the > > > hardware. > > > > It is also described in patch 3 in the series. I need to repeat this > > description in patch 2 too? > > > > Cite from patch 3: > > > > XR20M1172 register set is mostly compatible with SC16IS762, but it has > > a support for additional division rates of UART with special DLD register. > > The point is, if I got it correctly, to have a few words in the description > of the DT binding itself, so whoever reads the bindings (w/o even accessing > the Git history of the project) may understand this. Yes, each patch must independently justify the change - although in this case it is about the history not people reading the bindings as I was talking about the commit message and not the contents of the diff. It's especially true when I only get sent the bindings patch, although from other messages in this thread that may not have been intentional. Thanks, Conor.
Attachment:
signature.asc
Description: PGP signature
