From: Kalle Valo <kvalo@codeaurora.org>
To: Joe Perches <joe@perches.com>
Cc: yhchuang@realtek.com, Larry.Finger@lwfinger.net,
pkshih@realtek.com, tehuang@realtek.com, sgruszka@redhat.com,
johannes@sipsolutions.net, linux-wireless@vger.kernel.org
Subject: Re: [RFC v4 08/13] rtw88: debug files
Date: Thu, 15 Nov 2018 18:08:23 +0200 [thread overview]
Message-ID: <87zhua8gso.fsf@codeaurora.org> (raw)
In-Reply-To: <3946d05c0c026f3bdffaa35a686e5f12b8f3c59e.camel@perches.com> (Joe Perches's message of "Thu, 15 Nov 2018 07:19:56 -0800")
Joe Perches <joe@perches.com> writes:
> On Thu, 2018-11-15 at 16:15 +0200, Kalle Valo wrote:
>> Joe Perches <joe@perches.com> writes:
>>
>> > On Sat, 2018-10-13 at 13:28 -0700, Joe Perches wrote:
>> > > On Sat, 2018-10-13 at 18:23 +0300, Kalle Valo wrote:
>> > > > Joe Perches <joe@perches.com> writes:
>> > []
>> > > > > It's very unusual to have _all_ the logging under a CONFIG_<FOO>_DEBUG
>> > > > > config guard flag.
>> > > >
>> > > > For wireless drivers that is actually quite typical.
>> > >
>> > > No, it isn't.
>> > >
>> > > > IIRC at least ath6kl, ath9k and ath10k do that, most likely also others.
>> > >
>> > > No, they don't. Check again.
> []
>> > Kalle, did I miss a reply here?
>>
>> I didn't bother to waste my time after reading comments like "check
>> again" and "look harder".
>
> You are the maintainer here and you give advice that others are
> likely to follow.
>
> I was a bit frustrated after your fairly assertive, but wrong
> statement that the other drivers you maintain do the same thing.
Yeah, I can understand that. But do note that I'm a non-native english
speaker with an always full inbox, I don't have time nor skills to write
in a way expected by a native speaker. So please don't assume that I'm
assertive or other meanings from my style of text, just consider that my
writings as "simple english".
Also I do a lot of mistakes, and when I do please just clearly explain
it to me. I'm always grateful about that, all constructive help is very
welcome.
--
Kalle Valo
next prev parent reply other threads:[~2018-11-15 16:08 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-13 9:00 [RFC v4 00/13] rtw88: mac80211 driver for Realtek 802.11ac wireless network chips yhchuang
2018-10-13 9:00 ` [RFC v4 01/13] rtw88: main files yhchuang
2018-10-13 9:00 ` [RFC v4 02/13] rtw88: core files yhchuang
2018-10-13 9:00 ` [RFC v4 03/13] rtw88: hci files yhchuang
2018-10-13 9:00 ` [RFC v4 04/13] rtw88: trx files yhchuang
2018-10-13 9:00 ` [RFC v4 05/13] rtw88: mac files yhchuang
2018-10-16 8:22 ` Stanislaw Gruszka
2018-10-13 9:00 ` [RFC v4 06/13] rtw88: fw and efuse files yhchuang
2018-10-16 8:25 ` Stanislaw Gruszka
2018-10-13 9:00 ` [RFC v4 07/13] rtw88: phy files yhchuang
2018-10-13 9:00 ` [RFC v4 08/13] rtw88: debug files yhchuang
2018-10-13 14:19 ` Joe Perches
2018-10-13 15:23 ` Kalle Valo
2018-10-13 20:28 ` Joe Perches
2018-11-04 4:45 ` Joe Perches
2018-11-15 14:15 ` Kalle Valo
2018-11-15 15:19 ` Joe Perches
2018-11-15 16:08 ` Kalle Valo [this message]
2018-10-13 9:00 ` [RFC v4 09/13] rtw88: chip files yhchuang
2018-10-13 9:00 ` [RFC v4 10/13] rtw88: 8822B init table yhchuang
2018-10-13 9:00 ` [RFC v4 11/13] rtw88: 8822C " yhchuang
2018-10-13 9:00 ` [RFC v4 12/13] rtw88: Kconfig & Makefile yhchuang
2018-10-13 9:00 ` [RFC v4 13/13] rtw88: add support for Realtek 802.11ac wireless chips yhchuang
2018-10-16 8:28 ` [RFC v4 00/13] rtw88: mac80211 driver for Realtek 802.11ac wireless network chips Stanislaw Gruszka
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=87zhua8gso.fsf@codeaurora.org \
--to=kvalo@codeaurora.org \
--cc=Larry.Finger@lwfinger.net \
--cc=joe@perches.com \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=pkshih@realtek.com \
--cc=sgruszka@redhat.com \
--cc=tehuang@realtek.com \
--cc=yhchuang@realtek.com \
/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;
as well as URLs for NNTP newsgroup(s).