Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Mar 2008 22:06:55 GMT
From:      Andrew Thompson <thompsa@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 137031 for review
Message-ID:  <200803062206.m26M6tlk053862@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=137031

Change 137031 by thompsa@thompsa_peeps on 2008/03/06 22:06:48

	Allow setting the channel in monitor mode, AP scanning/auth are
	already taken care of by their respective firmware commands.
	
	Discussed with:		sam

Affected files ...

.. //depot/projects/wifi/sys/dev/wpi/if_wpi.c#19 edit

Differences ...

==== //depot/projects/wifi/sys/dev/wpi/if_wpi.c#19 (text+ko) ====

@@ -3667,8 +3667,17 @@
 		break;
 
 	case WPI_SET_CHAN:
-		/* XXX Is this needed ? */
-		DPRINTF(("Ignoring WPI_SET_CHAN\n"));
+		/*
+		 * Only need to set the channel in Monitor mode. The scanning
+		 * firmware command lists the channel and the AP channel is
+		 * set in wpi_auth().
+		 */
+		if (ic->ic_opmode == IEEE80211_M_MONITOR) {
+			error = wpi_config(sc);
+			if (error != 0)
+				device_printf(sc->sc_dev,
+				    "error %d settting channel\n", error);
+		}
 		break;
 
 	case WPI_AUTH:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200803062206.m26M6tlk053862>