diff mbox series

[1/2] dt-bindings: arm: sunxi: Correct the descriptions for Pine64 boards

Message ID d2943d9f4c99a239f86188eaf45a73972685c255.1713833436.git.dsimic@manjaro.org (mailing list archive)
State New, archived
Headers show
Series [1/2] dt-bindings: arm: sunxi: Correct the descriptions for Pine64 boards | expand

Commit Message

Dragan Simic April 23, 2024, 1 a.m. UTC
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.

Cc: Marek Kraus <gamiee@pine64.org>
Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---

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@manjaro.org/

 Documentation/devicetree/bindings/arm/sunxi.yaml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Rob Herring April 23, 2024, 3:10 p.m. UTC | #1
On Tue, 23 Apr 2024 03:00:19 +0200, Dragan Simic wrote:
> 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.
> 
> Cc: Marek Kraus <gamiee@pine64.org>
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> ---
> 
> 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@manjaro.org/
> 
>  Documentation/devicetree/bindings/arm/sunxi.yaml | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Dragan Simic April 24, 2024, 6:38 a.m. UTC | #2
On 2024-04-23 17:10, Rob Herring wrote:
> On Tue, 23 Apr 2024 03:00:19 +0200, Dragan Simic wrote:
>> 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.
>> 
>> Cc: Marek Kraus <gamiee@pine64.org>
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>> 
>> 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@manjaro.org/
>> 
>>  Documentation/devicetree/bindings/arm/sunxi.yaml | 12 ++++++------
>>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> Reviewed-by: Rob Herring (Arm) <robh@kernel.org>

Thanks!
Dragan Simic May 22, 2024, 6:10 a.m. UTC | #3
Hello Icenowy,

On 2024-05-22 02:48, Icenowy Zheng wrote:
> 在 2024-04-23星期二的 03:00 +0200,Dragan Simic写道:
>> 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.
>> 
>> Cc: Marek Kraus <gamiee@pine64.org>
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>> 
>> 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@manjaro.org/
>> 
>>  Documentation/devicetree/bindings/arm/sunxi.yaml | 12 ++++++------
>>  1 file changed, 6 insertions(+), 6 deletions(-)
>> 
>> diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml
>> b/Documentation/devicetree/bindings/arm/sunxi.yaml
>> index 09d835db6db5..b66873ae2d71 100644
>> --- a/Documentation/devicetree/bindings/arm/sunxi.yaml
>> +++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
>> @@ -709,17 +709,17 @@ properties:
>>            - const: sochip,s3
>>            - const: allwinner,sun8i-v3
>>  
>> -      - description: Pine64 PineH64 model A
>> +      - description: Pine64 H64 Model A
>>          items:
>>            - const: pine64,pine-h64
>>            - const: allwinner,sun50i-h6
>>  
>> -      - description: Pine64 PineH64 model B
>> +      - description: Pine64 H64 Model B
> 
> Sorry for replying so late, but I don't think there is a Pine64 H64
> board. The Pine64 wiki calls it Pine H64. [1]
> 
> [1] https://wiki.pine64.org/wiki/PINE_H64

Good point, thanks.  Though, this board is really an exception to
the naming scheme employed for the Pine64 boards, so perhaps it would
actually be better to rename the board in the Pine64 wiki, by adding
"64" to "Pine", to ensure consistency.

Alas, the Pine64 wiki is currently in read-only mode, due to some
recent issues with the underlying hardware that runs it.  Migration to
another form of documentation for Pine64 boards is also a possibility,
which makes the updates even more complicated.

With all this in mind, I think it would be the best to rename the board
on the Pine64 side, to ensure consistency, and keep this patch as-is.
I'll make a mental note to do that on the Pine64 side once the current
situation with the Pine64 wiki is resolved.

>>          items:
>>            - const: pine64,pine-h64-model-b
>>            - const: allwinner,sun50i-h6
>>  
>> -      - description: Pine64 LTS
>> +      - description: Pine64 A64 LTS
>>          items:
>>            - const: pine64,pine64-lts
>>            - const: allwinner,sun50i-r18
>> @@ -748,17 +748,17 @@ properties:
>>            - const: pine64,pinephone
>>            - const: allwinner,sun50i-a64
>>  
>> -      - description: Pine64 PineTab, Development Sample
>> +      - description: Pine64 PineTab Developer Sample
>>          items:
>>            - const: pine64,pinetab
>>            - const: allwinner,sun50i-a64
>>  
>> -      - description: Pine64 PineTab, Early Adopter's batch (and
>> maybe later ones)
>> +      - description: Pine64 PineTab Early Adopter
>>          items:
>>            - const: pine64,pinetab-early-adopter
>>            - const: allwinner,sun50i-a64
>>  
>> -      - description: Pine64 SoPine Baseboard
>> +      - description: Pine64 SOPine
>>          items:
>>            - const: pine64,sopine-baseboard
>>            - const: pine64,sopine
>>
Andre Przywara May 22, 2024, 10:05 a.m. UTC | #4
On Wed, 22 May 2024 08:10:21 +0200
Dragan Simic <dsimic@manjaro.org> wrote:

