linux-kernel - Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Thu, 18 Apr 2024 19:12:35 +0200
From: Dragan Simic <dsimic@...jaro.org>
To: Conor Dooley <conor@...nel.org>
Cc: Heiko Stübner <heiko@...ech.de>,
 linux-rockchip@...ts.infradead.org, linux-arm-kernel@...ts.infradead.org,
 devicetree@...r.kernel.org, robh+dt@...nel.org, krzk+dt@...nel.org,
 conor+dt@...nel.org, linux-kernel@...r.kernel.org, amadeus@....edu.cn,
 FUKAUMI Naoki <naoki@...xa.com>
Subject: Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions
 for Radxa boards

On 2024-04-18 19:04, Conor Dooley wrote:
> On Thu, Apr 18, 2024 at 06:59:42PM +0200, Heiko Stübner wrote:
>> Am Donnerstag, 18. April 2024, 18:26:19 CEST schrieb Dragan Simic:
>> > Correct the descriptions of a few Radxa boards, according to the up-to-date
>> > documentation from Radxa and the detailed explanation from Naoki. [1]  To sum
>> > it up, the short naming, as specified by Radxa, is preferred.
>> >
>> > [1] https://lore.kernel.org/linux-rockchip/B26C732A4DCEA9B3+282b8775-601b-4d4a-a513-4924b7940076@radxa.com/
>> >
>> > Suggested-by: FUKAUMI Naoki <naoki@...xa.com>
>> > Signed-off-by: Dragan Simic <dsimic@...jaro.org>
>> > ---
>> >  Documentation/devicetree/bindings/arm/rockchip.yaml | 8 ++++----
>> >  1 file changed, 4 insertions(+), 4 deletions(-)
>> >
>> > diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
>> > index fcf7316ecd74..ae58e25c29ec 100644
>> > --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
>> > +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
>> > @@ -767,22 +767,22 @@ properties:
>> >            - const: radxa,rockpis
>> >            - const: rockchip,rk3308
>> >
>> > -      - description: Radxa Rock2 Square
>> > +      - description: Radxa Rock 2 Square
>> 
>> I may be just blind, but what changed here?
> 
> There's now a space before the 2.

Exactly.  That's part of the Radxa's naming convention, which may be
seen as somewhat similar to the general rule of spacing out values and
their associated units, e.g. "2 MB" is in general preferred over "2MB"
in nearly any kind of a formally correct document.

As a side note, there's even so-called "half space" as a typographical
convention for spacing out the values and the associated units.  That
makes formatted text with such spacing a bit nicer. :)