[PATCH v2 0/2] Correct the model names and descriptions for Pine64 boards

From: Dragan Simic
Date: Sun May 26 2024 - 20:08:48 EST


This series corrects and, in a couple of cases, additionally simplifies
a bit the model names and the descriptions of a few Pine64 boards and
devices based on Allwinner SoCs, according to their official names used
in the Pine64 wiki and on the official Pine64 website. [1][2][3]

These corrections ensure consistency between the officially used Pine64
board names and the names and descriptions in the source code. These
changes complete the correction of the model names and descriptions of
the Pine64 boards and devices, which was started earlier with the Pine64
boards and devices based on Rockchip SoCs. [4][5]

Changes to the model names may cause certain issues if some scripts misuse
/proc/device-tree/model to detect the board they're executed on. Though,
the right way to detect a board is to use /proc/device-tree/compatible
instead, because its contents is part of the ABI. Such scripts, if they
actually exist in the field, should be improved to use the right way to
detect the board model.

Link to v1: https://lore.kernel.org/linux-sunxi/d2943d9f4c99a239f86188eaf45a73972685c255.1713833436.git.dsimic@xxxxxxxxxxx/T/#u

Changes in v2:
- Introduced this cover letter, partially by absorbing the notes from
the original versions of the individual patches
- Collected a couple of tags, which should still apply after the v2
changes described in the following bullet point
- As pointed out by Icenowy [6] and Andre, [7] historical and factual
accuracy trumps the naming consistency; after checking with Pine64,
it was concluded not to perform any retroactive naming cleanups, but
to preserve the history, so the v2 switches to the historically and
factually accurate board names, and introduces a few additional naming
corrections, for completeness
- Adjusted the patch descriptions according to the v2 changes

[1] https://wiki.pine64.org/wiki/PINE_A64
[2] https://wiki.pine64.org/wiki/PINE_H64
[3] https://pine64.org/devices/
[4] https://lore.kernel.org/linux-rockchip/ec124dab2b1a8776aa39177ecce34babca3a50e2.1713832790.git.dsimic@xxxxxxxxxxx/
[5] https://lore.kernel.org/linux-rockchip/06ce014a1dedff11a785fe523056b3b8ffdf21ee.1713832790.git.dsimic@xxxxxxxxxxx/
[6] https://lore.kernel.org/linux-sunxi/057b4a5504656bb7455ead39768d9e7167fb724b.camel@xxxxxxxxxx/
[7] https://lore.kernel.org/linux-sunxi/20240522110507.51b12966@xxxxxxxxxxxxxxxxxxxxxxxxxxx/

Dragan Simic (2):
dt-bindings: arm: sunxi: Correct the descriptions for Pine64 boards
arm64: dts: allwinner: Correct the model names for Pine64 boards

Documentation/devicetree/bindings/arm/sunxi.yaml | 16 ++++++++--------
.../boot/dts/allwinner/sun50i-a64-pine64-lts.dts | 2 +-
.../dts/allwinner/sun50i-a64-pine64-plus.dts | 2 +-
.../boot/dts/allwinner/sun50i-a64-pine64.dts | 2 +-
.../boot/dts/allwinner/sun50i-a64-pinebook.dts | 2 +-
.../sun50i-a64-pinetab-early-adopter.dts | 2 +-
.../boot/dts/allwinner/sun50i-a64-pinetab.dts | 2 +-
.../allwinner/sun50i-a64-sopine-baseboard.dts | 2 +-
.../dts/allwinner/sun50i-h6-pine-h64-model-b.dts | 2 +-
.../boot/dts/allwinner/sun50i-h6-pine-h64.dts | 2 +-
10 files changed, 17 insertions(+), 17 deletions(-)

Range-diff against v1:
1: 8deaf9d7d1b7 ! 1: 84cd1c708637 dt-bindings: arm: sunxi: Correct the descriptions for Pine64 boards
@@ Commit message
dt-bindings: arm: sunxi: Correct the descriptions for Pine64 boards

Correct the descriptions of a few Pine64 boards and devices, according
- to their official names used on the Pine64 wiki. This ensures consistency
- between the officially used names and the names in the source code.
+ to their official names used in the Pine64 wiki and on the official Pine64
+ website. [1][2][3] This ensures consistency between the officially used
+ names and the names in the source code.

+ [1] https://wiki.pine64.org/wiki/PINE_A64
+ [2] https://wiki.pine64.org/wiki/PINE_H64
+ [3] https://pine64.org/devices/
+
+ Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
Cc: Marek Kraus <gamiee@xxxxxxxxxx>
Signed-off-by: Dragan Simic <dsimic@xxxxxxxxxxx>

-
- ## Notes ##
- This completes the correction of the descriptions of the Pine64 boards
- and devices, which was started with the Pine64 boards and devices based
- on Rockchip SoCs. [1]
-
- [1] https://lore.kernel.org/linux-rockchip/ec124dab2b1a8776aa39177ecce34babca3a50e2.1713832790.git.dsimic@xxxxxxxxxxx/
-
## Documentation/devicetree/bindings/arm/sunxi.yaml ##
@@ Documentation/devicetree/bindings/arm/sunxi.yaml: properties:
+ - const: olimex,a64-teres-i
+ - const: allwinner,sun50i-a64
+
+- - description: Pine64
++ - description: Pine64 PINE A64
+ items:
+ - const: pine64,pine64
+ - const: allwinner,sun50i-a64
+
+- - description: Pine64+
++ - description: Pine64 PINE A64+
+ items:
+ - const: pine64,pine64-plus
+ - const: allwinner,sun50i-a64
+
+ - description: Pine64 PineCube
+ items:
+ - const: pine64,pinecube
- const: sochip,s3
- const: allwinner,sun8i-v3

