2022-07-13 20:59:39

by Pali Rohár

[permalink] [raw]
Subject: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
indicate SoC in example.

Signed-off-by: Pali Rohár <[email protected]>
---
.../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
index f6d6642d81c0..d2ca008de266 100644
--- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
+++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
@@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
Boards using a SoC of the Marvell Armada 37xx family must carry the
following root node property:

+ - compatible: must contain "marvell,armada3700"
+
+In addition, boards using the Marvell Armada 3710 SoC shall have the
+following property before the previous one:
+
- compatible: must contain "marvell,armada3710"

In addition, boards using the Marvell Armada 3720 SoC shall have the
@@ -13,7 +18,7 @@ following property before the previous one:

Example:

-compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
+compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";


Power management
--
2.20.1


2022-07-18 21:01:08

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On Wed, 13 Jul 2022 22:01:23 +0200, Pali Roh?r wrote:
> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> indicate SoC in example.
>
> Signed-off-by: Pali Roh?r <[email protected]>
> ---
> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>

Acked-by: Rob Herring <[email protected]>

2022-08-08 20:59:46

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

PING?

On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> indicate SoC in example.
>
> Signed-off-by: Pali Rohár <[email protected]>
> ---
> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> index f6d6642d81c0..d2ca008de266 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> Boards using a SoC of the Marvell Armada 37xx family must carry the
> following root node property:
>
> + - compatible: must contain "marvell,armada3700"
> +
> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> +following property before the previous one:
> +
> - compatible: must contain "marvell,armada3710"
>
> In addition, boards using the Marvell Armada 3720 SoC shall have the
> @@ -13,7 +18,7 @@ following property before the previous one:
>
> Example:
>
> -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
>
>
> Power management
> --
> 2.20.1
>

2022-08-09 06:02:02

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On 08/08/2022 23:23, Pali Rohár wrote:
> PING?
>
> On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
>> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
>> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
>> indicate SoC in example.
>>
>> Signed-off-by: Pali Rohár <[email protected]>
>> ---
>> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
>> 1 file changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> index f6d6642d81c0..d2ca008de266 100644
>> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
>> Boards using a SoC of the Marvell Armada 37xx family must carry the
>> following root node property:
>>
>> + - compatible: must contain "marvell,armada3700"
>> +
>> +In addition, boards using the Marvell Armada 3710 SoC shall have the
>> +following property before the previous one:
>> +

The change is an ABI break, which looks reasonable, but still platform
maintainer should comment on it. Especially on the aspect why the
marvell,armada3710 fallback was chosen at the first place.

Unfortunately none of Marvel folks were CCed here, so you will need to
resend it. Otherwise no one will apply it, regardless of our ack.

Best regards,
Krzysztof

2022-08-09 13:36:05

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On Tuesday 09 August 2022 08:58:50 Krzysztof Kozlowski wrote:
> On 08/08/2022 23:23, Pali Rohár wrote:
> > PING?
> >
> > On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> >> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> >> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> >> indicate SoC in example.
> >>
> >> Signed-off-by: Pali Rohár <[email protected]>
> >> ---
> >> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> >> 1 file changed, 6 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >> index f6d6642d81c0..d2ca008de266 100644
> >> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> >> Boards using a SoC of the Marvell Armada 37xx family must carry the
> >> following root node property:
> >>
> >> + - compatible: must contain "marvell,armada3700"
> >> +
> >> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> >> +following property before the previous one:
> >> +
>
> The change is an ABI break, which looks reasonable, but still platform
> maintainer should comment on it. Especially on the aspect why the
> marvell,armada3710 fallback was chosen at the first place.

I do not think this is ABI break but rather incorrect documentation and
bug in some board dts files.

> Unfortunately none of Marvel folks were CCed here, so you will need to
> resend it. Otherwise no one will apply it, regardless of our ack.

Done.

2022-08-09 13:37:36

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

+ Andrew, Gregory, Sebastian

please comment on this

On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> indicate SoC in example.
>
> Signed-off-by: Pali Rohár <[email protected]>
> ---
> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> index f6d6642d81c0..d2ca008de266 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> Boards using a SoC of the Marvell Armada 37xx family must carry the
> following root node property:
>
> + - compatible: must contain "marvell,armada3700"
> +
> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> +following property before the previous one:
> +
> - compatible: must contain "marvell,armada3710"
>
> In addition, boards using the Marvell Armada 3720 SoC shall have the
> @@ -13,7 +18,7 @@ following property before the previous one:
>
> Example:
>
> -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
>
>
> Power management
> --
> 2.20.1
>

2022-08-09 15:19:27

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On Tue, Aug 09, 2022 at 03:12:52PM +0200, Pali Roh?r wrote:
> + Andrew, Gregory, Sebastian
>
> please comment on this

Krzysztof said:

> The change is an ABI break, which looks reasonable, but still
> platform maintainer should comment on it. Especially on the aspect
> why the marvell,armada3710 fallback was chosen at the first place.

I was not involved in the armada37xx enablement. I have no idea why
marvell,armada3710 fallback was chosen.

Gregory might know, he is the author of this file.

It does however seem reasonable. And a grep does suggest this
compatible is purely cosmetic, it is not matched on anywhere.

Andrew

2022-08-09 16:09:10

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On 09/08/2022 16:13, Pali Rohár wrote:
> On Tuesday 09 August 2022 08:58:50 Krzysztof Kozlowski wrote:
>> On 08/08/2022 23:23, Pali Rohár wrote:
>>> PING?
>>>
>>> On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
>>>> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
>>>> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
>>>> indicate SoC in example.
>>>>
>>>> Signed-off-by: Pali Rohár <[email protected]>
>>>> ---
>>>> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
>>>> 1 file changed, 6 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>>>> index f6d6642d81c0..d2ca008de266 100644
>>>> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>>>> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
>>>> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
>>>> Boards using a SoC of the Marvell Armada 37xx family must carry the
>>>> following root node property:
>>>>
>>>> + - compatible: must contain "marvell,armada3700"
>>>> +
>>>> +In addition, boards using the Marvell Armada 3710 SoC shall have the
>>>> +following property before the previous one:
>>>> +
>>
>> The change is an ABI break, which looks reasonable, but still platform
>> maintainer should comment on it. Especially on the aspect why the
>> marvell,armada3710 fallback was chosen at the first place.
>
> I do not think this is ABI break but rather incorrect documentation and
> bug in some board dts files.

This documentation is the ABI, especially if actually implemented in the
DTS files, therefore by all definitions of DT ABI this is an ABI break.

What you are saying about "incorrect documentation and bug in some DTS
files" is a good reason to break the ABI, but it is still a break.

Best regards,
Krzysztof

2022-08-09 16:36:08

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On Tuesday 09 August 2022 18:46:04 Krzysztof Kozlowski wrote:
> On 09/08/2022 16:13, Pali Rohár wrote:
> > On Tuesday 09 August 2022 08:58:50 Krzysztof Kozlowski wrote:
> >> On 08/08/2022 23:23, Pali Rohár wrote:
> >>> PING?
> >>>
> >>> On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> >>>> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> >>>> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> >>>> indicate SoC in example.
> >>>>
> >>>> Signed-off-by: Pali Rohár <[email protected]>
> >>>> ---
> >>>> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> >>>> 1 file changed, 6 insertions(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >>>> index f6d6642d81c0..d2ca008de266 100644
> >>>> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >>>> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> >>>> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> >>>> Boards using a SoC of the Marvell Armada 37xx family must carry the
> >>>> following root node property:
> >>>>
> >>>> + - compatible: must contain "marvell,armada3700"
> >>>> +
> >>>> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> >>>> +following property before the previous one:
> >>>> +
> >>
> >> The change is an ABI break, which looks reasonable, but still platform
> >> maintainer should comment on it. Especially on the aspect why the
> >> marvell,armada3710 fallback was chosen at the first place.
> >
> > I do not think this is ABI break but rather incorrect documentation and
> > bug in some board dts files.
>
> This documentation is the ABI, especially if actually implemented in the
> DTS files, therefore by all definitions of DT ABI this is an ABI break.

Ok.

> What you are saying about "incorrect documentation and bug in some DTS
> files" is a good reason to break the ABI, but it is still a break.
>
> Best regards,
> Krzysztof

I think it is now clear what we both mean, just we used different
terminology for the same thing.

2022-09-24 12:24:19

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

PING?

On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> indicate SoC in example.
>
> Signed-off-by: Pali Rohár <[email protected]>
> ---
> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> index f6d6642d81c0..d2ca008de266 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> Boards using a SoC of the Marvell Armada 37xx family must carry the
> following root node property:
>
> + - compatible: must contain "marvell,armada3700"
> +
> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> +following property before the previous one:
> +
> - compatible: must contain "marvell,armada3710"
>
> In addition, boards using the Marvell Armada 3720 SoC shall have the
> @@ -13,7 +18,7 @@ following property before the previous one:
>
> Example:
>
> -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
>
>
> Power management
> --
> 2.20.1
>

2022-09-24 14:12:44

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On 08/08/2022 22:23, Pali Rohár wrote:
> PING?
>

You got the ack, what are you pinging us for?

Best regards,
Krzysztof

2022-09-24 14:59:20

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On Saturday 24 September 2022 15:50:38 Krzysztof Kozlowski wrote:
> On 08/08/2022 22:23, Pali Rohár wrote:
> > PING?
> >
>
> You got the ack, what are you pinging us for?
>
> Best regards,
> Krzysztof
>
So, who can take this patch?

2022-09-24 16:58:50

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

On 24/09/2022 16:30, Pali Rohár wrote:
> On Saturday 24 September 2022 15:50:38 Krzysztof Kozlowski wrote:
>> On 08/08/2022 22:23, Pali Rohár wrote:
>>> PING?
>>>
>>
>> You got the ack, what are you pinging us for?
>>
>> Best regards,
>> Krzysztof
>>
> So, who can take this patch?

This looks like platform bindings, so the platform maintainers. Would be
easier for them to pick it up, if they were CCed. :)

