LKML Archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
@ 2024-04-18 16:26 Dragan Simic
  2024-04-18 16:26 ` [PATCH 2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 boards Dragan Simic
                   ` (3 more replies)
  0 siblings, 4 replies; 12+ messages in thread
From: Dragan Simic @ 2024-04-18 16:26 UTC (permalink / raw
  To: linux-rockchip
  Cc: heiko, linux-arm-kernel, devicetree, robh+dt, krzk+dt, conor+dt,
	linux-kernel, amadeus, FUKAUMI Naoki

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@radxa.com>
Signed-off-by: Dragan Simic <dsimic@manjaro.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
@@ -705,7 +705,7 @@ properties:
           - const: qnap,ts433
           - const: rockchip,rk3568
 
-      - description: Radxa Compute Module 3(CM3)
+      - description: Radxa Compute Module 3 (CM3)
         items:
           - enum:
               - radxa,cm3-io
@@ -767,22 +767,22 @@ properties:
           - const: radxa,rockpis
           - const: rockchip,rk3308
 
-      - description: Radxa Rock2 Square
+      - description: Radxa Rock 2 Square
         items:
           - const: radxa,rock2-square
           - const: rockchip,rk3288
 
       - description: Radxa ROCK3 Model A
         items:
           - const: radxa,rock3a
           - const: rockchip,rk3568
 
-      - description: Radxa ROCK 5 Model A
+      - description: Radxa ROCK 5A
         items:
           - const: radxa,rock-5a
           - const: rockchip,rk3588s
 
-      - description: Radxa ROCK 5 Model B
+      - description: Radxa ROCK 5B
         items:
           - const: radxa,rock-5b
           - const: rockchip,rk3588

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 boards
  2024-04-18 16:26 [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Dragan Simic
@ 2024-04-18 16:26 ` Dragan Simic
  2024-04-18 16:59 ` [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Heiko Stübner
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 12+ messages in thread
From: Dragan Simic @ 2024-04-18 16:26 UTC (permalink / raw
  To: linux-rockchip
  Cc: heiko, linux-arm-kernel, devicetree, robh+dt, krzk+dt, conor+dt,
	linux-kernel, amadeus, FUKAUMI Naoki

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@radxa.com>
Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---
 arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts  | 2 +-
 arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
index 1fe8b2a0ed75..1c26f04cbfcb 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
@@ -7,7 +7,7 @@
 #include "rk3588.dtsi"
 
 / {
-	model = "Radxa ROCK 5 Model B";
+	model = "Radxa ROCK 5B";
 	compatible = "radxa,rock-5b", "rockchip,rk3588";
 
 	aliases {
diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts
index 00afb90d4eb1..4c7f2a2fdb11 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts
@@ -8,7 +8,7 @@
 #include "rk3588s.dtsi"
 
 / {
-	model = "Radxa ROCK 5 Model A";
+	model = "Radxa ROCK 5A";
 	compatible = "radxa,rock-5a", "rockchip,rk3588s";
 
 	aliases {

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 16:26 [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Dragan Simic
  2024-04-18 16:26 ` [PATCH 2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 boards Dragan Simic
@ 2024-04-18 16:59 ` Heiko Stübner
  2024-04-18 17:04   ` Conor Dooley
  2024-04-22 14:32 ` Rob Herring
  2024-04-23 15:15 ` Heiko Stuebner
  3 siblings, 1 reply; 12+ messages in thread
From: Heiko Stübner @ 2024-04-18 16:59 UTC (permalink / raw
  To: linux-rockchip, Dragan Simic
  Cc: linux-arm-kernel, devicetree, robh+dt, krzk+dt, conor+dt,
	linux-kernel, amadeus, FUKAUMI Naoki

Hi Dragan,

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@radxa.com>
> Signed-off-by: Dragan Simic <dsimic@manjaro.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?


Heiko



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 16:59 ` [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Heiko Stübner
@ 2024-04-18 17:04   ` Conor Dooley
  2024-04-18 17:07     ` Heiko Stübner
  2024-04-18 17:12     ` Dragan Simic
  0 siblings, 2 replies; 12+ messages in thread
From: Conor Dooley @ 2024-04-18 17:04 UTC (permalink / raw
  To: Heiko Stübner
  Cc: linux-rockchip, Dragan Simic, linux-arm-kernel, devicetree,
	robh+dt, krzk+dt, conor+dt, linux-kernel, amadeus, FUKAUMI Naoki

[-- Attachment #1: Type: text/plain, Size: 1355 bytes --]

On Thu, Apr 18, 2024 at 06:59:42PM +0200, Heiko Stübner wrote:
> Hi Dragan,
> 
> 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@radxa.com>
> > Signed-off-by: Dragan Simic <dsimic@manjaro.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.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 17:04   ` Conor Dooley
@ 2024-04-18 17:07     ` Heiko Stübner
  2024-04-18 17:12     ` Dragan Simic
  1 sibling, 0 replies; 12+ messages in thread
From: Heiko Stübner @ 2024-04-18 17:07 UTC (permalink / raw
  To: Conor Dooley
  Cc: linux-rockchip, Dragan Simic, linux-arm-kernel, devicetree,
	robh+dt, krzk+dt, conor+dt, linux-kernel, amadeus, FUKAUMI Naoki

Am Donnerstag, 18. April 2024, 19:04:44 CEST schrieb Conor Dooley:
> On Thu, Apr 18, 2024 at 06:59:42PM +0200, Heiko Stübner wrote:
> > Hi Dragan,
> > 
> > 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@radxa.com>
> > > Signed-off-by: Dragan Simic <dsimic@manjaro.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.

ah, now I see it too :-)



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 17:04   ` Conor Dooley
  2024-04-18 17:07     ` Heiko Stübner
@ 2024-04-18 17:12     ` Dragan Simic
  2024-04-19 14:09       ` Conor Dooley
  1 sibling, 1 reply; 12+ messages in thread
From: Dragan Simic @ 2024-04-18 17:12 UTC (permalink / raw
  To: Conor Dooley
  Cc: Heiko Stübner, linux-rockchip, linux-arm-kernel, devicetree,
	robh+dt, krzk+dt, conor+dt, linux-kernel, amadeus, FUKAUMI Naoki

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@radxa.com>
>> > Signed-off-by: Dragan Simic <dsimic@manjaro.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. :)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 17:12     ` Dragan Simic
@ 2024-04-19 14:09       ` Conor Dooley
  2024-04-21 13:54         ` Dragan Simic
  0 siblings, 1 reply; 12+ messages in thread
From: Conor Dooley @ 2024-04-19 14:09 UTC (permalink / raw
  To: Dragan Simic
  Cc: Heiko Stübner, linux-rockchip, linux-arm-kernel, devicetree,
	robh+dt, krzk+dt, conor+dt, linux-kernel, amadeus, FUKAUMI Naoki

[-- Attachment #1: Type: text/plain, Size: 2099 bytes --]

On Thu, Apr 18, 2024 at 07:12:35PM +0200, Dragan Simic wrote:
> 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@radxa.com>
> > > > Signed-off-by: Dragan Simic <dsimic@manjaro.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. :)

Let's not introduce half spaces into the bindings though :)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-19 14:09       ` Conor Dooley
@ 2024-04-21 13:54         ` Dragan Simic
  0 siblings, 0 replies; 12+ messages in thread
From: Dragan Simic @ 2024-04-21 13:54 UTC (permalink / raw
  To: Conor Dooley
  Cc: Heiko Stübner, linux-rockchip, linux-arm-kernel, devicetree,
	robh+dt, krzk+dt, conor+dt, linux-kernel, amadeus, FUKAUMI Naoki

On 2024-04-19 16:09, Conor Dooley wrote:
> On Thu, Apr 18, 2024 at 07:12:35PM +0200, Dragan Simic wrote:
>> 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@radxa.com>
>> > > > Signed-off-by: Dragan Simic <dsimic@manjaro.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. :)
> 
> Let's not introduce half spaces into the bindings though :)

Of course. :)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 16:26 [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Dragan Simic
  2024-04-18 16:26 ` [PATCH 2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 boards Dragan Simic
  2024-04-18 16:59 ` [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Heiko Stübner
@ 2024-04-22 14:32 ` Rob Herring
  2024-04-22 17:02   ` Dragan Simic
  2024-04-23 15:15 ` Heiko Stuebner
  3 siblings, 1 reply; 12+ messages in thread
From: Rob Herring @ 2024-04-22 14:32 UTC (permalink / raw
  To: Dragan Simic
  Cc: amadeus, conor+dt, krzk+dt, linux-arm-kernel, linux-kernel,
	robh+dt, heiko, linux-rockchip, devicetree, FUKAUMI Naoki


On Thu, 18 Apr 2024 18:26:19 +0200, Dragan Simic wrote:
> 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@radxa.com>
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> ---
>  Documentation/devicetree/bindings/arm/rockchip.yaml | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-22 14:32 ` Rob Herring
@ 2024-04-22 17:02   ` Dragan Simic
  0 siblings, 0 replies; 12+ messages in thread
From: Dragan Simic @ 2024-04-22 17:02 UTC (permalink / raw
  To: Rob Herring
  Cc: amadeus, conor+dt, krzk+dt, linux-arm-kernel, linux-kernel,
	robh+dt, heiko, linux-rockchip, devicetree, FUKAUMI Naoki

On 2024-04-22 16:32, Rob Herring wrote:
> On Thu, 18 Apr 2024 18:26:19 +0200, Dragan Simic wrote:
>> 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@radxa.com>
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>>  Documentation/devicetree/bindings/arm/rockchip.yaml | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> Acked-by: Rob Herring (Arm) <robh@kernel.org>

Thanks!

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-18 16:26 [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Dragan Simic
                   ` (2 preceding siblings ...)
  2024-04-22 14:32 ` Rob Herring
@ 2024-04-23 15:15 ` Heiko Stuebner
  2024-04-24  6:39   ` Dragan Simic
  3 siblings, 1 reply; 12+ messages in thread
From: Heiko Stuebner @ 2024-04-23 15:15 UTC (permalink / raw
  To: linux-rockchip, Dragan Simic
  Cc: Heiko Stuebner, krzk+dt, amadeus, linux-arm-kernel, FUKAUMI Naoki,
	robh+dt, linux-kernel, devicetree, conor+dt

On Thu, 18 Apr 2024 18:26:19 +0200, Dragan Simic wrote:
> 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/
> 
> 
> [...]

Applied, thanks!

[1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
      commit: d78084cdb5a959a5759efed9b3bc3182d005073c
[2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 boards
      commit: 45e831033f7a00a14f64afa1e34c476a9ff0f9f0

Best regards,
-- 
Heiko Stuebner <heiko@sntech.de>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards
  2024-04-23 15:15 ` Heiko Stuebner
@ 2024-04-24  6:39   ` Dragan Simic
  0 siblings, 0 replies; 12+ messages in thread
From: Dragan Simic @ 2024-04-24  6:39 UTC (permalink / raw
  To: Heiko Stuebner
  Cc: linux-rockchip, krzk+dt, amadeus, linux-arm-kernel, FUKAUMI Naoki,
	robh+dt, linux-kernel, devicetree, conor+dt

On 2024-04-23 17:15, Heiko Stuebner wrote:
> On Thu, 18 Apr 2024 18:26:19 +0200, Dragan Simic wrote:
>> 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/
>> 
>> 
>> [...]
> 
> Applied, thanks!
> 
> [1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa 
> boards
>       commit: d78084cdb5a959a5759efed9b3bc3182d005073c
> [2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 
> boards
>       commit: 45e831033f7a00a14f64afa1e34c476a9ff0f9f0

Thanks!

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2024-04-24  6:39 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-18 16:26 [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Dragan Simic
2024-04-18 16:26 ` [PATCH 2/2] arm64: dts: rockchip: Correct the model names for Radxa ROCK 5 boards Dragan Simic
2024-04-18 16:59 ` [PATCH 1/2] dt-bindings: arm: rockchip: Correct the descriptions for Radxa boards Heiko Stübner
2024-04-18 17:04   ` Conor Dooley
2024-04-18 17:07     ` Heiko Stübner
2024-04-18 17:12     ` Dragan Simic
2024-04-19 14:09       ` Conor Dooley
2024-04-21 13:54         ` Dragan Simic
2024-04-22 14:32 ` Rob Herring
2024-04-22 17:02   ` Dragan Simic
2024-04-23 15:15 ` Heiko Stuebner
2024-04-24  6:39   ` Dragan Simic

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 read-only IMAP folder(s) and NNTP newsgroup(s).