commit 26e04a86ef1901183ace26bade4cb559945dcb1a
author Prithvi Pai <ppai@nvidia.com> Wed Apr 02 13:01:22 2025 +0530
committer Prithvi Pai <ppai@nvidia.com> Fri Apr 11 08:38:17 2025 +0530
tree 0fbea3a7749e95fd9ae0cac9057c34b957886833
parent 6981b7ffb8a81bd1ea90c8deba8466accbe2693b

Remove duplicate functions to get smbios slot counts

Currently there are duplicate functions to get CPU count,
dimm count, pcie slot count, tpm count and firmware inventory
record counts. The current patch refactors the same and removes
the duplicate.

Tested:
1) DBus tree with this changes looks as before

`- /xyz/openbmc_project
   |- /xyz/openbmc_project/Smbios
   | `- /xyz/openbmc_project/Smbios/MDR_V2
   |- /xyz/openbmc_project/inventory
   |`- /xyz/openbmc_project/inventory/system
   |`- /xyz/openbmc_project/inventory/system/chassis
   |`- /xyz/openbmc_project/inventory/system/chassis/motherboard
   |`- /xyz/openbmc_project/inventory/system/chassis/motherboard/bios
   `- /xyz/openbmc_project/software
   `- /xyz/openbmc_project/software/UEFI

Change-Id: Ie77be54ac240819df785f93272cc2b8398b6c2d0
Signed-off-by: Prithvi Pai <ppai@nvidia.com>

include/mdrv2.hpp

diff --git a/include/mdrv2.hpp b/include/mdrv2.hpp
index c60633c..9776523 100644
--- a/include/mdrv2.hpp
+++ b/include/mdrv2.hpp
@@ -189,11 +189,7 @@
     inline uint8_t smbiosValidFlag(uint8_t index);
     void systemInfoUpdate(void);
 
-    std::optional<size_t> getTotalCpuSlot(void);
-    std::optional<size_t> getTotalDimmSlot(void);
-    std::optional<size_t> getTotalPcieSlot(void);
-    std::optional<size_t> getTotalTpm(void);
-    std::optional<size_t> getTotalFirmwareInventory(void);
+    std::optional<size_t> getTotalSmbiosEntries(uint8_t smbiosType);
     std::vector<std::unique_ptr<Cpu>> cpus;
     std::vector<std::unique_ptr<Dimm>> dimms;
     std::vector<std::unique_ptr<Pcie>> pcies;