New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 701131 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[Enguarde/Squawks] audio can be heard via both 3.5mm headphone & speaker at the same time

Project Member Reported by helenzhang@google.com, Mar 13 2017

Issue description

Chrome OS :9334.9.0, 58.0.3029.18
Kernel version: 4.4.44-07310-ga1da0ad

What steps will reproduce the problem?
( not sure about exact steps) 
(1) Sign in 
(2) Please any audio via speaker 
(3) Plug in usb headphone 
(4) Plug in 3.5 mm headphone 
(5) Unplug usb headphone 

What is the expected result?
Audio should not play via both 3.5mm headphone or speaker  

What happens instead?
Audio plays via both 3.5mm headphone and speaker 

cras_test_client --dump_s and audio setting show speaker is selected 

Always reproducible if it happens. Reboot will not resolve the issue even if unplug 3.5mm headphone before reboot and plug afterwards. 
 
Cc: cychiang@chromium.org rjahagir@chromium.org ka...@chromium.org pgangishetty@chromium.org sontis@chromium.org
Summary: [Enguarde] audio can be heard via both 3.5mm headphone & speaker at the same time (was: [Enguarde] audio can be heard via both 3.5mm headphone & speaker)
crash log, system log and screenshot:
https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cr/701131/

similar like the  issue 681891 . 
Cc: jen...@chromium.org
Owner: cychiang@chromium.org
Hi Helen, can this be reproduced on Squawks ?
What was the reproduce rate ?
Thanks!
could you also provide audio_diagnostic result when the issue happens ?
generate_logs is broken.
Labels: -proj-enguarde Proj-Rambi
Summary: [Enguarde/Squawks] audio can be heard via both 3.5mm headphone & speaker at the same time (was: [Enguarde] audio can be heard via both 3.5mm headphone & speaker at the same time)
Hi Jimmy, it's reproducible in squawks too. 

Steps: 
(1) Sign in 
(2) Play any audio via speaker 
(3) Manually click 'Speaker(internal)' at audio uebertray 
    ( looks like this is the step caused the issue) 
(4) Plug in usb headphone 
(5) Plug in 3.5 mm headphone 
(6) Unplug usb headphone 

audio_diagnostic_result file is at:
https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cr/701131/

Swanky has the same issue with similar steps. 
Owner: hychao@chromium.org
Status: Started (was: Untriaged)
We repro the problem.
Thanks Helen!

Comment 8 by hychao@chromium.org, Mar 15 2017

After some test work, I found this is a problem in machine driver. Probably another issue when we move kernel 3.10 -> 4.4

stop cras; stop ui;

localhost ~ # amixer -c1 cset numid=69 off
numid=69,iface=MIXER,name='Headphone Switch'
  ; type=BOOLEAN,access=rw------,values=1
  : values=off

// plug 3.5mm jack

localhost ~ # amixer -c1 cget numid=69
numid=69,iface=MIXER,name='Headphone Switch'
  ; type=BOOLEAN,access=rw------,values=1
  : values=on



Comment 9 by hychao@chromium.org, Mar 15 2017

Labels: M-57
Fix uploaded:

https://chromium-review.googlesource.com/455896 audio-configs: Fix byt-max98090 UCM for headphone switch        

We'll need to cherry-pick this for a few boards on M57.
Project Member

Comment 10 by bugdroid1@chromium.org, Mar 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/48919e119639d8dd96d401822cff4e8fe259ee88

commit 48919e119639d8dd96d401822cff4e8fe259ee88
Author: Hsin-Yu Chao <hychao@chromium.org>
Date: Wed Mar 15 19:39:23 2017

audio-configs: Fix byt-max98090 UCM for headphone switch

The 'Headphone Switch' will turn on/off by machine driver when
jack is plugged/unplugged. We need to use 'Headphone left/right
Switch' to actually mute/unmute the headphone node.

BUG= chromium:701131 
TEST=On Squawks, plug 3.5mm headphone, select to Speaker and
play YouTube. Unplug headphone then re-plug it, verify speaker
is still the active output node and no audio comming out from
headphone.

Change-Id: I88b3d1588cd60ba8d0cd50550d19581c04e55de5
Reviewed-on: https://chromium-review.googlesource.com/455896
Commit-Ready: Hsinyu Chao <hychao@chromium.org>
Tested-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>

