From: "Justin P. Mattock" <justinmattock@gmail.com>
To: trivial@kernel.org
Cc: linux-kernel@vger.kernel.org, gregkh@suse.de,
"Justin P. Mattock" <justinmattock@gmail.com>
Subject: [PATCH 03/15]drivers:staging:rtl8187se:r8180_dm.c Fix some comments.
Date: Thu, 30 Dec 2010 21:48:39 -0800 [thread overview]
Message-ID: <1293774531-5294-4-git-send-email-justinmattock@gmail.com> (raw)
In-Reply-To: <1293774531-5294-3-git-send-email-justinmattock@gmail.com>
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
next prev parent reply other threads:[~2010-12-31 5:48 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
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 ` Justin P. Mattock [this message]
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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1293774531-5294-4-git-send-email-justinmattock@gmail.com \
--to=justinmattock@gmail.com \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=trivial@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox