Kevin - Audio recorded & played through Onboard speakers is too low though volume set to the max |
|||||||||||
Issue descriptionChrome Version: 56.0.2924.101 Platform: 9000.86.0 (Official Build) stable-channel kevin ARC version 3731316 Firmware Google_Kevin.8785.149.0 What steps will reproduce the problem? (1) Record audio through Onboard microphone (used Simple Audio Recorder & Online voice recorder) (2) Play the recorded audio through the onboard speakers. What is the expected result? Should be able to hear good What happens instead? Volume is too low though set to max level Please use labels and text to provide additional information. For graphics-related bugs, please copy/paste the contents of the about:gpu page at the end of this report.
,
Feb 15 2017
It seems that ADC1 is not controlled at all. I will check it.
,
Feb 15 2017
I found the cause: When we init device for wake on device, the default system gain is overwritten from 20dB to 0dB because the range of wake on device is 0~0 dB. This is the same issue as part of issue 60744 . The fix is to store the target gain no matter what the current range is. I will fix it and cherry-pick the fix to 56 and 57 after approval.
,
Feb 15 2017
the related issue is crosbug.com/p/60744 in partner issue tracker.
,
Feb 15 2017
The fix is uploaded: https://chromium-review.googlesource.com/430527 CRAS: system_state - Store target capture gain
,
Feb 28 2017
+bhthompson The fix in #5 was just merged. I think it might be too late to merge to 56. But 57 should be ok ? Thanks!
,
Feb 28 2017
This bug requires manual review: We are only 13 days from stable. Please contact the milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 1 2017
+ketakid for 57
,
Mar 2 2017
Hi ketakid, we would like to merge CL in #5 to R57 to fix internal microphone gain too low issue. There are more user feedback complaining about this so I hope this can be fixed in R57. Thanks!
,
Mar 4 2017
IMO this is a release blocker, so I added the label in case we missed it. Hope this can be merged to R57 soon. Thanks!
,
Mar 6 2017
Has this been merged to and validated on master at least a day?
,
Mar 6 2017
Approving merge to M57 Chrome OS.
,
Mar 7 2017
Thanks! Verified on image 9344.0 on kevin and chell. I will merge the CLs.
,
Mar 7 2017
,
Mar 7 2017
,
Mar 8 2017
The internal mic test can catch this, but kevin is not running perbuild test so we did not notice it.
,
Mar 10 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 10 2017
Verified on 57.0.2987.100 beta / 9202.51.0(Official Build) beta-channel kevin
,
Mar 14 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 15 2017
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by pgangishetty@google.com
, Feb 14 2017