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

Add motorola ocean #3

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

ultra-azu
Copy link
Member

No description provided.

Comment "Earpiece playback"

ConflictingDevice [
"Headphones"
Copy link
Member

Choose a reason for hiding this comment

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

Quinary devices should not conflict with Primary devices

Comment "Headphones playback"

ConflictingDevice [
"Earpiece"
Copy link
Member

Choose a reason for hiding this comment

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

Quinary devices should not conflict with Primary devices

"Mic2"
]

Include.smes.File "/codecs/msm8953-wcd/HeadsetMicEnableSeq.conf"
Copy link
Member

Choose a reason for hiding this comment

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

Check #5

}
}

#Speaker is also the Earpiece
Copy link
Member

Choose a reason for hiding this comment

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

If speaker and earpiece is same than you should keep Speaker, and remove Earpiece i don't see any advantage of keep both of them.

Copy link

Choose a reason for hiding this comment

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

Also see #8

CapturePCM "hw:${CardId},1"
CaptureChannels 2
CapturePriority 400
JackControl "Headset Jack"

Choose a reason for hiding this comment

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

Suggested change
JackControl "Headset Jack"
JackControl "Mic Jack"

PlaybackPCM "hw:${CardId},0"
PlaybackPriority 300
PlaybackChannels 2
JackControl "Headset Jack"

Choose a reason for hiding this comment

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

Suggested change
JackControl "Headset Jack"
JackControl "Headphone Jack"

@barni2000
Copy link
Member

On motorola-potter has one speaker for earpiece and speaker but both codec is connected to it, for higher volume it using speaker codec for voice calls it uses different path. Is not wcd codec used for earpiece?

]

Value {
PlaybackPCM "hw:${CardId},2"
Copy link
Member

Choose a reason for hiding this comment

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

I think it should be 0

@barni2000 barni2000 force-pushed the master branch 2 times, most recently from 6b5b7a5 to 20ccb0c Compare January 25, 2025 22:12
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