Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: kvalo=codeaurora.org@mg.codeaurora.org
DMARC-Filter: OpenDMARC Filter v1.4.1 smtp.codeaurora.org D52E9C4338F
From:   Kalle Valo <kvalo@codeaurora.org>
To:     Martin Fuzzey <martin.fuzzey@flowbird.group>
Cc:     linux-wireless@vger.kernel.org,
        Amitkumar Karwar <amitkarwar@gmail.com>,
        Siva Rebbagondla <siva8118@gmail.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Prameela Rani Garnepudi <prameela.j04cs@gmail.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/3] rsi: fix startup and P2P mode
References: <1630337206-12410-1-git-send-email-martin.fuzzey@flowbird.group>
Date:   Tue, 21 Sep 2021 08:49:05 +0300
In-Reply-To: <1630337206-12410-1-git-send-email-martin.fuzzey@flowbird.group>
        (Martin Fuzzey's message of "Mon, 30 Aug 2021 17:26:43 +0200")
Message-ID: <875yuuwka6.fsf@codeaurora.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Martin Fuzzey <martin.fuzzey@flowbird.group> writes:

> This series fixes an occasional startup failure when BT
> coexistence is enabled and makes P2P mode work
> (the two are unrelated but sending as a series since the same files
> are modified).
>
> Tested on RS9116 in P2P client mode against an Android phone (S8, Android 9)
>
> For an unknown reason GO mode does not work against the phone but it
> does work against a second RS9116 device.
> In this case Android does not send a response to the 3rd WPA handshake
> packet. It is difficult to debug this without the phone wpa_supplicant
> logs which I don't have.
>
> Regression tested OK for STA and AP modes.
>
> V2:
>   * rebase against wireless-drivers-next
>   * add cc: stable to patches 2 & 3
>
> Martin Fuzzey (3):
>   rsi: fix occasional initialisation failure with BT coex
>   rsi: fix key enabled check causing unwanted encryption for vap_id > 0
>   rsi: fix rate mask set leading to P2P failure

BTW thanks for the outstanding cover letter and commit logs in the
patches, it was so easy for me to understand the background and the
level of testing these patches have had. If everyone would write their
patches like this my job would be a lot easier :)

I'm going to use this patchset as an example in the submittingpatches
wiki page.

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

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches