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

[audio] Avoid taking over phone audio path option #38

Closed

Conversation

htruong
Copy link
Contributor

@htruong htruong commented Mar 10, 2018

Normally the Android auto head unit would take over the phone's audio.
In many cases, this is undesirable because the existing Bluetooth
connection in the phone with the car stereo might do a better job of
handling audio/media control. This patch includes the option to make
the head unit to be as close to a screen-only device as possible.

When the option is enabled,
OpenAuto will not take over media and navigation voice channel.

@htruong htruong force-pushed the do_not_interfere_for_f1xpl branch 2 times, most recently from a8ca32f to 113a45d Compare March 10, 2018 23:18
Normally the Android auto head unit would take over the phone's audio.
In many cases this is undesirable because the existing bluetooth
connection in the phone with the car stereo might do a better job of
handling audio/media control. This patch includes the option to make
the head unit to be as close to a screen-only device as possible.

When the option is enabled,
OpenAuto will not take over media and navigation voice channel.
@htruong htruong force-pushed the do_not_interfere_for_f1xpl branch from 113a45d to ef4efbc Compare March 10, 2018 23:20
@htruong
Copy link
Contributor Author

htruong commented Mar 10, 2018

@f1xpl I have fixed the "on by default" problem, it is now off by default, unless the user opt to turn it on.

@f1xpl
Copy link
Owner

f1xpl commented Mar 11, 2018

Based on your pull request I made changes to allow disable/enable music and speech audio channels separately.

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.

2 participants