Search Linux Wireless

Re: [PATCH 3/3] wcn36xx: Implement downstream compliant beacon filtering

To: Loic Poulain <loic.poulain@xxxxxxxxxx>
Subject: Re: [PATCH 3/3] wcn36xx: Implement downstream compliant beacon filtering
From: "Bryan O'Donoghue" <bryan.odonoghue@xxxxxxxxxx>
Date: Tue, 14 Dec 2021 11:59:01 +0000
Cc: kvalo@xxxxxxxxxxxxxx, linux-wireless@xxxxxxxxxxxxxxx, wcn36xx@xxxxxxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, benl@xxxxxxxxxxxx
In-reply-to: <CAMZdPi82xqRGn+oZwVX5wgutEDO1qpufSra2oBW9S4pALHHAdw@mail.gmail.com>
References: <20211213135127.1656258-1-bryan.odonoghue@linaro.org> <20211213135127.1656258-4-bryan.odonoghue@linaro.org> <CAMZdPi82xqRGn+oZwVX5wgutEDO1qpufSra2oBW9S4pALHHAdw@mail.gmail.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.1.1

On 14/12/2021 10:31, Loic Poulain wrote:

Shouldn't this be configured only if the vif type is NL80211_IFTYPE_STATION?

It shouldn't make a difference since BMPS will only be reached by an STAbut, I have no objection to adding the if.


---
bod

References:
- [PATCH 0/3] wcn36xx: Implement explicit beacon filter tables
  - From: Bryan O'Donoghue
- [PATCH 3/3] wcn36xx: Implement downstream compliant beacon filtering
  - From: Bryan O'Donoghue
- Re: [PATCH 3/3] wcn36xx: Implement downstream compliant beacon filtering
  - From: Loic Poulain