Search Linux Wireless

Re: [PATCH v4 02/12] wcn36xx: Run suspend for the first ieee80211_vif

To: Loic Poulain <loic.poulain@xxxxxxxxxx>, "Bryan O'Donoghue" <bryan.odonoghue@xxxxxxxxxx>
Subject: Re: [PATCH v4 02/12] wcn36xx: Run suspend for the first ieee80211_vif
From: "Bryan O'Donoghue" <pure.logic@xxxxxxxxxxxxxxxxx>
Date: Tue, 1 Jun 2021 11:10:53 +0100
Cc: Kalle Valo <kvalo@xxxxxxxxxxxxxx>, wcn36xx@xxxxxxxxxxxxxxxxxxx, linux-wireless@xxxxxxxxxxxxxxx, Shawn Guo <shawn.guo@xxxxxxxxxx>, Benjamin Li <benl@xxxxxxxxxxxx>, Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
In-reply-to: <CAMZdPi9VOPdz_4nhjDC1o49eobRPZRkniKtWc4ZLOiGEH3nP6w@mail.gmail.com>
References: <20210601024920.1424144-1-bryan.odonoghue@linaro.org> <20210601024920.1424144-3-bryan.odonoghue@linaro.org> <CAMZdPi9VOPdz_4nhjDC1o49eobRPZRkniKtWc4ZLOiGEH3nP6w@mail.gmail.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1

On 01/06/2021 08:09, Loic Poulain wrote:

Where is the balanced mutex_unlock?


Ah - I've mixed patch #2 and patch #3

References:
- [PATCH v4 00/12] wcn36xx: Enable downstream consistent Wake on Lan
  - From: Bryan O'Donoghue
- [PATCH v4 02/12] wcn36xx: Run suspend for the first ieee80211_vif
  - From: Bryan O'Donoghue
- Re: [PATCH v4 02/12] wcn36xx: Run suspend for the first ieee80211_vif
  - From: Loic Poulain