Re: [PATCH] iwlwifi: Move each system tracepoints to their own header

2015-04-12 Thread Emmanuel Grumbach
On Sun, Apr 12, 2015 at 8:37 PM, Emmanuel Grumbach  wrote:
> On Wed, Apr 8, 2015 at 10:44 AM, Johannes Berg
>  wrote:
>> On Tue, 2015-04-07 at 12:13 -0400, Steven Rostedt wrote:
>>> Every tracing file must have its own TRACE_SYSTEM defined.
>>> The iwlwifi tracepoint header broke this and added in the middle
>>> of the file:
>> [...]
>>> Unfortunately, this broke new code in the ftrace infrastructure.
>>> Moving each of these TRACE_SYSTEMs into their own trace file with
>>> just one TRACE_SYSTEM per file fixes the issue.
>>>
>>> Cc: Johannes Berg 
>>> Signed-off-by: Steven Rostedt 
>>
>>
>>>  drivers/net/wireless/iwlwifi/iwl-devtrace-data.h   |  79 
>>>  drivers/net/wireless/iwlwifi/iwl-devtrace-io.h | 155 
>>>  .../net/wireless/iwlwifi/iwl-devtrace-iwlwifi.h| 200 ++
>>>  drivers/net/wireless/iwlwifi/iwl-devtrace-msg.h|  97 +
>>>  drivers/net/wireless/iwlwifi/iwl-devtrace-ucode.h  |  81 
>>>  drivers/net/wireless/iwlwifi/iwl-devtrace.h| 438 
>>> +
>>>  6 files changed, 618 insertions(+), 432 deletions(-)
>>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-data.h
>>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-io.h
>>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-iwlwifi.h
>>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-msg.h
>>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-ucode.h
>>
>> Looks good to me.
>>
>> Reviewed-by: Johannes Berg 
>>
>
> I had to make quite a few modifications since this patch was not based
> on iwlwifi-next, but I picked it up in our internal tree.

Wait - you seemed to want to apply that on your tree because of
dependencies (which I can understand). So how do you suggest to
proceed?
You can apply your original version of the patch on your tree, but
then it'll collapse when it'll meet iwlwifi-next.git.
I can try to have the modified version of the patch into 4.1 but I
doubt I'll make it. If I do make it, it'd allow you to work on top of
4.1 whenever that'll be ready? What is your timeline?

Or maybe you can pull iwlwifi-next (which would be odd I have to say)?
What do you suggest?
FWIW - I am talking about:

commit 7e1223b50089ab5901215d2fd8c61b42c7cfe034
Author: Emmanuel Grumbach 
Date:   Tue Feb 3 20:11:48 2015 +0200

iwlwifi: mvm: new Alive / error table API

The new API slightly changes the layout of the version of
the firmware - prepare for that.

Signed-off-by: Emmanuel Grumbach 
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


http://worldfreedom.ch/

2015-04-12 Thread contact
http://worldfreedom.ch/
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: DRIVER FOR INTEL2200BG

2015-04-12 Thread Julian Calaby
Hi,

On Mon, Apr 13, 2015 at 10:39 AM, J.H. Schmidt  wrote:
> My IBM(Lenovo) laptop running XP crashed a year ago and I changed to Lupuplus 
> which
> runs on RAM.No problems with older Sprint modem and D link router running Wi 
> Fi.Enter
> a nearby lightning strike which killed the router.Century link gave us a new 
> modem router
> combo which works fine with with 2 computes r running 7 and 2 IMacs.I can't 
> get the
> Linux machine to work with the new Century Link combo modem router-Any ideas?

I'm not sure I understand your problem.

1. You're running some RAM based Linux distro on your Thinkpad
2. It has an Intel 2200bg card
3. You've gotten a new router
4. Now it doesn't connect

If you were connecting to the old router using WiFi, your card is
already working.

I'm guessing your new router is much newer than the old one. Is it an
"n" router? (I.e. supports 802.11n, WiFi speeds above 54Mbit/s) Some
of them default to "n" only operation and need a setting turned on to
allow "b" or "g" devices to connect.

