commit 4491419217d426791003facc749f600692d9b2a6
author Zev Weiss <zev@bewilderbeest.net> Tue Mar 11 07:59:30 2025 +0000
committer Zev Weiss <zev@bewilderbeest.net> Mon Mar 24 15:29:01 2025 +0000
tree 1cc85eeef470f354def6e5f619b6a3c413ecaa52
parent ab00bd8b43cbfba82c0baf5afe9f4c52ae934ca1

Add handling for liquidflow & pressure sensors

When pressure [1] and liquidflow [2] were added to
phosphor-dbus-interfaces, bmcweb's unit-handling code and list of dbus
sensor paths weren't updated accordingly; let's add them now.

Tested: pressure and liquidflow sensors on dbus now appear (including
appropriate units) in the redfish hierarchy and on the webui-vue sensors
page.

[1] https://github.com/openbmc/phosphor-dbus-interfaces/commit/69d821b0368b8c6943a91c7f96528e2c0047f432
[2] https://github.com/openbmc/phosphor-dbus-interfaces/commit/6a8507d06e172d8d29c0459f0a0d078553d2ecc7

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I1dbe869d6c642eaeebad8605adad50315c52ad3d

redfish-core/include/utils/sensor_utils.hpp

diff --git a/redfish-core/include/utils/sensor_utils.hpp b/redfish-core/include/utils/sensor_utils.hpp
index 733e95a..718df34 100644
--- a/redfish-core/include/utils/sensor_utils.hpp
+++ b/redfish-core/include/utils/sensor_utils.hpp
@@ -208,6 +208,14 @@
     {
         return "J";
     }
+    if (sensorType == "liquidflow")
+    {
+        return "L/min";
+    }
+    if (sensorType == "pressure")
+    {
+        return "Pa";
+    }
     return "";
 }
 
@@ -253,6 +261,14 @@
     {
         return sensor::ReadingType::EnergyJoules;
     }
+    if (sensorType == "liquidflow")
+    {
+        return sensor::ReadingType::LiquidFlowLPM;
+    }
+    if (sensorType == "pressure")
+    {
+        return sensor::ReadingType::PressurePa;
+    }
     return sensor::ReadingType::Invalid;
 }
 

redfish-core/lib/sensors.hpp

diff --git a/redfish-core/lib/sensors.hpp b/redfish-core/lib/sensors.hpp
index 2804b86..2675e7f 100644
--- a/redfish-core/lib/sensors.hpp
+++ b/redfish-core/lib/sensors.hpp
@@ -80,6 +80,8 @@
         "/xyz/openbmc_project/sensors/fan_pwm",
         "/xyz/openbmc_project/sensors/altitude",
         "/xyz/openbmc_project/sensors/energy",
+        "/xyz/openbmc_project/sensors/liquidflow",
+        "/xyz/openbmc_project/sensors/pressure",
         "/xyz/openbmc_project/sensors/utilization"});
     } else {
       return  std::to_array<std::string_view>({"/xyz/openbmc_project/sensors/power",