I actually hoped they will pick up also fix for maintainers entry:
https://lore.kernel.org/all/875yi5g96s.fsf@BL-laptop/
...

Best regards,
Krzysztof

2022-10-09 12:14:29

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

Gregory: ping

On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> indicate SoC in example.
>
> Signed-off-by: Pali Rohár <[email protected]>
> ---
> .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> index f6d6642d81c0..d2ca008de266 100644
> --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> Boards using a SoC of the Marvell Armada 37xx family must carry the
> following root node property:
>
> + - compatible: must contain "marvell,armada3700"
> +
> +In addition, boards using the Marvell Armada 3710 SoC shall have the
> +following property before the previous one:
> +
> - compatible: must contain "marvell,armada3710"
>
> In addition, boards using the Marvell Armada 3720 SoC shall have the
> @@ -13,7 +18,7 @@ following property before the previous one:
>
> Example:
>
> -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
>
>
> Power management
> --
> 2.20.1
>

2022-11-01 23:28:17

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

PING?

On Sunday 09 October 2022 13:56:58 Pali Rohár wrote:
> Gregory: ping
>
> On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> > Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> > Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> > indicate SoC in example.
> >
> > Signed-off-by: Pali Rohár <[email protected]>
> > ---
> > .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > index f6d6642d81c0..d2ca008de266 100644
> > --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> > Boards using a SoC of the Marvell Armada 37xx family must carry the
> > following root node property:
> >
> > + - compatible: must contain "marvell,armada3700"
> > +
> > +In addition, boards using the Marvell Armada 3710 SoC shall have the
> > +following property before the previous one:
> > +
> > - compatible: must contain "marvell,armada3710"
> >
> > In addition, boards using the Marvell Armada 3720 SoC shall have the
> > @@ -13,7 +18,7 @@ following property before the previous one:
> >
> > Example:
> >
> > -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> > +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
> >
> >
> > Power management
> > --
> > 2.20.1
> >

2022-11-28 19:31:05

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

PING?

On Tuesday 01 November 2022 23:43:00 Pali Rohár wrote:
> PING?
>
> On Sunday 09 October 2022 13:56:58 Pali Rohár wrote:
> > Gregory: ping
> >
> > On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> > > Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> > > Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> > > indicate SoC in example.
> > >
> > > Signed-off-by: Pali Rohár <[email protected]>
> > > ---
> > > .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> > > 1 file changed, 6 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > index f6d6642d81c0..d2ca008de266 100644
> > > --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> > > Boards using a SoC of the Marvell Armada 37xx family must carry the
> > > following root node property:
> > >
> > > + - compatible: must contain "marvell,armada3700"
> > > +
> > > +In addition, boards using the Marvell Armada 3710 SoC shall have the
> > > +following property before the previous one:
> > > +
> > > - compatible: must contain "marvell,armada3710"
> > >
> > > In addition, boards using the Marvell Armada 3720 SoC shall have the
> > > @@ -13,7 +18,7 @@ following property before the previous one:
> > >
> > > Example:
> > >
> > > -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> > > +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
> > >
> > >
> > > Power management
> > > --
> > > 2.20.1
> > >

2022-12-16 18:39:34

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

PING?

