public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/15]staging:rtl8187se trivial fix typos in comments.
@ 2010-12-31  5:48 Justin P. Mattock
  2010-12-31  5:48 ` [PATCH 01/15]drivers:staging:rtl8187se:r8180_93cx6.h Fix some comments Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh





Below is a set of patches that fix some comments in either a typo or makes the comment
more understanding(from the best that I can decipher).. 
Please let me know if there is anything I missed, and I'll fix it up
and resend.

 drivers/staging/rtl8187se/ieee80211/dot11d.c       |    2 +-
 drivers/staging/rtl8187se/ieee80211/ieee80211.h    |   10 ++--
 .../staging/rtl8187se/ieee80211/ieee80211_crypt.c  |    4 +-
 drivers/staging/rtl8187se/ieee80211/ieee80211_rx.c |    4 +-
 .../rtl8187se/ieee80211/ieee80211_softmac.c        |   58 ++++++++++----------
 drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c |    2 +-
 drivers/staging/rtl8187se/r8180.h                  |    6 +-
 drivers/staging/rtl8187se/r8180_93cx6.h            |    2 +-
 drivers/staging/rtl8187se/r8180_core.c             |   38 +++++++-------
 drivers/staging/rtl8187se/r8180_dm.c               |   24 ++++----
 drivers/staging/rtl8187se/r8180_hw.h               |    8 ++--
 drivers/staging/rtl8187se/r8180_rtl8225z2.c        |    4 +-
 drivers/staging/rtl8187se/r8180_wx.c               |    4 +-
 drivers/staging/rtl8187se/r8180_wx.h               |    2 +-
 drivers/staging/rtl8187se/r8185b_init.c            |    8 ++--
 15 files changed, 88 insertions(+), 88 deletions(-)


Justin P. Mattock

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [PATCH 01/15]drivers:staging:rtl8187se:r8180_93cx6.h Fix some comments.
  2010-12-31  5:48 [PATCH 00/15]staging:rtl8187se trivial fix typos in comments Justin P. Mattock
@ 2010-12-31  5:48 ` Justin P. Mattock
  2010-12-31  5:48   ` [PATCH 02/15]drivers:staging:rtl8187se:r8180_core.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_93cx6.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_93cx6.h b/drivers/staging/rtl8187se/r8180_93cx6.h
index 79e7391..4a3e248 100644
--- a/drivers/staging/rtl8187se/r8180_93cx6.h
+++ b/drivers/staging/rtl8187se/r8180_93cx6.h
@@ -7,7 +7,7 @@
 	Parts of this driver are based on the rtl8180 driver skeleton from Patric Schenke & Andres Salomon
 	Parts of this driver are based on the Intel Pro Wireless 2100 GPL driver
 
-	We want to tanks the Authors of such projects and the Ndiswrapper project Authors.
+	We want to thanks the Authors of such projects and the Ndiswrapper project Authors.
 */
 
 /*This files contains card eeprom (93c46 or 93c56) programming routines*/
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 02/15]drivers:staging:rtl8187se:r8180_core.c Fix some comments.
  2010-12-31  5:48 ` [PATCH 01/15]drivers:staging:rtl8187se:r8180_93cx6.h Fix some comments Justin P. Mattock
@ 2010-12-31  5:48   ` Justin P. Mattock
  2010-12-31  5:48     ` [PATCH 03/15]drivers:staging:rtl8187se:r8180_dm.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_core.c |   38 ++++++++++++++++----------------
 1 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_core.c b/drivers/staging/rtl8187se/r8180_core.c
index 70ab008..d688679 100644
--- a/drivers/staging/rtl8187se/r8180_core.c
+++ b/drivers/staging/rtl8187se/r8180_core.c
@@ -17,7 +17,7 @@
 
    Some ideas borrowed from the 8139too.c driver included in linux kernel.
 
-   We (I?) want to thanks the Authors of those projecs and also the
+   We (I?) want to thanks the Authors of those projects and also the
    Ndiswrapper's project Authors.
 
    A big big thanks goes also to Realtek corp. for their help in my attempt to
@@ -1380,7 +1380,7 @@ long TranslateToDbm8185(u8 SignalStrengthIndex)
 
 /*
  * Perform signal smoothing for dynamic mechanism.
- * This is different with PerformSignalSmoothing8185 in smoothing fomula.
+ * This is different with PerformSignalSmoothing8185 in smoothing formula.
  * No dramatic adjustion is apply because dynamic mechanism need some degree
  * of correctness. Ported from 8187B.
  */
@@ -1591,7 +1591,7 @@ void rtl8180_rx(struct net_device *dev)
 		priv->RSSI = RSSI;
 		/* SQ translation formula is provided by SD3 DZ. 2006.06.27 */
 		if (quality >= 127)
-			quality = 1; /*0; */ /* 0 will cause epc to show signal zero , walk aroud now; */
+			quality = 1; /*0; */ /* 0 will cause epc to show signal zero , workaround for now; */
 		else if (quality < 27)
 			quality = 100;
 		else
@@ -1620,7 +1620,7 @@ void rtl8180_rx(struct net_device *dev)
 				/* Perform signal smoothing for dynamic
 				 * mechanism on demand. This is different
 				 * with PerformSignalSmoothing8185 in smoothing
-				 * fomula. No dramatic adjustion is apply
+				 * formula. No dramatic adjustion is apply
 				 * because dynamic mechanism need some degree
 				 * of correctness. */
 				PerformUndecoratedSignalSmoothing8185(priv, bCckRate);