I'd investigate the settings of the router and ensure that
compatibility for older ("g") devices is turned on.

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: pull-request: mac80211-next 2015-04-10

2015-04-12 Thread David Miller
From: Johannes Berg 
Date: Fri, 10 Apr 2015 13:18:58 +0200

> I realize it's late, but what I've got right now has so many fixes that
> I'd be sending even after the merge window, so it felt like there was
> little point in holding it up until after then. The new "intermediate
> queue" feature is unused right now so seems safe enough also, and having
> it in will help modify drivers to use it.
> There's also some rate control refactoring that, as far as I can tell,
> is uncontroversial and over the last couple of weeks has been has been
> reviewed and tested by people who use it, so I'm including it.
> 
> Let me know if you want me to change anything here or if there are any
> other problems.

Pulled, thanks Johannes.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


DRIVER FOR INTEL2200BG

2015-04-12 Thread J.H. Schmidt
My IBM(Lenovo) laptop running XP crashed a year ago and I changed to Lupuplus 
which runs on RAM.No problems with older Sprint modem and D link router running 
Wi Fi.Enter a nearby lightning strike which killed the router.Century link gave 
us a new modem router combo which works fine with with 2 computes r running 7 
and 2 IMacs.I can't get the Linux machine to work with the new Century Link 
combo modem router-Any ideas?--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 5/5 v4] netfilter: Fix format string of nfnetlink_log proc file

2015-04-12 Thread Richard Weinberger
The printed values are all of type unsigned integer, therefore use
%u instead of %d. Otherwise an user can face negative values.

Signed-off-by: Richard Weinberger 
---
 net/netfilter/nfnetlink_log.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c
