Searched hist:"3 c208464" (Results 1 – 3 of 3) sorted by relevance
| /openbmc/phosphor-power/phosphor-power-supply/test/ |
| H A D | power_supply_tests.cpp | 3c208464 Wed May 13 16:25:58 CDT 2020 Brandon Wyman <bjwyman@gmail.com> psu-ng: Updates for clearFaults
It was discovered that the power supplies may not always have the
in1_crit "file" available in the HWMON directory. This file may be
missing if certain commands are not supported. It was also discovered
that it can be missing if the power supply falsely indicates it supports
PEC, but actually does not. Changing the readString call to read from
the in1_input file, which typically should be present (READ_VIN).
Move the clearFaults function from the hpp file to the cpp file, as:
1. It is getting a bit lengthy.
2. Adding a try/catch for ReadFailure to avoid crashing the application
if there is a ReadFailure, which is in the phosphor::logging
namespace already used in the cpp file.
Update the test case to expect a read from in1_input, and return a value
that more closely resembles an actual READ_VIN value.
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I9747317c68040cdce6bb80922e3928be55376c44
|
| /openbmc/phosphor-power/phosphor-power-supply/ |
| H A D | power_supply.hpp | 3c208464 Wed May 13 16:25:58 CDT 2020 Brandon Wyman <bjwyman@gmail.com> psu-ng: Updates for clearFaults
It was discovered that the power supplies may not always have the
in1_crit "file" available in the HWMON directory. This file may be
missing if certain commands are not supported. It was also discovered
that it can be missing if the power supply falsely indicates it supports
PEC, but actually does not. Changing the readString call to read from
the in1_input file, which typically should be present (READ_VIN).
Move the clearFaults function from the hpp file to the cpp file, as:
1. It is getting a bit lengthy.
2. Adding a try/catch for ReadFailure to avoid crashing the application
if there is a ReadFailure, which is in the phosphor::logging
namespace already used in the cpp file.
Update the test case to expect a read from in1_input, and return a value
that more closely resembles an actual READ_VIN value.
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I9747317c68040cdce6bb80922e3928be55376c44
|
| H A D | power_supply.cpp | 3c208464 Wed May 13 16:25:58 CDT 2020 Brandon Wyman <bjwyman@gmail.com> psu-ng: Updates for clearFaults
It was discovered that the power supplies may not always have the
in1_crit "file" available in the HWMON directory. This file may be
missing if certain commands are not supported. It was also discovered
that it can be missing if the power supply falsely indicates it supports
PEC, but actually does not. Changing the readString call to read from
the in1_input file, which typically should be present (READ_VIN).
Move the clearFaults function from the hpp file to the cpp file, as:
1. It is getting a bit lengthy.
2. Adding a try/catch for ReadFailure to avoid crashing the application
if there is a ReadFailure, which is in the phosphor::logging
namespace already used in the cpp file.
Update the test case to expect a read from in1_input, and return a value
that more closely resembles an actual READ_VIN value.
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I9747317c68040cdce6bb80922e3928be55376c44
|