commit 0961b1155df7b146bcaccbb5691eb35b18af3bc3
author Michael Shen <gpgpgp@google.com> Tue Feb 22 11:06:33 2022 +0800
committer Ed Tanous <ed@tanous.net> Thu Mar 03 07:11:40 2022 +0000
tree 3dc4381520c74df2ebc4d10337c40baa5414395c
parent 4e98940d4f5ddf341c447f81c224738315e13b0b

Rename function `formatFRU` to `formatIPMIFRU`

`FruDevice` can parse FRU content more than IPMI format.
Rename the function to a more specific name.

Signed-off-by: Michael Shen <gpgpgp@google.com>
Change-Id: If24f9cb0d51bb18c924b800fee595f52f85673dc

src/FruDevice.cpp

diff --git a/src/FruDevice.cpp b/src/FruDevice.cpp
index 3774a2d..dc7b77f 100644
--- a/src/FruDevice.cpp
+++ b/src/FruDevice.cpp
@@ -654,7 +654,7 @@
     std::shared_ptr<sdbusplus::asio::connection>& systemBus)
 {
     boost::container::flat_map<std::string, std::string> formattedFRU;
-    resCodes res = formatFRU(device, formattedFRU);
+    resCodes res = formatIPMIFRU(device, formattedFRU);
     if (res == resCodes::resErr)
     {
         std::cerr << "failed to parse FRU for device at bus " << bus
@@ -830,7 +830,7 @@
         return false;
     }
     // verify legal fru by running it through fru parsing logic
-    if (formatFRU(fru, tmp) != resCodes::resOK)
+    if (formatIPMIFRU(fru, tmp) != resCodes::resOK)
     {
         std::cerr << "Invalid fru format during writeFRU\n";
         return false;

src/FruUtils.cpp

diff --git a/src/FruUtils.cpp b/src/FruUtils.cpp
index 940f8c8..1152f87 100644
--- a/src/FruUtils.cpp
+++ b/src/FruUtils.cpp
@@ -256,8 +256,9 @@
     return true;
 }
 
-resCodes formatFRU(const std::vector<uint8_t>& fruBytes,
-                   boost::container::flat_map<std::string, std::string>& result)
+resCodes
+    formatIPMIFRU(const std::vector<uint8_t>& fruBytes,
+                  boost::container::flat_map<std::string, std::string>& result)
 {
     resCodes ret = resCodes::resOK;
     if (fruBytes.size() <= fruBlockSize)