Skip to content

Commit

Permalink
Removed m_device in the xbutil advanced command since its mentioned in (
Browse files Browse the repository at this point in the history
#8203) (#8303)

the sub commands

Signed-off-by: Manoj Takasi <[email protected]>
Co-authored-by: Manoj Takasi <[email protected]>
(cherry picked from commit 5dee762)
  • Loading branch information
ManojTakasi authored Jul 24, 2024
1 parent cb8b00d commit 6990cad
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 7 deletions.
4 changes: 2 additions & 2 deletions src/runtime_src/core/tools/xbutil2/OO_AieClockFreq.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down
4 changes: 2 additions & 2 deletions src/runtime_src/core/tools/xbutil2/OO_AieRegRead.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down
3 changes: 1 addition & 2 deletions src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ SubCmdAdvanced::SubCmdAdvanced(bool _isHidden, bool _isDepricated, bool _isPreli
setIsPreliminary(_isPreliminary);

m_commonOptions.add_options()
("device,d", boost::program_options::value<decltype(m_device)>(&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")
;

Expand Down Expand Up @@ -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;
}

Expand Down
1 change: 0 additions & 1 deletion src/runtime_src/core/tools/xbutil2/SubCmdAdvanced.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ class SubCmdAdvanced : public SubCmd {

private:
bool m_help;
std::string m_device;
};

#endif
Expand Down

0 comments on commit 6990cad

Please sign in to comment.