commit e3338528bbf422655610e7414a5c7b886558d135
author James Feist <james.feist@linux.intel.com> Tue Sep 15 15:40:30 2020 -0700
committer James Feist <james.feist@linux.intel.com> Tue Sep 15 23:21:02 2020 +0000
tree e0463feaef13ff226d51f92ca11c306737a551c0
parent d3da1280dd996d1c4677fdfb2c98bb95ed11f718

Move dbus connection into base sensor object

This seems common through most sensors, might as
well move it for consistency.

Tested: sensors operational as normal

Change-Id: I4e6c55cbb7171ee51700015f8c2ef8c05c90fb4e
Signed-off-by: James Feist <james.feist@linux.intel.com>

src/ExitAirTempSensor.cpp

diff --git a/src/ExitAirTempSensor.cpp b/src/ExitAirTempSensor.cpp
index 7197c4a..41d9a19 100644
--- a/src/ExitAirTempSensor.cpp
+++ b/src/ExitAirTempSensor.cpp
@@ -166,9 +166,9 @@
     Sensor(boost::replace_all_copy(sensorName, " ", "_"),
            std::move(thresholdData), sensorConfiguration,
            "xyz.openbmc_project.Configuration.ExitAirTemp", cfmMaxReading,
-           cfmMinReading, PowerState::on),
+           cfmMinReading, conn, PowerState::on),
     std::enable_shared_from_this<CFMSensor>(), parent(parent),
-    dbusConnection(conn), objServer(objectServer)
+    objServer(objectServer)
 {
     sensorInterface =
         objectServer.add_interface("/xyz/openbmc_project/sensors/cfm/" + name,
@@ -494,9 +494,8 @@
     Sensor(boost::replace_all_copy(sensorName, " ", "_"),
            std::move(thresholdData), sensorConfiguration,
            "xyz.openbmc_project.Configuration.ExitAirTemp", exitAirMaxReading,
-           exitAirMinReading, PowerState::on),
-    std::enable_shared_from_this<ExitAirTempSensor>(), dbusConnection(conn),
-    objServer(objectServer)
+           exitAirMinReading, conn, PowerState::on),
+    std::enable_shared_from_this<ExitAirTempSensor>(), objServer(objectServer)
 {
     sensorInterface = objectServer.add_interface(
         "/xyz/openbmc_project/sensors/temperature/" + name,