Hello,
This patch series adds initial Device Trees for Realtek RTD1319 SoC and
Realtek Pym Particles EVB.
This v4 is an update of James' v3, incorporating pending review comments.
Upstreaming progress being tracked at:
https://en.opensuse.org/HCL:Realtek_DHC
Latest experimental patches at:
https://github.com/afaerber/linux/commits/rtd1295-next
Have a lot of fun!
Cheers,
Andreas
v3 -> v4:
* Updated Realtek copyright for files changed in v3
* Updated PMU compatible (Robin)
* Changed compatible, renamed .dts
* Updated bindings schema and prepended refactoring
Cc: [email protected]
Cc: Rob Herring <[email protected]>
Cc: James Tai <[email protected]>
Andreas Färber (1):
dt-bindings: arm: realtek: Convert comments to descriptions
James Tai (2):
dt-bindings: arm: realtek: Document RTD1319 and Realtek Pym Particles
EVB
arm64: dts: realtek: Add RTD1319 SoC and Realtek Pym Particles EVB
.../devicetree/bindings/arm/realtek.yaml | 30 ++-
arch/arm64/boot/dts/realtek/Makefile | 2 +
.../boot/dts/realtek/rtd1319-pymparticles.dts | 43 ++++
arch/arm64/boot/dts/realtek/rtd1319.dtsi | 12 +
arch/arm64/boot/dts/realtek/rtd13xx.dtsi | 213 ++++++++++++++++++
5 files changed, 288 insertions(+), 12 deletions(-)
create mode 100644 arch/arm64/boot/dts/realtek/rtd1319-pymparticles.dts
create mode 100644 arch/arm64/boot/dts/realtek/rtd1319.dtsi
create mode 100644 arch/arm64/boot/dts/realtek/rtd13xx.dtsi
--
2.26.2
Turn the SoC-level comments into description properties.
Signed-off-by: Andreas Färber <[email protected]>
---
v4: New
.../devicetree/bindings/arm/realtek.yaml | 24 +++++++++----------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/realtek.yaml b/Documentation/devicetree/bindings/arm/realtek.yaml
index 845f9c76d6f7..0b388016bbcd 100644
--- a/Documentation/devicetree/bindings/arm/realtek.yaml
+++ b/Documentation/devicetree/bindings/arm/realtek.yaml
@@ -14,21 +14,21 @@ properties:
const: '/'
compatible:
oneOf:
- # RTD1195 SoC based boards
- - items:
+ - description: RTD1195 SoC based boards
+ items:
- enum:
- mele,x1000 # MeLE X1000
- realtek,horseradish # Realtek Horseradish EVB
- const: realtek,rtd1195
- # RTD1293 SoC based boards
- - items:
+ - description: RTD1293 SoC based boards
+ items:
- enum:
- synology,ds418j # Synology DiskStation DS418j
- const: realtek,rtd1293
- # RTD1295 SoC based boards
- - items:
+ - description: RTD1295 SoC based boards
+ items:
- enum:
- mele,v9 # MeLE V9
- probox2,ava # ProBox2 AVA
@@ -36,21 +36,21 @@ properties:
- zidoo,x9s # Zidoo X9S
- const: realtek,rtd1295
- # RTD1296 SoC based boards
- - items:
+ - description: RTD1296 SoC based boards
+ items:
- enum:
- synology,ds418 # Synology DiskStation DS418
- const: realtek,rtd1296
- # RTD1395 SoC based boards
- - items:
+ - description: RTD1395 SoC based boards
+ items:
- enum:
- bananapi,bpi-m4 # Banana Pi BPI-M4
- realtek,lion-skin # Realtek Lion Skin EVB
- const: realtek,rtd1395
- # RTD1619 SoC based boards
- - items:
+ - description: RTD1619 SoC based boards
+ items:
- enum:
- realtek,mjolnir # Realtek Mjolnir EVB
- const: realtek,rtd1619
--
2.26.2
From: James Tai <[email protected]>
Add Device Trees for Realtek RTD1319 SoC family, RTD1319 SoC and
Realtek Pym Particles EVB.
Signed-off-by: James Tai <[email protected]>
Signed-off-by: Andreas Färber <[email protected]>
---
v3 -> v4:
* Updated Realtek copyright for 2 out of 3 files from v3
* Renamed from rtd1319-pymparticle.dts to rtd1319-pymparticles.dts
* Updated compatible from pymparticle to pym-particles
* Updated PMU compatible from armv8-pmuv3 to cortex-a55-pmu (Robin)
v2 -> v3:
* Add virtual maintenance interrupt for architecture timer
* Correct the GIC redistributor address range
v1 -> v2:
* Reserve the boot ROM address
* Reserve boot loader address
* Reserve audio/video FW address
* Reserve RPC and ring buffer address
* Reserve TEE address
* Support 1 GiB RAM by default
* Reduce rbus range to 2 MiB
* Apply the syscon for ISO,MISC,CRT,SB2,SCPU_WRAPPER
* Adjust compatible strings order in document
arch/arm64/boot/dts/realtek/Makefile | 2 +
.../boot/dts/realtek/rtd1319-pymparticles.dts | 43 ++++
arch/arm64/boot/dts/realtek/rtd1319.dtsi | 12 +
arch/arm64/boot/dts/realtek/rtd13xx.dtsi | 213 ++++++++++++++++++
4 files changed, 270 insertions(+)
create mode 100644 arch/arm64/boot/dts/realtek/rtd1319-pymparticles.dts
create mode 100644 arch/arm64/boot/dts/realtek/rtd1319.dtsi
create mode 100644 arch/arm64/boot/dts/realtek/rtd13xx.dtsi
diff --git a/arch/arm64/boot/dts/realtek/Makefile b/arch/arm64/boot/dts/realtek/Makefile
index ef8d8fcbaa05..83708596726d 100644
--- a/arch/arm64/boot/dts/realtek/Makefile
+++ b/arch/arm64/boot/dts/realtek/Makefile
@@ -9,6 +9,8 @@ dtb-$(CONFIG_ARCH_REALTEK) += rtd1295-zidoo-x9s.dtb
dtb-$(CONFIG_ARCH_REALTEK) += rtd1296-ds418.dtb
+dtb-$(CONFIG_ARCH_REALTEK) += rtd1319-pymparticles.dtb
+
dtb-$(CONFIG_ARCH_REALTEK) += rtd1395-bpi-m4.dtb
dtb-$(CONFIG_ARCH_REALTEK) += rtd1395-lionskin.dtb
diff --git a/arch/arm64/boot/dts/realtek/rtd1319-pymparticles.dts b/arch/arm64/boot/dts/realtek/rtd1319-pymparticles.dts
new file mode 100644
index 000000000000..e0b3c3707a85
--- /dev/null
+++ b/arch/arm64/boot/dts/realtek/rtd1319-pymparticles.dts
@@ -0,0 +1,43 @@
+// SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause)
+/*
+ * Copyright (c) 2019-2020 Realtek Semiconductor Corp.
+ */
+
+/dts-v1/;
+
+#include "rtd1319.dtsi"
+
+/ {
+ compatible = "realtek,pym-particles", "realtek,rtd1319";
+ model = "Realtek Pym Particles EVB";
+
+ memory@2e000 {
+ device_type = "memory";
+ reg = <0x2e000 0x3ffd2000>; /* boot ROM to 1 GiB or 2 GiB */
+ };
+
+ chosen {
+ stdout-path = "serial0:460800n8";
+ };
+
+ aliases {
+ serial0 = &uart0;
+ serial1 = &uart1;
+ serial2 = &uart2;
+ };
+};
+
+/* debug console (J1) */
+&uart0 {
+ status = "okay";
+};
+
+/* M.2 slots (CON2, CON8) and J14 */
+&uart1 {
+ status = "disabled";
+};
+
+/* GPIO connector (T1) */
+&uart2 {
+ status = "disabled";
+};
diff --git a/arch/arm64/boot/dts/realtek/rtd1319.dtsi b/arch/arm64/boot/dts/realtek/rtd1319.dtsi
new file mode 100644
index 000000000000..1dcee00009cd
--- /dev/null
+++ b/arch/arm64/boot/dts/realtek/rtd1319.dtsi
@@ -0,0 +1,12 @@
+// SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause)
+/*
+ * Realtek RTD1319 SoC
+ *
+ * Copyright (c) 2019 Realtek Semiconductor Corp.
+ */
+
+#include "rtd13xx.dtsi"
+
+/ {
+ compatible = "realtek,rtd1319";
+};
diff --git a/arch/arm64/boot/dts/realtek/rtd13xx.dtsi b/arch/arm64/boot/dts/realtek/rtd13xx.dtsi
new file mode 100644
index 000000000000..8c5b6fc7b8eb
--- /dev/null
+++ b/arch/arm64/boot/dts/realtek/rtd13xx.dtsi
@@ -0,0 +1,213 @@
+// SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause)
+/*
+ * Realtek RTD13xx SoC family
+ *
+ * Copyright (c) 2019-2020 Realtek Semiconductor Corp.
+ */
+
+/memreserve/ 0x0000000000000000 0x000000000002e000; /* Boot ROM */
+/memreserve/ 0x000000000002e000 0x0000000000100000; /* Boot loader */
+/memreserve/ 0x000000000f400000 0x0000000000500000; /* Video FW */
+/memreserve/ 0x000000000f900000 0x0000000000500000; /* Audio FW */
+/memreserve/ 0x0000000010000000 0x0000000000014000; /* Audio FW RAM */
+
+#include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+
+/ {
+ interrupt-parent = <&gic>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ rpc_comm: rpc@3f000 {
+ reg = <0x3f000 0x1000>;
+ };
+
+ rpc_ringbuf: rpc@1ffe000 {
+ reg = <0x1ffe000 0x4000>;
+ };
+
+ tee: tee@10100000 {
+ reg = <0x10100000 0xf00000>;
+ no-map;
+ };
+ };
+
+ cpus {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ cpu0: cpu@0 {
+ device_type = "cpu";
+ compatible = "arm,cortex-a55";
+ reg = <0x0>;
+ enable-method = "psci";
+ next-level-cache = <&l2>;
+ };
+
+ cpu1: cpu@100 {
+ device_type = "cpu";
+ compatible = "arm,cortex-a55";
+ reg = <0x100>;
+ enable-method = "psci";
+ next-level-cache = <&l2>;
+ };
+
+ cpu2: cpu@200 {
+ device_type = "cpu";
+ compatible = "arm,cortex-a55";
+ reg = <0x200>;
+ enable-method = "psci";
+ next-level-cache = <&l2>;
+ };
+
+ cpu3: cpu@300 {
+ device_type = "cpu";
+ compatible = "arm,cortex-a55";
+ reg = <0x300>;
+ enable-method = "psci";
+ next-level-cache = <&l2>;
+ };
+
+ l2: l2-cache {
+ compatible = "cache";
+ };
+ };
+
+ timer {
+ compatible = "arm,armv8-timer";
+ interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>,
+ <GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>,
+ <GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>,
+ <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>,
+ <GIC_PPI 9 IRQ_TYPE_LEVEL_LOW>;
+ };
+
+ arm_pmu: pmu {
+ compatible = "arm,cortex-a55-pmu";
+ interrupts = <GIC_PPI 7 IRQ_TYPE_LEVEL_LOW>;
+ interrupt-affinity = <&cpu0>, <&cpu1>, <&cpu2>, <&cpu3>;
+ };
+
+ psci {
+ compatible = "arm,psci-1.0";
+ method = "smc";
+ };
+
+ osc27M: osc {
+ compatible = "fixed-clock";
+ clock-frequency = <27000000>;
+ clock-output-names = "osc27M";
+ #clock-cells = <0>;
+ };
+
+ soc {
+ compatible = "simple-bus";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x00000000 0x00000000 0x0002e000>, /* boot ROM */
+ <0xff100000 0xff100000 0x00200000>, /* GIC */
+ <0x98000000 0x98000000 0x00200000>; /* rbus */
+
+ rbus: bus@98000000 {
+ compatible = "simple-bus";
+ reg = <0x98000000 0x200000>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x0 0x98000000 0x200000>;
+
+ crt: syscon@0 {
+ compatible = "syscon", "simple-mfd";
+ reg = <0x0 0x1000>;
+ reg-io-width = <4>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x0 0x0 0x1000>;
+ };
+
+ iso: syscon@7000 {
+ compatible = "syscon", "simple-mfd";
+ reg = <0x7000 0x1000>;
+ reg-io-width = <4>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x0 0x7000 0x1000>;
+ };
+
+ sb2: syscon@1a000 {
+ compatible = "syscon", "simple-mfd";
+ reg = <0x1a000 0x1000>;
+ reg-io-width = <4>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x0 0x1a000 0x1000>;
+ };
+
+ misc: syscon@1b000 {
+ compatible = "syscon", "simple-mfd";
+ reg = <0x1b000 0x1000>;
+ reg-io-width = <4>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x0 0x1b000 0x1000>;
+ };
+
+ scpu_wrapper: syscon@1d000 {
+ compatible = "syscon", "simple-mfd";
+ reg = <0x1d000 0x1000>;
+ reg-io-width = <4>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0x0 0x1d000 0x1000>;
+ };
+ };
+
+ gic: interrupt-controller@ff100000 {
+ compatible = "arm,gic-v3";
+ reg = <0xff100000 0x10000>,
+ <0xff140000 0x80000>;
+ interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-controller;
+ #interrupt-cells = <3>;
+ };
+ };
+};
+
+&iso {
+ uart0: serial0@800 {
+ compatible = "snps,dw-apb-uart";
+ reg = <0x800 0x400>;
+ reg-shift = <2>;
+ reg-io-width = <4>;
+ interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
+ clock-frequency = <432000000>;
+ status = "disabled";
+ };
+};
+
+&misc {
+ uart1: serial1@200 {
+ compatible = "snps,dw-apb-uart";
+ reg = <0x200 0x400>;
+ reg-shift = <2>;
+ reg-io-width = <4>;
+ interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
+ clock-frequency = <432000000>;
+ status = "disabled";
+ };
+
+ uart2: serial2@400 {
+ compatible = "snps,dw-apb-uart";
+ reg = <0x400 0x400>;
+ reg-shift = <2>;
+ reg-io-width = <4>;
+ interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
+ clock-frequency = <432000000>;
+ status = "disabled";
+ };
+};
--
2.26.2
From: James Tai <[email protected]>
Define compatible strings for Realtek RTD1319 SoC and Realtek Pym Particles
EVB.
Signed-off-by: James Tai <[email protected]>
Signed-off-by: Andreas Färber <[email protected]>
---
v3 -> v4:
* Renamed compatible from pymparticle to pym-particles
* Turned SoC comment into description
v2 -> v3: Unchanged
v1 -> v2:
* Put string in alphabetical order
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 0b388016bbcd..e36e87df3521 100644
--- a/Documentation/devicetree/bindings/arm/realtek.yaml
+++ b/Documentation/devicetree/bindings/arm/realtek.yaml
@@ -42,6 +42,12 @@ properties:
- synology,ds418 # Synology DiskStation DS418
- const: realtek,rtd1296
+ - description: RTD1319 SoC based boards
+ items:
+ - enum:
+ - realtek,pym-particles # Realtek Pym Particles EVB
+ - const: realtek,rtd1319
+
- description: RTD1395 SoC based boards
items:
- enum:
--
2.26.2
Am 21.06.20 um 01:32 schrieb Andreas Färber:
> diff --git a/arch/arm64/boot/dts/realtek/rtd13xx.dtsi b/arch/arm64/boot/dts/realtek/rtd13xx.dtsi
> new file mode 100644
> index 000000000000..8c5b6fc7b8eb
> --- /dev/null
> +++ b/arch/arm64/boot/dts/realtek/rtd13xx.dtsi
[...]
> +&iso {
> + uart0: serial0@800 {
Node name should be serial, not serial0.
> + compatible = "snps,dw-apb-uart";
> + reg = <0x800 0x400>;
> + reg-shift = <2>;
> + reg-io-width = <4>;
> + interrupts = <GIC_SPI 68 IRQ_TYPE_LEVEL_HIGH>;
> + clock-frequency = <432000000>;
> + status = "disabled";
> + };
> +};
> +
> +&misc {
> + uart1: serial1@200 {
Ditto, serial.
> + compatible = "snps,dw-apb-uart";
> + reg = <0x200 0x400>;
> + reg-shift = <2>;
> + reg-io-width = <4>;
> + interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
> + clock-frequency = <432000000>;
> + status = "disabled";
> + };
> +
> + uart2: serial2@400 {
Ditto.
> + compatible = "snps,dw-apb-uart";
> + reg = <0x400 0x400>;
> + reg-shift = <2>;
> + reg-io-width = <4>;
> + interrupts = <GIC_SPI 90 IRQ_TYPE_LEVEL_HIGH>;
> + clock-frequency = <432000000>;
> + status = "disabled";
> + };
> +};
Regards,
Andreas
--
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer
HRB 36809 (AG Nürnberg)
Hi James,
Am 21.06.20 um 01:32 schrieb Andreas Färber:
> From: James Tai <[email protected]>
>
> Add Device Trees for Realtek RTD1319 SoC family, RTD1319 SoC and
> Realtek Pym Particles EVB.
>
> Signed-off-by: James Tai <[email protected]>
> Signed-off-by: Andreas Färber <[email protected]>
> ---
> v3 -> v4:
> * Updated Realtek copyright for 2 out of 3 files from v3
> * Renamed from rtd1319-pymparticle.dts to rtd1319-pymparticles.dts
> * Updated compatible from pymparticle to pym-particles
> * Updated PMU compatible from armv8-pmuv3 to cortex-a55-pmu (Robin)
>
> v2 -> v3:
> * Add virtual maintenance interrupt for architecture timer
> * Correct the GIC redistributor address range
[...]
> diff --git a/arch/arm64/boot/dts/realtek/rtd13xx.dtsi b/arch/arm64/boot/dts/realtek/rtd13xx.dtsi
> new file mode 100644
> index 000000000000..8c5b6fc7b8eb
> --- /dev/null
> +++ b/arch/arm64/boot/dts/realtek/rtd13xx.dtsi
[...]
> + gic: interrupt-controller@ff100000 {
> + compatible = "arm,gic-v3";
> + reg = <0xff100000 0x10000>,
> + <0xff140000 0x80000>;
> + interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>;
In my testing this appears to cause the following error:
[ 2.239858] irq: type mismatch, failed to map hwirq-25 for
interrupt-controller@ff100000!
...
[ 3.505649] kvm [1]: IPA Size Limit: 40bits
[ 3.506051] kvm [1]: GICv3: no GICV resource entry
[ 3.506058] kvm [1]: disabling GICv2 emulation
[ 3.506081] kvm [1]: GIC system register CPU interface enabled
[ 3.506175] kvm [1]: vgic interrupt IRQ1
[ 3.506293] kvm [1]: Hyp mode initialized successfully
If I change it to IRQ_TYPE_LEVEL_LOW, that error goes away:
[ 3.506030] kvm [1]: IPA Size Limit: 40bits
[ 3.506430] kvm [1]: GICv3: no GICV resource entry
[ 3.506437] kvm [1]: disabling GICv2 emulation
[ 3.506459] kvm [1]: GIC system register CPU interface enabled
[ 3.506551] kvm [1]: vgic interrupt IRQ1
[ 3.506672] kvm [1]: Hyp mode initialized successfully
In-tree RTD1619 has it as HIGH, too, but doesn't show above error:
[ 2.918973] kvm [1]: IPA Size Limit: 40bits
[ 2.919345] kvm [1]: GICv3: no GICV resource entry
[ 2.919352] kvm [1]: disabling GICv2 emulation
[ 2.919373] kvm [1]: GIC system register CPU interface enabled
[ 2.919522] kvm [1]: vgic interrupt IRQ1
[ 2.919700] kvm [1]: Hyp mode initialized successfully
RTD1619 doesn't show an error either if I change it to LOW though:
[ 2.918843] kvm [1]: IPA Size Limit: 40bits
[ 2.919212] kvm [1]: GICv3: no GICV resource entry
[ 2.919218] kvm [1]: disabling GICv2 emulation
[ 2.919240] kvm [1]: GIC system register CPU interface enabled
[ 2.919390] kvm [1]: vgic interrupt IRQ1
[ 2.919567] kvm [1]: Hyp mode initialized successfully
The GICv3 bindings example does have it as 4 == HIGH, but so does the
GICv2 binding example, and yet we used LOW == 8 for in-tree RTD139x,
RTD129x and RTD1195. The downstream BSP uses value 4 == HIGH for both
RTD16xx and RTD13xx - is it possible this was never actually tested?
Thanks in advance for clarifying the correct interrupt polarity.
> + interrupt-controller;
> + #interrupt-cells = <3>;
> + };
[snip]
Regards,
Andreas
--
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer
HRB 36809 (AG Nürnberg)
> -----Original Message-----
> From: linux-realtek-soc <[email protected]> On
> Behalf Of Andreas Farber
> Sent: Sunday, June 21, 2020 7:32 AM
> To: [email protected]
> Cc: [email protected]; Rob Herring <[email protected]>;
> [email protected]; [email protected]; Andreas
> Färber <[email protected]>
> Subject: [PATCH v4 1/3] dt-bindings: arm: realtek: Convert comments to
> descriptions
>
> Turn the SoC-level comments into description properties.
>
> Signed-off-by: Andreas Färber <[email protected]>
> ---
> v4: New
>
> .../devicetree/bindings/arm/realtek.yaml | 24 +++++++++----------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/realtek.yaml
> b/Documentation/devicetree/bindings/arm/realtek.yaml
> index 845f9c76d6f7..0b388016bbcd 100644
> --- a/Documentation/devicetree/bindings/arm/realtek.yaml
> +++ b/Documentation/devicetree/bindings/arm/realtek.yaml
> @@ -14,21 +14,21 @@ properties:
> const: '/'
> compatible:
> oneOf:
> - # RTD1195 SoC based boards
> - - items:
> + - description: RTD1195 SoC based boards
> + items:
> - enum:
> - mele,x1000 # MeLE X1000
> - realtek,horseradish # Realtek Horseradish EVB
> - const: realtek,rtd1195
>
> - # RTD1293 SoC based boards
> - - items:
> + - description: RTD1293 SoC based boards
> + items:
> - enum:
> - synology,ds418j # Synology DiskStation DS418j
> - const: realtek,rtd1293
>
> - # RTD1295 SoC based boards
> - - items:
> + - description: RTD1295 SoC based boards
> + items:
> - enum:
> - mele,v9 # MeLE V9
> - probox2,ava # ProBox2 AVA @@ -36,21 +36,21 @@
> properties:
> - zidoo,x9s # Zidoo X9S
> - const: realtek,rtd1295
>
> - # RTD1296 SoC based boards
> - - items:
> + - description: RTD1296 SoC based boards
> + items:
> - enum:
> - synology,ds418 # Synology DiskStation DS418
> - const: realtek,rtd1296
>
> - # RTD1395 SoC based boards
> - - items:
> + - description: RTD1395 SoC based boards
> + items:
> - enum:
> - bananapi,bpi-m4 # Banana Pi BPI-M4
> - realtek,lion-skin # Realtek Lion Skin EVB
> - const: realtek,rtd1395
>
> - # RTD1619 SoC based boards
> - - items:
> + - description: RTD1619 SoC based boards
> + items:
> - enum:
> - realtek,mjolnir # Realtek Mjolnir EVB
> - const: realtek,rtd1619
> --
> 2.26.2
Acked-by: James Tai <[email protected]>
> -----Original Message-----
> From: Andreas Färber <[email protected]>
> Sent: Sunday, June 21, 2020 7:32 AM
> To: [email protected]
> Cc: [email protected]; [email protected]; James
> Tai [戴志峰] <[email protected]>; Andreas Färber <[email protected]>;
> Rob Herring <[email protected]>; [email protected]
> Subject: [PATCH v4 2/3] dt-bindings: arm: realtek: Document RTD1319 and
> Realtek Pym Particles EVB
>
> From: James Tai <[email protected]>
>
> Define compatible strings for Realtek RTD1319 SoC and Realtek Pym Particles
> EVB.
>
> Signed-off-by: James Tai <[email protected]>
> Signed-off-by: Andreas Färber <[email protected]>
> ---
> v3 -> v4:
> * Renamed compatible from pymparticle to pym-particles
> * Turned SoC comment into description
>
> v2 -> v3: Unchanged
>
> v1 -> v2:
> * Put string in alphabetical order
>
> 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 0b388016bbcd..e36e87df3521 100644
> --- a/Documentation/devicetree/bindings/arm/realtek.yaml
> +++ b/Documentation/devicetree/bindings/arm/realtek.yaml
> @@ -42,6 +42,12 @@ properties:
> - synology,ds418 # Synology DiskStation DS418
> - const: realtek,rtd1296
>
> + - description: RTD1319 SoC based boards
> + items:
> + - enum:
> + - realtek,pym-particles # Realtek Pym Particles EVB
> + - const: realtek,rtd1319
> +
> - description: RTD1395 SoC based boards
> items:
> - enum:
> --
> 2.26.2
Acked-by: James Tai <[email protected]>