From: Sudeep Holla <sudeep.holla@arm.com>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Vincenzo Frascino <vincenzo.frascino@arm.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Sudeep Holla <sudeep.holla@arm.com>,
linux-arm-kernel@lists.infradead.org,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Liviu Dudau <liviu.dudau@arm.com>,
Lorenzo Pieralisi <lpieralisi@kernel.org>,
Russell King <linux@armlinux.org.uk>,
Will Deacon <will@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Subject: Re: [PATCH v5 1/8] dt-bindings: arm: Add Morello compatibility
Date: Tue, 18 Feb 2025 15:11:01 +0000 [thread overview]
Message-ID: <Z7SjBWme-HhNYwtV@bogus> (raw)
In-Reply-To: <CACRpkda-J_NHC7Te=Shk0A-35qWms3xeM2MggdGM0ze3Gt0KMw@mail.gmail.com>
On Fri, Feb 14, 2025 at 11:38:54AM +0100, Linus Walleij wrote:
> Hi Vincenzo,
>
> thanks for your patch!
>
> On Thu, Feb 13, 2025 at 7:03 PM Vincenzo Frascino
> <vincenzo.frascino@arm.com> 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/
> >
> > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com>
> > ---
> > Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml | 4 ++++
>
> I was thinking, that since the .dtsi and .dts files are not reusing
> any of the Juno .dtsi (correct me if I'm wrong!) this should not
> be in vexpress-juno.yaml, instead perhaps you should create a new
> morello.yaml file?
>
It is me who suggested to put it along with other vexpress stuff as I
wasn't sure how much of vexpress bindings will be reused here when
Vincenzo started this. I agree it can be a separate binding on its own
as I don't see much commonality now with the vexpress bindings.
Just a note, though the file is named arm,vexpress-juno.yaml, it also
carries bindings for all Vexpress based Arm Ltd boards(both 32-bit and
64-bit ones), but they all use common vexpress bindings in general.
I just thought of highlighting that so that the expectation to reuse
this file is not to check commonality in juno dtsi files but to check
the vexpress binding reuse. Hope we are aligned with that.
--
Regards,
Sudeep
next prev parent reply other threads:[~2025-02-18 15:11 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-13 18:03 [PATCH v5 0/8] arm64: dts: Add Arm Morello support Vincenzo Frascino
2025-02-13 18:03 ` [PATCH v5 1/8] dt-bindings: arm: Add Morello compatibility Vincenzo Frascino
2025-02-14 10:38 ` Linus Walleij
2025-02-18 15:11 ` Sudeep Holla [this message]
2025-02-13 18:03 ` [PATCH v5 2/8] dt-bindings: arm: Add Morello fvp compatibility Vincenzo Frascino
2025-02-14 8:12 ` Krzysztof Kozlowski
2025-02-18 18:07 ` Vincenzo Frascino
2025-02-19 7:13 ` Krzysztof Kozlowski
2025-02-19 11:30 ` Vincenzo Frascino
2025-02-19 19:53 ` Krzysztof Kozlowski
2025-02-13 18:03 ` [PATCH v5 3/8] dt-bindings: arm: Add Rainier compatibility Vincenzo Frascino
2025-02-14 8:12 ` Krzysztof Kozlowski
2025-02-13 18:03 ` [PATCH v5 4/8] perf: arm_pmuv3: Add support for ARM Rainier PMU Vincenzo Frascino
2025-02-14 8:13 ` Krzysztof Kozlowski
2025-02-13 18:03 ` [PATCH v5 5/8] arm64: dts: morello: Add support for common functionalities Vincenzo Frascino
2025-02-13 18:03 ` [PATCH v5 6/8] arm64: dts: morello: Add support for soc dts Vincenzo Frascino
2025-02-13 18:03 ` [PATCH v5 7/8] arm64: dts: morello: Add support for fvp dts Vincenzo Frascino
2025-02-13 18:03 ` [PATCH v5 8/8] MAINTAINERS: Add Vincenzo Frascino as Arm Morello Maintainer Vincenzo Frascino
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Z7SjBWme-HhNYwtV@bogus \
--to=sudeep.holla@arm.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=liviu.dudau@arm.com \
--cc=lpieralisi@kernel.org \
--cc=mark.rutland@arm.com \
--cc=robh@kernel.org \
--cc=vincenzo.frascino@arm.com \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.