Change sensor Value to double
Generally most demons we've looked at take the int64 values and
immediately translates the value to a double. As the individual demons
are doing the floating point computation anyway, there doesn't seem to be
a large benefit to keeping separating the value on dbus, only to recompute
in each demon, causing extra overhead for consumers. In some cases this
causes an extra d-bus call, for instance when reading a threshold a
consumer to know the value needs to also access the value field to get
the scale, which is arguably more expensive than having the property be a
double in the first place.
Change-Id: If47c19f77286e3764d167a6843ba91e2ca1769b8
Signed-off-by: James Feist <james.feist@linux.intel.com>
diff --git a/xyz/openbmc_project/Sensor/Threshold/Critical.interface.yaml b/xyz/openbmc_project/Sensor/Threshold/Critical.interface.yaml
index af640fb..0c2a220 100644
--- a/xyz/openbmc_project/Sensor/Threshold/Critical.interface.yaml
+++ b/xyz/openbmc_project/Sensor/Threshold/Critical.interface.yaml
@@ -15,11 +15,11 @@
properties:
- name: CriticalHigh
- type: int64
+ type: double
description: >
The upper bound of the critical threshold.
- name: CriticalLow
- type: int64
+ type: double
description: >
The lower bound of the critical threshold.
- name: CriticalAlarmHigh