Hi,

> Hello Icenowy,
> 
> On 2024-05-22 02:48, Icenowy Zheng wrote:
> > 在 2024-04-23星期二的 03:00 +0200,Dragan Simic写道:  
> >> 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.
> >> 
> >> Cc: Marek Kraus <gamiee@pine64.org>
> >> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> >> ---
> >> 
> >> 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@manjaro.org/
> >> 
> >>  Documentation/devicetree/bindings/arm/sunxi.yaml | 12 ++++++------
> >>  1 file changed, 6 insertions(+), 6 deletions(-)
> >> 
> >> diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml
> >> b/Documentation/devicetree/bindings/arm/sunxi.yaml
> >> index 09d835db6db5..b66873ae2d71 100644
> >> --- a/Documentation/devicetree/bindings/arm/sunxi.yaml
> >> +++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
> >> @@ -709,17 +709,17 @@ properties:
> >>            - const: sochip,s3
> >>            - const: allwinner,sun8i-v3
> >>  
> >> -      - description: Pine64 PineH64 model A
> >> +      - description: Pine64 H64 Model A
> >>          items:
> >>            - const: pine64,pine-h64
> >>            - const: allwinner,sun50i-h6
> >>  
> >> -      - description: Pine64 PineH64 model B
> >> +      - description: Pine64 H64 Model B  
> > 
> > Sorry for replying so late, but I don't think there is a Pine64 H64
> > board. The Pine64 wiki calls it Pine H64. [1]
> > 
> > [1] https://wiki.pine64.org/wiki/PINE_H64  
> 
> Good point, thanks.  Though, this board is really an exception to
> the naming scheme employed for the Pine64 boards, so perhaps it would
> actually be better to rename the board in the Pine64 wiki, by adding
> "64" to "Pine", to ensure consistency.

I am sorry, but I don't think this is how it works. The board is really
called "Pine H64", that's printed on the board [1] and everywhere else [2].
That's a choice the manufacturer made, and renaming some Wiki page won't
change that. I understand the engineer's desire to make everything nice and
consistent ;-) , but I am afraid that's not our call. After all this file
is to document the device naming, not to be an example of consistent
naming schemes.

Cheers,
Andre

[1] https://linux-sunxi.org/images/5/53/Pineh64_top.jpg
[2] https://pine64.org/devices/pine_h64_model_a/
> 
> Alas, the Pine64 wiki is currently in read-only mode, due to some
> recent issues with the underlying hardware that runs it.  Migration to
> another form of documentation for Pine64 boards is also a possibility,
> which makes the updates even more complicated.
> 
> With all this in mind, I think it would be the best to rename the board
> on the Pine64 side, to ensure consistency, and keep this patch as-is.
> I'll make a mental note to do that on the Pine64 side once the current
> situation with the Pine64 wiki is resolved.
> 
> >>          items:
> >>            - const: pine64,pine-h64-model-b
> >>            - const: allwinner,sun50i-h6
> >>  
> >> -      - description: Pine64 LTS
> >> +      - description: Pine64 A64 LTS
> >>          items:
> >>            - const: pine64,pine64-lts
> >>            - const: allwinner,sun50i-r18
> >> @@ -748,17 +748,17 @@ properties:
> >>            - const: pine64,pinephone
> >>            - const: allwinner,sun50i-a64
> >>  
> >> -      - description: Pine64 PineTab, Development Sample
> >> +      - description: Pine64 PineTab Developer Sample
> >>          items:
> >>            - const: pine64,pinetab
> >>            - const: allwinner,sun50i-a64
> >>  
> >> -      - description: Pine64 PineTab, Early Adopter's batch (and
> >> maybe later ones)
> >> +      - description: Pine64 PineTab Early Adopter
> >>          items:
> >>            - const: pine64,pinetab-early-adopter
> >>            - const: allwinner,sun50i-a64
> >>  
> >> -      - description: Pine64 SoPine Baseboard
> >> +      - description: Pine64 SOPine
> >>          items:
> >>            - const: pine64,sopine-baseboard
> >>            - const: pine64,sopine
> >>   
>
Dragan Simic May 22, 2024, 1:06 p.m. UTC | #5
Hello Andre,