@@ -1646,7 +1646,7 @@ void rtl8180_rx(struct net_device *dev)
 
 		if (first) {
 			if (!priv->rx_skb_complete) {
-				/* seems that HW sometimes fails to reiceve and
+				/* seems that HW sometimes fails to receive and
 				   doesn't provide the last descriptor */
 				dev_kfree_skb_any(priv->rx_skb);
 				priv->stats.rxnolast++;
@@ -1784,9 +1784,9 @@ rate) {
 	/*
 	 * This function doesn't require lock because we make
 	 * sure it's called with the tx_lock already acquired.
-	 * this come from the kernel's hard_xmit callback (through
-	 * the ieee stack, or from the try_wake_queue (again through
-	 * the ieee stack.
+	 * this comes from the kernel's hard_xmit callback through
+	 * the ieee stack, or from the try_wake_queue(again through
+	 * the ieee stack).
 	 */
 	priority = AC2Q(skb->priority);
 	spin_lock_irqsave(&priv->tx_lock, flags);
@@ -1815,7 +1815,7 @@ rate) {
  * If the ring is full packet are dropped (for data frame the queue
  * is stopped before this can happen). For this reason it is better
  * if the descriptors are larger than the largest management frame
- * we intend to TX: i'm unsure what the HW does if it will not found
+ * we intend to TX: i'm unsure what the HW does if it has not found
  * the last fragment of a frame because it has been dropped...
  * Since queues for Management and Data frames are different we
  * might use a different lock than tx_lock (for example mgmt_tx_lock)
@@ -1916,8 +1916,8 @@ void rtl8180_prepare_beacon(struct net_device *dev)
 }
 
 /* 
- * This function do the real dirty work: it enqueues a TX command
- * descriptor in the ring buffer, copyes the frame in a TX buffer
+ * This function does the real dirty work: it enqueues a TX command
+ * descriptor in the ring buffer, copies the frame in a TX buffer
  * and kicks the NIC to ensure it does the DMA transfer.
  */
 short rtl8180_tx(struct net_device *dev, u8* txbuf, int len, int priority,
@@ -2092,7 +2092,7 @@ short rtl8180_tx(struct net_device *dev, u8* txbuf, int len, int priority,
 
 		if (remain == len && !descfrag) {
 			ownbit_flag = false;
-			*tail = *tail | (1<<29) ; /* fist segment of the packet */
+			*tail = *tail | (1<<29) ; /* first segment of the packet */
 			*tail = *tail | (len);
 		} else {
 			ownbit_flag = true;
@@ -2367,7 +2367,7 @@ void rtl8180_wmm_param_update(struct work_struct *work)
 				u8		u1bAIFS;
 				u32		u4bAcParam;
 				pAcParam = (PAC_PARAM)(&AcParam);
-				/* Retrive paramters to udpate. */
+				/* Retrieve paramters to update. */
 				u1bAIFS = pAcParam->f.AciAifsn.f.AIFSN * (((mode&IEEE_G) == IEEE_G) ? 9 : 20) + aSifsTime;
 				u4bAcParam = ((((u32)(pAcParam->f.TXOPLimit))<<AC_PARAM_TXOP_LIMIT_OFFSET)|
 					      (((u32)(pAcParam->f.Ecw.f.ECWmax))<<AC_PARAM_ECW_MAX_OFFSET)|
@@ -2403,7 +2403,7 @@ void rtl8180_wmm_param_update(struct work_struct *work)
 			u8		u1bAIFS;
 			u32		u4bAcParam;
 
-			/* Retrive paramters to udpate. */
+			/* Retrieve paramters to update. */
 			eACI = pAcParam->f.AciAifsn.f.ACI;
 			/* Mode G/A: slotTimeTimer = 9; Mode B: 20 */
 			u1bAIFS = pAcParam->f.AciAifsn.f.AIFSN * (((mode&IEEE_G) == IEEE_G) ? 9 : 20) + aSifsTime;
@@ -2613,7 +2613,7 @@ short rtl8180_init(struct net_device *dev)
 	DMESG("Channel plan is %d\n", priv->channel_plan);
 	rtl8180_set_channel_map(priv->channel_plan, priv->ieee80211);
 
-	/* FIXME: these constants are placed in a bad pleace. */
+	/* FIXME: these constants are placed in a bad place. */
 	priv->txbuffsize = 2048;	/* 1024; */
 	priv->txringcount = 32;		/* 32; */
 	priv->rxbuffersize = 2048;	/* 1024; */
@@ -2689,7 +2689,7 @@ short rtl8180_init(struct net_device *dev)
 	priv->bTxPowerTrack = false;
 	priv->ThermalMeter = 0;
 	priv->FalseAlarmRegValue = 0;
-	priv->RegDigOfdmFaUpTh = 0xc; /* Upper threhold of OFDM false alarm, which is used in DIG. */
+	priv->RegDigOfdmFaUpTh = 0xc; /* Upper threshold of OFDM false alarm, which is used in DIG. */
 	priv->DIG_NumberFallbackVote = 0;
 	priv->DIG_NumberUpgradeVote = 0;
 	priv->LastSignalStrengthInPercent = 0;
@@ -2885,7 +2885,7 @@ short rtl8180_init(struct net_device *dev)
 		priv->chtxpwr_ofdm[i+1] = (word & 0xff00) >> 8;
 	}
 
-	/* 3Read crystal calibtration and thermal meter indication on 87SE. */
+	/* 3Read crystal calibration and thermal meter indication on 87SE. */
 	eeprom_93cx6_read(&eeprom, EEPROM_RSV>>1, &tmpu16);
 
 	/* Crystal calibration for Xin and Xout resp. */
@@ -3128,7 +3128,7 @@ void rtl8180_adapter_start(struct net_device *dev)
 
 	/*
 	 * The following is very strange. seems to be that 1 means test mode,
-	 * but we need to acknolwledges the nic when a packet is ready
+	 * but we need to acknowledge the nic when a packet is ready
 	 * although we set it to 0
 	 */
 
@@ -3883,7 +3883,7 @@ void rtl8180_tx_isr(struct net_device *dev, int pri, short error)
 	 * If the packet previous of the nic pointer has been
 	 * processed this doesn't matter: it will be checked
 	 * here at the next round. Anyway if no more packet are
-	 * TXed no memory leak occour at all.
+	 * TXed no memory leak occur at all.
 	 */
 
 	switch (pri) {
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 03/15]drivers:staging:rtl8187se:r8180_dm.c Fix some comments.
  2010-12-31  5:48   ` [PATCH 02/15]drivers:staging:rtl8187se:r8180_core.c " Justin P. Mattock
@ 2010-12-31  5:48     ` Justin P. Mattock
  2010-12-31  5:48       ` [PATCH 04/15]drivers:staging:rtl8187se:r8180.h " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_dm.c |   24 ++++++++++++------------
 1 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_dm.c b/drivers/staging/rtl8187se/r8180_dm.c
index fc49078..a0619b8 100644
--- a/drivers/staging/rtl8187se/r8180_dm.c
+++ b/drivers/staging/rtl8187se/r8180_dm.c
@@ -6,7 +6,7 @@
 
 //
 //	Description:
-//		Return TRUE if we shall perform High Power Mecahnism, FALSE otherwise.
+//		Return TRUE if we shall perform High Power Mechanism, FALSE otherwise.
 //
 //+by amy 080312
 #define RATE_ADAPTIVE_TIMER_PERIOD      300
@@ -35,7 +35,7 @@ bool CheckHighPower(struct net_device *dev)
 //		See also DoRxHighPower() and SetTxPowerLevel8185() for reference.
 //
 //	Note:
-//		The reason why we udpate Tx power level here instead of DoRxHighPower()
+//		The reason why we update Tx power level here instead of DoRxHighPower()
 //		is the number of IO to change Tx power is much more than channel TR switch
 //		and they are related to OFDM and MAC registers.
 //		So, we don't want to update it so frequently in per-Rx packet base.
@@ -78,7 +78,7 @@ DoTxHighPower(
 		priv->bToUpdateTxPwr = true;
 		u1bTmp= read_nic_byte(dev, CCK_TXAGC);
 
-		// If it never enter High Power.
+		// If it never enters High Power.
 		if( CckTxPwrIdx == u1bTmp)
 		{
 		u1bTmp = (u1bTmp > 16) ? (u1bTmp -16): 0;  // 8dbm
@@ -145,7 +145,7 @@ void rtl8180_tx_pw_wq (struct work_struct *work)
 
 //
 //	Description:
-//		Return TRUE if we shall perform DIG Mecahnism, FALSE otherwise.
+//		Return TRUE if we shall perform DIG Mechanism, FALSE otherwise.
 //
 bool
 CheckDig(
@@ -650,7 +650,7 @@ StaRateAdaptive87SE(
 
 	//
 	// For Netgear case, I comment out the following signal strength estimation,
-	// which can results in lower rate to transmit when sample is NOT enough (e.g. PING request).
+	// which can result in lower rate to transmit when sample is NOT enough (e.g. PING request).
 	// 2007.04.09, by Roger.
 	//
 
@@ -687,7 +687,7 @@ StaRateAdaptive87SE(
 	{
 		//2 For Test Upgrading mechanism
 		// Note:
-		// 	Sometimes the throughput is upon on the capability bwtween the AP and NIC,
+		// 	Sometimes the throughput is upon on the capability between the AP and NIC,
 		// 	thus the low data rate does not improve the performance.
 		// 	We randomly upgrade the data rate and check if the retry rate is improved.
 
@@ -990,7 +990,7 @@ StaRateAdaptive87SE(
 				bUpdateInitialGain = true;
 
 			// The difference in throughput between 48Mbps and 36Mbps is 8M.
-			// So, we must be carefully in this rate scale. Isaiah 2008-02-15.
+			// So, we must be careful in this rate scale. Isaiah 2008-02-15.
 			//
 			if(  ((priv->CurrentOperaRate == 72) || (priv->CurrentOperaRate == 48) || (priv->CurrentOperaRate == 36)) &&
 				(priv->FailTxRateCount > 2) )
@@ -1498,11 +1498,11 @@ SwAntennaDiversity(
 		// didn't changed by HW evaluation.
 		// 2008.02.27.
 		//
-		// [TRC Dell Lab] SignalStrength is inaccuracy. Isaiah 2008-03-05
+		// [TRC Dell Lab] SignalStrength is inaccurate. Isaiah 2008-03-05
 		// For example, Throughput of aux is better than main antenna(about 10M v.s 2M),
 		// but AdRxSignalStrength is less than main.
-		// Our guess is that main antenna have lower throughput and get many change
-		// to receive more CCK packets(ex.Beacon) which have stronger SignalStrength.
+		// Our guess is that main antenna has lower throughput and gets many changes
+		// to receive more CCK packets(ex.Beacon) which has stronger SignalStrength.
 		//
 		if( (!priv->bHWAdSwitched) && (bSwCheckSS))
 		{
@@ -1526,7 +1526,7 @@ SwAntennaDiversity(
 			priv->bAdSwitchedChecking = false;
 			// Increase Rx signal strength threshold if necessary.
 			if(	(priv->AdRxSignalStrength > (priv->AdRxSsThreshold + 10)) && // Signal is much stronger than current threshold
-				priv->AdRxSsThreshold <= priv->AdMaxRxSsThreshold) // Current threhold is not yet reach upper limit.
+				priv->AdRxSsThreshold <= priv->AdMaxRxSsThreshold) // Current threshold is not yet reach upper limit.
 			{
 				priv->AdRxSsThreshold = (priv->AdRxSsThreshold + priv->AdRxSignalStrength) / 2;
 				priv->AdRxSsThreshold = (priv->AdRxSsThreshold > priv->AdMaxRxSsThreshold) ?
@@ -1555,7 +1555,7 @@ SwAntennaDiversity(
 
 //
 //	Description:
-//		Return TRUE if we shall perform Tx Power Tracking Mecahnism, FALSE otherwise.
+//		Return TRUE if we shall perform Tx Power Tracking Mechanism, FALSE otherwise.
 //
 bool
 CheckTxPwrTracking(	struct net_device *dev)
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 04/15]drivers:staging:rtl8187se:r8180.h Fix some comments.
  2010-12-31  5:48     ` [PATCH 03/15]drivers:staging:rtl8187se:r8180_dm.c " Justin P. Mattock
@ 2010-12-31  5:48       ` Justin P. Mattock
  2010-12-31  5:48         ` [PATCH 05/15]drivers:staging:rtl8187se:r8180_hw.h " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180.h |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180.h b/drivers/staging/rtl8187se/r8180.h
index d15bdf6..dbc1ce4 100644
--- a/drivers/staging/rtl8187se/r8180.h
+++ b/drivers/staging/rtl8187se/r8180.h
@@ -11,7 +11,7 @@
 
    Parts of this driver are based on the Intel Pro Wireless 2100 GPL driver
 
-   We want to tanks the Authors of those projects and the Ndiswrapper
+   We want to thanks the Authors of those projects and the Ndiswrapper
    project Authors.
 */
 
@@ -235,7 +235,7 @@ typedef struct _link_detect_t
 	u32				LastNumTxUnicast;
 	u32				LastNumRxUnicast;
 
-	bool				bBusyTraffic;    //when it is set to 1, UI cann't scan at will.
+	bool				bBusyTraffic;    //when it is set to 1, UI can't scan at will.
 }link_detect_t, *plink_detect_t;
 
 //YJ,modified,080828,end
@@ -552,7 +552,7 @@ typedef struct r8180_priv
 	bool				bDigMechanism; // TRUE if DIG is enabled, FALSE ow.
 	bool				bRegHighPowerMechanism; // For High Power Mechanism. 061010, by rcnjko.
 	u32					FalseAlarmRegValue;
-	u8					RegDigOfdmFaUpTh; // Upper threhold of OFDM false alarm, which is used in DIG.
+	u8					RegDigOfdmFaUpTh; // Upper threshold of OFDM false alarm, which is used in DIG.
 	u8					DIG_NumberFallbackVote;
 	u8					DIG_NumberUpgradeVote;
 	// For HW antenna diversity, added by Roger, 2008.01.30.
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 05/15]drivers:staging:rtl8187se:r8180_hw.h Fix some comments.
  2010-12-31  5:48       ` [PATCH 04/15]drivers:staging:rtl8187se:r8180.h " Justin P. Mattock
@ 2010-12-31  5:48         ` Justin P. Mattock
  2010-12-31  5:48           ` [PATCH 06/15]drivers:staging:rtl8187se:r8180_rtl8225z2.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_hw.h |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_hw.h b/drivers/staging/rtl8187se/r8180_hw.h
index 3fca144..34aa664 100644
--- a/drivers/staging/rtl8187se/r8180_hw.h
+++ b/drivers/staging/rtl8187se/r8180_hw.h
@@ -10,7 +10,7 @@
 	Parts of this driver are based on the Intel Pro Wireless
 	2100 GPL driver.
 
-	We want to tanks the Authors of those projects
+	We want to thanks the Authors of those projects
 	and the Ndiswrapper project Authors.
 */
 
@@ -181,7 +181,7 @@
 #define RF_SW_CFG_SI		BIT1
 #define EIFS			0x2D			/* Extended InterFrame Space Timer, in unit of 4 us.	*/
 
-#define BRSR			0x34			/* Basic rate set										*/
+#define BRSR			0x34			/* Basic rate set */
 
 #define IMR 0x006C
 #define ISR 0x003C
@@ -286,7 +286,7 @@
 #define CR_MulRW    ((1 << 0))
 
 #define IMR_Dot11hInt	((1 << 25))			/*802.11h Measurement Interrupt					*/
-#define IMR_BcnDmaInt	((1 << 24))			/*Beacon DMA Interrupt */ /*What differenct between BcnDmaInt and BcnInt???	*/
+#define IMR_BcnDmaInt	((1 << 24))			/*Beacon DMA Interrupt */ /*What difference between BcnDmaInt and BcnInt???	*/
 #define IMR_WakeInt		((1 << 23))			/*Wake Up Interrupt								*/
 #define IMR_TXFOVW		((1 << 22))			/*Tx FIFO Overflow Interrupt					*/
 #define IMR_TimeOut1	((1 << 21))			/*Time Out Interrupt 1							*/
@@ -313,7 +313,7 @@
 #define IMR_TimeOut3	((1 << 0))			/*Time Out Interrupt 3							*/
 #define IMR_TMGDOK      ((1 << 30))
 #define ISR_Dot11hInt	((1 << 25))			/*802.11h Measurement Interrupt					*/
-#define ISR_BcnDmaInt	((1 << 24))			/*Beacon DMA Interrupt	*/ /*What differenct between BcnDmaInt and BcnInt???	*/
+#define ISR_BcnDmaInt	((1 << 24))			/*Beacon DMA Interrupt	*/ /*What difference between BcnDmaInt and BcnInt???	*/
 #define ISR_WakeInt		((1 << 23))			/*Wake Up Interrupt								*/
 #define ISR_TXFOVW		((1 << 22))			/*Tx FIFO Overflow Interrupt					*/
 #define ISR_TimeOut1	((1 << 21))			/*Time Out Interrupt 1							*/
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 06/15]drivers:staging:rtl8187se:r8180_rtl8225z2.c Fix some comments.
  2010-12-31  5:48         ` [PATCH 05/15]drivers:staging:rtl8187se:r8180_hw.h " Justin P. Mattock
@ 2010-12-31  5:48           ` Justin P. Mattock
  2010-12-31  5:48             ` [PATCH 07/15]drivers:staging:rtl8187se:r8180_wx.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_rtl8225z2.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_rtl8225z2.c b/drivers/staging/rtl8187se/r8180_rtl8225z2.c
index 2a2afd5..47297ab 100644
--- a/drivers/staging/rtl8187se/r8180_rtl8225z2.c
+++ b/drivers/staging/rtl8187se/r8180_rtl8225z2.c
@@ -479,7 +479,7 @@ s8 DbmToTxPwrIdx(struct r8180_priv *priv, WIRELESS_MODE WirelessMode,
 
 	/*
 	 * TRUE if we want to use a default implementation.
-	 * We shall set it to FALSE when we have exact translation formular
+	 * We shall set it to FALSE when we have exact translation formula
 	 * for target IC. 070622, by rcnjko.
 	 */
 	if (bUseDefault) {
@@ -688,7 +688,7 @@ void rtl8225z2_rf_init(struct net_device *dev)
 
 	write_rtl8225(dev, 0x2, 0xc4d);
 
-	/* FIXME!! rtl8187 we have to check if calibrarion
+	/* FIXME!! rtl8187 we have to check if calibration
 	 * is successful and eventually cal. again (repeat
 	 * the two write on reg 2)
 	 */
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 07/15]drivers:staging:rtl8187se:r8180_wx.c Fix some comments.
  2010-12-31  5:48           ` [PATCH 06/15]drivers:staging:rtl8187se:r8180_rtl8225z2.c " Justin P. Mattock
@ 2010-12-31  5:48             ` Justin P. Mattock
  2010-12-31  5:48               ` [PATCH 08/15]drivers:staging:rtl8187se:r8180_wx.h " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_wx.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_wx.c b/drivers/staging/rtl8187se/r8180_wx.c
index 39ef7e0..ef1def6 100644
--- a/drivers/staging/rtl8187se/r8180_wx.c
+++ b/drivers/staging/rtl8187se/r8180_wx.c
@@ -13,7 +13,7 @@
 
 	Parts of this driver are based on the Intel Pro Wireless 2100 GPL driver.
 
-	We want to tanks the Authors of those projects and the Ndiswrapper
+	We want to thanks the Authors of those projects and the Ndiswrapper
 	project Authors.
 */
 
@@ -1529,7 +1529,7 @@ static inline int is_same_network(struct ieee80211_network *src,
 			(dst->capability & WLAN_CAPABILITY_BSS)));
 }
 
-/* WB modefied to show signal to GUI on 18-01-2008 */
+/* WB modified to show signal to GUI on 18-01-2008 */
 static struct iw_statistics *r8180_get_wireless_stats(struct net_device *dev)
 {
 	struct r8180_priv *priv = ieee80211_priv(dev);
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 08/15]drivers:staging:rtl8187se:r8180_wx.h Fix some comments.
  2010-12-31  5:48             ` [PATCH 07/15]drivers:staging:rtl8187se:r8180_wx.c " Justin P. Mattock
@ 2010-12-31  5:48               ` Justin P. Mattock
  2010-12-31  5:48                 ` [PATCH 09/15]drivers:staging:rtl8187se:r8185b_init.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8180_wx.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8180_wx.h b/drivers/staging/rtl8187se/r8180_wx.h
index 735d03d..4081914 100644
--- a/drivers/staging/rtl8187se/r8180_wx.h
+++ b/drivers/staging/rtl8187se/r8180_wx.h
@@ -7,7 +7,7 @@
 	Parts of this driver are based on the rtl8180 driver skeleton from Patric Schenke & Andres Salomon
 	Parts of this driver are based on the Intel Pro Wireless 2100 GPL driver
 
-	We want to tanks the Authors of such projects and the Ndiswrapper project Authors.
+	We want to thanks the Authors of such projects and the Ndiswrapper project Authors.
 */
 
 /* this file (will) contains wireless extension handlers*/
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 09/15]drivers:staging:rtl8187se:r8185b_init.c Fix some comments.
  2010-12-31  5:48               ` [PATCH 08/15]drivers:staging:rtl8187se:r8180_wx.h " Justin P. Mattock
@ 2010-12-31  5:48                 ` Justin P. Mattock
  2010-12-31  5:48                   ` [PATCH 10/15]drivers:staging:rtl8187se:ieee80211:dot11d.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/r8185b_init.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rtl8187se/r8185b_init.c b/drivers/staging/rtl8187se/r8185b_init.c
index 3bdf9b3..31e6810 100644
--- a/drivers/staging/rtl8187se/r8185b_init.c
+++ b/drivers/staging/rtl8187se/r8185b_init.c
@@ -1103,7 +1103,7 @@ ActUpdateChannelAccessSetting(
 				u8		u1bAIFS;
 				u32		u4bAcParam;
 
-				/*  Retrive paramters to udpate. */
+				/*  Retreive paramters to update. */
 				eACI = pAcParam->f.AciAifsn.f.ACI;
 				u1bAIFS = pAcParam->f.AciAifsn.f.AIFSN * ChnlAccessSetting->SlotTimeTimer + aSifsTime;
 				u4bAcParam = ((((u32)(pAcParam->f.TXOPLimit)) << AC_PARAM_TXOP_LIMIT_OFFSET)	|
@@ -1338,7 +1338,7 @@ MgntDisconnect(
 {
 	struct r8180_priv *priv = (struct r8180_priv *)ieee80211_priv(dev);
 	/*
-		Schedule an workitem to wake up for ps mode, 070109, by rcnjko.
+		Schedule a workitem to wake up for ps mode, 070109, by rcnjko.
 	*/
 
 	if (IS_DOT11D_ENABLE(priv->ieee80211))
@@ -1549,7 +1549,7 @@ IPSEnter(
 			Do not enter IPS in the following conditions:
 			(1) RF is already OFF or Sleep
 			(2) bSwRfProcessing (indicates the IPS is still under going)
-			(3) Connectted (only disconnected can trigger IPS)
+			(3) Connected (only disconnected can trigger IPS)
 			(4) IBSS (send Beacon)
 			(5) AP mode (send Beacon)
 		*/
@@ -1652,7 +1652,7 @@ void rtl8185b_adapter_start(struct net_device *dev)
 
 	/*
 		We assume RegWirelessMode has already been initialized before,
-		however, we has to validate the wireless mode here and provide a
+		however, we have to validate the wireless mode here and provide a
 		reasonable initialized value if necessary. 2005.01.13, by rcnjko.
 	*/
 	SupportedWirelessMode = GetSupportedWirelessMode8185(dev);
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 10/15]drivers:staging:rtl8187se:ieee80211:dot11d.c Fix some comments.
  2010-12-31  5:48                 ` [PATCH 09/15]drivers:staging:rtl8187se:r8185b_init.c " Justin P. Mattock
@ 2010-12-31  5:48                   ` Justin P. Mattock
  2010-12-31  5:48                     ` [PATCH 11/15]drivers:staging:rtl8187se:ieee80211:ieee80211_crypt.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/ieee80211/dot11d.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/rtl8187se/ieee80211/dot11d.c b/drivers/staging/rtl8187se/ieee80211/dot11d.c
index 309bb8b..0e93eb0 100644
--- a/drivers/staging/rtl8187se/ieee80211/dot11d.c
+++ b/drivers/staging/rtl8187se/ieee80211/dot11d.c
@@ -55,7 +55,7 @@ Dot11d_Reset(struct ieee80211_device *ieee)
 
 //
 //	Description:
-//		Update country IE from Beacon or Probe Resopnse
+//		Update country IE from Beacon or Probe Response
 //		and configure PHY for operation in the regulatory domain.
 //
 //	TODO:
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 11/15]drivers:staging:rtl8187se:ieee80211:ieee80211_crypt.c Fix some comments.
  2010-12-31  5:48                   ` [PATCH 10/15]drivers:staging:rtl8187se:ieee80211:dot11d.c " Justin P. Mattock
@ 2010-12-31  5:48                     ` Justin P. Mattock
  2010-12-31  5:48                       ` [PATCH 12/15]drivers:staging:rtl8187se:ieee80211:ieee80211.h " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 .../staging/rtl8187se/ieee80211/ieee80211_crypt.c  |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_crypt.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_crypt.c
index c8dbcb9..60b2f7e 100644
--- a/drivers/staging/rtl8187se/ieee80211/ieee80211_crypt.c
+++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_crypt.c
@@ -89,8 +89,8 @@ void ieee80211_crypt_delayed_deinit(struct ieee80211_device *ieee,
 	*crypt = NULL;
 
 	/* must not run ops->deinit() while there may be pending encrypt or
-	 * decrypt operations. Use a list of delayed deinits to avoid needing
-	 * locking. */
+	 * decrypt operations. Use a list of delayed deinits to avoid locking. 
+	 */
 
 	spin_lock_irqsave(&ieee->lock, flags);
 	list_add(&tmp->list, &ieee->crypt_deinit_list);
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 12/15]drivers:staging:rtl8187se:ieee80211:ieee80211.h Fix some comments.
  2010-12-31  5:48                     ` [PATCH 11/15]drivers:staging:rtl8187se:ieee80211:ieee80211_crypt.c " Justin P. Mattock
@ 2010-12-31  5:48                       ` Justin P. Mattock
  2010-12-31  5:48                         ` [PATCH 13/15]drivers:staging:rtl8187se:ieee80211:ieee80211_rx.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/ieee80211/ieee80211.h |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211.h b/drivers/staging/rtl8187se/ieee80211/ieee80211.h
index dc608c7..e54f88f 100644
--- a/drivers/staging/rtl8187se/ieee80211/ieee80211.h
+++ b/drivers/staging/rtl8187se/ieee80211/ieee80211.h
@@ -1106,7 +1106,7 @@ struct ieee80211_device {
 
 	/* used instead of hard_start_xmit (not softmac_hard_start_xmit)
 	 * if the IEEE_SOFTMAC_TX_QUEUE feature is used to TX data
-	 * frames. I the option IEEE_SOFTMAC_SINGLE_QUEUE is also set
+	 * frames. If the option IEEE_SOFTMAC_SINGLE_QUEUE is also set
 	 * then also management frames are sent via this callback.
 	 * This function can't sleep.
 	 */
@@ -1124,7 +1124,7 @@ struct ieee80211_device {
 
 	/* ask to the driver to retune the radio .
 	 * This function can sleep. the driver should ensure
-	 * the radio has been swithced before return.
+	 * the radio has been switched before return.
 	 */
 	void (*set_chan)(struct net_device *dev,short ch);
 
@@ -1138,9 +1138,9 @@ struct ieee80211_device {
 	 * it is called in a work_queue when swithcing to ad-hoc mode
 	 * or in behalf of iwlist scan when the card is associated
 	 * and root user ask for a scan.
-	 * the fucntion stop_scan should stop both the syncro and
+	 * the function stop_scan should stop both the syncro and
 	 * background scanning and can sleep.
-	 * The fucntion start_scan should initiate the background
+	 * The function start_scan should initiate the background
 	 * scanning and can't sleep.
 	 */
 	void (*scan_syncro)(struct net_device *dev);
@@ -1199,7 +1199,7 @@ struct ieee80211_device {
 /* Generate respones to probe requests */
 #define IEEE_SOFTMAC_PROBERS (1<<5)
 
-/* The ieee802.11 stack will manages the netif queue
+/* The ieee802.11 stack will manage the netif queue
  * wake/stop for the driver, taking care of 802.11
  * fragmentation. See softmac.c for details. */
 #define IEEE_SOFTMAC_TX_QUEUE (1<<7)
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 13/15]drivers:staging:rtl8187se:ieee80211:ieee80211_rx.c Fix some comments.
  2010-12-31  5:48                       ` [PATCH 12/15]drivers:staging:rtl8187se:ieee80211:ieee80211.h " Justin P. Mattock
@ 2010-12-31  5:48                         ` Justin P. Mattock
  2010-12-31  5:48                           ` [PATCH 14/15]drivers:staging:rtl8187se:ieee80211:ieee80211_softmac.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/ieee80211/ieee80211_rx.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_rx.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_rx.c
index 3a72449..6993a6a 100644
--- a/drivers/staging/rtl8187se/ieee80211/ieee80211_rx.c
+++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_rx.c
@@ -1373,9 +1373,9 @@ inline void ieee80211_process_probe_response(
 	}
 
 	// For Asus EeePc request,
-	// (1) if wireless adapter receive get any 802.11d country code in AP beacon,
+	// (1) if wireless adapter receive gets any 802.11d country code in AP beacon,
 	//	   wireless adapter should follow the country code.
-	// (2)  If there is no any country code in beacon,
+	// (2)  If there is not any country code in beacon,
 	//       then wireless adapter should do active scan from ch1~11 and
 	//       passive scan from ch12~14
 	if(ieee->bGlobalDomain)
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 14/15]drivers:staging:rtl8187se:ieee80211:ieee80211_softmac.c Fix some comments.
  2010-12-31  5:48                         ` [PATCH 13/15]drivers:staging:rtl8187se:ieee80211:ieee80211_rx.c " Justin P. Mattock
@ 2010-12-31  5:48                           ` Justin P. Mattock
  2010-12-31  5:48                             ` [PATCH 15/15]drivers:staging:rtl8187se:ieee80211:ieee80211_tx.c " Justin P. Mattock
  0 siblings, 1 reply; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 .../rtl8187se/ieee80211/ieee80211_softmac.c        |   58 ++++++++++----------
 1 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
index 652d879..e945e88 100644
--- a/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
+++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c
@@ -42,7 +42,7 @@ short ieee80211_is_shortslot(struct ieee80211_network net)
 	return (net.capability & WLAN_CAPABILITY_SHORT_SLOT);
 }
 
-/* returns the total length needed for pleacing the RATE MFIE
+/* returns the total length needed for placing the RATE MFIE
  * tag and the EXTENDED RATE MFIE tag if needed.
  * It encludes two bytes per tag for the tag itself and its len
  */
@@ -60,7 +60,7 @@ unsigned int ieee80211_MFIE_rate_len(struct ieee80211_device *ieee)
 	return rate_len;
 }
 
-/* pleace the MFIE rate, tag to the memory (double) poined.
+/* place the MFIE rate, tag to the memory(double) pointed.
  * Then it updates the pointer so that
  * it points after the new MFIE tag added.
  */
@@ -152,7 +152,7 @@ void enqueue_mgmt(struct ieee80211_device *ieee, struct sk_buff *skb)
 
 /*
  * if the queue is full but we have newer frames then
- * just overwrites the oldest.
+ * just overwrite the oldest.
  *
  * if (nh == ieee->mgmt_queue_tail)
  *		return -1;
@@ -456,8 +456,8 @@ void ieee80211_softmac_scan_syncro(struct ieee80211_device *ieee)
 		}while(!channel_map[ch]);
 		/* this function can be called in two situations
 		 * 1- We have switched to ad-hoc mode and we are
-		 *    performing a complete syncro scan before conclude
-		 *    there are no interesting cell and to create a
+		 *    performing a complete syncro scan before concluding
+		 *    there are no interesting cell's and to create a
 		 *    new one. In this case the link state is
 		 *    IEEE80211_NOLINK until we found an interesting cell.
 		 *    If so the ieee8021_new_net, called by the RX path
@@ -465,12 +465,12 @@ void ieee80211_softmac_scan_syncro(struct ieee80211_device *ieee)
 		 *    scanning
 		 * 2- We are linked and the root uses run iwlist scan.
 		 *    So we switch to IEEE80211_LINKED_SCANNING to remember
-		 *    that we are still logically linked (not interested in
+		 *    that we are still logically linked(not interested in
 		 *    new network events, despite for updating the net list,
 		 *    but we are temporarly 'unlinked' as the driver shall
-		 *    not filter RX frames and the channel is changing.
+		 *    not filter RX frames and the channel is changing).
 		 * So the only situation in witch are interested is to check
-		 * if the state become LINKED because of the #1 situation
+		 * if the state becomes LINKED because of the #1 situation
 		 */
 
 		if (ieee->state == IEEE80211_LINKED)
@@ -483,7 +483,7 @@ void ieee80211_softmac_scan_syncro(struct ieee80211_device *ieee)
 //			printk("====send probe request\n");
 			ieee80211_send_probe_requests(ieee);
 		}
-		/* this prevent excessive time wait when we
+		/* this prevents excessive time wait when we
 		 * need to wait for a syncro scan to end..
 		 */
 		if (ieee->sync_scan_hurryup)
@@ -519,8 +519,8 @@ void ieee80211_softmac_ips_scan_syncro(struct ieee80211_device *ieee)
         {
                 /* this function can be called in two situations
                  * 1- We have switched to ad-hoc mode and we are
-                 *    performing a complete syncro scan before conclude
-                 *    there are no interesting cell and to create a
+                 *    performing a complete syncro scan before concluding
+                 *    there are no interesting cells and to create a
                  *    new one. In this case the link state is
                  *    IEEE80211_NOLINK until we found an interesting cell.
                  *    If so the ieee8021_new_net, called by the RX path
@@ -528,10 +528,10 @@ void ieee80211_softmac_ips_scan_syncro(struct ieee80211_device *ieee)
                  *    scanning
                  * 2- We are linked and the root uses run iwlist scan.
                  *    So we switch to IEEE80211_LINKED_SCANNING to remember
-                 *    that we are still logically linked (not interested in
+                 *    that we are still logically linked(not interested in
                  *    new network events, despite for updating the net list,
                  *    but we are temporarly 'unlinked' as the driver shall
-                 *    not filter RX frames and the channel is changing.
+                 *    not filter RX frames and the channel is changing).
                  * So the only situation in witch are interested is to check
                  * if the state become LINKED because of the #1 situation
                  */
@@ -549,7 +549,7 @@ void ieee80211_softmac_ips_scan_syncro(struct ieee80211_device *ieee)
 //			printk("====send probe request\n");
 			ieee80211_send_probe_requests(ieee);
                 }
-		/* this prevent excessive time wait when we
+		/* this prevents excessive time wait when we
                  * need to wait for a syncro scan to end..
                  */
 //                if (ieee->sync_scan_hurryup)
@@ -1314,7 +1314,7 @@ inline void ieee80211_softmac_new_net(struct ieee80211_device *ieee, struct ieee
 
 	short apset,ssidset,ssidbroad,apmatch,ssidmatch;
 
-	/* we are interested in new new only if we are not associated
+	/* we are interested in new only if we are not associated
 	 * and we are not associating / authenticating
 	 */
 	if (ieee->state != IEEE80211_NOLINK)
@@ -1955,16 +1955,16 @@ associate_complete:
 
 /* following are for a simplier TX queue management.
  * Instead of using netif_[stop/wake]_queue the driver
- * will uses these two function (plus a reset one), that
- * will internally uses the kernel netif_* and takes
+ * will use these two functions (plus a reset one), that
+ * will internally use the kernel netif_* and takes
  * care of the ieee802.11 fragmentation.
  * So the driver receives a fragment per time and might
- * call the stop function when it want without take care
- * to have enough room to TX an entire packet.
- * This might be useful if each fragment need it's own
+ * call the stop function when it does not have enough
+ * room to TX an entire packet.
+ * This might be useful if each fragment needs it's own
  * descriptor, thus just keep a total free memory > than
  * the max fragmentation threshold is not enough.. If the
- * ieee802.11 stack passed a TXB struct then you needed
+ * ieee802.11 stack passed a TXB struct then you need
  * to keep N free descriptors where
  * N = MAX_PACKET_SIZE / MIN_FRAG_TRESHOLD
  * In this way you need just one and the 802.11 stack
@@ -2183,13 +2183,13 @@ void ieee80211_start_ibss_wq(struct work_struct *work)
 
 	if(ieee->state == IEEE80211_NOLINK)
 		ieee->current_network.channel = 10;
-	/* if not then the state is not linked. Maybe the user swithced to
+	/* if not then the state is not linked. Maybe the user switched to
 	 * ad-hoc mode just after being in monitor mode, or just after
-	 * being very few time in managed mode (so the card have had no
+	 * being short period of time in managed mode (so the card has had no
 	 * time to scan all the chans..) or we have just run up the iface
 	 * after setting ad-hoc mode. So we have to give another try..
 	 * Here, in ibss mode, should be safe to do this without extra care
-	 * (in bss mode we had to make sure no-one tryed to associate when
+	 * (in bss mode we had to make sure no-one tried to associate when
 	 * we had just checked the ieee->state and we was going to start the
 	 * scan) beacause in ibss mode the ieee80211_new_net function, when
 	 * finds a good net, just set the ieee->state to IEEE80211_LINKED,
@@ -2340,15 +2340,15 @@ void ieee80211_associate_retry_wq(struct work_struct *work)
 	if(ieee->state != IEEE80211_ASSOCIATING_RETRY)
 		goto exit;
 	/* until we do not set the state to IEEE80211_NOLINK
-	* there are no possibility to have someone else trying
-	* to start an association procdure (we get here with
+	* there is no possibility to have someone else trying
+	* to start an association procedure (we get here with
 	* ieee->state = IEEE80211_ASSOCIATING).
 	* When we set the state to IEEE80211_NOLINK it is possible
 	* that the RX path run an attempt to associate, but
 	* both ieee80211_softmac_check_all_nets and the
 	* RX path works with ieee->lock held so there are no
 	* problems. If we are still disassociated then start a scan.
-	* the lock here is necessary to ensure no one try to start
+	* the lock here is necessary to ensure no one tries to start
 	* an association procedure when we have just checked the
 	* state and we are going to start the scan.
 	*/
@@ -2484,8 +2484,8 @@ void ieee80211_start_protocol(struct ieee80211_device *ieee)
 
 
 	/* if the user set the MAC of the ad-hoc cell and then
-	 * switch to managed mode, shall we  make sure that association
-	 * attempts does not fail just because the user provide the essid
+	 * switch to managed mode, shall we make sure that association
+	 * attempts does not fail just because the user provides the essid
 	 * and the nic is still checking for the AP MAC ??
 	 */
 	switch (ieee->iw_mode) {
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [PATCH 15/15]drivers:staging:rtl8187se:ieee80211:ieee80211_tx.c Fix some comments.
  2010-12-31  5:48                           ` [PATCH 14/15]drivers:staging:rtl8187se:ieee80211:ieee80211_softmac.c " Justin P. Mattock
@ 2010-12-31  5:48                             ` Justin P. Mattock
  0 siblings, 0 replies; 16+ messages in thread
From: Justin P. Mattock @ 2010-12-31  5:48 UTC (permalink / raw)
  To: trivial; +Cc: linux-kernel, gregkh, Justin P. Mattock

Below are some patches that either fix a typo in a comment and/or fixes a
comment altogether to make more sense. If there is anything I missed let me know
and I'll resend.

Signed-off-by: Justin P. Mattock <justinmattock@gmail.com>

---
 drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c b/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c
index 6cb31e1..552115c 100644
--- a/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c
+++ b/drivers/staging/rtl8187se/ieee80211/ieee80211_tx.c
@@ -445,7 +445,7 @@ int ieee80211_rtl_xmit(struct sk_buff *skb,
 		(CFG_IEEE80211_COMPUTE_FCS | CFG_IEEE80211_RESERVE_FCS))
 			bytes_per_frag -= IEEE80211_FCS_LEN;
 
-		/* Each fragment may need to have room for encryptiong pre/postfix */
+		/* Each fragment may need to have room for encryption pre/postfix */
 		if (encrypt)
 			bytes_per_frag -= crypt->ops->extra_prefix_len +
 				crypt->ops->extra_postfix_len;
-- 
1.6.5.2.180.gc5b3e


^ permalink raw reply related	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2010-12-31  6:02 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-31  5:48 [PATCH 00/15]staging:rtl8187se trivial fix typos in comments Justin P. Mattock
2010-12-31  5:48 ` [PATCH 01/15]drivers:staging:rtl8187se:r8180_93cx6.h Fix some comments Justin P. Mattock
2010-12-31  5:48   ` [PATCH 02/15]drivers:staging:rtl8187se:r8180_core.c " Justin P. Mattock
2010-12-31  5:48     ` [PATCH 03/15]drivers:staging:rtl8187se:r8180_dm.c " Justin P. Mattock
2010-12-31  5:48       ` [PATCH 04/15]drivers:staging:rtl8187se:r8180.h " Justin P. Mattock
2010-12-31  5:48         ` [PATCH 05/15]drivers:staging:rtl8187se:r8180_hw.h " Justin P. Mattock
2010-12-31  5:48           ` [PATCH 06/15]drivers:staging:rtl8187se:r8180_rtl8225z2.c " Justin P. Mattock
2010-12-31  5:48             ` [PATCH 07/15]drivers:staging:rtl8187se:r8180_wx.c " Justin P. Mattock
2010-12-31  5:48               ` [PATCH 08/15]drivers:staging:rtl8187se:r8180_wx.h " Justin P. Mattock
2010-12-31  5:48                 ` [PATCH 09/15]drivers:staging:rtl8187se:r8185b_init.c " Justin P. Mattock
2010-12-31  5:48                   ` [PATCH 10/15]drivers:staging:rtl8187se:ieee80211:dot11d.c " Justin P. Mattock
2010-12-31  5:48                     ` [PATCH 11/15]drivers:staging:rtl8187se:ieee80211:ieee80211_crypt.c " Justin P. Mattock
2010-12-31  5:48                       ` [PATCH 12/15]drivers:staging:rtl8187se:ieee80211:ieee80211.h " Justin P. Mattock
2010-12-31  5:48                         ` [PATCH 13/15]drivers:staging:rtl8187se:ieee80211:ieee80211_rx.c " Justin P. Mattock
2010-12-31  5:48                           ` [PATCH 14/15]drivers:staging:rtl8187se:ieee80211:ieee80211_softmac.c " Justin P. Mattock
2010-12-31  5:48                             ` [PATCH 15/15]drivers:staging:rtl8187se:ieee80211:ieee80211_tx.c " Justin P. Mattock

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox