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.
BUG=none
TEST=none
Change-Id: Id7c563353ca3edfb96a8be2008b18f983ef14f69
Reviewed-on: https://chromium-review.googlesource.com/1714126
Tested-by: Brian Norris <briannorris@chromium.org>
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: Grant Grundler <grundler@chromium.org>
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 @@
continue
fi
- # 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;;