On 2024-05-22 12:05, Andre Przywara wrote:
> On Wed, 22 May 2024 08:10:21 +0200
> Dragan Simic <dsimic@manjaro.org> wrote:
>> On 2024-05-22 02:48, Icenowy Zheng wrote:
>> > 在 2024-04-23星期二的 03:00 +0200,Dragan Simic写道:
>> >> 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.
>> >>
>> >> Cc: Marek Kraus <gamiee@pine64.org>
>> >> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> >> ---
>> >>
>> >> 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@manjaro.org/
>> >>
>> >>  Documentation/devicetree/bindings/arm/sunxi.yaml | 12 ++++++------
>> >>  1 file changed, 6 insertions(+), 6 deletions(-)
>> >>
>> >> diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml
>> >> b/Documentation/devicetree/bindings/arm/sunxi.yaml
>> >> index 09d835db6db5..b66873ae2d71 100644
>> >> --- a/Documentation/devicetree/bindings/arm/sunxi.yaml
>> >> +++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
>> >> @@ -709,17 +709,17 @@ properties:
>> >>            - const: sochip,s3
>> >>            - const: allwinner,sun8i-v3
>> >>  
>> >> -      - description: Pine64 PineH64 model A
>> >> +      - description: Pine64 H64 Model A
>> >>          items:
>> >>            - const: pine64,pine-h64
>> >>            - const: allwinner,sun50i-h6
>> >>  
>> >> -      - description: Pine64 PineH64 model B
>> >> +      - description: Pine64 H64 Model B
>> >
>> > Sorry for replying so late, but I don't think there is a Pine64 H64
>> > board. The Pine64 wiki calls it Pine H64. [1]
>> >
>> > [1] https://wiki.pine64.org/wiki/PINE_H64
>> 
>> Good point, thanks.  Though, this board is really an exception to
>> the naming scheme employed for the Pine64 boards, so perhaps it would
>> actually be better to rename the board in the Pine64 wiki, by adding
>> "64" to "Pine", to ensure consistency.
> 
> I am sorry, but I don't think this is how it works. The board is really
> called "Pine H64", that's printed on the board [1] and everywhere else 
> [2].
> That's a choice the manufacturer made, and renaming some Wiki page 
> won't
> change that. I understand the engineer's desire to make everything nice 
> and
> consistent ;-) , but I am afraid that's not our call. After all this 
> file
> is to document the device naming, not to be an example of consistent
> naming schemes.
> 
> [1] https://linux-sunxi.org/images/5/53/Pineh64_top.jpg
> [2] https://pine64.org/devices/pine_h64_model_a/

Those are all valid concerns.  Though, when it comes to the silkscreened
labels on PCBs, sometimes they have no connection with the real, 
official
board names.  An almost funny example :) is the QuartzPro64 board, also
from Pine64. [3]

I'll check with Pine64 are there some intentions for unifying the 
officially
used board names, and if there aren't, I'll submit the v2 of this series
with the corrected board names.

[3] 
https://wiki.pine64.org/images/f/fe/Quartzpro64_whole_board_top_resized.jpeg

>> Alas, the Pine64 wiki is currently in read-only mode, due to some
>> recent issues with the underlying hardware that runs it.  Migration to
>> another form of documentation for Pine64 boards is also a possibility,
>> which makes the updates even more complicated.
>> 
>> With all this in mind, I think it would be the best to rename the 
>> board
>> on the Pine64 side, to ensure consistency, and keep this patch as-is.
>> I'll make a mental note to do that on the Pine64 side once the current
>> situation with the Pine64 wiki is resolved.
>> 
>> >>          items:
>> >>            - const: pine64,pine-h64-model-b
>> >>            - const: allwinner,sun50i-h6
>> >>  
>> >> -      - description: Pine64 LTS
>> >> +      - description: Pine64 A64 LTS
>> >>          items:
>> >>            - const: pine64,pine64-lts
>> >>            - const: allwinner,sun50i-r18
>> >> @@ -748,17 +748,17 @@ properties:
>> >>            - const: pine64,pinephone
>> >>            - const: allwinner,sun50i-a64
>> >>  
>> >> -      - description: Pine64 PineTab, Development Sample
>> >> +      - description: Pine64 PineTab Developer Sample
>> >>          items:
>> >>            - const: pine64,pinetab
>> >>            - const: allwinner,sun50i-a64
>> >>  
>> >> -      - description: Pine64 PineTab, Early Adopter's batch (and
>> >> maybe later ones)
>> >> +      - description: Pine64 PineTab Early Adopter
>> >>          items:
>> >>            - const: pine64,pinetab-early-adopter
>> >>            - const: allwinner,sun50i-a64
>> >>  
>> >> -      - description: Pine64 SoPine Baseboard
>> >> +      - description: Pine64 SOPine
>> >>          items:
>> >>            - const: pine64,sopine-baseboard
>> >>            - const: pine64,sopine
>> >>
>>
Dragan Simic May 26, 2024, 4:40 p.m. UTC | #6
Hello Andre and Icenowy,

