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

bsp_diff cleanup to mixin #2080

Conversation

haochuan1982
Copy link
Contributor

No description provided.

xinxin-wan and others added 8 commits October 9, 2024 22:12
Add the new input routing for the second microphone.

Test done:
Use the first mic for google assistant, and the second mic for
normal recording, both two microphones can work together.
The application used in the test: PSI recorder & KITCHENSINK.

Tracked-On: OAM-121996
Signed-off-by: Wan, Xinxin <[email protected]>
To list the Media volume and call volume separately in the sound
settings, to provide the separate control of volume

Tests: Verified by opening the settings->sound and could see the media
volume listed.

Tracked-On: OAM-122694
Signed-off-by: Deepa K G <[email protected]>
To list the Media volume and call volume separately in the sound
settings, to provide the separate control of volume

Tests: Verified by opening the settings->sound and could see the media
volume listed.

Tracked-On: OAM-122694
Signed-off-by: Deepa K G <[email protected]>
Camera app capture requires Built-In Back Mic. Add the Built-In Back Mic
support.

Test done:
Recording and playback via PSI
Recording via camera

Tracked-On: OAM-123095
Signed-off-by: Deepa K G <[email protected]>
Signed-off-by: sgnanase <[email protected]>
Signed-off-by: Wan, Xinxin <[email protected]>
Signed-off-by: Libin Yang <[email protected]>
This patch adds changes to keep alarm, notification
and system sounds also in the media group. Since
there are no seperate pcm devices to handle these,
there is delay while switching from one context to
other as it tries to close same pcm device and
open same pcm device again.

Test done:
A2dp playback with alarm

Tracked-On: OAM-123332
Signed-off-by: padmashree mandri <[email protected]>
This patch adds changes to keep alarm, notification
and system sounds also in the media group. Since
there are no seperate pcm devices to handle these,
there is delay while switching from one context to
other as it tries to close same pcm device and
open same pcm device again.

Test done:
A2dp playback with alarm

Tracked-On: OAM-123332
Signed-off-by: padmashree mandri <[email protected]>
Supporting zone3 and zone4 in audio_policy_xxx and
car_audio_configuration.xml

Tracked-On: OAM-124414
Signed-off-by: Libin Yang <[email protected]>
Supporting zone3 and zone4 in audio_hal_configuration.xml

Tracked-On: OAM-124414
Signed-off-by: Libin Yang <[email protected]>
@sysopenci sysopenci added Pending Developer Approval Pending Developer Approval Valid commit message Pending PR Review Pending PR Review Engineering Build Not Started Engineering Build Not Started labels Oct 9, 2024
@haochuan1982 haochuan1982 changed the title U mr0 master 2nd bsp patch bsp_diff cleanup to mixin Oct 9, 2024
Copy link
Contributor

@dyang23 dyang23 left a comment

Choose a reason for hiding this comment

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

LGTM

@sysopenci sysopenci added Developer Approved Pending Developer Approval Pending Developer Approval and removed Pending Developer Approval Pending Developer Approval Developer Approved labels Oct 10, 2024
padmashree9110 and others added 2 commits October 11, 2024 07:34
This patch adds changes to keep navigation and voice_out
also in the media group. Since there are no seperate pcm
devices to handle these, there is delay while switching
from one context to other as it tries to close same pcm
device and open same pcm device again.

Signed-off-by: padmashree mandri <[email protected]>
Tests done:
1. Media playback and capture
2. Video playback
3. HFP call

Tracked-On: OAM-125159
Signed-off-by: Haochuan Z Chen <[email protected]>
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

2 similar comments
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

FAILURE: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

1 similar comment
@sysopenci
Copy link

FAILURE: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful and removed Engineering Build Failed labels Oct 11, 2024
Copy link

@balajim001 balajim001 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@dyang23 dyang23 left a comment

Choose a reason for hiding this comment

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

LGTM

@sysopenci
Copy link

Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci merged commit b0d8f69 into projectceladon:celadon/u/mr0/master Oct 11, 2024
19 checks passed
@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon_umr0_master-merge/168

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants