check_ethernet: drop no-op 'duplex' check

This check is misleading: it's not really checking the value of the
'duplex' attribute at all. And even the check it *is* doing isn't really
effective -- for a valid Ethernet device, it will still get 'echo'd a
few lines down, since it
(a) won't be named 'wwan*' or 'lo' and
(b) it won't show up in the 'iw' listing

So since it's basically redundant and confusing, drop it.


Change-Id: Id7c563353ca3edfb96a8be2008b18f983ef14f69
Tested-by: Brian Norris <>
Commit-Ready: ChromeOS CL Exonerator Bot <>
Legacy-Commit-Queue: Commit Bot <>
Reviewed-by: Grant Grundler <>
diff --git a/recover_duts/hooks/check_ethernet.hook b/recover_duts/hooks/check_ethernet.hook
index 0f42e34..227297c 100755
--- a/recover_duts/hooks/check_ethernet.hook
+++ b/recover_duts/hooks/check_ethernet.hook
@@ -67,13 +67,6 @@
-    # lab interconnect is full-duplex and tells us so.
-    # devices w/o link won't get listed here
-    if cat "${device_path}/duplex" > /dev/null 2>&1 ; then
-       echo "${device}"
-       continue
-    fi
     # ignore "wwan*" devices: they are 3G/4G/LTE modems.
     case "${device}" in
       wwan*|lo) continue;;