index 5c79735..3ad9126 100644
--- a/net/netfilter/nfnetlink_log.c
+++ b/net/netfilter/nfnetlink_log.c
@@ -1007,7 +1007,7 @@ static int seq_show(struct seq_file *s, void *v)
 {
const struct nfulnl_instance *inst = v;
 
-   seq_printf(s, "%5d %6d %5d %1d %5d %6d %2d\n",
+   seq_printf(s, "%5u %6u %5u %1u %5u %6u %2u\n",
   inst->group_num,
   inst->peer_portid, inst->qlen,
   inst->copy_mode, inst->copy_range,
-- 
1.8.4.5

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 1/5 v4] netlink: Fix portid type in netlink_notify

2015-04-12 Thread Richard Weinberger
portid is an unsigned integer. Fix netlink_notify to
match all other portid user in the kernel.

Signed-off-by: Richard Weinberger 
---
 include/linux/netlink.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/netlink.h b/include/linux/netlink.h
index 02fc86d..6835c12 100644
--- a/include/linux/netlink.h
+++ b/include/linux/netlink.h
@@ -134,7 +134,7 @@ struct netlink_callback {
 
 struct netlink_notify {
struct net *net;
-   int portid;
+   u32 portid;
int protocol;
 };
 
-- 
1.8.4.5

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 4/5 v4] netfilter: Fix format string of nfnetlink_queue proc file

2015-04-12 Thread Richard Weinberger
The printed values are all of type unsigned integer, therefore use
%u instead of %d. Otherwise an user can face negative values.

Fixes:
$ cat /proc/net/netfilter/nfnetlink_queue
0  29508   278 2 65531 0 2004213241 -2129885586  1
1 -27747 0 2 65531 0 00  1
2 -27748 0 2 65531 0 00  1

Signed-off-by: Richard Weinberger 
---
 net/netfilter/nfnetlink_queue_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/netfilter/nfnetlink_queue_core.c 
b/net/netfilter/nfnetlink_queue_core.c
index 49267f8..0b98c74 100644
--- a/net/netfilter/nfnetlink_queue_core.c
+++ b/net/netfilter/nfnetlink_queue_core.c
@@ -1251,7 +1251,7 @@ static int seq_show(struct seq_file *s, void *v)
 {
const struct nfqnl_instance *inst = v;
 
-   seq_printf(s, "%5d %6d %5d %1d %5d %5d %5d %8d %2d\n",
+   seq_printf(s, "%5u %6u %5u %1u %5u %5u %5u %8u %2d\n",
   inst->queue_num,
   inst->peer_portid, inst->queue_total,
   inst->copy_mode, inst->copy_range,
-- 
1.8.4.5

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 3/5 v4] netfilter: Fix portid types

2015-04-12 Thread Richard Weinberger
The netlink portid is an unsigned integer, use this type
also in netfilter.

Signed-off-by: Richard Weinberger 
---
 net/netfilter/nfnetlink_log.c| 4 ++--
 net/netfilter/nfnetlink_queue_core.c | 7 +++
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c
index 51afea4..5c79735 100644
--- a/net/netfilter/nfnetlink_log.c
+++ b/net/netfilter/nfnetlink_log.c
@@ -63,7 +63,7 @@ struct nfulnl_instance {
struct timer_list timer;
struct net *net;
struct user_namespace *peer_user_ns;/* User namespace of the peer 
process */
-   int peer_portid;/* PORTID of the peer process */
+   u32 peer_portid;/* PORTID of the peer process */
 
/* configurable parameters */
unsigned int flushtimeout;  /* timeout until queue flush */
@@ -152,7 +152,7 @@ static void nfulnl_timer(unsigned long data);
 
 static struct nfulnl_instance *
 instance_create(struct net *net, u_int16_t group_num,
-   int portid, struct user_namespace *user_ns)
+   u32 portid, struct user_namespace *user_ns)
 {
struct nfulnl_instance *inst;
struct nfnl_log_net *log = nfnl_log_pernet(net);
diff --git a/net/netfilter/nfnetlink_queue_core.c 
b/net/netfilter/nfnetlink_queue_core.c
index 628afc3..49267f8 100644
--- a/net/netfilter/nfnetlink_queue_core.c
+++ b/net/netfilter/nfnetlink_queue_core.c
@@ -55,7 +55,7 @@ struct nfqnl_instance {
struct hlist_node hlist;/* global list of queues */
struct rcu_head rcu;
 
-   int peer_portid;
+   u32 peer_portid;
unsigned int queue_maxlen;
unsigned int copy_range;
unsigned int queue_dropped;
@@ -110,8 +110,7 @@ instance_lookup(struct nfnl_queue_net *q, u_int16_t 
queue_num)
 }
 
 static struct nfqnl_instance *
-instance_create(struct nfnl_queue_net *q, u_int16_t queue_num,
-   int portid)
+instance_create(struct nfnl_queue_net *q, u_int16_t queue_num, u32 portid)
 {
struct nfqnl_instance *inst;
unsigned int h;
@@ -870,7 +869,7 @@ static const struct nla_policy 
nfqa_verdict_batch_policy[NFQA_MAX+1] = {
 };
 
 static struct nfqnl_instance *
-verdict_instance_lookup(struct nfnl_queue_net *q, u16 queue_num, int nlportid)
+verdict_instance_lookup(struct nfnl_queue_net *q, u16 queue_num, u32 nlportid)
 {
struct nfqnl_instance *queue;
 
-- 
1.8.4.5

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 2/5 v4] nfc: Fix portid type in urelease_work

2015-04-12 Thread Richard Weinberger
portid is an unsigned integer. Fix urelease_work to
match all other portid user in the kernel.

Signed-off-by: Richard Weinberger 
---
 net/nfc/netlink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/nfc/netlink.c b/net/nfc/netlink.c
index 14a2d11..3763036 100644
--- a/net/nfc/netlink.c
+++ b/net/nfc/netlink.c
@@ -1584,7 +1584,7 @@ static const struct genl_ops nfc_genl_ops[] = {
 
 struct urelease_work {
struct  work_struct w;
-   int portid;
+   u32 portid;
 };
 
 static void nfc_urelease_event_work(struct work_struct *work)
-- 
1.8.4.5

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] iwlwifi: Move each system tracepoints to their own header

2015-04-12 Thread Emmanuel Grumbach
On Wed, Apr 8, 2015 at 10:44 AM, Johannes Berg
 wrote:
> On Tue, 2015-04-07 at 12:13 -0400, Steven Rostedt wrote:
>> Every tracing file must have its own TRACE_SYSTEM defined.
>> The iwlwifi tracepoint header broke this and added in the middle
>> of the file:
> [...]
>> Unfortunately, this broke new code in the ftrace infrastructure.
>> Moving each of these TRACE_SYSTEMs into their own trace file with
>> just one TRACE_SYSTEM per file fixes the issue.
>>
>> Cc: Johannes Berg 
>> Signed-off-by: Steven Rostedt 
>
>
>>  drivers/net/wireless/iwlwifi/iwl-devtrace-data.h   |  79 
>>  drivers/net/wireless/iwlwifi/iwl-devtrace-io.h | 155 
>>  .../net/wireless/iwlwifi/iwl-devtrace-iwlwifi.h| 200 ++
>>  drivers/net/wireless/iwlwifi/iwl-devtrace-msg.h|  97 +
>>  drivers/net/wireless/iwlwifi/iwl-devtrace-ucode.h  |  81 
>>  drivers/net/wireless/iwlwifi/iwl-devtrace.h| 438 
>> +
>>  6 files changed, 618 insertions(+), 432 deletions(-)
>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-data.h
>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-io.h
>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-iwlwifi.h
>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-msg.h
>>  create mode 100644 drivers/net/wireless/iwlwifi/iwl-devtrace-ucode.h
>
> Looks good to me.
>
> Reviewed-by: Johannes Berg 
>

I had to make quite a few modifications since this patch was not based
on iwlwifi-next, but I picked it up in our internal tree.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH V2] rtlwifi: misspelled code and comments corrected.

2015-04-12 Thread Cheolhyun Park
Signed-off-by: Cheolhyun Park 
---
 drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c | 2 +-
 drivers/net/wireless/rtlwifi/rtl8192de/dm.c   | 2 +-
 drivers/net/wireless/rtlwifi/rtl8821ae/dm.c   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c 
b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
index f5ee67c..1273997 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
@@ -312,7 +312,7 @@ static void rtl92c_dm_ctrl_initgain_by_rssi(struct 
ieee80211_hw *hw)
struct dig_t *digtable = &rtlpriv->dm_digtable;
u32 isbt;
 
-   /* modify DIG lower bound, deal with abnorally large false alarm */
+   /* modify DIG lower bound, deal with abnormally large false alarm */
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
digtable->large_fa_hit++;
if (digtable->forbidden_igi < digtable->cur_igvalue) {
diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
index a1be5a6..f209cda 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
@@ -420,7 +420,7 @@ static void rtl92d_dm_dig(struct ieee80211_hw *hw)
 "dm_DIG() Before: Recover_cnt=%d, rx_gain_min=%x\n",
 de_digtable->recover_cnt, de_digtable->rx_gain_min);
 
-   /* deal with abnorally large false alarm */
+   /* deal with abnormally large false alarm */
if (falsealm_cnt->cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
 "dm_DIG(): Abnormally false alarm case\n");
diff --git a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
index 0b2082d..342678d 100644
--- a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
@@ -873,7 +873,7 @@ static void rtl8821ae_dm_dig(struct ieee80211_hw *hw)
 
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
-"Abnornally false alarm case.\n");
+"Abnormally false alarm case.\n");
 
if (dm_digtable->large_fa_hit != 3)
dm_digtable->large_fa_hit++;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] rtlwifi: misspelled code and comments corrected.

2015-04-12 Thread Larry Finger

On 04/12/2015 11:04 AM, Cheolhyun Park wrote:

Signed-off-by: Cheolhyun Park 


When you send a corrected patch, it is common to indicate that this is a 
corrected patch by putting the version number inside the [] in the subject, and 
to indicate the changes inside the patch, but after the --- line. Neither of 
these annotations will be included in the commit message, but they help the 
maintainer keep track of the patch, and whether you have satisfied the 
reviewer's comments. Thus this patch should have had the subject "[PATCH V2] 
rtlwifi: ..." and you should have had the lines below after the ---:

---


V2 - Fixed the typo introduced in V1 as noted by Julia Lawall.
---



  drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c | 2 +-
  drivers/net/wireless/rtlwifi/rtl8192de/dm.c   | 2 +-
  drivers/net/wireless/rtlwifi/rtl8821ae/dm.c   | 2 +-
  3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c 
b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
index f5ee67c..1273997 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
@@ -312,7 +312,7 @@ static void rtl92c_dm_ctrl_initgain_by_rssi(struct 
ieee80211_hw *hw)
struct dig_t *digtable = &rtlpriv->dm_digtable;
u32 isbt;

-   /* modify DIG lower bound, deal with abnorally large false alarm */
+   /* modify DIG lower bound, deal with abnormally large false alarm */
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
digtable->large_fa_hit++;
if (digtable->forbidden_igi < digtable->cur_igvalue) {
diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
index a1be5a6..f209cda 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
@@ -420,7 +420,7 @@ static void rtl92d_dm_dig(struct ieee80211_hw *hw)
 "dm_DIG() Before: Recover_cnt=%d, rx_gain_min=%x\n",
 de_digtable->recover_cnt, de_digtable->rx_gain_min);

-   /* deal with abnorally large false alarm */
+   /* deal with abnormally large false alarm */
if (falsealm_cnt->cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
 "dm_DIG(): Abnormally false alarm case\n");
diff --git a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
index 0b2082d..342678d 100644
--- a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
@@ -873,7 +873,7 @@ static void rtl8821ae_dm_dig(struct ieee80211_hw *hw)

if (rtlpriv->falsealm_cnt.cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
-"Abnornally false alarm case.\n");
+"Abnormally false alarm case.\n");

if (dm_digtable->large_fa_hit != 3)
dm_digtable->large_fa_hit++;



--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] rtlwifi: misspelled code and comments corrected.