- - description: Pine64 PineH64 model A
-+ - description: Pine64 H64 Model A
++ - description: Pine64 PINE H64 Model A
items:
- const: pine64,pine-h64
- const: allwinner,sun50i-h6

- - description: Pine64 PineH64 model B
-+ - description: Pine64 H64 Model B
++ - description: Pine64 PINE H64 Model B
items:
- const: pine64,pine-h64-model-b
- const: allwinner,sun50i-h6

- - description: Pine64 LTS
-+ - description: Pine64 A64 LTS
++ - description: Pine64 PINE A64 LTS
items:
- const: pine64,pine64-lts
- const: allwinner,sun50i-r18
@@ Documentation/devicetree/bindings/arm/sunxi.yaml: properties:
- const: allwinner,sun50i-a64

- - description: Pine64 SoPine Baseboard
-+ - description: Pine64 SOPine
++ - description: Pine64 SOPINE
items:
- const: pine64,sopine-baseboard
- const: pine64,sopine
2: fe5c35b78bef ! 2: 4a988518e0db arm64: dts: allwinner: Correct the model names for Pine64 boards
@@ Commit message
arm64: dts: allwinner: Correct the model names for Pine64 boards

Correct the model names of a few Pine64 boards and devices, according
- to their official names used on the Pine64 wiki. This ensures consistency
- between the officially used names and the names in the source code.
+ to their official names used in the Pine64 wiki and on the official Pine64
+ website. [1][2][3] This ensures consistency between the officially used
+ names and the names in the source code.

+ [1] https://wiki.pine64.org/wiki/PINE_A64
+ [2] https://wiki.pine64.org/wiki/PINE_H64
+ [3] https://pine64.org/devices/
+
+ Acked-by: Jernej Skrabec <jernej.skrabec@xxxxxxxxx>
Cc: Marek Kraus <gamiee@xxxxxxxxxx>
Signed-off-by: Dragan Simic <dsimic@xxxxxxxxxxx>

-
- ## Notes ##
- This completes the correction of the model names of the Pine64 boards
- and devices, which was started with the Pine64 boards and devices based
- on Rockchip SoCs. [1]
-
- These improvements may cause certain issues if some scripts misuse
- /proc/device-tree/model to detect the board they're executed on. Though,
- the right way to detect a board is to use /proc/device-tree/compatible
- instead, because its contents is part of the ABI. Such scripts, if they
- actually exist in the field, should be improved to use the right way to
- detect the board model.
-
- [1] https://lore.kernel.org/linux-rockchip/06ce014a1dedff11a785fe523056b3b8ffdf21ee.1713832790.git.dsimic@xxxxxxxxxxx/
-
## arch/arm64/boot/dts/allwinner/sun50i-a64-pine64-lts.dts ##
@@
#include "sun50i-a64-sopine-baseboard.dts"

/ {
- model = "Pine64 LTS";
-+ model = "Pine64 A64 LTS";
++ model = "Pine64 PINE A64 LTS";
compatible = "pine64,pine64-lts", "allwinner,sun50i-r18",
"allwinner,sun50i-a64";


+ ## arch/arm64/boot/dts/allwinner/sun50i-a64-pine64-plus.dts ##
+@@
+ #include "sun50i-a64-pine64.dts"
+
+ / {
+- model = "Pine64+";
++ model = "Pine64 PINE A64+";
+ compatible = "pine64,pine64-plus", "allwinner,sun50i-a64";
+
+ /* TODO: Camera, touchscreen, etc. */
+
+ ## arch/arm64/boot/dts/allwinner/sun50i-a64-pine64.dts ##
+@@
+ #include <dt-bindings/gpio/gpio.h>
+
+ / {
+- model = "Pine64";
++ model = "Pine64 PINE A64";
+ compatible = "pine64,pine64", "allwinner,sun50i-a64";
+
+ aliases {
+
## arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts ##
@@
#include <dt-bindings/pwm/pwm.h>
@@ arch/arm64/boot/dts/allwinner/sun50i-a64-sopine-baseboard.dts

/ {
- model = "SoPine with baseboard";
-+ model = "Pine64 SOPine on Baseboard carrier board";
++ model = "Pine64 SOPINE on Baseboard carrier board";
compatible = "pine64,sopine-baseboard", "pine64,sopine",
"allwinner,sun50i-a64";


## arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64-model-b.dts ##
@@
/delete-node/ &reg_gmac_3v3;

/ {
- model = "Pine H64 model B";
-+ model = "Pine64 H64 Model B";
++ model = "Pine64 PINE H64 Model B";
compatible = "pine64,pine-h64-model-b", "allwinner,sun50i-h6";

wifi_pwrseq: pwrseq {

## arch/arm64/boot/dts/allwinner/sun50i-h6-pine-h64.dts ##
@@
#include <dt-bindings/gpio/gpio.h>

/ {
- model = "Pine H64 model A";
-+ model = "Pine64 H64 Model A";
++ model = "Pine64 PINE H64 Model A";
compatible = "pine64,pine-h64", "allwinner,sun50i-h6";

aliases {