2023-12-30 15:45:13

by Josua Mayer

Subject: [PATCH v4 02/11] dt-bindings: marvell: a38x: convert the soc compatibles description to yaml

Convert the existing txt binding for armada-38x socs to DT schema
format.

Note that the current bindings only document the SoC (armada380,
armada385, armada388). This is undesirable, instead there should be
entries for actual boards.

For now only convert to yaml, the content can be corrected separately.

Signed-off-by: Josua Mayer <[email protected]>
---
.../bindings/arm/marvell/armada-38x.yaml | 33 ++++++++++++++++++++++
1 file changed, 33 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
new file mode 100644
index 000000000000..5af222e6db18
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
@@ -0,0 +1,33 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/marvell/armada-38x.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Marvell Armada 38x Platforms
+
+maintainers:
+ - Gregory CLEMENT <[email protected]>
+
+properties:
+ $nodename:
+ const: '/'
+ compatible:
+ oneOf:
+
+ - description: Armada 380 SoC
+ items:
+ - const: marvell,armada380
+
+ - description: Armada 385 SoC
+ items:
+ - const: marvell,armada385
+ - const: marvell,armada380
+
+ - description: Armada 388 SoC
+ items:
+ - const: marvell,armada388
+ - const: marvell,armada385
+ - const: marvell,armada380
+
+additionalProperties: true

--
2.35.3

2024-01-04 08:40:01

by Krzysztof Kozlowski

Subject: Re: [PATCH v4 02/11] dt-bindings: marvell: a38x: convert the soc compatibles description to yaml

On 30/12/2023 16:44, Josua Mayer wrote:
> Convert the existing txt binding for armada-38x socs to DT schema
> format.
>

Where is the conversion? I don't see. Conversion means "remove and add"
in a diff. I see only second part.

> Note that the current bindings only document the SoC (armada380,
> armada385, armada388). This is undesirable, instead there should be
> entries for actual boards.
>
> For now only convert to yaml, the content can be corrected separately.
>
> Signed-off-by: Josua Mayer <[email protected]>
> ---
> .../bindings/arm/marvell/armada-38x.yaml | 33 ++++++++++++++++++++++
> 1 file changed, 33 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
> new file mode 100644
> index 000000000000..5af222e6db18
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
> @@ -0,0 +1,33 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/marvell/armada-38x.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Marvell Armada 38x Platforms
> +
> +maintainers:
> + - Gregory CLEMENT <[email protected]>
> +
> +properties:
> + $nodename:
> + const: '/'
> + compatible:
> + oneOf:
> +
> + - description: Armada 380 SoC
> + items:
> + - const: marvell,armada380
> +
> + - description: Armada 385 SoC
> + items:
> + - const: marvell,armada385
> + - const: marvell,armada380
> +

No, we have been here.

NAK.

Best regards,
Krzysztof

2024-01-04 12:14:52

by Josua Mayer

Subject: Re: [PATCH v4 02/11] dt-bindings: marvell: a38x: convert the soc compatibles description to yaml

Am 04.01.24 um 09:39 schrieb Krzysztof Kozlowski:
> On 30/12/2023 16:44, Josua Mayer wrote:
>> Convert the existing txt binding for armada-38x socs to DT schema
>> format.
>>
> Where is the conversion? I don't see. Conversion means "remove and add"
> in a diff. I see only second part.
>
> + - description: Armada 385 SoC
> + items:
> + - const: marvell,armada385
> + - const: marvell,armada380
> +
> No, we have been here.
>
> NAK.
I had intended to drop this patch, I should have seen it looking at draft before sending .... .

I will do as you requested first: convert and add some valid boards in one step.


Sincerely
Josua Mayer