chameleon_audio: cleanup USB test |
||||
Issue descriptionA fail in audio_AudioBasicUSBPlaybackRecord.suspend https://wmatrix.googleplex.com/testrun/unfiltered?test_ids=306904159 caused the failure on later test: audio_AudioVolume.speaker https://wmatrix.googleplex.com/testrun/unfiltered?test_ids=306922782 because USB is still plugged. In the start of the test, we call Reset on chameleond, it unplugs USB on chameleon side. However, on DUT side, it needs to re-enumerate USB to detect that USB is unplugged. We can do that in usb_facade_native.py in the init of USBFacadeNative. That way we can make sure USB is re-enumerated again in the start of the test.
,
Jun 7 2016
,
Jun 13 2016
I found that kernel 4.2 dwc2 driver implements disconnect function correctly. It controls software pull-up on USB port so Cros side can actually see the USB is unplugged. Since we are going to move to kernel 4.2 for bluetooth HID and for suspend/resume issue 618175, this issue will not be a problem anymore. Also, if kernel 4.2 runs fine, we can clean up usb facade which resets drivers of USB controller on Cros side to re-enumerate USB devices.
,
Sep 2 2016
Now that we have deployed kernel 4.2 to chameleon in the lab, it is time to cleanup the usb test. It will be helpful to do usb plug/unplug stress test.
,
Sep 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/52bfcdc9527192e527febe7eef2e2530a0000517 commit 52bfcdc9527192e527febe7eef2e2530a0000517 Author: Cheng-Yi Chiang <cychiang@chromium.org> Date: Fri Sep 02 08:07:09 2016 [autotest] audio_widget: Remove USBPlugHandler Since Chameleon with 4.2 kernel supports pull-up resistor control on USB port, we don't need USBPlugHandler to control USB facade to reset USB controller or bind/unbind driver. Remove USBPlugHandler and use DummyPlugHandler for USB widget link. The USB facade is not removed since it might be useful to control USB controller in other use cases. BUG= chromium:616712 TEST=run audio_AudioBasicUSBPlayback and audio_AudioBasicUSBRecord test on samus. Change-Id: I543736017a4b8df75499534e87ac431b914972e1 Reviewed-on: https://chromium-review.googlesource.com/380442 Commit-Ready: Cheng-Yi Chiang <cychiang@chromium.org> Tested-by: Cheng-Yi Chiang <cychiang@chromium.org> Reviewed-by: Wai-Hong Tam <waihong@google.com> [modify] https://crrev.com/52bfcdc9527192e527febe7eef2e2530a0000517/client/cros/chameleon/audio_widget_link.py [modify] https://crrev.com/52bfcdc9527192e527febe7eef2e2530a0000517/client/cros/chameleon/audio_widget.py [modify] https://crrev.com/52bfcdc9527192e527febe7eef2e2530a0000517/client/cros/chameleon/chameleon_audio_helper.py
,
Oct 20 2016
Old issues. New issues will be filed if tests fail. |
||||
►
Sign in to add a comment |
||||
Comment 1 by ka...@chromium.org
, Jun 2 2016