Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* SCMI support for sensors shared memory?
@ 2024-02-22 19:35 Florian Fainelli
  2024-02-23  6:15 ` Sudeep Holla
  0 siblings, 1 reply; 2+ messages in thread
From: Florian Fainelli @ 2024-02-22 19:35 UTC (permalink / raw)
  To: linux-arm-kernel@lists.infradead.org, Sudeep Holla,
	Cristian Marussi
  Cc: Jim Quinlan


[-- Attachment #1.1: Type: text/plain, Size: 290 bytes --]

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.

Is this something that you are working on by any chance, or should we 
contribute it once we have it ready?

Thanks!
-- 
Florian

[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4221 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: SCMI support for sensors shared memory?
  2024-02-22 19:35 SCMI support for sensors shared memory? Florian Fainelli
@ 2024-02-23  6:15 ` Sudeep Holla
  0 siblings, 0 replies; 2+ messages in thread
From: Sudeep Holla @ 2024-02-23  6:15 UTC (permalink / raw)
  To: Florian Fainelli
  Cc: linux-arm-kernel@lists.infradead.org, Sudeep Holla,
	Cristian Marussi, Jim Quinlan

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-02-23  6:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-22 19:35 SCMI support for sensors shared memory? Florian Fainelli
2024-02-23  6:15 ` Sudeep Holla

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox