[PATCH] firmware: arm_scmi: Add new sensor unit types

Artem Shimko posted 1 patch 3 months, 2 weeks ago
include/linux/scmi_protocol.h | 2 ++
1 file changed, 2 insertions(+)
[PATCH] firmware: arm_scmi: Add new sensor unit types
Posted by Artem Shimko 3 months, 2 weeks ago
Add support for two new sensor unit types introduced in Arm SCMI
- STATE_BINARY (0x5E): Binary state (1: enabled/on, 0: disabled/off)
- BYTES_SEC (0x5F): Bytes per second

These unit types are used for sensors that report binary state information
and data transfer rate measurements respectively.

Signed-off-by: Artem Shimko <a.shimko.dev@gmail.com>
---
Hello,

This patch adds definitions for two new sensor unit types that were
introduced in the Arm System Control and Management Interface (SCMI)
Specification version 4.0:

STATE_BINARY (0x5E) - Used for sensors that represent binary states
where:
1 indicates enabled/on state
0 indicates disabled/off state

BYTES_SEC (0x5F) - Used for sensors that measure data transfer rates
in bytes per second

These additions ensure compatibility with SCMIv4.0 and allow proper
handling of binary state sensors and bandwidth measurement sensors
in the Linux SCMI driver.

The patch follows the existing pattern for sensor unit type definitions
and maintains backward compatibility with previous SCMI versions.

Thank you for your consideration.

Best regards,
Artem Shimko

 include/linux/scmi_protocol.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/linux/scmi_protocol.h b/include/linux/scmi_protocol.h
index aafaac1496b0..629acb7831eb 100644
--- a/include/linux/scmi_protocol.h
+++ b/include/linux/scmi_protocol.h
@@ -488,6 +488,8 @@ enum scmi_sensor_class {
 	CUBIC_METERS_SEC = 0x5B,
 	MM_MERCURY = 0x5C,
 	RADIANS_SEC_SQUARED = 0x5D,
+	STATE_BINARY = 0x5E,
+	BYTES_SEC = 0x5F,
 	OEM_UNIT = 0xFF
 };
 
-- 
2.43.0
Re: [PATCH] firmware: arm_scmi: Add new sensor unit types
Posted by Sudeep Holla 3 months, 2 weeks ago
On Tue, Oct 21, 2025 at 07:52:11PM +0300, Artem Shimko wrote:
> Add support for two new sensor unit types introduced in Arm SCMI
> - STATE_BINARY (0x5E): Binary state (1: enabled/on, 0: disabled/off)
> - BYTES_SEC (0x5F): Bytes per second
> 
> These unit types are used for sensors that report binary state information
> and data transfer rate measurements respectively.
> 
> Signed-off-by: Artem Shimko <a.shimko.dev@gmail.com>
> ---
> Hello,
> 
> This patch adds definitions for two new sensor unit types that were
> introduced in the Arm System Control and Management Interface (SCMI)
> Specification version 4.0:
>

It is still alpha, we don't add anything into the kernel unless it is
of beta quality. Do you have a use or you are just adding to match the
spec ?

-- 
Regards,
Sudeep
Re: [PATCH] firmware: arm_scmi: Add new sensor unit types
Posted by Artem Shimko 4 days, 22 hours ago
On Wed, Oct 22, 2025 at 12:22 PM Sudeep Holla <sudeep.holla@arm.com> wrote:
> It is still alpha, we don't add anything into the kernel unless it is
> of beta quality.

Hi Sudeep,

Just a polite ping.

Is it still too early for 4.0 bet0?

Thank you!

Regards,
Artem
Re: [PATCH] firmware: arm_scmi: Add new sensor unit types
Posted by Sudeep Holla 4 days, 21 hours ago
On Tue, Feb 03, 2026 at 03:45:28PM +0300, Artem Shimko wrote:
> On Wed, Oct 22, 2025 at 12:22 PM Sudeep Holla <sudeep.holla@arm.com> wrote:
> > It is still alpha, we don't add anything into the kernel unless it is
> > of beta quality.
> 
> Hi Sudeep,
> 
> Just a polite ping.
> 
> Is it still too early for 4.0 bet0?
> 

v4.0 beta is out but I would take this with telemetry support that may
start using this. Let me know if there are any users before that.

-- 
Regards,
Sudeep
Re: [PATCH] firmware: arm_scmi: Add new sensor unit types
Posted by Artem Shimko 4 days, 18 hours ago
On Tue, Feb 3, 2026 at 5:25 PM Sudeep Holla <sudeep.holla@kernel.org> wrote:
>  Let me know if there are any users before that.
Hi Sudeep,

There are no users before that.

Thank you!

Regards,
Artem
Re: [PATCH] firmware: arm_scmi: Add new sensor unit types
Posted by Artem Shimko 3 months, 2 weeks ago
On Wed, Oct 22, 2025 at 12:22 PM Sudeep Holla <sudeep.holla@arm.com> wrote:

> It is still alpha, we don't add anything into the kernel unless it is
> of beta quality. Do you have a use or you are just adding to match the
> spec ?
Hi Sudeep,

I'm adding it to meet the specifications.

Trying to be useful

Thank you.

Best regards,
Artem