linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Mark Rutland <mark.rutland@arm.com>
To: Sudeep Holla <sudeep.holla@arm.com>
Cc: Vincenzo Frascino <vincenzo.frascino@arm.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Linus Walleij <linus.walleij@linaro.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>, Jessica Clarke <jrtc27@jrtc27.com>
Subject: Re: [PATCH v7 00/10] arm64: dts: Add Arm Morello support
Date: Tue, 25 Feb 2025 12:14:31 +0000	[thread overview]
Message-ID: <Z720J5kXEnj5sZn0@J2N7QTR9R3> (raw)
In-Reply-To: <Z72ydHCLxd-WGsJu@bogus>

Hi Sudeep,

On Tue, Feb 25, 2025 at 12:07:16PM +0000, Sudeep Holla wrote:
> On Mon, Feb 24, 2025 at 10:08:18AM +0000, Vincenzo Frascino wrote:
> > On 21/02/2025 18:54, Mark Rutland wrote:
> > > On Fri, Feb 21, 2025 at 06:03:39PM +0000, Vincenzo Frascino wrote:
> > >> This series adds dts support for the Arm Morello System Development
> > >> Platform.
> > >
> > > Do we actually need the dts for this board?
> > >
> > > I have one on my desk; it boots vanilla Debian 12 via UEFI + ACPI just
> > > fine, with the Debian 6.1.0-13-arm64 kernel.
> > >
> > > Is there something that we can only do with the DT? i.e. some
> > > functionality that isn't exposed via ACPI?
> > >
> > > How do you expect this DT to be used?
> >
> > There are functionalities that are not exposed via ACPI, e.g. gpu, dpu, i2c for
> > the phy, etc. My aim to have upstream support for all the hardware exposed by
> > the platform.
> 
> Does this address some of your concerns ? I do understand some of these
> are not well addressed in ACPI and hence people use DT as an alternative.

Yep; I'm happy with this so long as there's an actual functional reason
to have the DT, which it seems there is.

It would have been nice for that to be spelled out a bit clearer in the
cover / commit messages, but that's not important and doesn't need a
respin.

> I was thinking of queuing this in -next if all the bindings are acked.
> Let me know if you still have concerns and would like to avoid getting
> these merged. I will hold off then.

No need to hold off.

Sorry for the confusion; I should have been clearer with my questions.

Mark.


  reply	other threads:[~2025-02-25 12:54 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-21 18:03 [PATCH v7 00/10] arm64: dts: Add Arm Morello support Vincenzo Frascino
2025-02-21 18:03 ` [PATCH v7 01/10] arm64: Kconfig: Update description for CONFIG_ARCH_VEXPRESS Vincenzo Frascino
2025-02-27 15:27   ` Liviu Dudau
2025-02-21 18:03 ` [PATCH v7 02/10] dt-bindings: arm: Add Morello compatibility Vincenzo Frascino
2025-02-25 15:33   ` Linus Walleij
2025-02-25 15:33   ` Linus Walleij
2025-02-21 18:03 ` [PATCH v7 03/10] dt-bindings: arm: Add Morello fvp compatibility Vincenzo Frascino
2025-02-24 17:29   ` Rob Herring (Arm)
2025-02-25 15:32   ` Linus Walleij
2025-02-21 18:03 ` [PATCH v7 04/10] dt-bindings: arm: Add Rainier compatibility Vincenzo Frascino
2025-02-21 18:55   ` Mark Rutland
2025-02-24 10:42     ` Vincenzo Frascino
2025-02-21 18:03 ` [PATCH v7 05/10] dt-bindings: arm-pmu: Add support for ARM Rainier PMU Vincenzo Frascino
2025-02-25 15:28   ` Linus Walleij
2025-02-21 18:03 ` [PATCH v7 06/10] perf: arm_pmuv3: " Vincenzo Frascino
2025-02-27 10:10   ` Sudeep Holla
2025-02-27 17:14     ` Will Deacon
2025-02-27 17:19       ` Sudeep Holla
2025-02-21 18:03 ` [PATCH v7 07/10] arm64: dts: morello: Add support for common functionalities Vincenzo Frascino
2025-02-25 16:03   ` Linus Walleij
2025-02-27 11:08   ` Sudeep Holla
2025-02-21 18:03 ` [PATCH v7 08/10] arm64: dts: morello: Add support for soc dts Vincenzo Frascino
2025-02-25 16:03   ` Linus Walleij
2025-02-21 18:03 ` [PATCH v7 09/10] arm64: dts: morello: Add support for fvp dts Vincenzo Frascino
2025-02-25 16:04   ` Linus Walleij
2025-02-21 18:03 ` [PATCH v7 10/10] MAINTAINERS: Add Vincenzo Frascino as Arm Morello Maintainer Vincenzo Frascino
2025-02-25 16:04   ` Linus Walleij
2025-02-21 18:54 ` [PATCH v7 00/10] arm64: dts: Add Arm Morello support Mark Rutland
2025-02-24 10:08   ` Vincenzo Frascino
2025-02-25 12:07     ` Sudeep Holla
2025-02-25 12:14       ` Mark Rutland [this message]
2025-02-25 19:18         ` Sudeep Holla
2025-02-27 11:31 ` (subset) " Sudeep Holla
2025-03-01  7:05 ` Will Deacon

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=Z720J5kXEnj5sZn0@J2N7QTR9R3 \
    --to=mark.rutland@arm.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jrtc27@jrtc27.com \
    --cc=krzk+dt@kernel.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=robh@kernel.org \
    --cc=sudeep.holla@arm.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).