From 5dee762fe74f1c338aa191032a9fb48ea337d60e Mon Sep 17 00:00:00 2001 From: Manoj Takasi <133196374+ManojTakasi@users.noreply.github.com> Date: Mon, 3 Jun 2024 17:30:28 +0530 Subject: [PATCH] Removed m_device in the xbutil advanced command since its mentioned in (#8203) the sub commands Signed-off-by: Manoj Takasi Co-authored-by: Manoj Takasi --- src/runtime_src/core/tools/xbutil2/OO_AieClockFreq.cpp | 4 ++-- src/runtime_src/core/tools/xbutil2/OO_AieRegRead.cpp | 4 ++-- src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.cpp | 3 +-- src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.h | 1 - 4 files changed, 5 insertions(+), 7 deletions(-) diff --git a/src/runtime_src/core/tools/xbutil2/OO_AieClockFreq.cpp b/src/runtime_src/core/tools/xbutil2/OO_AieClockFreq.cpp index 3eaee7e79f6..cadc2abdd0e 100644 --- a/src/runtime_src/core/tools/xbutil2/OO_AieClockFreq.cpp +++ b/src/runtime_src/core/tools/xbutil2/OO_AieClockFreq.cpp @@ -118,8 +118,8 @@ OO_AieClockFreq::execute(const SubCmdOptions& _options) const po::variables_map vm; process_arguments(vm, _options); - // Exit if neither action or device specified - if(m_help || m_device.empty()) { + // Exit if action is specified + if(m_help) { printHelp(); return; } diff --git a/src/runtime_src/core/tools/xbutil2/OO_AieRegRead.cpp b/src/runtime_src/core/tools/xbutil2/OO_AieRegRead.cpp index a02bb029878..a42db0ca739 100644 --- a/src/runtime_src/core/tools/xbutil2/OO_AieRegRead.cpp +++ b/src/runtime_src/core/tools/xbutil2/OO_AieRegRead.cpp @@ -413,8 +413,8 @@ OO_AieRegRead::execute(const SubCmdOptions& _options) const po::variables_map vm; process_arguments(vm, _options); - // Exit if neither action or device specified - if(m_help || m_device.empty()) { + // Exit if action is specified + if(m_help) { printHelp(); return; } diff --git a/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.cpp b/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.cpp index 5ad9a0b6988..8b6ed13f3bf 100644 --- a/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.cpp +++ b/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.cpp @@ -44,7 +44,6 @@ SubCmdAdvanced::SubCmdAdvanced(bool _isHidden, bool _isDepricated, bool _isPreli setIsPreliminary(_isPreliminary); m_commonOptions.add_options() - ("device,d", boost::program_options::value(&m_device), "The Bus:Device.Function (e.g., 0000:d8:00.0) device of interest") ("help", boost::program_options::bool_switch(&m_help), "Help to use this sub-command") ; @@ -77,7 +76,7 @@ SubCmdAdvanced::execute(const SubCmdOptions& _options) const // No suboption print help if (!optionOption) { - printHelp(false, "", XBU::get_device_class(m_device, true)); + printHelp(); return; } diff --git a/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.h b/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.h index b34dc60a72c..870454e6676 100644 --- a/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.h +++ b/src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.h @@ -17,7 +17,6 @@ class SubCmdAdvanced : public SubCmd { private: bool m_help; - std::string m_device; }; #endif