linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Sudeep Holla <sudeep.holla@arm.com>
To: Mark Rutland <mark.rutland@arm.com>
Cc: Vincenzo Frascino <vincenzo.frascino@arm.com>,
	Sudeep Holla <sudeep.holla@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 19:18:46 +0000	[thread overview]
Message-ID: <20250225191846.5zopbxloo4zuoxp5@bogus> (raw)
In-Reply-To: <Z720J5kXEnj5sZn0@J2N7QTR9R3>

On Tue, Feb 25, 2025 at 12:14:31PM +0000, Mark Rutland wrote:
> 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 can add that info in my pull request as it is very valid point. One needs
to know why we are pushing DTS now after couple of years after the boards
are available and why DT over ACPI which is shipped with the device.

> > 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.
> 

No confusion as such, good to ask explicitly and get the motivation for
this series captured on the list. Since I had gone through this motion
even with N1SDP which for some reasons I don't know didn't appear on the
list, I didn't ask it and implicitly assumed similar reasoning.

So, it is good that you asked and got it answered for wider audience here.

-- 
Regards,
Sudeep


  reply	other threads:[~2025-02-25 19:31 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
2025-02-25 19:18         ` Sudeep Holla [this message]
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=20250225191846.5zopbxloo4zuoxp5@bogus \
    --to=sudeep.holla@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=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 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).