Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(436)

Unified Diff: devil/devil/android/tools/device_status.py

Issue 2914063002: devil: Add physical port num to device_status.py (Closed)
Patch Set: Add physical port num to device_status Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: devil/devil/android/tools/device_status.py
diff --git a/devil/devil/android/tools/device_status.py b/devil/devil/android/tools/device_status.py
index 159c6c5dbced0866eecc8b29094db61d3a1b276f..98c2f32a95186ac61aaf3a691604229bc4fd1cd9 100755
--- a/devil/devil/android/tools/device_status.py
+++ b/devil/devil/android/tools/device_status.py
@@ -24,8 +24,10 @@ from devil.android import device_list
from devil.android import device_utils
from devil.android.sdk import adb_wrapper
from devil.constants import exit_codes
+from devil.utils import find_usb_devices
from devil.utils import lsusb
from devil.utils import run_tests_helper
+from devil.utils import usb_hubs
logger = logging.getLogger(__name__)
@@ -107,6 +109,12 @@ def DeviceStatus(devices, blacklist):
}
usb_devices = set(lsusb.get_android_devices())
+ port_mapping = {}
+ for hub in find_usb_devices.GetAllPhysicalPortToSerialMaps(
+ usb_hubs.ALL_HUBS, fast=True):
+ # Reverse the mapping.
+ port_mapping.update({device: port for port, device in hub.iteritems()})
+
def blacklisting_device_status(device):
serial = device.adb.GetDeviceSerial()
adb_status = (
@@ -118,6 +126,7 @@ def DeviceStatus(devices, blacklist):
'serial': serial,
'adb_status': adb_status,
'usb_status': usb_status,
+ 'physical_port': port_mapping.get(serial, 'unknown'),
}
if not IsBlacklisted(serial, blacklist):
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698