diff mbox series

Subject: [PATCH v1 1/2] dt-bindings: arm: realtek: Add Realtek Pym Particles

Message ID 9511f256edfd41aea7e9910879df3a1b@realtek.com
State Changes Requested, archived
Headers show
Series Subject: [PATCH v1 1/2] dt-bindings: arm: realtek: Add Realtek Pym Particles | expand

Checks

Context Check Description
robh/checkpatch warning total: 2 errors, 0 warnings, 12 lines checked
robh/patch-applied fail build log

Commit Message

CY_Huang[黃鉦晏] March 31, 2023, 9:58 a.m. UTC
Define a compatible string for Realtek RTD1319 Pym Particles eval board.

Signed-off-by: cy.huang <mailto:cy.huang@realtek.com>
---

v1:
* RTD1319 SoC and Realtek PymParticle EVB

Documentation/devicetree/bindings/arm/realtek.yaml | 6 ++++++
1 file changed, 6 insertions(+)

--
2.39.0

Comments

Krzysztof Kozlowski March 31, 2023, 12:19 p.m. UTC | #1
On 31/03/2023 11:58, CY_Huang[黃鉦晏] wrote:
> Define a compatible string for Realtek RTD1319 Pym Particles eval board.
> 
> Signed-off-by: cy.huang <mailto:cy.huang@realtek.com>

Your patch has corrupted subject.
> ---
> 
> v1:
> * RTD1319 SoC and Realtek PymParticle EVB

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC.  It might happen, that command when run on an older
kernel, gives you outdated entries.  Therefore please be sure you base
your patches on recent Linux kernel.

> 
> Documentation/devicetree/bindings/arm/realtek.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/realtek.yaml b/Documentation/devicetree/bindings/arm/realtek.yaml
> index ddd9a85099e9..8d4e92e9eb4b 100644
> --- a/Documentation/devicetree/bindings/arm/realtek.yaml
> +++ b/Documentation/devicetree/bindings/arm/realtek.yaml
> @@ -55,6 +55,12 @@ properties:
>                - realtek,mjolnir # Realtek Mjolnir EVB
>            - const: realtek,rtd1619
> 
> +      # RTD1319 SoC based boards
> +      - items:
> +          - enum:
> +              - realtek,pym-particles # Realtek Pym-particles EVB
> +          - const: realtek,rtd1319

Wrong placement - these are ordered by SoC. Don't add stuff just at the end.

> +
> additionalProperties: true
> 
> ...
> --
> 2.39.0

Best regards,
Krzysztof
CY_Huang[黃鉦晏] April 6, 2023, 2:28 a.m. UTC | #2
I'll resend with correctness,
Thanks a lot 

Cy
^-----Original Message-----
^From: Krzysztof Kozlowski <krzk@kernel.org>
^Sent: Friday, March 31, 2023 8:19 PM
^To: CY_Huang[黃鉦晏] <cy.huang@realtek.com>; afaerber@suse.de
^Cc: devicetree@vger.kernel.org; robh+dt@kernel.org; James Tai [戴志峰]
^<james.tai@realtek.com>
^Subject: Re: Subject: [PATCH v1 1/2] dt-bindings: arm: realtek: Add Realtek Pym
^Particles
^
^
^External mail.
^
^
^
^On 31/03/2023 11:58, CY_Huang[黃鉦晏] wrote:
^> Define a compatible string for Realtek RTD1319 Pym Particles eval board.
^>
^> Signed-off-by: cy.huang <mailto:cy.huang@realtek.com>
^
^Your patch has corrupted subject.
^> ---
^>
^> v1:
^> * RTD1319 SoC and Realtek PymParticle EVB
^
^Please use scripts/get_maintainers.pl to get a list of necessary people and lists to
^CC.  It might happen, that command when run on an older kernel, gives you
^outdated entries.  Therefore please be sure you base your patches on recent Linux
^kernel.
^
^>
^> Documentation/devicetree/bindings/arm/realtek.yaml | 6 ++++++
^> 1 file changed, 6 insertions(+)
^>
^> diff --git a/Documentation/devicetree/bindings/arm/realtek.yaml
^> b/Documentation/devicetree/bindings/arm/realtek.yaml
^> index ddd9a85099e9..8d4e92e9eb4b 100644
^> --- a/Documentation/devicetree/bindings/arm/realtek.yaml
^> +++ b/Documentation/devicetree/bindings/arm/realtek.yaml
^> @@ -55,6 +55,12 @@ properties:
^>                - realtek,mjolnir # Realtek Mjolnir EVB
^>            - const: realtek,rtd1619
^>
^> +      # RTD1319 SoC based boards
^> +      - items:
^> +          - enum:
^> +              - realtek,pym-particles # Realtek Pym-particles EVB
^> +          - const: realtek,rtd1319
^
^Wrong placement - these are ordered by SoC. Don't add stuff just at the end.
^
^> +
^> additionalProperties: true
^>
^> ...
^> --
^> 2.39.0
^
^Best regards,
^Krzysztof
^
^
^------Please consider the environment before printing this e-mail.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/realtek.yaml b/Documentation/devicetree/bindings/arm/realtek.yaml
index ddd9a85099e9..8d4e92e9eb4b 100644
--- a/Documentation/devicetree/bindings/arm/realtek.yaml
+++ b/Documentation/devicetree/bindings/arm/realtek.yaml
@@ -55,6 +55,12 @@  properties:
               - realtek,mjolnir # Realtek Mjolnir EVB
           - const: realtek,rtd1619

+      # RTD1319 SoC based boards
+      - items:
+          - enum:
+              - realtek,pym-particles # Realtek Pym-particles EVB
+          - const: realtek,rtd1319
+
additionalProperties: true

...