Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Kalle Valo <kvalo@kernel.org>
To:     Ping-Ke Shih <pkshih@realtek.com>
Cc:     Jun ASAKA <JunASAKA@zzy040330.moe>,
        "Jes.Sorensen\@gmail.com" <Jes.Sorensen@gmail.com>,
        "davem\@davemloft.net" <davem@davemloft.net>,
        "edumazet\@google.com" <edumazet@google.com>,
        "kuba\@kernel.org" <kuba@kernel.org>,
        "pabeni\@redhat.com" <pabeni@redhat.com>,
        "linux-wireless\@vger.kernel.org" <linux-wireless@vger.kernel.org>,
        "netdev\@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel\@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v5] wifi: rtl8xxxu: fixing IQK failures for rtl8192eu
References: <20221207033926.11777-1-JunASAKA@zzy040330.moe>
        <2ac07b1d6e06443b95befb79d27549d2@realtek.com>
        <b4b65c74-792f-4df1-18bf-5c6f80845814@zzy040330.moe>
        <159ac3a296164b05b319bfb254a7901b@realtek.com>
Date:   Wed, 07 Dec 2022 14:31:28 +0200
In-Reply-To: <159ac3a296164b05b319bfb254a7901b@realtek.com> (Ping-Ke Shih's
        message of "Wed, 7 Dec 2022 03:55:16 +0000")
Message-ID: <87h6y7mm3j.fsf@kernel.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Ping-Ke Shih <pkshih@realtek.com> writes:

>> >> Fixing "Path A RX IQK failed" and "Path B RX IQK failed"
>> >> issues for rtl8192eu chips by replacing the arguments with
>> >> the ones in the updated official driver as shown below.
>> >> 1. https://github.com/Mange/rtl8192eu-linux-driver
>> >> 2. vendor driver version: 5.6.4
>> >>
>> >> Tested-by: Jun ASAKA <JunASAKA@zzy040330.moe>
>> >> Signed-off-by: Jun ASAKA <JunASAKA@zzy040330.moe>

The assumption is that the patch submitter tests the patches, so a
separate Tested-by tag is not needed. I'll remove it.

>> >> Reviewed-by: Ping-Ke Shih <pkshih@realtek.com>
>> >> ---
>> >> v5:
>> >>   - no modification.
>> > Then, why do you need v5?
>> Well,=C2=A0 I just want to add the "Reviewed-By" line to the commit mess=
age.
>> Sorry for the noise if there is no need to do that.
>
> No need to add "Reviewed-By". Kalle will add it when this patch gets merg=
ed.

Yeah. And to be precise patchwork actually does that automatically, a
very nice feature :)

--=20
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatc=
hes