2015-04-12 Thread Cheolhyun Park
Signed-off-by: Cheolhyun Park 
---
 drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c | 2 +-
 drivers/net/wireless/rtlwifi/rtl8192de/dm.c   | 2 +-
 drivers/net/wireless/rtlwifi/rtl8821ae/dm.c   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c 
b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
index f5ee67c..1273997 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
@@ -312,7 +312,7 @@ static void rtl92c_dm_ctrl_initgain_by_rssi(struct 
ieee80211_hw *hw)
struct dig_t *digtable = &rtlpriv->dm_digtable;
u32 isbt;
 
-   /* modify DIG lower bound, deal with abnorally large false alarm */
+   /* modify DIG lower bound, deal with abnormally large false alarm */
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
digtable->large_fa_hit++;
if (digtable->forbidden_igi < digtable->cur_igvalue) {
diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
index a1be5a6..f209cda 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
@@ -420,7 +420,7 @@ static void rtl92d_dm_dig(struct ieee80211_hw *hw)
 "dm_DIG() Before: Recover_cnt=%d, rx_gain_min=%x\n",
 de_digtable->recover_cnt, de_digtable->rx_gain_min);
 
-   /* deal with abnorally large false alarm */
+   /* deal with abnormally large false alarm */
if (falsealm_cnt->cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
 "dm_DIG(): Abnormally false alarm case\n");
