2013-01-22 16:49:11

by Stanislaw Gruszka

[permalink] [raw]
Subject: [PATCH] rt2x00: print chip and firmware version by default

Signed-off-by: Stanislaw Gruszka <[email protected]>
Tested-by: Xose Vazquez Perez <[email protected]>
---
drivers/net/wireless/rt2x00/rt2x00.h | 2 +-
drivers/net/wireless/rt2x00/rt2x00mac.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
index 3e37c19..bef449c 100644
--- a/drivers/net/wireless/rt2x00/rt2x00.h
+++ b/drivers/net/wireless/rt2x00/rt2x00.h
@@ -92,7 +92,7 @@
#define NOTICE(__dev, __msg, __args...) \
DEBUG_PRINTK(__dev, KERN_NOTICE, "Notice", __msg, ##__args)
#define INFO(__dev, __msg, __args...) \
- DEBUG_PRINTK(__dev, KERN_INFO, "Info", __msg, ##__args)
+ DEBUG_PRINTK_MSG(__dev, KERN_INFO, "Info", __msg, ##__args)
#define DEBUG(__dev, __msg, __args...) \
DEBUG_PRINTK(__dev, KERN_DEBUG, "Debug", __msg, ##__args)
#define EEPROM(__dev, __msg, __args...) \
diff --git a/drivers/net/wireless/rt2x00/rt2x00mac.c b/drivers/net/wireless/rt2x00/rt2x00mac.c
index ed7a1bb..20c6ecc 100644
--- a/drivers/net/wireless/rt2x00/rt2x00mac.c
+++ b/drivers/net/wireless/rt2x00/rt2x00mac.c
@@ -731,9 +731,9 @@ int rt2x00mac_conf_tx(struct ieee80211_hw *hw,
queue->aifs = params->aifs;
queue->txop = params->txop;

- INFO(rt2x00dev,
- "Configured TX queue %d - CWmin: %d, CWmax: %d, Aifs: %d, TXop: %d.\n",
- queue_idx, queue->cw_min, queue->cw_max, queue->aifs, queue->txop);
+ DEBUG(rt2x00dev,
+ "Configured TX queue %d - CWmin: %d, CWmax: %d, Aifs: %d, TXop: %d.\n",
+ queue_idx, queue->cw_min, queue->cw_max, queue->aifs, queue->txop);

return 0;
}
--
1.7.4.4



2013-01-22 22:22:58

by Gertjan van Wingerde

[permalink] [raw]
Subject: Re: [PATCH] rt2x00: print chip and firmware version by default

Xose,

Sent from my iPad

On 22 jan. 2013, at 22:12, Xose Vazquez Perez <[email protected]> wrote:

> On 01/22/2013 07:58 PM, Gertjan van Wingerde wrote:
>
>> On 22 jan. 2013, at 17:49, Stanislaw Gruszka <[email protected]> wrote:
>>
>>> Signed-off-by: Stanislaw Gruszka <[email protected]>
>>> Tested-by: Xose Vazquez Perez <[email protected]>
>>> ---
>>> drivers/net/wireless/rt2x00/rt2x00.h | 2 +-
>>> drivers/net/wireless/rt2x00/rt2x00mac.c | 6 +++---
>>> 2 files changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
>>> index 3e37c19..bef449c 100644
>>> --- a/drivers/net/wireless/rt2x00/rt2x00.h
>>> +++ b/drivers/net/wireless/rt2x00/rt2x00.h
>>> @@ -92,7 +92,7 @@
>>> #define NOTICE(__dev, __msg, __args...) \
>>> DEBUG_PRINTK(__dev, KERN_NOTICE, "Notice", __msg, ##__args)
>>> #define INFO(__dev, __msg, __args...) \
>>> - DEBUG_PRINTK(__dev, KERN_INFO, "Info", __msg, ##__args)
>>> + DEBUG_PRINTK_MSG(__dev, KERN_INFO, "Info", __msg, ##__args)
>>> #define DEBUG(__dev, __msg, __args...) \
>>> DEBUG_PRINTK(__dev, KERN_DEBUG, "Debug", __msg, ##__args)
>>> #define EEPROM(__dev, __msg, __args...) \
>
>> I think this change is good, but we should also change WARNING and
>> NOTICE. Otherwise they won't be displayed if CONFIG_RT2X00_DEBUG is
>> disabled, but INFO will, which would be strange.
>
> I believe this patch is so useful and trivial enough to be included,
> as is, in the next 3.8-rc .
>
> The next steps can be done for 3.9

To be honest, I wouldn't classify this as a bug, so 3.8-rc is out of the question. And doing a respin to add the other log levels can easily be done rather quickly, so there's no need to split it up.

---
Gertjan

2013-01-22 19:06:48

by Gertjan van Wingerde

[permalink] [raw]
Subject: Re: [PATCH] rt2x00: print chip and firmware version by default

Hi Stanislaw,

Sent from my iPad

On 22 jan. 2013, at 17:49, Stanislaw Gruszka <[email protected]> wrote:

> Signed-off-by: Stanislaw Gruszka <[email protected]>
> Tested-by: Xose Vazquez Perez <[email protected]>
> ---
> drivers/net/wireless/rt2x00/rt2x00.h | 2 +-
> drivers/net/wireless/rt2x00/rt2x00mac.c | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
> index 3e37c19..bef449c 100644
> --- a/drivers/net/wireless/rt2x00/rt2x00.h
> +++ b/drivers/net/wireless/rt2x00/rt2x00.h
> @@ -92,7 +92,7 @@
> #define NOTICE(__dev, __msg, __args...) \
> DEBUG_PRINTK(__dev, KERN_NOTICE, "Notice", __msg, ##__args)
> #define INFO(__dev, __msg, __args...) \
> - DEBUG_PRINTK(__dev, KERN_INFO, "Info", __msg, ##__args)
> + DEBUG_PRINTK_MSG(__dev, KERN_INFO, "Info", __msg, ##__args)
> #define DEBUG(__dev, __msg, __args...) \
> DEBUG_PRINTK(__dev, KERN_DEBUG, "Debug", __msg, ##__args)
> #define EEPROM(__dev, __msg, __args...) \

I think this change is good, but we should also change WARNING and NOTICE. Otherwise they won't be displayed if CONFIG_RT2X00_DEBUG is disabled, but INFO will, which would be strange.

> diff --git a/drivers/net/wireless/rt2x00/rt2x00mac.c b/drivers/net/wireless/rt2x00/rt2x00mac.c
> index ed7a1bb..20c6ecc 100644
> --- a/drivers/net/wireless/rt2x00/rt2x00mac.c
> +++ b/drivers/net/wireless/rt2x00/rt2x00mac.c
> @@ -731,9 +731,9 @@ int rt2x00mac_conf_tx(struct ieee80211_hw *hw,
> queue->aifs = params->aifs;
> queue->txop = params->txop;
>
> - INFO(rt2x00dev,
> - "Configured TX queue %d - CWmin: %d, CWmax: %d, Aifs: %d, TXop: %d.\n",
> - queue_idx, queue->cw_min, queue->cw_max, queue->aifs, queue->txop);
> + DEBUG(rt2x00dev,
> + "Configured TX queue %d - CWmin: %d, CWmax: %d, Aifs: %d, TXop: %d.\n",
> + queue_idx, queue->cw_min, queue->cw_max, queue->aifs, queue->txop);
>
> return 0;
> }
> --
> 1.7.4.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

2013-01-22 21:12:38

by Xose Vazquez Perez

[permalink] [raw]
Subject: Re: [PATCH] rt2x00: print chip and firmware version by default

On 01/22/2013 07:58 PM, Gertjan van Wingerde wrote:

> On 22 jan. 2013, at 17:49, Stanislaw Gruszka <[email protected]> wrote:
>
>> Signed-off-by: Stanislaw Gruszka <[email protected]>
>> Tested-by: Xose Vazquez Perez <[email protected]>
>> ---
>> drivers/net/wireless/rt2x00/rt2x00.h | 2 +-
>> drivers/net/wireless/rt2x00/rt2x00mac.c | 6 +++---
>> 2 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
>> index 3e37c19..bef449c 100644
>> --- a/drivers/net/wireless/rt2x00/rt2x00.h
>> +++ b/drivers/net/wireless/rt2x00/rt2x00.h
>> @@ -92,7 +92,7 @@
>> #define NOTICE(__dev, __msg, __args...) \
>> DEBUG_PRINTK(__dev, KERN_NOTICE, "Notice", __msg, ##__args)
>> #define INFO(__dev, __msg, __args...) \
>> - DEBUG_PRINTK(__dev, KERN_INFO, "Info", __msg, ##__args)
>> + DEBUG_PRINTK_MSG(__dev, KERN_INFO, "Info", __msg, ##__args)
>> #define DEBUG(__dev, __msg, __args...) \
>> DEBUG_PRINTK(__dev, KERN_DEBUG, "Debug", __msg, ##__args)
>> #define EEPROM(__dev, __msg, __args...) \
>

> I think this change is good, but we should also change WARNING and
> NOTICE. Otherwise they won't be displayed if CONFIG_RT2X00_DEBUG is
> disabled, but INFO will, which would be strange.

I believe this patch is so useful and trivial enough to be included,
as is, in the next 3.8-rc .

The next steps can be done for 3.9

2013-01-23 18:03:35

by Gertjan van Wingerde

[permalink] [raw]
Subject: Re: [PATCH] rt2x00: print chip and firmware version by default

Hi Stanislaw,

Sent from my iPad

On 22 jan. 2013, at 23:19, Stanislaw Gruszka <[email protected]> wrote:

> On Tue, Jan 22, 2013 at 10:12:32PM +0100, Xose Vazquez Perez wrote:
>> On 01/22/2013 07:58 PM, Gertjan van Wingerde wrote:
>>
>>> On 22 jan. 2013, at 17:49, Stanislaw Gruszka <[email protected]> wrote:
>>>
>>>> Signed-off-by: Stanislaw Gruszka <[email protected]>
>>>> Tested-by: Xose Vazquez Perez <[email protected]>
>>>> ---
>>>> drivers/net/wireless/rt2x00/rt2x00.h | 2 +-
>>>> drivers/net/wireless/rt2x00/rt2x00mac.c | 6 +++---
>>>> 2 files changed, 4 insertions(+), 4 deletions(-)
>>>>
>>>> diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
>>>> index 3e37c19..bef449c 100644
>>>> --- a/drivers/net/wireless/rt2x00/rt2x00.h
>>>> +++ b/drivers/net/wireless/rt2x00/rt2x00.h
>>>> @@ -92,7 +92,7 @@
>>>> #define NOTICE(__dev, __msg, __args...) \
>>>> DEBUG_PRINTK(__dev, KERN_NOTICE, "Notice", __msg, ##__args)
>>>> #define INFO(__dev, __msg, __args...) \
>>>> - DEBUG_PRINTK(__dev, KERN_INFO, "Info", __msg, ##__args)
>>>> + DEBUG_PRINTK_MSG(__dev, KERN_INFO, "Info", __msg, ##__args)
>>>> #define DEBUG(__dev, __msg, __args...) \
>>>> DEBUG_PRINTK(__dev, KERN_DEBUG, "Debug", __msg, ##__args)
>>>> #define EEPROM(__dev, __msg, __args...) \
>>
>>> I think this change is good, but we should also change WARNING and
>>> NOTICE. Otherwise they won't be displayed if CONFIG_RT2X00_DEBUG is
>>> disabled, but INFO will, which would be strange.
>
> We have those "TX status report missed" messages which spam logs,
> I think we should move them to DEBUG level then.
>
> NOTICE is only used in suspend/resume, it should be DEBUG too and NOTICE
> macro could be removed.
>
>> I believe this patch is so useful and trivial enough to be included,
>> as is, in the next 3.8-rc .
>>
>> The next steps can be done for 3.9
>
> If Gertjan will be so kind and ACK this patch, I'll post further
> changes as incremental patch :-)

As I mentioned to Xose already yesterday, I think this is only appropriate for -next, so we can do this properly. I can see this being handled as a series of patches, which first change the loglevels of the existing log lines, then remove NOTICE, and them change the macro of WARNING to not be covered by CONFIG_RT2X00_DEBUG.

---
Gertjan

2013-01-22 22:19:35

by Stanislaw Gruszka

[permalink] [raw]
Subject: Re: [PATCH] rt2x00: print chip and firmware version by default

On Tue, Jan 22, 2013 at 10:12:32PM +0100, Xose Vazquez Perez wrote:
> On 01/22/2013 07:58 PM, Gertjan van Wingerde wrote:
>
> > On 22 jan. 2013, at 17:49, Stanislaw Gruszka <[email protected]> wrote:
> >
> >> Signed-off-by: Stanislaw Gruszka <[email protected]>
> >> Tested-by: Xose Vazquez Perez <[email protected]>
> >> ---
> >> drivers/net/wireless/rt2x00/rt2x00.h | 2 +-
> >> drivers/net/wireless/rt2x00/rt2x00mac.c | 6 +++---
> >> 2 files changed, 4 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/drivers/net/wireless/rt2x00/rt2x00.h b/drivers/net/wireless/rt2x00/rt2x00.h
> >> index 3e37c19..bef449c 100644
> >> --- a/drivers/net/wireless/rt2x00/rt2x00.h
> >> +++ b/drivers/net/wireless/rt2x00/rt2x00.h
> >> @@ -92,7 +92,7 @@
> >> #define NOTICE(__dev, __msg, __args...) \
> >> DEBUG_PRINTK(__dev, KERN_NOTICE, "Notice", __msg, ##__args)
> >> #define INFO(__dev, __msg, __args...) \
> >> - DEBUG_PRINTK(__dev, KERN_INFO, "Info", __msg, ##__args)
> >> + DEBUG_PRINTK_MSG(__dev, KERN_INFO, "Info", __msg, ##__args)
> >> #define DEBUG(__dev, __msg, __args...) \
> >> DEBUG_PRINTK(__dev, KERN_DEBUG, "Debug", __msg, ##__args)
> >> #define EEPROM(__dev, __msg, __args...) \
> >
>
> > I think this change is good, but we should also change WARNING and
> > NOTICE. Otherwise they won't be displayed if CONFIG_RT2X00_DEBUG is
> > disabled, but INFO will, which would be strange.

We have those "TX status report missed" messages which spam logs,
I think we should move them to DEBUG level then.

NOTICE is only used in suspend/resume, it should be DEBUG too and NOTICE
macro could be removed.

> I believe this patch is so useful and trivial enough to be included,
> as is, in the next 3.8-rc .
>
> The next steps can be done for 3.9

If Gertjan will be so kind and ACK this patch, I'll post further
changes as incremental patch :-)

Stanislaw