Skip to content
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

alsa_settings: MTLP_RVP_SDW.sh: add rt711 GE control #1206

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

bardliao
Copy link
Contributor

MTLP_RVP_SDW uese rt711 and it needs to overwritre the jack detection type.

@bardliao bardliao requested a review from marc-hb June 18, 2024 12:56
@bardliao bardliao requested a review from a team as a code owner June 18, 2024 12:56
kv2019i
kv2019i previously approved these changes Jun 18, 2024
marc-hb
marc-hb previously approved these changes Jun 18, 2024
Copy link
Collaborator

@marc-hb marc-hb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fredoh9 ok for you?

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 18, 2024

MTL https://sof-ci.01.org/softestpr/PR1206/build523/devicetest/index.html all green but there's no MTLP_RVP_SDW model in PR testing, only in daily test runs.

CAVS25 https://sof-ci.01.org/softestpr/PR1206/build522/devicetest/index.html is totally empty.
LNL https://sof-ci.01.org/softestpr/PR1206/build521/devicetest/index.html had only two devices. Should not be related but worrying. Will re-run later when CI is less busy. EDIT: it was just busy.

cc:

@@ -1,5 +1,9 @@
set -e

# override jack detection mode to headset
# related linux pr: https://github.com/thesofproject/linux/pull/4969
amixer -c sofsoundwire cset name='rt711 GE49 Selected Mode' 2 || true
Copy link
Collaborator

@marc-hb marc-hb Jun 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just realized that this is a copy/paste of 90e0929 in LNLM_SDW_AIOC.sh

Can you please add a duplication warning at the top of the file? Something short and generic, no need to go into a detailed comparison about what is copied from LNLM_SDW_AIOC versus what has diverged. Something like: # A lot of this file is duplicated in LNLM_SDW_AIOC.sh and possibly others.

Also add the same warning at the top of LNLM_SDW_AIOC.sh, which will avoid future commits like 90e0929 missing the duplication and forgetting to update some files.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just realized that this is a copy/paste of 90e0929 in LNLM_SDW_AIOC.sh

Can you please add a duplication warning at the top of the file? Something short and generic, no need to go into a detailed comparison about what is copied from LNLM_SDW_AIOC versus what has diverged. Something like: # A lot of this file is duplicated in LNLM_SDW_AIOC.sh and possibly others.

Also add the same warning at the top of LNLM_SDW_AIOC.sh, which will avoid future commits like 90e0929 missing the duplication and forgetting to update some files.

Done

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 18, 2024

SOFCI TEST

@marc-hb
Copy link
Collaborator

marc-hb commented Jun 19, 2024

MTLP_RVP_SDW uese rt711 and it needs to overwritre the jack detection
type.

Signed-off-by: Bard Liao <[email protected]>
@bardliao bardliao dismissed stale reviews from marc-hb and kv2019i via cf5e4c1 June 19, 2024 01:25
Copy link
Collaborator

@marc-hb marc-hb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay, I missed your last force-push.

Trivial one-liner, easy to fix or revert if needed. 2 days in review, others got their chance. Let's merge this.

@marc-hb marc-hb merged commit f3109df into thesofproject:main Jun 20, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants