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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Krzysztof,

On 2025-03-05 10:36, Krzysztof Kozlowski wrote:
On 05/03/2025 09:45, Dragan Simic wrote:

diff --git a/Documentation/devicetree/bindings/dts-coding-style.rst b/Documentation/devicetree/bindings/dts-coding-style.rst
index 8a68331075a0..15de3ede2d9c 100644
--- a/Documentation/devicetree/bindings/dts-coding-style.rst
+++ b/Documentation/devicetree/bindings/dts-coding-style.rst
@@ -133,6 +133,12 @@ The above-described ordering follows this approach: 3. Status is the last information to annotate that device node is or is not
    finished (board resources are needed).

+The above-described ordering specifies the preferred ordering of property +groups, while the individual properties inside each group shall use natural +sort order by the property name. More specifically, natural sort order shall +apply to multi-digit numbers found inside the property names, while alpha-
+numerical ordering shall apply otherwise.

The last sentence was not here and I don't get the point. Natural sort
order should be always preferred over alpha-numerical for properties.
About which other case ("...apply otherwise.") are you thinking?

Yes, I added that sentence in the v2 to, hopefully, clarify the natural
sort order itself a bit.  I've researched the natural sort order a bit
further, and it technically applies only to the multi-digit numbers found
inside the sorted strings.  That's what I wanted to explain, and "shall
apply otherwise" refers to applying the alpha-numerical sort order to the
remainders of the sorted strings, i.e. to everything but the multi-digit
numbers found in the property names.




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux