| commit | 0eee0c193d641bcb2ff63dd180b216866f8bfb33 | [log] [tgz] |
|---|---|---|
| author | Zbigniew Kurzynski <zbigniew.kurzynski@intel.com> | Thu Jun 18 14:20:08 2020 +0200 |
| committer | Zbigniew Kurzynski <zbigniew.kurzynski@intel.com> | Thu Jun 25 06:05:33 2020 +0000 |
| tree | c38c62bef8acb5fa7b22da1756ee2a244b1528b2 | |
| parent | 62084c81755eae0a4fe59e1c8d255d376fe3d8f4 [diff] |
Fix for sensor labeling
In case when more than one item is available for the same type of sensor
and the same channel number, only one dbus sensor was created instead of
few.
This change fixes this shortcoming by adding the ‘item’ suffix to the sensor
label and allowing to distinguish created sensors from each other for
different items. In case of 'input' item the suffix is omitted.
For example, when 3 items are available like: power1 _average, power1 _cap,
power1 _input and the name file contains text ‘Power’, before this change
only one sensor will be created under name:
|-/xyz/openbmc_project/sensors/power/Power_CPU1
After this change a 3 separate sensors will be created:
|-/xyz/openbmc_project/sensors/power/Power_Average_CPU1
|-/xyz/openbmc_project/sensors/power/Power_Cap_CPU1
|-/xyz/openbmc_project/sensors/power/Power_CPU1
With this commit the name of each sensor will follow the title convention,
where each word starts with upper case.
Tested:
Tested manually with different items for the same sensor reading.
Signed-off-by: Zbigniew Kurzynski <zbigniew.kurzynski@intel.com>
Change-Id: I981d216e92fbee3b2a45a251e3359e6d41923b7d
dbus-sensors is a collection of sensor applications that provide the xyz.openbmc_project.Sensor collection of interfaces. They read sensor values from hwmon, d-bus, or direct driver access to provide readings. Some advance non-sensor features such as fan presence, pwm control, and automatic cpu detection (x86) are also supported.
runtime re-configurable from d-bus (entity-manager or the like)
isolated: each sensor type is isolated into its own daemon, so a bug in one sensor is unlikely to affect another, and single sensor modifications are possible
async single-threaded: uses sdbusplus/asio bindings
multiple data inputs: hwmon, d-bus, direct driver access