probe: remove ".*.auto" suffix in probed result |
||||||||||
Issue descriptionThe audio codec driver might add ".*.auto" suffix in the probed result. That confused us because the same component might return the different probed result. Since the suffix is not related to the hardware component, we remove it in the probed result.
,
Nov 16 2016
This should be a quick fix, right
,
Nov 17 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/factory/+/48650880e4309727bd861821432c9a710f6a4cc7 commit 48650880e4309727bd861821432c9a710f6a4cc7 Author: Chih-Yu Huang <akahuang@google.com> Date: Thu Nov 17 09:57:08 2016 gooftool: Remove ".*.auto" suffix in probe result. When the platform add the device, it would add the .*.auto suffix to the device name. It causes the probe module gets the different value form the samecomponent. Therefore we remove the auto-generated suffix. BUG= chromium:659921 TEST=py/gooftool/probe_unittest.py Change-Id: I6ff55f8dfaa5831afd596420a384b0b2b92f9c8c Reviewed-on: https://chromium-review.googlesource.com/411633 Commit-Ready: Chih-Yu Huang <akahuang@chromium.org> Tested-by: Chih-Yu Huang <akahuang@chromium.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org> [modify] https://crrev.com/48650880e4309727bd861821432c9a710f6a4cc7/py/gooftool/probe_unittest.py [modify] https://crrev.com/48650880e4309727bd861821432c9a710f6a4cc7/py/gooftool/probe.py
,
Nov 18 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/factory/+/d8b3c0edb2b611007860cd21806dee12a5d9bc05 commit d8b3c0edb2b611007860cd21806dee12a5d9bc05 Author: Chih-Yu Huang <akahuang@google.com> Date: Thu Nov 17 09:57:08 2016 CHERRY-PICK: gooftool: Remove ".*.auto" suffix in probe result. When the platform add the device, it would add the .*.auto suffix to the device name. It causes the probe module gets the different value form the samecomponent. Therefore we remove the auto-generated suffix. BUG= chromium:659921 ,chrome-os-partner:58950 TEST=py/gooftool/probe_unittest.py Change-Id: Ie3a6b02892deb406609e2934c0891e472cafcb3e Original-Change-Id: I6ff55f8dfaa5831afd596420a384b0b2b92f9c8c Reviewed-on: https://chromium-review.googlesource.com/411633 Commit-Ready: Chih-Yu Huang <akahuang@chromium.org> Tested-by: Chih-Yu Huang <akahuang@chromium.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/412448 Reviewed-by: Chih-Yu Huang <akahuang@chromium.org> Commit-Queue: Chih-Yu Huang <akahuang@chromium.org> [modify] https://crrev.com/d8b3c0edb2b611007860cd21806dee12a5d9bc05/py/gooftool/probe_unittest.py [modify] https://crrev.com/d8b3c0edb2b611007860cd21806dee12a5d9bc05/py/gooftool/probe.py
,
Nov 22 2016
Fixed at comment 3
,
Feb 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/factory/+/05d23d254c49299c46a3ae96d9f0933ea55fc6e3 commit 05d23d254c49299c46a3ae96d9f0933ea55fc6e3 Author: Chih-Yu Huang <akahuang@google.com> Date: Fri Feb 10 05:09:56 2017 gooftool: Remove ".*.auto" suffix in probe result. When the platform add the device, it would add the .*.auto suffix to the device name. It causes the probe module gets the different value form the samecomponent. Therefore we remove the auto-generated suffix. BUG= chromium:659921 TEST=py/gooftool/probe_unittest.py Change-Id: I6ff55f8dfaa5831afd596420a384b0b2b92f9c8c Reviewed-on: https://chromium-review.googlesource.com/411633 Commit-Ready: Chih-Yu Huang <akahuang@chromium.org> Tested-by: Chih-Yu Huang <akahuang@chromium.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/440707 Commit-Queue: Hung-Te Lin <hungte@chromium.org> Tested-by: Hung-Te Lin <hungte@chromium.org> [modify] https://crrev.com/05d23d254c49299c46a3ae96d9f0933ea55fc6e3/py/gooftool/probe_unittest.py [modify] https://crrev.com/05d23d254c49299c46a3ae96d9f0933ea55fc6e3/py/gooftool/probe.py
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by cychiang@chromium.org
, Oct 27 2016