Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <d12fc3f83ace9285f7e1153003b14bc351ce95de.camel@collabora.com>
Subject: Re: [PATCHv6 4/7] dt-bindings: arm: rockchip: add initial rk3588
 boards
From:   Christopher Obbard <chris.obbard@collabora.com>
To:     Sebastian Reichel <sebastian.reichel@collabora.com>,
        Heiko Stuebner <heiko@sntech.de>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Marc Zyngier <maz@kernel.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        Benjamin Gaignard <benjamin.gaignard@collabora.com>,
        linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, kernel@collabora.com,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Date:   Thu, 05 Jan 2023 19:20:19 +0000
In-Reply-To: <20221214182247.79824-5-sebastian.reichel@collabora.com>
References: <20221214182247.79824-1-sebastian.reichel@collabora.com>
         <20221214182247.79824-5-sebastian.reichel@collabora.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.46.2-1 
MIME-Version: 1.0
Precedence: bulk

On Wed, 2022-12-14 at 19:22 +0100, Sebastian Reichel wrote:
> Add DT binding documentation for the Rockchip RK3588 EVB1,
> Radxa Rock 5 Model A and B.
>=20
> Co-Developed-by: Christopher Obbard <chris.obbard@collabora.com>
> Signed-off-by: Christopher Obbard <chris.obbard@collabora.com>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
> ---
> =C2=A0.../devicetree/bindings/arm/rockchip.yaml=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0 | 15
> +++++++++++++++
> =C2=A01 file changed, 15 insertions(+)
>=20
> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml
> b/Documentation/devicetree/bindings/arm/rockchip.yaml
> index c6c69a4e3777..49d442afe67f 100644
> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
> @@ -620,6 +620,16 @@ properties:
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: rad=
xa,rock3a
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: roc=
kchip,rk3568
> =C2=A0
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - description: Radxa Rock 5 Model A
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 items:
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: radxa,ro=
ck-5a
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: rockchip=
,rk3588s
> +
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - description: Radxa Rock 5 Model B

Hi Sebastian,

To better match other boards from this vendor (e.g. RADXA ROCK 4)

would it be best to capitalise this (and obviously the model field in
the respective DTS patches) the same?

So it would become "Radxa ROCK 5A" and "Radxa ROCK 5B".

It's also capitalised this way on the documentation website [1].

(I think the vendor has specifically dropped the Pi reference from
these boards, but I don't know why)

Thanks!
Chris

[1]: https://wiki.radxa.com/Rock5/5B


> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 items:
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: radxa,ro=
ck-5b
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: rockchip=
,rk3588
> +
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - description: Rikomagic MK808 v1
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 items:
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: rik=
omagic,mk808
> @@ -704,6 +714,11 @@ properties:
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: roc=
kchip,rk3399-sapphire-excavator
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: roc=
kchip,rk3399
> =C2=A0
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - description: Rockchip RK3588 Evaluation=
 board
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 items:
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: rockchip=
,rk3588-evb1-v10
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: rockchip=
,rk3588
> +
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - description: Rockchip RV1108 Evalu=
ation board
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 items:
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 - const: roc=
kchip,rv1108-evb
> --=20
> 2.39.0
>=20
>=20