commit 62084c81755eae0a4fe59e1c8d255d376fe3d8f4
author Josh Lehan <krellan@google.com> Mon Jun 22 15:27:12 2020 -0700
committer Josh Lehan <krellan@google.com> Wed Jun 24 20:34:41 2020 +0000
tree 435b54cb4bbc657a4af93acc735b4ff2955a4ab9
parent 9a44547ef3ccb02eb6a3dd4f4b5b01f2c5db7402

psusensor: Update sensorTypes and sync with pmbusNames

Trivial patch, to update sensorTypes array,
keeping it in sync with pmbusNames array,
and sorting both arrays alphabetically.

Question, why are there two arrays here?
The sensorTypes array could be programmatically
generated from pmbusNames easily.
If interest, I will make another patch to do this.

Question, what is special about the "pmbus" driver name,
which does not correspond to a sensor chip, unlike the others?

Question, why is pmbusNames in lowercase but sensorTypes
is in all caps, except for "pmbus"?
Is there a reason for this,
or is it just a convention that evolved that way?

Signed-off-by: Josh Lehan <krellan@google.com>
Change-Id: I9fbc4fb65a0c3a69d43eef2e873169e522f11748

src/PSUSensorMain.cpp

diff --git a/src/PSUSensorMain.cpp b/src/PSUSensorMain.cpp
index 03af046..616aec7 100644
--- a/src/PSUSensorMain.cpp
+++ b/src/PSUSensorMain.cpp
@@ -40,7 +40,10 @@
 
 static constexpr bool DEBUG = false;
 
-static constexpr std::array<const char*, 8> sensorTypes = {
+static constexpr std::array<const char*, 13> sensorTypes = {
+    "xyz.openbmc_project.Configuration.ADM1272",
+    "xyz.openbmc_project.Configuration.ADM1278",
+    "xyz.openbmc_project.Configuration.INA219",
     "xyz.openbmc_project.Configuration.INA230",
     "xyz.openbmc_project.Configuration.ISL68137",
     "xyz.openbmc_project.Configuration.MAX16601",
@@ -48,10 +51,12 @@
     "xyz.openbmc_project.Configuration.MAX20734",
     "xyz.openbmc_project.Configuration.MAX20796",
     "xyz.openbmc_project.Configuration.MAX34451",
-    "xyz.openbmc_project.Configuration.pmbus"};
+    "xyz.openbmc_project.Configuration.pmbus",
+    "xyz.openbmc_project.Configuration.PXE1610",
+    "xyz.openbmc_project.Configuration.RAA228228"};
 
 static std::vector<std::string> pmbusNames = {
-    "adm1272",  "adm1278",  "isl68137", "ina219",   "ina230",
+    "adm1272",  "adm1278",  "ina219",   "ina230",   "isl68137",
     "max16601", "max20730", "max20734", "max20796", "max34451",
     "pmbus",    "pxe1610",  "raa228228"};