commit 32453e9b9a4b1f8233efc68646cf317c2e8f7173
author Brandon Wyman <bjwyman@gmail.com> Wed Dec 15 19:00:14 2021 +0000
committer Shawn McCarney <shawnmm@us.ibm.com> Thu Jan 27 19:14:40 2022 +0000
tree 96b0122bae28bd9e12b9be887ec2f2aff412e3ae
parent b70eae9af652c38eb6aa8f2e4e71075106fa41d9

psu-ng: Continue reading after readFail

Update the PMBus::read() function to allow for not creating journal
trace and elog, but default to continuing to trace and elog, the
previous behavior.

If we reach the limit of read failures that results in a communication
error log, continue to read, but stop logging failures.

If communication restores, we may be able to detect what caused the read
failure, or otherwise detect or clear new faults.

Change-Id: If59b86211ab54c31248ede78f8f117b607298923
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>

phosphor-power-supply/test/mock.hpp

diff --git a/phosphor-power-supply/test/mock.hpp b/phosphor-power-supply/test/mock.hpp
index 25f48c1..cf4d3ae 100644
--- a/phosphor-power-supply/test/mock.hpp
+++ b/phosphor-power-supply/test/mock.hpp
@@ -19,7 +19,8 @@
   public:
     virtual ~MockedPMBus() = default;
 
-    MOCK_METHOD(uint64_t, read, (const std::string& name, Type type),
+    MOCK_METHOD(uint64_t, read,
+                (const std::string& name, Type type, bool errTrace),
                 (override));
     MOCK_METHOD(std::string, readString, (const std::string& name, Type type),
                 (override));