[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-kip/chromeos-base/chromeos-bsp-kip/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-glimmer/chromeos-base/chromeos-bsp-glimmer/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-orco/chromeos-base/chromeos-bsp-orco/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-winky/chromeos-base/chromeos-bsp-winky/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-expresso/chromeos-base/chromeos-bsp-expresso/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-swanky/chromeos-base/chromeos-bsp-swanky/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-ninja/chromeos-base/chromeos-bsp-ninja/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-banjo/chromeos-base/chromeos-bsp-banjo/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-rambi/chromeos-base/chromeos-bsp-rambi/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-sumo/chromeos-base/chromeos-bsp-sumo/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-clapper/chromeos-base/chromeos-bsp-clapper/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-quawks/chromeos-base/chromeos-bsp-quawks/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-heli/chromeos-base/chromeos-bsp-heli/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-candy/chromeos-base/chromeos-bsp-candy/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-enguarde/chromeos-base/chromeos-bsp-enguarde/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-squawks/chromeos-base/chromeos-bsp-squawks/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/48919e119639d8dd96d401822cff4e8fe259ee88/overlay-gnawty/chromeos-base/chromeos-bsp-gnawty/files/audio-config/ucm-config/byt-max98090/HiFi.conf

Comment 11 by ka...@chromium.org, Mar 15 2017

Labels: -ReleaseBlock-Beta ReleaseBlock-Stable
Cc: bhthompson@chromium.org
Labels: Merge-Approved-58
Prepared https://chromium-review.googlesource.com/#/c/455758/ for M58

After verified the #10 fix in canary, I'll cherry-pick to M58.

Comment 15 by ketakid@google.com, Mar 16 2017

Labels: Merge-Approved-57
Please merge to M57 after M58. dgreid@ mentioned this doesn't include the ebuild changes needed. Will you be updating the CL?
Yes. the Tot ebuild change: https://chromium-review.googlesource.com/#/c/455678/
I verified this on ChromeOS-test-R59-9373.0.0. Will proceed merge for M58 and 57

However with this fixed, I observe another issue. Filed at  crbug.com/702468 
Project Member

Comment 18 by bugdroid1@chromium.org, Mar 17 2017

Labels: merge-merged-release-R58-9334.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/c995aeff0d7da9925ad51f34094628c968b87327

commit c995aeff0d7da9925ad51f34094628c968b87327
Author: Hsin-Yu Chao <hychao@chromium.org>
Date: Fri Mar 17 03:12:13 2017

audio-configs: Fix byt-max98090 UCM for headphone switch

The 'Headphone Switch' will turn on/off by machine driver when
jack is plugged/unplugged. We need to use 'Headphone left/right
Switch' to actually mute/unmute the headphone node.

BUG= chromium:701131 
TEST=On Squawks, plug 3.5mm headphone, select to Speaker and
play YouTube. Unplug headphone then re-plug it, verify speaker
is still the active output node and no audio comming out from
headphone.

Change-Id: I88b3d1588cd60ba8d0cd50550d19581c04e55de5
Reviewed-on: https://chromium-review.googlesource.com/455896
Commit-Ready: Hsinyu Chao <hychao@chromium.org>
Tested-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
(cherry picked from commit 48919e119639d8dd96d401822cff4e8fe259ee88)
Reviewed-on: https://chromium-review.googlesource.com/455758
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
Commit-Queue: Hsinyu Chao <hychao@chromium.org>

[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-ninja/chromeos-base/chromeos-bsp-ninja/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-winky/chromeos-base/chromeos-bsp-winky/chromeos-bsp-winky-0.0.1-r17.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-kip/chromeos-base/chromeos-bsp-kip/chromeos-bsp-kip-0.0.1-r18.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-quawks/chromeos-base/chromeos-bsp-quawks/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-rambi/chromeos-base/chromeos-bsp-rambi/chromeos-bsp-rambi-0.0.1-r21.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-swanky/chromeos-base/chromeos-bsp-swanky/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-heli/chromeos-base/chromeos-bsp-heli/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-sumo/chromeos-base/chromeos-bsp-sumo/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-glimmer/chromeos-base/chromeos-bsp-glimmer/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-orco/chromeos-base/chromeos-bsp-orco/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-expresso/chromeos-base/chromeos-bsp-expresso/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-clapper/chromeos-base/chromeos-bsp-clapper/chromeos-bsp-clapper-0.0.1-r23.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-clapper/chromeos-base/chromeos-bsp-clapper/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-swanky/chromeos-base/chromeos-bsp-swanky/chromeos-bsp-swanky-0.0.1-r21.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-candy/chromeos-base/chromeos-bsp-candy/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-quawks/chromeos-base/chromeos-bsp-quawks/chromeos-bsp-quawks-0.0.1-r15.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-winky/chromeos-base/chromeos-bsp-winky/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-squawks/chromeos-base/chromeos-bsp-squawks/chromeos-bsp-squawks-0.0.1-r18.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-candy/chromeos-base/chromeos-bsp-candy/chromeos-bsp-candy-0.0.1-r14.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-heli/chromeos-base/chromeos-bsp-heli/chromeos-bsp-heli-0.0.1-r9.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-enguarde/chromeos-base/chromeos-bsp-enguarde/chromeos-bsp-enguarde-0.0.1-r16.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-ninja/chromeos-base/chromeos-bsp-ninja/chromeos-bsp-ninja-0.0.1-r9.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-orco/chromeos-base/chromeos-bsp-orco/chromeos-bsp-orco-0.0.1-r9.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-expresso/chromeos-base/chromeos-bsp-expresso/chromeos-bsp-expresso-0.0.1-r12.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-kip/chromeos-base/chromeos-bsp-kip/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-gnawty/chromeos-base/chromeos-bsp-gnawty/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-banjo/chromeos-base/chromeos-bsp-banjo/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-sumo/chromeos-base/chromeos-bsp-sumo/chromeos-bsp-sumo-0.0.1-r13.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-glimmer/chromeos-base/chromeos-bsp-glimmer/chromeos-bsp-glimmer-0.0.1-r25.ebuild
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-gnawty/chromeos-base/chromeos-bsp-gnawty/chromeos-bsp-gnawty-0.0.1-r15.ebuild
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-rambi/chromeos-base/chromeos-bsp-rambi/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-enguarde/chromeos-base/chromeos-bsp-enguarde/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[modify] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-squawks/chromeos-base/chromeos-bsp-squawks/files/audio-config/ucm-config/byt-max98090/HiFi.conf
[rename] https://crrev.com/c995aeff0d7da9925ad51f34094628c968b87327/overlay-banjo/chromeos-base/chromeos-bsp-banjo/chromeos-bsp-banjo-0.0.1-r11.ebuild

Project Member

Comment 19 by bugdroid1@chromium.org, Mar 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/4c9ff46c2f3e9872221519a9c686a0e16b0d4663

commit 4c9ff46c2f3e9872221519a9c686a0e16b0d4663
Author: Hsin-Yu Chao <hychao@chromium.org>
Date: Fri Mar 17 05:28:21 2017

audio-configs: Uprev ebuild version for rambi series

So the UCM changes in commit 3ea52b2c can be built.

BUG= chromium:701131 
TEST=None

Change-Id: I7a43bc30059b528c98d7474646818ae4b5a573d9
Reviewed-on: https://chromium-review.googlesource.com/455678
Commit-Ready: Hsinyu Chao <hychao@chromium.org>
Tested-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>

[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-quawks/chromeos-base/chromeos-bsp-quawks/chromeos-bsp-quawks-0.0.1-r15.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-squawks/chromeos-base/chromeos-bsp-squawks/chromeos-bsp-squawks-0.0.1-r18.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-orco/chromeos-base/chromeos-bsp-orco/chromeos-bsp-orco-0.0.1-r9.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-candy/chromeos-base/chromeos-bsp-candy/chromeos-bsp-candy-0.0.1-r14.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-banjo/chromeos-base/chromeos-bsp-banjo/chromeos-bsp-banjo-0.0.1-r11.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-swanky/chromeos-base/chromeos-bsp-swanky/chromeos-bsp-swanky-0.0.1-r21.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-heli/chromeos-base/chromeos-bsp-heli/chromeos-bsp-heli-0.0.1-r9.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-kip/chromeos-base/chromeos-bsp-kip/chromeos-bsp-kip-0.0.1-r18.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-clapper/chromeos-base/chromeos-bsp-clapper/chromeos-bsp-clapper-0.0.1-r23.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-rambi/chromeos-base/chromeos-bsp-rambi/chromeos-bsp-rambi-0.0.1-r21.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-glimmer/chromeos-base/chromeos-bsp-glimmer/chromeos-bsp-glimmer-0.0.1-r25.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-enguarde/chromeos-base/chromeos-bsp-enguarde/chromeos-bsp-enguarde-0.0.1-r16.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-winky/chromeos-base/chromeos-bsp-winky/chromeos-bsp-winky-0.0.1-r17.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-gnawty/chromeos-base/chromeos-bsp-gnawty/chromeos-bsp-gnawty-0.0.1-r15.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-ninja/chromeos-base/chromeos-bsp-ninja/chromeos-bsp-ninja-0.0.1-r9.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-expresso/chromeos-base/chromeos-bsp-expresso/chromeos-bsp-expresso-0.0.1-r12.ebuild
[rename] https://crrev.com/4c9ff46c2f3e9872221519a9c686a0e16b0d4663/overlay-sumo/chromeos-base/chromeos-bsp-sumo/chromeos-bsp-sumo-0.0.1-r13.ebuild

hychao@ please merge this to M57 today if possible. We will be kicking off the stable RC build this weekend.
Labels: -Merge-Approved-57 -Merge-Approved-58
Status: Fixed (was: Started)
This should be fixed in M58 and M57 now, per 702468

Comment 22 by son...@google.com, Mar 20 2017

Status: Verified (was: Fixed)
Verified on build 9202.54.0

Sign in to add a comment