New issue
Advanced search Search tips

Issue 718688 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

ec: usb_updater2: Add support for serial number matching

Project Member Reported by drinkcat@chromium.org, May 5 2017

Issue description

Being able to match a specific device using serial numbers would be useful for servov4/micro updating over USB, using usb_updater2.

See https://chromium-review.googlesource.com/c/458468/11/extra/usb_updater/usb_updater2.c#310 .
 
Project Member

Comment 1 by sheriffbot@chromium.org, May 7 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Pri-2 -Hotlist-Recharge-Cold Pri-1
Owner: nsanders@chromium.org
Status: Assigned (was: Untriaged)
May need to move servo v4 to a newer branch and this would be required to make that happen. I'll take a look and fix if it's straightforward.
Project Member

Comment 3 by bugdroid1@chromium.org, May 10 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/1356a1a5b747bb9159e8e71db03a9f93a196b245

commit 1356a1a5b747bb9159e8e71db03a9f93a196b245
Author: Nick Sanders <nsanders@chromium.org>
Date: Thu May 10 09:27:11 2018

usb_updater2: match by serial number

Add a serial number match to allow update of servo or
other usb devices, where mutliple devices may be attached
to one system.

BRANCH=None
BUG= chromium:718688 
TEST=usb_updater2 -d 18d1:501b --serial=[C1706311869|fail]

Signed-off-by: Nick Sanders <nsanders@chromium.org>
Change-Id: Id6ec944139e685948433af703555fd185e91a9d4
Reviewed-on: https://chromium-review.googlesource.com/1050971
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>

[modify] https://crrev.com/1356a1a5b747bb9159e8e71db03a9f93a196b245/extra/usb_updater/usb_updater2.c

Status: Fixed (was: Assigned)

Sign in to add a comment