Re: [PATCH trivial 2/4] rtlwifi: rtl8821ae: Spelling s/wrtie_addr/write_addr/
From: Geert Uytterhoeven
Date: Fri Feb 17 2017 - 11:20:06 EST
On Fri, Feb 17, 2017 at 5:14 PM, Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote:
> On 02/17/2017 09:36 AM, Geert Uytterhoeven wrote:
>> As per symmetry with _rtl8821ae_dbi_read(), "wrtie" is not a funky
>> register acronym, but a simple misspelling of "write".
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
>> Cc: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
>> Cc: Chaoming Li <chaoming_li@xxxxxxxxxxxxxx>
>> Cc: netdev@xxxxxxxxxxxxxxx
> I'm not sure that it is possible to have a typo in a variable name. That
> seems to be within an author's prerogative. Nonetheless, the patch is OK
> other than it was sent to the wrong destination. Patches for this driver
MAINTAINERS has your name for drivers/net/wireless/realtek/rtlwifi/.
Perhaps that line should be dropped?
> should go through linux-wireless@xxxxxxxxxxxxxxxx Kalle Valo is the wireless
> maintainer, but it is not necessary to send to him directly as he usually
> picks up the patches from Patchwork.
I guess Jiri (trivial) will pick it up...
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds