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

alcatel-idol3: Add headphones support #8

Merged
merged 7 commits into from
Mar 9, 2022

Conversation

vknecht
Copy link
Member

@vknecht vknecht commented Sep 20, 2021

No description provided.

@stephan-gh
Copy link
Member

I'd like to see alcatel-idol347 first to be honest. :)

@stephan-gh
Copy link
Member

Oh it's the same I guess. Nevermind :D

ucm2/alcatel-idol3/HiFi.conf Outdated Show resolved Hide resolved
ucm2/alcatel-idol3/HiFi.conf Outdated Show resolved Hide resolved
Add variable support for MultiMedia1, MultiMedia2 and MultiMedia4 ports.
Keep default values for those respectively at:
- PRI_MI2S_RX
- TERT_MI2S_TX
- PRI_MI2S_RX
which is the usual usecase for most msm8916 phones.
idol3 and idol347 use SEC_MI2S_RX port instead of PRI_MI2S_RX.
Also set MultiMedia4 to QUAT_MI2S_RX instead of PRI_MI2S_RX.
Make use of related UCM variables to override default values.
Add support for VoiceRXPort and VoiceTXPort variables.
Keep default values for those respectively at:
- PRI_MI2S_RX
- TERT_MI2S_TX
which is the usual usecase for most msm8916 phones

Those will be used as ALSA q6voice-dai controls.
Add support for setting q6voice-dai VoiceRXPort and VoiceTXPort controls
Keep default values for those respectively at:
- PRI_MI2S_RX
- TERT_MI2S_TX
which is the usual usecase for most msm8916 phones
idol3 and idol347 use SEC_MI2S_RX instead of PRI_MI2S_RX port.
Set this value through VoiceRXPort variable.
Use generic pin switch mechanism instead of a tfa9897 specific control.
@stephan-gh stephan-gh merged commit 8d49bb1 into msm8916-mainline:master Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants