PERFORCE change 137031 for review

Andrew Thompson thompsa at FreeBSD.org
Thu Mar 6 22:06:57 UTC 2008


http://perforce.freebsd.org/chv.cgi?CH=137031

Change 137031 by thompsa at 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:


More information about the p4-projects mailing list