devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sudeep Holla <sudeep.holla@arm.com>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Nikunj Kela <quic_nkela@quicinc.com>,
	Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Sudeep Holla <sudeep.holla@arm.com>,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org, devicetree@vger.kernel.org,
	"Prasad Sodagudi (QUIC)" <quic_psodagud@quicinc.com>,
	srinivas.kandagatla@linaro.org, vincent.guittot@linaro.org,
	ulf.hansson@linaro.org
Subject: Re: DT Query on "New Compatible vs New Property"
Date: Wed, 24 Jan 2024 10:23:40 +0000	[thread overview]
Message-ID: <ZbDlLJRHu2ebdxc6@bogus> (raw)
In-Reply-To: <20240123161231.GG19029@thinkpad>

On Tue, Jan 23, 2024 at 09:42:31PM +0530, Manivannan Sadhasivam wrote:
> On Thu, Dec 14, 2023 at 07:18:25AM -0800, Nikunj Kela wrote:
> > 
> > HW is exactly the same. Let me give more insight on the setup. We have been
> > using the HW in virtual environment but now the ownership of certain
> > resources (e.g. clock controller etc.) is handed over to a different VM(non
> > Linux VM). Earlier the ownership of the resources was local to the same
> > VM(Linux VM) via passthrough mode so it could directly access them however
> > now Linux VM talks to non-Linux VM for its operations for resources that it
> > doesn't own anymore via some interface(shared memory/doorbell).  So shall we
> > use property like 'qcom, controlled-remotely' or do we need a new compatible
> > for such setup?
> >

I did see the mention of SCMI somewhere in the thread, hence the interest.
What specific resources are we talking here: clocks, reset, power domains,
regulators ? If so I don't understand the need for any new compatible
"qcom, controlled-remotely' or any change in the driver. The DT has standard
bindings for these and drivers would be requesting these resources using
std framework apis. If it is a clock controller in the host Linux VM or
if it is SCMI controlled clock in a non Linux VM must not matter for the
individual drivers right ? Sorry if I am missing something obvious here ?

> 
> Krzysztof, just a ping on this thread.
> 
> To summarise, the hardware is exactly same. We can consider the case of UFS. The
> UFS controller is exactly same in this proposed setup but the resources of the
> UFS controller are taken care by the VM. So instead of enabling the resources
> one by one, Linux kernel will just ask the VM to do so using an SCMI command.
>

I don't understand why you need to change the UFS controller driver to switch
to SCMI driver resource model from self/host Linux driven model.

> Due to this difference, we need to make the changes in the UFS controller
> driver. So we want to know if we can use a different compatible for the UFS
> controller altogether in DT (this will allow Linux kernel to have a separate
> driver and will simplify things) or just use a property like
> "remotely-controlled" to let the driver detect this setup and take action
> accordingly.
>

I would say the DT should be set accordingly before the Linux boots to point
all the resources to SCMI instead of self hosted various controller/provider
nodes in the DT. I don't understand why the compatible for a device need to
change if the OS resource handling model changes. The resource nodes just
points to a different provider node instead.

