Re: [PATCH v3] docs: dt-bindings: Specify ordering for properties within groups

From: Krzysztof Kozlowski
Date: Fri Mar 07 2025 - 07:41:50 EST


On 07/03/2025 10:50, Dragan Simic wrote:
>>> Applying strict alpha-numerical ordering can result in property lists
>>> that
>>> are suboptimal from the usability standpoint. For the provided
>>> example,
>>> which stems from a real-world DT, [2][3][4] applying strict
>>> alpha-numerical
>>> ordering produces the following undesirable result:
>>
>> BTW, your entire commit msg still has incorrect wrapping. Please use
>> standard editors which understand Git commit msg style (which I believe
>> is equal to Linux submitting patches).
>
> Sorry, but what makes the wrapping wrong? I wrap patch descriptions
> at or well before 78 columns, which I belive is the desired way to do
> so. The longest line in this patch description is 77 characters long,
> which is still below 78 characters.

Because it is not correct. Read submitting patches. Run checkpatch.

>
> Also, please note that I don't "just use" some random editor, but I use
> a highly customized Vim setup, in which I over time wrote about 46 KB
> of Vimscript code in my ~/.vimrc, all by hand. I've even contributed
> smaller patches to Vim.

OK, so not random, but incorrectly configured. Different tools, same result.


Best regards,
Krzysztof