From 4253fcaa5af69241ddc06ae72d3f94fd3b8818c9 Mon Sep 17 00:00:00 2001 From: Brandon Wyman Date: Thu, 27 Jul 2017 15:37:55 -0500 Subject: Switch which GPIO value sets air or water cooled During the coding and testing of this application there was some miscommunication on the jumper position for a Witherspoon system. The system was jumpered for Air when intended to be jumpered for the Water cooled position. Further examination of the documentation and the jumper settings on systems found that the logic was reversed. This change should correct the problem and report the correct cooling properties. Resolves openbmc/openbmc#2041 Change-Id: I7e25bd72342fde40df1fec8f6a1a444eac401f09 Signed-off-by: Brandon Wyman --- cooling-type/cooling_type.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cooling-type/cooling_type.cpp b/cooling-type/cooling_type.cpp index c1f96aa..3cb19ab 100644 --- a/cooling-type/cooling_type.cpp +++ b/cooling-type/cooling_type.cpp @@ -68,11 +68,11 @@ void CoolingType::readGpio(const std::string& gpioPath, unsigned int keycode) // TODO openbmc/phosphor-fan-presence#6 if (value > 0) { - setWaterCooled(); + setAirCooled(); } else { - setAirCooled(); + setWaterCooled(); } } -- cgit v1.2.1