[PATCH v2 3/8] dt-bindings: marvell: a38x: add solidrun armada 388 clearfog boards
Conor Dooley
conor at kernel.org
Sun Dec 24 06:42:42 PST 2023
On Sun, Dec 24, 2023 at 03:38:45PM +0100, Josua Mayer wrote:
> Add DT compatible for SolidRun Armada-388 based Clearfog Base and Pro
> boards.
>
> While they are just two variants, we have 3 device-tree:
> - armada-388-clearfog.dts
> - armada-388-clearfog-base.dts
> - armada-388-clearfog-pro.dts
>
> While the first one is identical to the pro variant, the filename is
> not, and neither are the compatible strings.
>
> The first is compatible to armada-380, 385, 388 and
> solidrun,clearfog-a1.
> The other two are compatible to the same, plus an additional string
> including the pro/base as suffix.
>
> Add two bindings, one for the explicit pro and base variants using a
> total of 5 compatibles, and one for the generic name using just 4.
>
> Signed-off-by: Josua Mayer <josua at solid-run.com>
There's nothing here that explains to me _why_ there are three sets of
compatibles for only two boards. Why is this the case?
Also, binding patches need to be sent to the devicetree mailing list.
get_maintainer.pl should have told you that, no?
> ---
> .../bindings/arm/marvell/armada-38x.yaml | 20 +++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml b/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
> index 5bc69c4fe34d..25c29b466094 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-38x.yaml
> @@ -38,4 +38,24 @@ properties:
> - const: marvell,armada385
> - const: marvell,armada380
>
> + - description: |
This isn't needed, there's no formatting to preserve.
Cheers,
Conor.
> + SolidRun Armada 388 SoM based clearfog board. This is
> + equivalent to clearfog pro, but lacks the "pro" label
> + in dtb filename and compatible strings.
> + items:
> + - const: solidrun,clearfog-a1
> + - const: marvell,armada388
> + - const: marvell,armada385
> + - const: marvell,armada380
> +
> + - description: SolidRun Armada 388 SoM based clearfog board variants
> + items:
> + - enum:
> + - solidrun,clearfog-base-a1
> + - solidrun,clearfog-pro-a1
> + - const: solidrun,clearfog-a1
> + - const: marvell,armada388
> + - const: marvell,armada385
> + - const: marvell,armada380
> +
> additionalProperties: true
> --
> 2.35.3
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20231224/4c74af18/attachment.sig>
More information about the linux-arm-kernel
mailing list