-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: remove clx he loopback sub command #3059
Conversation
Pull Request Test Coverage Report for Build 7227872530
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the commit message as to why loopback sub command is being removed.
Ananda: updated commit meassge with comments |
I don't see "CXL host exerciser doesn’t support loopback feature, so remove lpbk sub command" in the commit message. I even pulled your branch locally and checked. |
CXL host exerciser doesn’t support loopback feature, so remove lpbk sub command Signed-off-by: anandaravuri <ananda.ravuri@intel.com>
### Description A recent checkin began relying on sys/pci.h for its definition of PCI_STD_NUM_BARS. It seems that some distributions don't have a version of that file that defines PCI_STD_NUM_BARS. This change defines it to 6 when not found. ### Collateral (docs, reports, design examples, case IDs): - [ ] Document Update Required? (Specify FIM/AFU/Scripts) ### Tests added: ### Tests run: CI Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
16fb491
to
b1d1182
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updated message.
CXL host exerciser doesn’t support loopback feature, so remove lpbk sub command