* [PATCH] dt-bindings: arm: sunxi: sort alphabetically
@ 2023-12-21 14:06 Andre Przywara
2023-12-21 14:45 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Andre Przywara @ 2023-12-21 14:06 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland
Cc: linux-arm-kernel, devicetree, linux-sunxi
The sunxi.yaml file, holding all known board compatible names for
devices with Allwinner SoCs, is apparently sorted alphabetically, by the
"description" entry. However there are a few outliers.
Re-order the entries to get a strict alphabetical ordering, so that:
$ grep description: sunxi.yaml | sort -fc
returns empty-handed. There is no change otherwise.
Signed-off-by: Andre Przywara <andre.przywara@arm.com>
---
Based on sunxi/dt-for-6.8
.../devicetree/bindings/arm/sunxi.yaml | 172 +++++++++---------
1 file changed, 86 insertions(+), 86 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
index a9d8e85565b89..0bf24f4882240 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
@@ -51,16 +51,16 @@ properties:
- const: allwinner,parrot
- const: allwinner,sun8i-a33
- - description: Anbernic RG-Nano
- items:
- - const: anbernic,rg-nano
- - const: allwinner,sun8i-v3s
-
- description: Amarula A64 Relic
items:
- const: amarula,a64-relic
- const: allwinner,sun50i-a64
+ - description: Anbernic RG-Nano
+ items:
+ - const: anbernic,rg-nano
+ - const: allwinner,sun8i-v3s
+
- description: Auxtek T003 A10s HDMI TV Stick
items:
- const: allwinner,auxtek-t003
@@ -96,6 +96,11 @@ properties:
- const: sinovoip,bpi-m2-berry
- const: allwinner,sun8i-r40
+ - description: BananaPi M2 Magic
+ items:
+ - const: sinovoip,bananapi-m2m
+ - const: allwinner,sun8i-a33
+
- description: BananaPi M2 Plus
items:
- const: sinovoip,bpi-m2-plus
@@ -116,11 +121,6 @@ properties:
- const: bananapi,bpi-m2-plus-v1.2
- const: allwinner,sun50i-h5
- - description: BananaPi M2 Magic
- items:
- - const: sinovoip,bananapi-m2m
- - const: allwinner,sun8i-a33
-
- description: BananaPi M2 Ultra
items:
- const: sinovoip,bpi-m2-ultra
@@ -230,6 +230,18 @@ properties:
- const: sochip,s3
- const: allwinner,sun8i-v3
+ - description: Emlid Neutis N5 Developer Board
+ items:
+ - const: emlid,neutis-n5-devboard
+ - const: emlid,neutis-n5
+ - const: allwinner,sun50i-h5
+
+ - description: Emlid Neutis N5H3 Developer Board
+ items:
+ - const: emlid,neutis-n5h3-devboard
+ - const: emlid,neutis-n5h3
+ - const: allwinner,sun8i-h3
+
- description: Empire Electronix D709 Tablet
items:
- const: empire-electronix,d709
@@ -306,16 +318,26 @@ properties:
- const: gemei,g9
- const: allwinner,sun4i-a10
- - description: Hyundai A7HD
+ - description: HAOYU Electronics Marsboard A10
items:
- - const: hyundai,a7hd
+ - const: haoyu,a10-marsboard
- const: allwinner,sun4i-a10
+ - description: HAOYU Electronics Marsboard A20
+ items:
+ - const: haoyu,a20-marsboard
+ - const: allwinner,sun7i-a20
+
- description: HSG H702
items:
- const: hsg,h702
- const: allwinner,sun5i-a13
+ - description: Hyundai A7HD
+ items:
+ - const: hyundai,a7hd
+ - const: allwinner,sun4i-a10
+
- description: I12 TV Box
items:
- const: allwinner,i12-tvbox
@@ -343,11 +365,6 @@ properties:
- const: primux,inet86dz
- const: allwinner,sun8i-a23
- - description: iNet-9F Rev 03
- items:
- - const: inet-tek,inet9f-rev03
- - const: allwinner,sun4i-a10
-
- description: iNet-97F Rev 02
items:
- const: primux,inet97fv2
@@ -358,6 +375,11 @@ properties:
- const: primux,inet98v-rev2
- const: allwinner,sun5i-a13
+ - description: iNet-9F Rev 03
+ items:
+ - const: inet-tek,inet9f-rev03
+ - const: allwinner,sun4i-a10
+
- description: iNet D978 Rev 02 Tablet
items:
- const: primux,inet-d978-rev2
@@ -472,15 +494,10 @@ properties:
- const: lamobo,lamobo-r1
- const: allwinner,sun7i-a20
- - description: HAOYU Electronics Marsboard A10
- items:
- - const: haoyu,a10-marsboard
- - const: allwinner,sun4i-a10
-
- - description: HAOYU Electronics Marsboard A20
+ - description: MangoPi MQ-R board
items:
- - const: haoyu,a20-marsboard
- - const: allwinner,sun7i-a20
+ - const: widora,mangopi-mq-r-t113
+ - const: allwinner,sun8i-t113s
- description: MapleBoard MP130
items:
@@ -557,18 +574,6 @@ properties:
- const: msi,primo81
- const: allwinner,sun6i-a31s
- - description: Emlid Neutis N5 Developer Board
- items:
- - const: emlid,neutis-n5-devboard
- - const: emlid,neutis-n5
- - const: allwinner,sun50i-h5
-
- - description: Emlid Neutis N5H3 Developer Board
- items:
- - const: emlid,neutis-n5h3-devboard
- - const: emlid,neutis-n5h3
- - const: allwinner,sun8i-h3
-
- description: NextThing Co. CHIP
items:
- const: nextthing,chip
@@ -638,11 +643,6 @@ properties:
- const: olimex,a20-olinuxino-lime
- const: allwinner,sun7i-a20
- - description: Olimex A20-OlinuXino LIME (with eMMC)
- items:
- - const: olimex,a20-olinuxino-lime-emmc
- - const: allwinner,sun7i-a20
-
- description: Olimex A20-OlinuXino LIME2
items:
- const: olimex,a20-olinuxino-lime2
@@ -653,6 +653,11 @@ properties:
- const: olimex,a20-olinuxino-lime2-emmc
- const: allwinner,sun7i-a20
+ - description: Olimex A20-OlinuXino LIME (with eMMC)
+ items:
+ - const: olimex,a20-olinuxino-lime-emmc
+ - const: allwinner,sun7i-a20
+
- description: Olimex A20-OlinuXino Micro
items:
- const: olimex,a20-olinuxino-micro
@@ -703,6 +708,17 @@ properties:
- const: pine64,pine64-plus
- const: allwinner,sun50i-a64
+ - description: Pine64 LTS
+ items:
+ - const: pine64,pine64-lts
+ - const: allwinner,sun50i-r18
+ - const: allwinner,sun50i-a64
+
+ - description: Pine64 Pinebook
+ items:
+ - const: pine64,pinebook
+ - const: allwinner,sun50i-a64
+
- description: Pine64 PineCube
items:
- const: pine64,pinecube
@@ -719,20 +735,9 @@ properties:
- const: pine64,pine-h64-model-b
- const: allwinner,sun50i-h6
- - description: Pine64 LTS
- items:
- - const: pine64,pine64-lts
- - const: allwinner,sun50i-r18
- - const: allwinner,sun50i-a64
-
- - description: Pine64 Pinebook
- items:
- - const: pine64,pinebook
- - const: allwinner,sun50i-a64
-
- - description: Pine64 PinePhone Developer Batch (1.0)
+ - description: Pine64 PinePhone (1.2)
items:
- - const: pine64,pinephone-1.0
+ - const: pine64,pinephone-1.2
- const: pine64,pinephone
- const: allwinner,sun50i-a64
@@ -742,9 +747,9 @@ properties:
- const: pine64,pinephone
- const: allwinner,sun50i-a64
- - description: Pine64 PinePhone (1.2)
+ - description: Pine64 PinePhone Developer Batch (1.0)
items:
- - const: pine64,pinephone-1.2
+ - const: pine64,pinephone-1.0
- const: pine64,pinephone
- const: allwinner,sun50i-a64
@@ -835,6 +840,12 @@ properties:
- const: sinlinx,sina33
- const: allwinner,sun8i-a33
+ - description: SL631 Action Camera with IMX179
+ items:
+ - const: allwinner,sl631-imx179
+ - const: allwinner,sl631
+ - const: allwinner,sun8i-v3
+
- description: SourceParts PopStick v1.1
items:
- const: sourceparts,popstick-v1.1
@@ -842,12 +853,6 @@ properties:
- const: allwinner,suniv-f1c200s
- const: allwinner,suniv-f1c100s
- - description: SL631 Action Camera with IMX179
- items:
- - const: allwinner,sl631-imx179
- - const: allwinner,sl631
- - const: allwinner,sun8i-v3
-
- description: Tanix TX6
items:
- const: oranth,tanix-tx6
@@ -883,11 +888,6 @@ properties:
- const: wexler,tab7200
- const: allwinner,sun7i-a20
- - description: MangoPi MQ-R board
- items:
- - const: widora,mangopi-mq-r-t113
- - const: allwinner,sun8i-t113s
-
- description: WITS A31 Colombus Evaluation Board
items:
- const: wits,colombus
@@ -903,11 +903,6 @@ properties:
- const: wobo,a10s-wobo-i5
- const: allwinner,sun5i-a10s
- - description: Yones TopTech BS1078 v2 Tablet
- items:
- - const: yones-toptech,bs1078-v2
- - const: allwinner,sun6i-a31s
-
- description: X96 Mate TV box
items:
- const: hechuang,x96-mate
@@ -998,6 +993,21 @@ properties:
- const: xunlong,orangepi-zero
- const: allwinner,sun8i-h2-plus
+ - description: Xunlong OrangePi Zero 2
+ items:
+ - const: xunlong,orangepi-zero2
+ - const: allwinner,sun50i-h616
+
+ - description: Xunlong OrangePi Zero 2W
+ items:
+ - const: xunlong,orangepi-zero2w
+ - const: allwinner,sun50i-h618
+
+ - description: Xunlong OrangePi Zero 3
+ items:
+ - const: xunlong,orangepi-zero3
+ - const: allwinner,sun50i-h618
+
- description: Xunlong OrangePi Zero Plus
items:
- const: xunlong,orangepi-zero-plus
@@ -1013,19 +1023,9 @@ properties:
- const: xunlong,orangepi-zero-plus2-h3
- const: allwinner,sun8i-h3
- - description: Xunlong OrangePi Zero 2
- items:
- - const: xunlong,orangepi-zero2
- - const: allwinner,sun50i-h616
-
- - description: Xunlong OrangePi Zero 2W
- items:
- - const: xunlong,orangepi-zero2w
- - const: allwinner,sun50i-h618
-
- - description: Xunlong OrangePi Zero 3
+ - description: Yones TopTech BS1078 v2 Tablet
items:
- - const: xunlong,orangepi-zero3
- - const: allwinner,sun50i-h618
+ - const: yones-toptech,bs1078-v2
+ - const: allwinner,sun6i-a31s
additionalProperties: true
base-commit: a4aebe936554dac6a91e5d091179c934f8325708
prerequisite-patch-id: b72637774a9de3328cc561ae4bb3137bba25fafd
prerequisite-patch-id: 1a3327044ae61af705ffd79028687f3f6629edf4
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: arm: sunxi: sort alphabetically
2023-12-21 14:06 [PATCH] dt-bindings: arm: sunxi: sort alphabetically Andre Przywara
@ 2023-12-21 14:45 ` Krzysztof Kozlowski
2023-12-21 15:07 ` Andre Przywara
0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-21 14:45 UTC (permalink / raw)
To: Andre Przywara, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Chen-Yu Tsai, Jernej Skrabec, Samuel Holland
Cc: linux-arm-kernel, devicetree, linux-sunxi
On 21/12/2023 15:06, Andre Przywara wrote:
> The sunxi.yaml file, holding all known board compatible names for
> devices with Allwinner SoCs, is apparently sorted alphabetically, by the
> "description" entry. However there are a few outliers.
>
> Re-order the entries to get a strict alphabetical ordering, so that:
> $ grep description: sunxi.yaml | sort -fc
> returns empty-handed. There is no change otherwise.
>
> Signed-off-by: Andre Przywara <andre.przywara@arm.com>
What was the existing order? Aren't this just shuffling again things?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: arm: sunxi: sort alphabetically
2023-12-21 14:45 ` Krzysztof Kozlowski
@ 2023-12-21 15:07 ` Andre Przywara
2023-12-21 15:38 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Andre Przywara @ 2023-12-21 15:07 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, linux-arm-kernel, devicetree,
linux-sunxi
On Thu, 21 Dec 2023 15:45:17 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
Hi,
> On 21/12/2023 15:06, Andre Przywara wrote:
> > The sunxi.yaml file, holding all known board compatible names for
> > devices with Allwinner SoCs, is apparently sorted alphabetically, by the
> > "description" entry. However there are a few outliers.
> >
> > Re-order the entries to get a strict alphabetical ordering, so that:
> > $ grep description: sunxi.yaml | sort -fc
> > returns empty-handed. There is no change otherwise.
> >
> > Signed-off-by: Andre Przywara <andre.przywara@arm.com>
>
> What was the existing order? Aren't this just shuffling again things?
It assume the intention was to order "alphabetically by description", but
there were some out-of-place entries which apparently slipped through
review.
Shall I add a comment at the top to note the order?
Cheers,
Andre
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: arm: sunxi: sort alphabetically
2023-12-21 15:07 ` Andre Przywara
@ 2023-12-21 15:38 ` Krzysztof Kozlowski
2023-12-21 16:29 ` Andre Przywara
0 siblings, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-21 15:38 UTC (permalink / raw)
To: Andre Przywara
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, linux-arm-kernel, devicetree,
linux-sunxi
On 21/12/2023 16:07, Andre Przywara wrote:
> On Thu, 21 Dec 2023 15:45:17 +0100
> Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
>
> Hi,
>
>> On 21/12/2023 15:06, Andre Przywara wrote:
>>> The sunxi.yaml file, holding all known board compatible names for
>>> devices with Allwinner SoCs, is apparently sorted alphabetically, by the
>>> "description" entry. However there are a few outliers.
>>>
>>> Re-order the entries to get a strict alphabetical ordering, so that:
>>> $ grep description: sunxi.yaml | sort -fc
>>> returns empty-handed. There is no change otherwise.
>>>
>>> Signed-off-by: Andre Przywara <andre.przywara@arm.com>
>>
>> What was the existing order? Aren't this just shuffling again things?
>
> It assume the intention was to order "alphabetically by description", but
> there were some out-of-place entries which apparently slipped through
> review.
>
> Shall I add a comment at the top to note the order?
Some files like this are ordered by compatible, some by class or family
of devices, so first you need to be sure there is no existing order.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: arm: sunxi: sort alphabetically
2023-12-21 15:38 ` Krzysztof Kozlowski
@ 2023-12-21 16:29 ` Andre Przywara
2023-12-22 8:09 ` Krzysztof Kozlowski
0 siblings, 1 reply; 6+ messages in thread
From: Andre Przywara @ 2023-12-21 16:29 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, linux-arm-kernel, devicetree,
linux-sunxi, Maxime Ripard
On Thu, 21 Dec 2023 16:38:41 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
Hi,
(CC:ing Maxime for potential historic context)
> On 21/12/2023 16:07, Andre Przywara wrote:
> > On Thu, 21 Dec 2023 15:45:17 +0100
> > Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
> >
> > Hi,
> >
> >> On 21/12/2023 15:06, Andre Przywara wrote:
> >>> The sunxi.yaml file, holding all known board compatible names for
> >>> devices with Allwinner SoCs, is apparently sorted alphabetically, by the
> >>> "description" entry. However there are a few outliers.
> >>>
> >>> Re-order the entries to get a strict alphabetical ordering, so that:
> >>> $ grep description: sunxi.yaml | sort -fc
> >>> returns empty-handed. There is no change otherwise.
> >>>
> >>> Signed-off-by: Andre Przywara <andre.przywara@arm.com>
> >>
> >> What was the existing order? Aren't this just shuffling again things?
> >
> > It assume the intention was to order "alphabetically by description", but
> > there were some out-of-place entries which apparently slipped through
> > review.
> >
> > Shall I add a comment at the top to note the order?
>
> Some files like this are ordered by compatible, some by class or family
> of devices, so first you need to be sure there is no existing order.
That's what I did, of course:
- there is no order in the compatible strings, not in the most specific
name, nor in the fallback names
- new boards have always been added somewhere in the middle, never at the
end, so it's not "oldest boards first"
- the SoCs used in the boards are all over the place, so it's not sorted
by SoC
- I see no rhyme or reason in the board types: we have tablets, devboards,
TV boxes all mixed up happily.
But: from the 197 listed boards, a movement of 13 boards, often by just one
position, makes the list ordered by description.
So I assumed this was the intention, and fixed the list accordingly. Happy
to stand corrected if the maintainers have a better idea.
Cheers,
Andre
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: arm: sunxi: sort alphabetically
2023-12-21 16:29 ` Andre Przywara
@ 2023-12-22 8:09 ` Krzysztof Kozlowski
0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-22 8:09 UTC (permalink / raw)
To: Andre Przywara
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, linux-arm-kernel, devicetree,
linux-sunxi, Maxime Ripard
On 21/12/2023 17:29, Andre Przywara wrote:
>>> Shall I add a comment at the top to note the order?
>>
>> Some files like this are ordered by compatible, some by class or family
>> of devices, so first you need to be sure there is no existing order.
>
> That's what I did, of course:
> - there is no order in the compatible strings, not in the most specific
> name, nor in the fallback names
> - new boards have always been added somewhere in the middle, never at the
> end, so it's not "oldest boards first"
> - the SoCs used in the boards are all over the place, so it's not sorted
> by SoC
> - I see no rhyme or reason in the board types: we have tablets, devboards,
> TV boxes all mixed up happily.
>
> But: from the 197 listed boards, a movement of 13 boards, often by just one
> position, makes the list ordered by description.
>
> So I assumed this was the intention, and fixed the list accordingly. Happy
> to stand corrected if the maintainers have a better idea.
With short comment at the top about sorting:
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-12-22 8:09 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-21 14:06 [PATCH] dt-bindings: arm: sunxi: sort alphabetically Andre Przywara
2023-12-21 14:45 ` Krzysztof Kozlowski
2023-12-21 15:07 ` Andre Przywara
2023-12-21 15:38 ` Krzysztof Kozlowski
2023-12-21 16:29 ` Andre Przywara
2023-12-22 8:09 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).