commit 0a4c5e879d967342f055937bd80908b7c465c2a7
author Qi Jiang <qijiang@google.com> Fri Apr 13 12:13:14 2018 -0700
committer Qi Jiang <qijiang@google.com> Wed Apr 18 00:05:02 2018 +0000
tree 6cdf9b6299761714f7139ccdcce5c4c14cdc7ac7
parent 69aa61eaab8636dfbd34eb7fc1bccc2a0d23cea0

Merge "Add more descriptions to AP init function" am: 9bbc302234
am: 26db65abce

Test: None
Bug: 65563975

Change-Id: I30bad517963027efee0487f9e9f37e5acf14cd53
(cherry picked from commit cb4406b2cb864f30606dd10772cf6132c0ddac5a)

acts/framework/acts/controllers/access_point.py

diff --git a/acts/framework/acts/controllers/access_point.py b/acts/framework/acts/controllers/access_point.py
index d293ec6..f537bc8 100755
--- a/acts/framework/acts/controllers/access_point.py
+++ b/acts/framework/acts/controllers/access_point.py
@@ -145,8 +145,11 @@
         Bring down hostapd if instance is running, bring down all bridge
         interfaces.
         """
-        # Stop hostapd instance if running
         try:
+            # This is necessary for Gale/Whirlwind flashed with dev channel image
+            # Unused interfaces such as existing hostapd daemon, guest, mesh
+            # interfaces need to be brought down as part of the AP initialization
+            # process, otherwise test would fail.
             try:
                 self.ssh.run('stop hostapd')
             except job.Error: