diff mbox series

[15/28] dt-bindings: arm: mrvl: Add missing compatible strings

Message ID 20200317093922.20785-16-lkundrak@v3.sk (mailing list archive)
State New, archived
Headers show
Series DT: Improve validation for Marvell SoCs | expand

Commit Message

Lubomir Rintel March 17, 2020, 9:39 a.m. UTC
Add compatible strings for the boards we have in tree. At the same time,
fix the MMP3 compatible string: the preferred vendor name for Marvell is
"marvell", not "mrvl", and indeed "marvell,mmp3" has been actively used,
not "mrvl,mmp3".

Fixes: 95aecb71b84e ("dt-bindings: arm: mrvl: Document MMP3 compatible string")
Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
---
 Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Andrew Lunn March 17, 2020, 1:54 p.m. UTC | #1
On Tue, Mar 17, 2020 at 10:39:09AM +0100, Lubomir Rintel wrote:
> Add compatible strings for the boards we have in tree. At the same time,
> fix the MMP3 compatible string: the preferred vendor name for Marvell is
> "marvell", not "mrvl", and indeed "marvell,mmp3" has been actively used,
> not "mrvl,mmp3".
> 
> Fixes: 95aecb71b84e ("dt-bindings: arm: mrvl: Document MMP3 compatible string")
> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>

What is implied by the Fixes: tag? Do you expect this to be
back ported? Is anything actually broken here?

Apart from that:

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
Rob Herring (Arm) March 27, 2020, 7:06 p.m. UTC | #2
On Tue, 17 Mar 2020 10:39:09 +0100, Lubomir Rintel wrote:
> Add compatible strings for the boards we have in tree. At the same time,
> fix the MMP3 compatible string: the preferred vendor name for Marvell is
> "marvell", not "mrvl", and indeed "marvell,mmp3" has been actively used,
> not "mrvl,mmp3".
> 
> Fixes: 95aecb71b84e ("dt-bindings: arm: mrvl: Document MMP3 compatible string")
> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
> ---
>  Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 

Applied, thanks.

Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml b/Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml
index 818dfe6de5120..3235ec9e9bad1 100644
--- a/Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml
+++ b/Documentation/devicetree/bindings/arm/mrvl/mrvl.yaml
@@ -28,8 +28,11 @@  properties:
         items:
           - enum:
               - mrvl,mmp2-brownstone
+              - olpc,xo-1.75
           - const: mrvl,mmp2
       - description: MMP3 based boards
         items:
-          - const: mrvl,mmp3
+          - enum:
+              - dell,wyse-ariel
+          - const: marvell,mmp3
 ...