commit 5e0cbc24e99a2dc336f076cc273140b53f2b1ae7
author Manojkiran Eda <manojkiran.eda@gmail.com> Tue Aug 12 12:15:38 2025 +0530
committer Manojkiran Eda <manojkiran.eda@gmail.com> Thu Aug 28 14:37:07 2025 +0530
tree 6367a5f8ad37f0d4b23df5f323762a6f2403a020
parent ae91a56d09bf50fe45b1760f29a81e188e07e163

Remove error trace from BIOS attribute updates

Commit 4e29f80bf85c3a8197bb6c3183f25face059f29c introduced an error
trace in the serialization function for a condition that is not
actually erroneous. This message is printed on every BIOS attribute
update, adding noise without providing meaningful information. The
trace is now removed to reduce unnecessary log clutter.

Change-Id: I632f8c98da5a4418c4715b6b88e95f1025183a54
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>

src/manager_serialize.cpp

diff --git a/src/manager_serialize.cpp b/src/manager_serialize.cpp
index 806b138..f3441eb 100644
--- a/src/manager_serialize.cpp
+++ b/src/manager_serialize.cpp
@@ -41,9 +41,9 @@
  *                       across code levels
  */
 template <class Archive>
-void save(Archive& archive, const Manager& entry, const std::uint32_t version)
+void save(Archive& archive, const Manager& entry,
+          const std::uint32_t /*version*/)
 {
-    lg2::error("Save is called with version {VER}", "VER", version);
     archive(entry.sdbusplus::xyz::openbmc_project::BIOSConfig::server::Manager::
                 baseBIOSTable(),
             entry.sdbusplus::xyz::openbmc_project::BIOSConfig::server::Manager::