-
Notifications
You must be signed in to change notification settings - Fork 59
test: add test jack detection if DSP is suspended #1321
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
test: add test jack detection if DSP is suspended #1321
Conversation
50eabd5 to
e548c1d
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.
Pull Request Overview
Adds a new test case to verify jack detection functionality when the DSP is in D3 suspended state. The test uses a USB relay to simulate physical jack plug/unplug events and validates that both jack detection status and DSP power state changes are handled correctly.
- Implements automated jack detection testing using USB relay hardware
- Validates DSP state transitions during jack events while suspended
- Includes comprehensive error handling and configuration options
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| # shellcheck disable=SC1091 source=case-lib/lib.sh | ||
| source "${TESTLIB}/lib.sh" | ||
| # shellcheck disable=SC1091 source=case-lib/relay.sh |
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.
| # shellcheck disable=SC1091 source=case-lib/relay.sh | |
| # shellcheck source=case-lib/relay.sh |
When someone runs shellcheck outside the top directory, you do want shellcheck to warn them that it cannot find this file.
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.
@marc-hb I applied same as for lib.sh
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.
I commented only once but I think disable=SC1091 should be removed in both cases.
- Either people run shellcheck -x from the top-level directory as they should. in that case
disable=SC1091is not needed because shellcheck successfully includes the files. - Or, people forget
-xor run shellcheck from a different directory. Then, you want the first warning (out of many other warnings) to be "could not find include files". So, you don't wantdisable=SC1091there either because you don't want to lose the most important warning.
So you never want disable=SC1091
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 Marc, I agree
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.
Afraid disable=SC1091 came back :-(
It's in two other files as well.
d676234 to
fd03264
Compare
fd03264 to
04418c9
Compare
redzynix
left a comment
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.
Some minor changes needed
bfa8930 to
df930dd
Compare
Add a new test case to test jack detection during DSP is in D3 state. The test will check if the jack detection status is updated correctly when the jack is plugged in and unplugged and also if DSP status is changing correctly as well. For unplug/plug headset jack is using a USB relay. https://github.com/darrylb123/usbrelay Signed-off-by: Artur Wilczak <arturx.wilczak@intel.com>
df930dd to
1a2f2a7
Compare
Add a new test case to test jack detection during DSP is in D3 state.
The test will check if the jack detection status is updated correctly when the jack is plugged in and unplugged and also if DSP status is changing correctly as well.
For unplug/plug headset jack is using a USB relay.
https://github.com/darrylb123/usbrelay