Skip to content

Conversation

@fredoh9
Copy link
Contributor

@fredoh9 fredoh9 commented May 8, 2024

LNL RVP keep toggling between headphone and headset. Overrides to headset only for the test.

Related kernel PR: thesofproject/linux#4969

@fredoh9 fredoh9 requested a review from a team as a code owner May 8, 2024 18:57
@fredoh9 fredoh9 requested review from marc-hb and plbossart May 8, 2024 18:57
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.

Is this not going to fail without thesofproject/linux#4969 ?

We want to keep a reasonable amount of backwards compatibility.

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.

Also: add a comment with the URL. Without context it's impossible to guess why this is done.

@fredoh9 fredoh9 force-pushed the fix/lnl_sdw_aioc_headset_mode branch from 0ef13de to 35751d7 Compare May 8, 2024 21:42
@fredoh9 fredoh9 requested a review from marc-hb May 8, 2024 21:42
marc-hb
marc-hb previously approved these changes May 8, 2024
@marc-hb marc-hb requested review from bardliao, kv2019i, lyakh and ujfalusi May 8, 2024 22:46
bardliao
bardliao previously approved these changes May 9, 2024
@marc-hb marc-hb marked this pull request as draft May 9, 2024 16:43
LNL RVP keep toggling between headphone and headset. Overrides to headset
only for the test.

Signed-off-by: Fred Oh <fred.oh@linux.intel.com>
@fredoh9 fredoh9 dismissed stale reviews from bardliao and marc-hb via ac6a188 May 9, 2024 16:47
@fredoh9 fredoh9 force-pushed the fix/lnl_sdw_aioc_headset_mode branch from 35751d7 to ac6a188 Compare May 9, 2024 16:47
@fredoh9 fredoh9 marked this pull request as ready for review May 9, 2024 16:52
@fredoh9 fredoh9 requested review from bardliao, marc-hb and plbossart May 9, 2024 16:53
Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

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

thanks @fredoh9

@fredoh9 fredoh9 merged commit 90e0929 into thesofproject:main May 10, 2024
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.

4 participants