James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 1 | /* |
| 2 | // Copyright (c) 2017 Intel Corporation |
| 3 | // |
| 4 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | // you may not use this file except in compliance with the License. |
| 6 | // You may obtain a copy of the License at |
| 7 | // |
| 8 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | // |
| 10 | // Unless required by applicable law or agreed to in writing, software |
| 11 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | // See the License for the specific language governing permissions and |
| 14 | // limitations under the License. |
| 15 | */ |
| 16 | |
Patrick Venture | ca44b2f | 2019-10-31 11:02:26 -0700 | [diff] [blame] | 17 | #include "ADCSensor.hpp" |
| 18 | |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 19 | #include <unistd.h> |
| 20 | |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 21 | #include <boost/algorithm/string/predicate.hpp> |
| 22 | #include <boost/algorithm/string/replace.hpp> |
| 23 | #include <boost/date_time/posix_time/posix_time.hpp> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 24 | #include <cmath> |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 25 | #include <filesystem> |
| 26 | #include <fstream> |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 27 | #include <iostream> |
| 28 | #include <limits> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 29 | #include <memory> |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 30 | #include <optional> |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 31 | #include <sdbusplus/asio/connection.hpp> |
| 32 | #include <sdbusplus/asio/object_server.hpp> |
| 33 | #include <string> |
Patrick Venture | 96e97db | 2019-10-31 13:44:38 -0700 | [diff] [blame] | 34 | #include <vector> |
Patrick Venture | 2da370e | 2019-11-01 11:51:31 -0700 | [diff] [blame] | 35 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 36 | static constexpr unsigned int sensorPollMs = 500; |
| 37 | static constexpr size_t warnAfterErrorCount = 10; |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 38 | static constexpr unsigned int gpioBridgeEnableMs = 20; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 39 | // scaling factor from hwmon |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 40 | static constexpr unsigned int sensorScaleFactor = 1000; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 41 | |
James Feist | cc6d4fe | 2018-11-14 16:38:26 -0800 | [diff] [blame] | 42 | static constexpr double roundFactor = 10000; // 3 decimal places |
James Feist | ce3fca4 | 2018-11-21 12:58:24 -0800 | [diff] [blame] | 43 | static constexpr double maxReading = 20; |
| 44 | static constexpr double minReading = 0; |
James Feist | cc6d4fe | 2018-11-14 16:38:26 -0800 | [diff] [blame] | 45 | |
James Feist | d870587 | 2019-02-08 13:26:09 -0800 | [diff] [blame] | 46 | ADCSensor::ADCSensor(const std::string& path, |
| 47 | sdbusplus::asio::object_server& objectServer, |
| 48 | std::shared_ptr<sdbusplus::asio::connection>& conn, |
| 49 | boost::asio::io_service& io, const std::string& sensorName, |
| 50 | std::vector<thresholds::Threshold>&& _thresholds, |
James Feist | 71d31b2 | 2019-01-02 16:57:54 -0800 | [diff] [blame] | 51 | const double scaleFactor, PowerState readState, |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 52 | const std::string& sensorConfiguration, |
Jae Hyun Yoo | 1cbd1c6 | 2019-07-29 15:02:43 -0700 | [diff] [blame] | 53 | std::optional<BridgeGpio>&& bridgeGpio) : |
James Feist | 930fcde | 2019-05-28 12:58:43 -0700 | [diff] [blame] | 54 | Sensor(boost::replace_all_copy(sensorName, " ", "_"), |
James Feist | ce3fca4 | 2018-11-21 12:58:24 -0800 | [diff] [blame] | 55 | std::move(_thresholds), sensorConfiguration, |
| 56 | "xyz.openbmc_project.Configuration.ADC", maxReading, minReading), |
Brad Bishop | fbb44ad | 2019-11-08 09:42:37 -0500 | [diff] [blame] | 57 | objServer(objectServer), inputDev(io, open(path.c_str(), O_RDONLY)), |
| 58 | waitTimer(io), path(path), errCount(0), scaleFactor(scaleFactor), |
| 59 | bridgeGpio(std::move(bridgeGpio)), readState(std::move(readState)), |
| 60 | thresholdTimer(io, this) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 61 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 62 | sensorInterface = objectServer.add_interface( |
| 63 | "/xyz/openbmc_project/sensors/voltage/" + name, |
| 64 | "xyz.openbmc_project.Sensor.Value"); |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 65 | if (thresholds::hasWarningInterface(thresholds)) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 66 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 67 | thresholdInterfaceWarning = objectServer.add_interface( |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 68 | "/xyz/openbmc_project/sensors/voltage/" + name, |
| 69 | "xyz.openbmc_project.Sensor.Threshold.Warning"); |
| 70 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 71 | if (thresholds::hasCriticalInterface(thresholds)) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 72 | { |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 73 | thresholdInterfaceCritical = objectServer.add_interface( |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 74 | "/xyz/openbmc_project/sensors/voltage/" + name, |
| 75 | "xyz.openbmc_project.Sensor.Threshold.Critical"); |
| 76 | } |
James Feist | 078f232 | 2019-03-08 11:09:05 -0800 | [diff] [blame] | 77 | association = objectServer.add_interface( |
James Feist | 2adc95c | 2019-09-30 14:55:28 -0700 | [diff] [blame] | 78 | "/xyz/openbmc_project/sensors/voltage/" + name, association::interface); |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 79 | setInitialProperties(conn); |
| 80 | setupRead(); |
James Feist | 71d31b2 | 2019-01-02 16:57:54 -0800 | [diff] [blame] | 81 | |
| 82 | // setup match |
| 83 | setupPowerMatch(conn); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 84 | } |
| 85 | |
| 86 | ADCSensor::~ADCSensor() |
| 87 | { |
| 88 | // close the input dev to cancel async operations |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 89 | inputDev.close(); |
| 90 | waitTimer.cancel(); |
James Feist | 251c782 | 2018-09-12 12:54:15 -0700 | [diff] [blame] | 91 | objServer.remove_interface(thresholdInterfaceWarning); |
| 92 | objServer.remove_interface(thresholdInterfaceCritical); |
| 93 | objServer.remove_interface(sensorInterface); |
James Feist | 078f232 | 2019-03-08 11:09:05 -0800 | [diff] [blame] | 94 | objServer.remove_interface(association); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 95 | } |
| 96 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 97 | void ADCSensor::setupRead(void) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 98 | { |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 99 | if (bridgeGpio.has_value()) |
| 100 | { |
Jae Hyun Yoo | 12bbae0 | 2019-07-22 17:24:09 -0700 | [diff] [blame] | 101 | (*bridgeGpio).set(1); |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 102 | // In case a channel has a bridge circuit,we have to turn the bridge on |
| 103 | // prior to reading a value at least for one scan cycle to get a valid |
| 104 | // value. Guarantee that the HW signal can be stable, the HW signal |
| 105 | // could be instability. |
| 106 | waitTimer.expires_from_now( |
| 107 | boost::posix_time::milliseconds(gpioBridgeEnableMs)); |
| 108 | waitTimer.async_wait([&](const boost::system::error_code& ec) { |
| 109 | if (ec == boost::asio::error::operation_aborted) |
| 110 | { |
| 111 | return; // we're being canceled |
| 112 | } |
| 113 | boost::asio::async_read_until( |
| 114 | inputDev, readBuf, '\n', |
| 115 | [&](const boost::system::error_code& ec, |
| 116 | std::size_t /*bytes_transfered*/) { handleResponse(ec); }); |
| 117 | }); |
| 118 | } |
| 119 | else |
| 120 | { |
| 121 | boost::asio::async_read_until( |
| 122 | inputDev, readBuf, '\n', |
| 123 | [&](const boost::system::error_code& ec, |
| 124 | std::size_t /*bytes_transfered*/) { handleResponse(ec); }); |
| 125 | } |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 126 | } |
| 127 | |
James Feist | d870587 | 2019-02-08 13:26:09 -0800 | [diff] [blame] | 128 | void ADCSensor::handleResponse(const boost::system::error_code& err) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 129 | { |
| 130 | if (err == boost::system::errc::bad_file_descriptor) |
| 131 | { |
| 132 | return; // we're being destroyed |
| 133 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 134 | std::istream responseStream(&readBuf); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 135 | |
| 136 | if (!err) |
| 137 | { |
| 138 | std::string response; |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 139 | std::getline(responseStream, response); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 140 | |
| 141 | // todo read scaling factors from configuration |
| 142 | try |
| 143 | { |
Josh Lehan | 833661a | 2020-03-04 17:35:41 -0800 | [diff] [blame^] | 144 | double nvalue = std::stod(response); |
James Feist | cc6d4fe | 2018-11-14 16:38:26 -0800 | [diff] [blame] | 145 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 146 | nvalue = (nvalue / sensorScaleFactor) / scaleFactor; |
James Feist | cc6d4fe | 2018-11-14 16:38:26 -0800 | [diff] [blame] | 147 | nvalue = std::round(nvalue * roundFactor) / roundFactor; |
| 148 | |
Josh Lehan | 833661a | 2020-03-04 17:35:41 -0800 | [diff] [blame^] | 149 | updateValue(nvalue); |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 150 | errCount = 0; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 151 | } |
James Feist | b6c0b91 | 2019-07-09 12:21:44 -0700 | [diff] [blame] | 152 | catch (std::invalid_argument&) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 153 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 154 | errCount++; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 155 | } |
| 156 | } |
James Feist | 3459d01 | 2020-01-30 13:17:43 -0800 | [diff] [blame] | 157 | else if (readState == PowerState::on && !isPowerOn()) |
| 158 | { |
| 159 | errCount = 0; |
| 160 | updateValue(std::numeric_limits<double>::quiet_NaN()); |
| 161 | } |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 162 | else |
| 163 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 164 | errCount++; |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 165 | } |
James Feist | dc6c55f | 2018-10-31 12:53:20 -0700 | [diff] [blame] | 166 | // only print once |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 167 | if (errCount == warnAfterErrorCount) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 168 | { |
James Feist | dc6c55f | 2018-10-31 12:53:20 -0700 | [diff] [blame] | 169 | std::cerr << "Failure to read sensor " << name << " at " << path |
| 170 | << " ec:" << err << "\n"; |
| 171 | } |
| 172 | |
| 173 | if (errCount >= warnAfterErrorCount) |
| 174 | { |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 175 | updateValue(0); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 176 | } |
| 177 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 178 | responseStream.clear(); |
| 179 | inputDev.close(); |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 180 | if (bridgeGpio.has_value()) |
| 181 | { |
Jae Hyun Yoo | 12bbae0 | 2019-07-22 17:24:09 -0700 | [diff] [blame] | 182 | (*bridgeGpio).set(0); |
Zhu, Yunge | a5b1bbc | 2019-04-09 19:49:36 -0400 | [diff] [blame] | 183 | } |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 184 | int fd = open(path.c_str(), O_RDONLY); |
Jae Hyun Yoo | ef9665a | 2019-10-10 10:26:39 -0700 | [diff] [blame] | 185 | if (fd < 0) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 186 | { |
| 187 | return; // we're no longer valid |
| 188 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 189 | inputDev.assign(fd); |
| 190 | waitTimer.expires_from_now(boost::posix_time::milliseconds(sensorPollMs)); |
James Feist | d870587 | 2019-02-08 13:26:09 -0800 | [diff] [blame] | 191 | waitTimer.async_wait([&](const boost::system::error_code& ec) { |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 192 | if (ec == boost::asio::error::operation_aborted) |
| 193 | { |
| 194 | return; // we're being canceled |
| 195 | } |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 196 | setupRead(); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 197 | }); |
| 198 | } |
| 199 | |
Jae Hyun Yoo | 9ced0a3 | 2018-10-25 10:42:39 -0700 | [diff] [blame] | 200 | void ADCSensor::checkThresholds(void) |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 201 | { |
James Feist | 71d31b2 | 2019-01-02 16:57:54 -0800 | [diff] [blame] | 202 | if (readState == PowerState::on && !isPowerOn()) |
| 203 | { |
| 204 | return; |
| 205 | } |
James Feist | 46342ec | 2019-03-06 14:03:41 -0800 | [diff] [blame] | 206 | |
| 207 | thresholds::checkThresholdsPowerDelay(this, thresholdTimer); |
James Feist | 6714a25 | 2018-09-10 15:26:18 -0700 | [diff] [blame] | 208 | } |