diff --git a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
index 0b2082d..342678d 100644
--- a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
@@ -873,7 +873,7 @@ static void rtl8821ae_dm_dig(struct ieee80211_hw *hw)
 
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
-"Abnornally false alarm case.\n");
+"Abnormally false alarm case.\n");
 
if (dm_digtable->large_fa_hit != 3)
dm_digtable->large_fa_hit++;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] rtlwifi: misspelled code and comments corrected.

2015-04-12 Thread Julia Lawall


On Sun, 12 Apr 2015, Cheolhyun Park wrote:

> Signed-off-by: Cheolhyun Park 
> ---
>  drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c | 2 +-
>  drivers/net/wireless/rtlwifi/rtl8192de/dm.c   | 2 +-
>  drivers/net/wireless/rtlwifi/rtl8821ae/dm.c   | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c 
> b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
> index f5ee67c..1273997 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
> @@ -312,7 +312,7 @@ static void rtl92c_dm_ctrl_initgain_by_rssi(struct 
> ieee80211_hw *hw)
>   struct dig_t *digtable = &rtlpriv->dm_digtable;
>   u32 isbt;
>
> - /* modify DIG lower bound, deal with abnorally large false alarm */
> + /* modify DIG lower bound, deal with abnormally arge false alarm */

You have converted large to arge.

