diff mbox series

[v15,02/16] dt-bindings: add Canaan boards compatible strings

Message ID 20210203125913.390949-3-damien.lemoal@wdc.com
State Superseded
Headers show
Series None | expand

Commit Message

Damien Le Moal Feb. 3, 2021, 12:58 p.m. UTC
Introduce the file riscv/canaan.yaml to document compatible strings
related to the Canaan Kendryte K210 SoC. The compatible string
"canaan,kendryte-k210" used to indicate the use of this SoC to the
early SoC init code is added. This new file also defines the compatible
strings of all supported boards based on this SoC.

Cc: Rob Herring <robh@kernel.org>
Cc: devicetree@vger.kernel.org
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
Reviewed-by: Atish Patra <atish.patra@wdc.com>
---
 .../devicetree/bindings/riscv/canaan.yaml     | 47 +++++++++++++++++++
 1 file changed, 47 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/riscv/canaan.yaml

Comments

Rob Herring (Arm) Feb. 4, 2021, 11:46 p.m. UTC | #1
On Wed, Feb 03, 2021 at 09:58:59PM +0900, Damien Le Moal wrote:
> Introduce the file riscv/canaan.yaml to document compatible strings

> related to the Canaan Kendryte K210 SoC. The compatible string

> "canaan,kendryte-k210" used to indicate the use of this SoC to the

> early SoC init code is added. This new file also defines the compatible

> strings of all supported boards based on this SoC.

> 

> Cc: Rob Herring <robh@kernel.org>

> Cc: devicetree@vger.kernel.org

> Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>

> Reviewed-by: Atish Patra <atish.patra@wdc.com>

> ---

>  .../devicetree/bindings/riscv/canaan.yaml     | 47 +++++++++++++++++++

>  1 file changed, 47 insertions(+)

>  create mode 100644 Documentation/devicetree/bindings/riscv/canaan.yaml

> 

> diff --git a/Documentation/devicetree/bindings/riscv/canaan.yaml b/Documentation/devicetree/bindings/riscv/canaan.yaml

> new file mode 100644

> index 000000000000..f8f3f286bd55

> --- /dev/null

> +++ b/Documentation/devicetree/bindings/riscv/canaan.yaml

> @@ -0,0 +1,47 @@

> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)

> +%YAML 1.2

> +---

> +$id: http://devicetree.org/schemas/riscv/canaan.yaml#

> +$schema: http://devicetree.org/meta-schemas/core.yaml#

> +

> +title: Canaan SoC-based boards

> +

> +maintainers:

> +  - Damien Le Moal <damien.lemoal@wdc.com>

> +

> +description:

> +  Canaan Kendryte K210 SoC-based boards

> +

> +properties:

> +  $nodename:

> +    const: '/'

> +  compatible:

> +    oneOf:

> +      - items:

> +          - const: sipeed,maix-bit

> +          - const: sipeed,maix-bitm

> +          - const: canaan,kendryte-k210

> +

> +      - items:

> +          - const: sipeed,maix-go

> +          - const: canaan,kendryte-k210

> +

> +      - items:

> +          - const: sipeed,maix-dock-m1

> +          - const: sipeed,maix-dock-m1w

> +          - const: canaan,kendryte-k210

> +

> +      - items:

> +          - const: sipeed,maixduino

> +          - const: canaan,kendryte-k210

> +

> +      - items:

> +          - const: canaan,kendryte-kd233

> +          - const: canaan,kendryte-k210


You could combine all the 2 entry cases. Some boards do this form though 
to add a description on each one. Either way,

Reviewed-by: Rob Herring <robh@kernel.org>


> +

> +      - items:

> +          - const: canaan,kendryte-k210

> +

> +additionalProperties: true

> +

> +...

> -- 

> 2.29.2

>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/riscv/canaan.yaml b/Documentation/devicetree/bindings/riscv/canaan.yaml
new file mode 100644
index 000000000000..f8f3f286bd55
--- /dev/null
+++ b/Documentation/devicetree/bindings/riscv/canaan.yaml
@@ -0,0 +1,47 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/riscv/canaan.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Canaan SoC-based boards
+
+maintainers:
+  - Damien Le Moal <damien.lemoal@wdc.com>
+
+description:
+  Canaan Kendryte K210 SoC-based boards
+
+properties:
+  $nodename:
+    const: '/'
+  compatible:
+    oneOf:
+      - items:
+          - const: sipeed,maix-bit
+          - const: sipeed,maix-bitm
+          - const: canaan,kendryte-k210
+
+      - items:
+          - const: sipeed,maix-go
+          - const: canaan,kendryte-k210
+
+      - items:
+          - const: sipeed,maix-dock-m1
+          - const: sipeed,maix-dock-m1w
+          - const: canaan,kendryte-k210
+
+      - items:
+          - const: sipeed,maixduino
+          - const: canaan,kendryte-k210
+
+      - items:
+          - const: canaan,kendryte-kd233
+          - const: canaan,kendryte-k210
+
+      - items:
+          - const: canaan,kendryte-k210
+
+additionalProperties: true
+
+...