| Message ID | 20241223162029.326997-2-vincenzo.frascino@arm.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | arm64: dts: Add Arm Morello support | expand |
On 23/12/2024 17:20, Vincenzo Frascino wrote: > Add compatibility to Arm Morello System Development Platform. > > Note: Morello is at the same time the name of an Architecture [1], an SoC > [2] and a Board [2]. > To distinguish in between Architecture/SoC and Board we refer to the first > as arm,morello and to the second as arm,morello-sdp. > > [1] https://developer.arm.com/Architectures/Morello > [2] https://www.morello-project.org/ > > Cc: Linus Walleij <linus.walleij@linaro.org> > Cc: Rob Herring <robh@kernel.org> > Cc: Krzysztof Kozlowski <krzk+dt@kernel.org> > Cc: Conor Dooley <conor+dt@kernel.org> > Cc: Liviu Dudau <liviu.dudau@arm.com> > Cc: Sudeep Holla <sudeep.holla@arm.com> > Cc: Lorenzo Pieralisi <lpieralisi@kernel.org> > Cc: Russell King <linux@armlinux.org.uk> > Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> > --- > Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > index 8dd6b6446394..7934d2c4cba0 100644 > --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > @@ -118,6 +118,9 @@ properties: > items: > - const: arm,foundation-aarch64 > - const: arm,vexpress > + - description: Arm Morello System Development Platform > + items: > + - const: arm,morello-sdp Also: const: arm,morello Unless you claim that this cannot be used in any other product than SDP, but this then needs explanation in commit msg. Best regards, Krzysztof
On 23/12/2024 16:32, Krzysztof Kozlowski wrote: > On 23/12/2024 17:20, Vincenzo Frascino wrote: >> Add compatibility to Arm Morello System Development Platform. >> >> Note: Morello is at the same time the name of an Architecture [1], an SoC >> [2] and a Board [2]. >> To distinguish in between Architecture/SoC and Board we refer to the first >> as arm,morello and to the second as arm,morello-sdp. >> >> [1] https://developer.arm.com/Architectures/Morello >> [2] https://www.morello-project.org/ >> >> Cc: Linus Walleij <linus.walleij@linaro.org> >> Cc: Rob Herring <robh@kernel.org> >> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org> >> Cc: Conor Dooley <conor+dt@kernel.org> >> Cc: Liviu Dudau <liviu.dudau@arm.com> >> Cc: Sudeep Holla <sudeep.holla@arm.com> >> Cc: Lorenzo Pieralisi <lpieralisi@kernel.org> >> Cc: Russell King <linux@armlinux.org.uk> >> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> >> --- >> Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml >> index 8dd6b6446394..7934d2c4cba0 100644 >> --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml >> +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml >> @@ -118,6 +118,9 @@ properties: >> items: >> - const: arm,foundation-aarch64 >> - const: arm,vexpress >> + - description: Arm Morello System Development Platform >> + items: >> + - const: arm,morello-sdp > Also: > const: arm,morello > > Unless you claim that this cannot be used in any other product than SDP, > but this then needs explanation in commit msg. > Morello is a Prototype architecture and there are no plans to use it outside of SDP hence I did not add arm,morello. I thought it was clear since the note states that Architecture, SoC and Board have the same name, if not, happy do address it in v3. > Best regards, > Krzysztof
On 23/12/2024 17:41, Vincenzo Frascino wrote: > > > On 23/12/2024 16:32, Krzysztof Kozlowski wrote: >> On 23/12/2024 17:20, Vincenzo Frascino wrote: >>> Add compatibility to Arm Morello System Development Platform. >>> >>> Note: Morello is at the same time the name of an Architecture [1], an SoC >>> [2] and a Board [2]. >>> To distinguish in between Architecture/SoC and Board we refer to the first >>> as arm,morello and to the second as arm,morello-sdp. >>> >>> [1] https://developer.arm.com/Architectures/Morello >>> [2] https://www.morello-project.org/ >>> >>> Cc: Linus Walleij <linus.walleij@linaro.org> >>> Cc: Rob Herring <robh@kernel.org> >>> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org> >>> Cc: Conor Dooley <conor+dt@kernel.org> >>> Cc: Liviu Dudau <liviu.dudau@arm.com> >>> Cc: Sudeep Holla <sudeep.holla@arm.com> >>> Cc: Lorenzo Pieralisi <lpieralisi@kernel.org> >>> Cc: Russell King <linux@armlinux.org.uk> >>> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> >>> --- >>> Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml >>> index 8dd6b6446394..7934d2c4cba0 100644 >>> --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml >>> +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml >>> @@ -118,6 +118,9 @@ properties: >>> items: >>> - const: arm,foundation-aarch64 >>> - const: arm,vexpress >>> + - description: Arm Morello System Development Platform >>> + items: >>> + - const: arm,morello-sdp >> Also: >> const: arm,morello >> >> Unless you claim that this cannot be used in any other product than SDP, >> but this then needs explanation in commit msg. >> > > Morello is a Prototype architecture and there are no plans to use it outside of > SDP hence I did not add arm,morello. > > I thought it was clear since the note states that Architecture, SoC and Board > have the same name, if not, happy do address it in v3. No, it's not clear because that statement still does not close possibility of creating "Morello SDP v2" board. You should have two compatibles, even if there are no plans to add new boards. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml index 8dd6b6446394..7934d2c4cba0 100644 --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml @@ -118,6 +118,9 @@ properties: items: - const: arm,foundation-aarch64 - const: arm,vexpress + - description: Arm Morello System Development Platform + items: + - const: arm,morello-sdp arm,vexpress,position: description: When daughterboards are stacked on one site, their position
Add compatibility to Arm Morello System Development Platform. Note: Morello is at the same time the name of an Architecture [1], an SoC [2] and a Board [2]. To distinguish in between Architecture/SoC and Board we refer to the first as arm,morello and to the second as arm,morello-sdp. [1] https://developer.arm.com/Architectures/Morello [2] https://www.morello-project.org/ Cc: Linus Walleij <linus.walleij@linaro.org> Cc: Rob Herring <robh@kernel.org> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org> Cc: Conor Dooley <conor+dt@kernel.org> Cc: Liviu Dudau <liviu.dudau@arm.com> Cc: Sudeep Holla <sudeep.holla@arm.com> Cc: Lorenzo Pieralisi <lpieralisi@kernel.org> Cc: Russell King <linux@armlinux.org.uk> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> --- Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml | 3 +++ 1 file changed, 3 insertions(+)