Skip to contentSkip to conversation

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

PATCH CY_Huang[黃鉦晏] 2 participants 3 messages Mar 31, 2023 – Apr 6, 2023

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

Define a compatible string for Realtek RTD1319 Pym Particles eval board.

---

v1:
* RTD1319 SoC and Realtek PymParticle EVB

Documentation/devicetree/bindings/arm/realtek.yaml | 6 ++++++

1 file changed, 6 insertions(+)
Documentation/devicetree/bindings/arm/realtek.yaml +0 -0
index ddd9a85099e9..8d4e92e9eb4b 100644
@@ -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

...
--
2.39.0

#1 lore raw
Quoted 3 lines · CY_Huang[黃鉦晏]

Define a compatible string for Realtek RTD1319 Pym Particles eval board.

Your patch has corrupted subject.

Quoted 4 lines

---

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.

Quoted 12 lines

Documentation/devicetree/bindings/arm/realtek.yaml | 6 ++++++

1 file changed, 6 insertions(+)
Documentation/devicetree/bindings/arm/realtek.yaml +0 -0
index ddd9a85099e9..8d4e92e9eb4b 100644
@@ -55,6 +55,12 @@ properties:

               - realtek,mjolnir # Realtek Mjolnir EVB
           - const: realtek,rtd1619

Quoted 5 lines

+      # 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.

Quoted 6 lines

+
additionalProperties: true

...
--
2.39.0

Best regards,
Krzysztof

#2 lore raw

I'll resend with correctness,
Thanks a lot

Cy
^-----Original Message-----
^From: Krzysztof Kozlowski <[email protected]>
^Sent: Friday, March 31, 2023 8:19 PM
^To: CY_Huang[黃鉦晏] <[email protected]>; [email protected]
^Cc: [email protected]; [email protected]; James Tai [戴志峰] ^<[email protected]>
^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:[email protected]>
^
^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.