Re: [PATCH 12/20] staging: wfx: align semantic of beacon filter with other filters

From: Dan Carpenter
Date: Thu Apr 16 2020 - 10:20:29 EST


On Wed, Apr 15, 2020 at 06:11:39PM +0200, Jerome Pouiller wrote:
> From: Jérôme Pouiller <jerome.pouiller@xxxxxxxxxx>
>
> Filters provided by HIF API are sometime inclusive, sometime exclusive.
>
> This patch align the behavior and name of the beacon filter with the
> other filters. Also avoid double negation: "disable filter"

Hooray! I have been wanting to suggest this every time I see the
->disable_beacon_filter name, especially for patch 7/20.

regards,
dan carpenter