-- 
Regards,
Sudeep

  parent reply	other threads:[~2024-01-24 10:23 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-12 17:45 DT Query on "New Compatible vs New Property" Nikunj Kela
2023-12-12 19:01 ` Krzysztof Kozlowski
2023-12-12 19:06   ` Nikunj Kela
2023-12-14  6:17     ` Manivannan Sadhasivam
2023-12-14  7:49       ` Krzysztof Kozlowski
2023-12-14 15:18         ` Nikunj Kela
2024-01-23 16:12           ` Manivannan Sadhasivam
2024-01-24  8:02             ` Krzysztof Kozlowski
2024-01-24  8:39               ` Manivannan Sadhasivam
2024-01-24  8:45                 ` Krzysztof Kozlowski
2024-01-24  8:53                   ` Manivannan Sadhasivam
2024-01-24  9:01                     ` Krzysztof Kozlowski
2024-01-24  9:27                       ` Manivannan Sadhasivam
2024-01-24  9:40                         ` Krzysztof Kozlowski
2024-01-24 10:36                           ` Manivannan Sadhasivam
2024-01-24 10:23             ` Sudeep Holla [this message]
2024-01-24 10:45               ` Manivannan Sadhasivam
2024-01-24 11:02                 ` Sudeep Holla
2024-01-24 12:27                   ` Nikunj Kela
2024-01-24 12:48                     ` Sudeep Holla
2024-01-24 13:17                       ` Nikunj Kela
2024-01-24 13:38                         ` Vincent Guittot
2024-01-24 14:04                           ` Sudeep Holla
2024-01-24 14:28                             ` Nikunj Kela
2024-01-24 17:24                               ` Sudeep Holla
2024-01-24 17:33                                 ` Nikunj Kela
2024-02-26 14:22                                   ` Nikunj Kela
2024-02-28 13:27                                     ` Ulf Hansson
2024-02-28 14:02                                       ` Sudeep Holla
2024-02-28 14:20                                         ` Krzysztof Kozlowski
2024-02-28 16:09                                           ` Sudeep Holla
2024-02-28 16:22                                             ` Ulf Hansson
2024-02-28 17:11                                               ` Srinivas Kandagatla
2024-03-01 11:53                                                 ` Ulf Hansson
2024-03-04 11:01                                                   ` Sudeep Holla
2024-03-12 16:52                                                     ` Nikunj Kela
2024-03-12 16:58                                                       ` Trilok Soni
2024-03-12 17:08                                                         ` Nikunj Kela
2024-03-12 17:21                                                         ` Srinivas Kandagatla
2024-03-12 17:25                                                           ` Trilok Soni
2024-03-13  9:19                                                             ` Ulf Hansson
2024-03-13  9:31                                                               ` Nikunj Kela
2024-03-13 11:21                                                               ` Srinivas Kandagatla
2024-03-13 11:49                                                             ` Srinivas Kandagatla
2024-03-13 22:40                                                               ` Trilok Soni
2024-04-10 16:53                                                                 ` Nikunj Kela
2024-04-11  9:29                                                                   ` Sudeep Holla
2024-03-13 11:04                                                       ` Sudeep Holla
2024-03-13 13:04                                                         ` Srinivas Kandagatla
2024-03-14 10:55                                                           ` Sudeep Holla
2024-03-14 12:35                                                             ` Nikunj Kela
2024-03-14 15:38                                                               ` Sudeep Holla
2024-03-16 19:30                                                                 ` Trilok Soni
2024-03-19 10:17                                                                   ` Srinivas Kandagatla
2024-03-19 12:00                                                                     ` Sudeep Holla
2024-03-19 14:40                                                                       ` Srinivas Kandagatla
2024-03-19 15:17                                                                         ` Sudeep Holla
2024-03-19 15:41                                                                           ` Srinivas Kandagatla
2024-03-19 16:13                                                                             ` Sudeep Holla
2024-04-10 16:55                                                                               ` Nikunj Kela
2024-04-10 17:13                                                                                 ` Krzysztof Kozlowski
2024-04-10 17:24                                                                                   ` Nikunj Kela
2024-04-11 15:44                                                                                     ` Conor Dooley
2024-04-11 15:55                                                                                       ` Nikunj Kela
2024-04-11 19:29                                                                                         ` Krzysztof Kozlowski
2024-04-12 10:16                                                                                           ` Sudeep Holla
2024-04-11  9:23                                                                                 ` Sudeep Holla
2024-04-11 15:59                                                                                   ` Nikunj Kela
2024-04-12 10:12                                                                                     ` Sudeep Holla
2024-01-24 14:01                         ` Sudeep Holla

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=ZbDlLJRHu2ebdxc6@bogus \
    --to=sudeep.holla@arm.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=quic_nkela@quicinc.com \
    --cc=quic_psodagud@quicinc.com \
    --cc=robh+dt@kernel.org \
    --cc=srinivas.kandagatla@linaro.org \
    --cc=ulf.hansson@linaro.org \
    --cc=vincent.guittot@linaro.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).