On Monday 28 November 2022 19:18:29 Pali Rohár wrote:
> PING?
>
> On Tuesday 01 November 2022 23:43:00 Pali Rohár wrote:
> > PING?
> >
> > On Sunday 09 October 2022 13:56:58 Pali Rohár wrote:
> > > Gregory: ping
> > >
> > > On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> > > > Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> > > > Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> > > > indicate SoC in example.
> > > >
> > > > Signed-off-by: Pali Rohár <[email protected]>
> > > > ---
> > > > .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> > > > 1 file changed, 6 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > > index f6d6642d81c0..d2ca008de266 100644
> > > > --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > > @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> > > > Boards using a SoC of the Marvell Armada 37xx family must carry the
> > > > following root node property:
> > > >
> > > > + - compatible: must contain "marvell,armada3700"
> > > > +
> > > > +In addition, boards using the Marvell Armada 3710 SoC shall have the
> > > > +following property before the previous one:
> > > > +
> > > > - compatible: must contain "marvell,armada3710"
> > > >
> > > > In addition, boards using the Marvell Armada 3720 SoC shall have the
> > > > @@ -13,7 +18,7 @@ following property before the previous one:
> > > >
> > > > Example:
> > > >
> > > > -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> > > > +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
> > > >
> > > >
> > > > Power management
> > > > --
> > > > 2.20.1
> > > >

2023-01-22 11:46:06

by Pali Rohár

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

PING?

On Friday 16 December 2022 19:20:01 Pali Rohár wrote:
> PING?
>
> On Monday 28 November 2022 19:18:29 Pali Rohár wrote:
> > PING?
> >
> > On Tuesday 01 November 2022 23:43:00 Pali Rohár wrote:
> > > PING?
> > >
> > > On Sunday 09 October 2022 13:56:58 Pali Rohár wrote:
> > > > Gregory: ping
> > > >
> > > > On Wednesday 13 July 2022 22:01:23 Pali Rohár wrote:
> > > > > Distinguish between Armada 3700 family, Armada 3710 SoC and Armada 3720 SoC.
> > > > > Armada 3720 DB is name of the board with Armada 3720 SoC, so correctly
> > > > > indicate SoC in example.
> > > > >
> > > > > Signed-off-by: Pali Rohár <[email protected]>
> > > > > ---
> > > > > .../devicetree/bindings/arm/marvell/armada-37xx.txt | 7 ++++++-
> > > > > 1 file changed, 6 insertions(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > > > index f6d6642d81c0..d2ca008de266 100644
> > > > > --- a/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > > > +++ b/Documentation/devicetree/bindings/arm/marvell/armada-37xx.txt
> > > > > @@ -4,6 +4,11 @@ Marvell Armada 37xx Platforms Device Tree Bindings
> > > > > Boards using a SoC of the Marvell Armada 37xx family must carry the
> > > > > following root node property:
> > > > >
> > > > > + - compatible: must contain "marvell,armada3700"
> > > > > +
> > > > > +In addition, boards using the Marvell Armada 3710 SoC shall have the
> > > > > +following property before the previous one:
> > > > > +
> > > > > - compatible: must contain "marvell,armada3710"
> > > > >
> > > > > In addition, boards using the Marvell Armada 3720 SoC shall have the
> > > > > @@ -13,7 +18,7 @@ following property before the previous one:
> > > > >
> > > > > Example:
> > > > >
> > > > > -compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3710";
> > > > > +compatible = "marvell,armada-3720-db", "marvell,armada3720", "marvell,armada3700";
> > > > >
> > > > >
> > > > > Power management
> > > > > --
> > > > > 2.20.1
> > > > >

2023-02-01 08:28:03

by Gregory CLEMENT

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: marvell: Update Armada 37xx platform bindings

Hello,

> On Tue, Aug 09, 2022 at 03:12:52PM +0200, Pali Rohár wrote:
>> + Andrew, Gregory, Sebastian
>>
>> please comment on this
>
> Krzysztof said:
>
>> The change is an ABI break, which looks reasonable, but still
>> platform maintainer should comment on it. Especially on the aspect
>> why the marvell,armada3710 fallback was chosen at the first place.
>
> I was not involved in the armada37xx enablement. I have no idea why
> marvell,armada3710 fallback was chosen.
>
> Gregory might know, he is the author of this file.
>
> It does however seem reasonable. And a grep does suggest this
> compatible is purely cosmetic, it is not matched on anywhere.

This compatible was for the single core version of the Armada 37xx.
I think the idea behind using it as fallback was the fact that if it
works for aramda 3720 then it works for armada 3710.

Gregory

>
> Andrew

--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com