On 2024-05-22 15:06, Dragan Simic wrote:
> On 2024-05-22 12:05, Andre Przywara wrote:
>> On Wed, 22 May 2024 08:10:21 +0200
>> Dragan Simic <dsimic@manjaro.org> wrote:
>>> On 2024-05-22 02:48, Icenowy Zheng wrote:
>>> > 在 2024-04-23星期二的 03:00 +0200,Dragan Simic写道:
>>> >> 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.
>>> >>
>>> >> [...]
>>> >>
>>> >> -      - description: Pine64 PineH64 model A
>>> >> +      - description: Pine64 H64 Model A
>>> >>          items:
>>> >>            - const: pine64,pine-h64
>>> >>            - const: allwinner,sun50i-h6
>>> >>  
>>> >> -      - description: Pine64 PineH64 model B
>>> >> +      - description: Pine64 H64 Model B
>>> >
>>> > Sorry for replying so late, but I don't think there is a Pine64 H64
>>> > board. The Pine64 wiki calls it Pine H64. [1]
>>> >
>>> > [1] https://wiki.pine64.org/wiki/PINE_H64
>>> 
>>> Good point, thanks.  Though, this board is really an exception to
>>> the naming scheme employed for the Pine64 boards, so perhaps it would
>>> actually be better to rename the board in the Pine64 wiki, by adding
>>> "64" to "Pine", to ensure consistency.
>> 
>> I am sorry, but I don't think this is how it works. The board is 
>> really
>> called "Pine H64", that's printed on the board [1] and everywhere else 
>> [2].
>> That's a choice the manufacturer made, and renaming some Wiki page 
>> won't
>> change that. I understand the engineer's desire to make everything 
>> nice and
>> consistent ;-) , but I am afraid that's not our call. After all this 
>> file
>> is to document the device naming, not to be an example of consistent
>> naming schemes.
>> 
>> [1] https://linux-sunxi.org/images/5/53/Pineh64_top.jpg
>> [2] https://pine64.org/devices/pine_h64_model_a/
> 
> Those are all valid concerns.  Though, when it comes to the 
> silkscreened
> labels on PCBs, sometimes they have no connection with the real, 
> official
> board names.  An almost funny example :) is the QuartzPro64 board, also
> from Pine64. [3]
> 
> I'll check with Pine64 are there some intentions for unifying the 
> officially
> used board names, and if there aren't, I'll submit the v2 of this 
> series
> with the corrected board names.
> 
> [3] 
> https://wiki.pine64.org/images/f/fe/Quartzpro64_whole_board_top_resized.jpeg

It's been decided that the history should be preserved, so I'll prepare
and submit the v2 of this series with the original Pine64 board names 
that
are currently used officially.

>>> Alas, the Pine64 wiki is currently in read-only mode, due to some
>>> recent issues with the underlying hardware that runs it.  Migration 
>>> to
>>> another form of documentation for Pine64 boards is also a 
>>> possibility,
>>> which makes the updates even more complicated.
>>> 
>>> With all this in mind, I think it would be the best to rename the 
>>> board
>>> on the Pine64 side, to ensure consistency, and keep this patch as-is.
>>> I'll make a mental note to do that on the Pine64 side once the 
>>> current
>>> situation with the Pine64 wiki is resolved.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
index 09d835db6db5..b66873ae2d71 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
@@ -709,17 +709,17 @@  properties:
           - const: sochip,s3
           - const: allwinner,sun8i-v3
 
-      - description: Pine64 PineH64 model A
+      - description: Pine64 H64 Model A
         items:
           - const: pine64,pine-h64
           - const: allwinner,sun50i-h6
 
-      - description: Pine64 PineH64 model B
+      - description: Pine64 H64 Model B
         items:
           - const: pine64,pine-h64-model-b
           - const: allwinner,sun50i-h6
 
-      - description: Pine64 LTS
+      - description: Pine64 A64 LTS
         items:
           - const: pine64,pine64-lts
           - const: allwinner,sun50i-r18
@@ -748,17 +748,17 @@  properties:
           - const: pine64,pinephone
           - const: allwinner,sun50i-a64
 
-      - description: Pine64 PineTab, Development Sample
+      - description: Pine64 PineTab Developer Sample
         items:
           - const: pine64,pinetab
           - const: allwinner,sun50i-a64
 
-      - description: Pine64 PineTab, Early Adopter's batch (and maybe later ones)
+      - description: Pine64 PineTab Early Adopter
         items:
           - const: pine64,pinetab-early-adopter
           - const: allwinner,sun50i-a64
 
-      - description: Pine64 SoPine Baseboard
+      - description: Pine64 SOPine
         items:
           - const: pine64,sopine-baseboard
           - const: pine64,sopine