[PATCH v4 2/2] dt-bindings: arm: rockchip: Add EmbedFire LubanCat 1
Krzysztof Kozlowski
krzysztof.kozlowski at linaro.org
Wed Dec 21 04:15:06 PST 2022
On 21/12/2022 13:12, ty wrote:
> Hi, Kozlowski and Stübner,
>
> Thank you very much for pointing me in the right direction.
>
>> So how did you solve the comment "Does not look like properly ordered.
>> Don't add stuff at the end of files/lists."? What did you change to
>> solve it?
>
>> for context sorting order in that file is alphabetical by description
>
> I finally got it. It should be put right after the "Elgin RV1108 R1".
>
>> You do not wrap at 75, but at some other place. Just put it to editor
>> and check where is wrapping (count characters).
>
> The original message was:
>
>> from: DHDAXCW <lasstp5011 at gmail.com>
>>
>> LubanCat 1 is a Rockchip RK3566 SBC based
>> is developed by EmbedFire Electronics Co., Ltd.
>> Mini Linux Card Type Cheap Computer Development Board
>>
>> Signed-off-by: Yuteng Zhong <zonyitoo at gmail.com>
>> Signed-off-by: DHDAXCW <lasstp5011 at gmail.com>
>
> The longest line is "Mini Linux Card Type Cheap Computer Development
> Board", which has 53 characters. Or ... I should use the whole line as
> much as possible? Then I see why I was doing wrong. It will be fixed
> in the PATCH v5.
Yes,
because
such
commit
msg
is
also
within
75
characters
but
it
is
not
correct,
right?
Best regards,
Krzysztof
More information about the Linux-rockchip
mailing list