Remove unused DumpType from Notify implementation.
Each type of dump is hosted in its own service so DumpType
is no more needed to differentiate between dumps in Notify.
So removing from implementation to match the interface.
Changes are tested with creation of a system dump.
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
Change-Id: Ib34b2235f7b7a6331e52f3dc55660bd6082aa29a
diff --git a/dump-extensions/openpower-dumps/dump_manager_system.cpp b/dump-extensions/openpower-dumps/dump_manager_system.cpp
index 5d5890d..52bb956 100644
--- a/dump-extensions/openpower-dumps/dump_manager_system.cpp
+++ b/dump-extensions/openpower-dumps/dump_manager_system.cpp
@@ -18,15 +18,9 @@
using namespace phosphor::logging;
using namespace sdbusplus::xyz::openbmc_project::Common::Error;
-void Manager::notify(NewDump::DumpType dumpType, uint32_t dumpId, uint64_t size)
+void Manager::notify(uint32_t dumpId, uint64_t size)
{
- if (dumpType != NewDump::DumpType::System)
- {
- log<level::ERR>("Only system dump is supported",
- entry("DUMPTYPE=%d", dumpType));
- return;
- }
// Get the timestamp
std::time_t timeStamp = std::time(nullptr);