julia

>   if (rtlpriv->falsealm_cnt.cnt_all > 1) {
>   digtable->large_fa_hit++;
>   if (digtable->forbidden_igi < digtable->cur_igvalue) {
> diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c 
> b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
> index a1be5a6..f209cda 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
> @@ -420,7 +420,7 @@ static void rtl92d_dm_dig(struct ieee80211_hw *hw)
>"dm_DIG() Before: Recover_cnt=%d, rx_gain_min=%x\n",
>de_digtable->recover_cnt, de_digtable->rx_gain_min);
>
> - /* deal with abnorally large false alarm */
> + /* deal with abnormally large false alarm */
>   if (falsealm_cnt->cnt_all > 1) {
>   RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
>"dm_DIG(): Abnormally false alarm case\n");
> diff --git a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c 
> b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
> index 0b2082d..342678d 100644
> --- a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
> +++ b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
> @@ -873,7 +873,7 @@ static void rtl8821ae_dm_dig(struct ieee80211_hw *hw)
>
>   if (rtlpriv->falsealm_cnt.cnt_all > 1) {
>   RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
> -  "Abnornally false alarm case.\n");
> +  "Abnormally false alarm case.\n");
>
>   if (dm_digtable->large_fa_hit != 3)
>   dm_digtable->large_fa_hit++;
> --
> 1.9.1
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] rtlwifi: misspelled code and comments corrected.

2015-04-12 Thread Cheolhyun Park
Signed-off-by: Cheolhyun Park 
---
 drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c | 2 +-
 drivers/net/wireless/rtlwifi/rtl8192de/dm.c   | 2 +-
 drivers/net/wireless/rtlwifi/rtl8821ae/dm.c   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c 
b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
index f5ee67c..1273997 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192c/dm_common.c
@@ -312,7 +312,7 @@ static void rtl92c_dm_ctrl_initgain_by_rssi(struct 
ieee80211_hw *hw)
struct dig_t *digtable = &rtlpriv->dm_digtable;
u32 isbt;
 
-   /* modify DIG lower bound, deal with abnorally large false alarm */
+   /* modify DIG lower bound, deal with abnormally arge false alarm */
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
digtable->large_fa_hit++;
if (digtable->forbidden_igi < digtable->cur_igvalue) {
diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
index a1be5a6..f209cda 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/dm.c
@@ -420,7 +420,7 @@ static void rtl92d_dm_dig(struct ieee80211_hw *hw)
 "dm_DIG() Before: Recover_cnt=%d, rx_gain_min=%x\n",
 de_digtable->recover_cnt, de_digtable->rx_gain_min);
 
-   /* deal with abnorally large false alarm */
+   /* deal with abnormally large false alarm */
if (falsealm_cnt->cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
 "dm_DIG(): Abnormally false alarm case\n");
diff --git a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c 
b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
index 0b2082d..342678d 100644
--- a/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
+++ b/drivers/net/wireless/rtlwifi/rtl8821ae/dm.c
@@ -873,7 +873,7 @@ static void rtl8821ae_dm_dig(struct ieee80211_hw *hw)
 
if (rtlpriv->falsealm_cnt.cnt_all > 1) {
RT_TRACE(rtlpriv, COMP_DIG, DBG_LOUD,
-"Abnornally false alarm case.\n");
+"Abnormally false alarm case.\n");
 
if (dm_digtable->large_fa_hit != 3)
dm_digtable->large_fa_hit++;
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html