[PATCH v2 1/2] dt-bindings: arm: rockchip: add Radxa ROCK 3C

Chukun Pan posted 2 patches 1 year, 7 months ago
[PATCH v2 1/2] dt-bindings: arm: rockchip: add Radxa ROCK 3C
Posted by Chukun Pan 1 year, 7 months ago
The Radxa ROCK 3C is a similar board to the
Radxa ROCK 3A with the Rockchip RK3566 SoC.
Add devicetree binding documentation for it.

Signed-off-by: Chukun Pan <amadeus@jmu.edu.cn>
Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
index 1bbbaf81134b..e04c213a0dee 100644
--- a/Documentation/devicetree/bindings/arm/rockchip.yaml
+++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
@@ -799,6 +799,11 @@ properties:
           - const: radxa,rock3a
           - const: rockchip,rk3568
 
+      - description: Radxa ROCK 3C
+        items:
+          - const: radxa,rock-3c
+          - const: rockchip,rk3566
+
       - description: Radxa ROCK 5A
         items:
           - const: radxa,rock-5a
-- 
2.25.1
Re: [PATCH v2 1/2] dt-bindings: arm: rockchip: add Radxa ROCK 3C
Posted by Krzysztof Kozlowski 1 year, 7 months ago
On 28/04/2024 14:36, Chukun Pan wrote:
> The Radxa ROCK 3C is a similar board to the
> Radxa ROCK 3A with the Rockchip RK3566 SoC.
> Add devicetree binding documentation for it.

For future:

Please wrap commit message according to Linux coding style / submission
process (neither too early nor over the limit):
https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/submitting-patches.rst#L597

> 
> Signed-off-by: Chukun Pan <amadeus@jmu.edu.cn>
> Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
> index 1bbbaf81134b..e04c213a0dee 100644
> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
> @@ -799,6 +799,11 @@ properties:
>            - const: radxa,rock3a
>            - const: rockchip,rk3568
>  
> +      - description: Radxa ROCK 3C

There was some big renaming of these boards. I assume you are using
correct naming?

Best regards,
Krzysztof
Re: [PATCH v2 1/2] dt-bindings: arm: rockchip: add Radxa ROCK 3C
Posted by Chukun Pan 1 year, 7 months ago
>> The Radxa ROCK 3C is a similar board to the
>> Radxa ROCK 3A with the Rockchip RK3566 SoC.
>> Add devicetree binding documentation for it.

> For future:
>
> Please wrap commit message according to Linux coding style / submission
> process (neither too early nor over the limit)

Should I compress these three lines into two?

> There was some big renaming of these boards. I assume you are using
> correct naming?

According to the link below, the model name is now correct:
https://lore.kernel.org/lkml/B26C732A4DCEA9B3+282b8775-601b-4d4a-a513-4924b7940076@radxa.com/

Thanks,
Chukun

-- 
2.25.1
Re: [PATCH v2 1/2] dt-bindings: arm: rockchip: add Radxa ROCK 3C
Posted by Dragan Simic 1 year, 7 months ago
Hello Krzysztof,

On 2024-04-28 18:37, Krzysztof Kozlowski wrote:
> On 28/04/2024 14:36, Chukun Pan wrote:
>> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml 
>> b/Documentation/devicetree/bindings/arm/rockchip.yaml
>> index 1bbbaf81134b..e04c213a0dee 100644
>> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
>> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
>> @@ -799,6 +799,11 @@ properties:
>>            - const: radxa,rock3a
>>            - const: rockchip,rk3568
>> 
>> +      - description: Radxa ROCK 3C
> 
> There was some big renaming of these boards. I assume you are using
> correct naming?

I can confirm that the naming above is fine.