From: Sudeep Holla <sudeep.holla@arm.com>
To: Florian Fainelli <florian.fainelli@broadcom.com>
Cc: "linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Sudeep Holla <sudeep.holla@arm.com>,
Cristian Marussi <cristian.marussi@arm.com>,
Jim Quinlan <james.quinlan@broadcom.com>
Subject: Re: SCMI support for sensors shared memory?
Date: Fri, 23 Feb 2024 06:15:22 +0000 [thread overview]
Message-ID: <20240223061522.lmhmwobub34mbfgd@bogus> (raw)
In-Reply-To: <4ed10434-0620-4fd3-b595-61f2c6080722@broadcom.com>
On Thu, Feb 22, 2024 at 11:35:21AM -0800, Florian Fainelli wrote:
> Hi Sudeep, Christian,
>
> AFAICT there is no support within scmi-hwmon.c for a sensors shared memory,
> even though the SCMI specification describes how to use it.
>
Correct, in fact lots of other features which includes statistic information
that use additional shared memory like this one is not supported in the
kernel driver. Though there was request to have such a feature when the
specification was initially developed, it was soon realised that most of
the platforms can't afford to have large amounts of shared memory needed
for this purpose. Yes even 512 or 1kB can be considered large on some
platforms that use dedicated SRAM for this SCMI communication as the
co-processor incharge of providing SCMI platform may not have access to
DRAM and hence DRAM curv eout is not an option.
So AFAIU, there was no known platform requiring this feature until now,
hence we never added it to the kernel as we can't test the same.
> Is this something that you are working on by any chance, or should we
> contribute it once we have it ready?
>
We are not working on it. So feel free to contribute, happy to review and
get that merged if required/supported on certain platforms.
--
Regards,
Sudeep
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
prev parent reply other threads:[~2024-02-23 6:16 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-22 19:35 SCMI support for sensors shared memory? Florian Fainelli
2024-02-23 6:15 ` Sudeep Holla [this message]
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=20240223061522.lmhmwobub34mbfgd@bogus \
--to=sudeep.holla@arm.com \
--cc=cristian.marussi@arm.com \
--cc=florian.fainelli@broadcom.com \
--cc=james.quinlan@broadcom.com \
--cc=linux-arm-kernel@lists.infradead.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