[PATCH 02/15]drivers:staging:rtl8187se:r8180_core.c Fix some comments.

Previous thread: [PATCH] Fix the HGA framebuffer driver by Brent Cook on Thursday, December 30, 2010 - 10:43 pm. (1 message)

Next thread: Re: is there a fix for ubuntu with intel GPU booting to blank screen? by Anca Emanuel on Thursday, December 30, 2010 - 11:32 pm. (7 messages)
From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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
--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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 ...
From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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 ...
From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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 = ...
From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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

--

From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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 ...
From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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 ...
From: Justin P. Mattock
Date: Thursday, December 30, 2010 - 10:48 pm

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 ...
Previous thread: [PATCH] Fix the HGA framebuffer driver by Brent Cook on Thursday, December 30, 2010 - 10:43 pm. (1 message)

Next thread: Re: is there a fix for ubuntu with intel GPU booting to blank screen? by Anca Emanuel on Thursday, December 30, 2010 - 11:32 pm. (7 messages)