Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: Larry Finger <larry.finger@gmail.com>
Subject: Re: [PATCH 1/3] staging: r8188eu: Remove all 5GHz network types
To:     "Fabio M. De Francesco" <fmdefrancesco@gmail.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Phillip Potter <phil@philpotter.co.uk>,
        Martin Kaiser <martin@kaiser.cx>,
        Michael Straube <straube.linux@gmail.com>,
        Fabio Aiuto <fabioaiuto83@gmail.com>,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
References: <20210812002519.23678-1-fmdefrancesco@gmail.com>
 <20210812002519.23678-2-fmdefrancesco@gmail.com>
From:   Larry Finger <Larry.Finger@lwfinger.net>
Message-ID: <13cdcd41-20f6-3b8e-0515-5066b1fdb49c@lwfinger.net>
Date:   Wed, 11 Aug 2021 21:09:49 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.12.0
MIME-Version: 1.0
In-Reply-To: <20210812002519.23678-2-fmdefrancesco@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 8/11/21 7:25 PM, Fabio M. De Francesco wrote:
> Remove all 5Ghz network types. r8188eu works on
> 802.11bgn standards and on 2.4Ghz band.
> 
> Signed-off-by: Fabio M. De Francesco<fmdefrancesco@gmail.com>
> ---
>   drivers/staging/r8188eu/core/rtw_ap.c         | 13 ++---------
>   drivers/staging/r8188eu/core/rtw_ieee80211.c  | 19 +++------------
>   drivers/staging/r8188eu/core/rtw_ioctl_set.c  |  2 +-
>   drivers/staging/r8188eu/core/rtw_mlme.c       | 10 --------
>   drivers/staging/r8188eu/core/rtw_p2p.c        |  2 +-
>   drivers/staging/r8188eu/core/rtw_wlan_util.c  | 15 ++----------
>   drivers/staging/r8188eu/hal/odm.c             |  1 -
>   drivers/staging/r8188eu/include/ieee80211.h   | 23 ++-----------------
>   drivers/staging/r8188eu/include/wifi.h        |  2 +-
>   drivers/staging/r8188eu/include/wlan_bssdef.h |  2 +-
>   10 files changed, 13 insertions(+), 76 deletions(-)

Fabio,

This patch is fine, but it does conflict with my series fixing the smatch 
warnings. Whichever set Greg takes first will apply cleanly, but the second will 
fail. This problem is due to many people working at the same time.

Acked-by: Larry Finger <Larry.Finger@lwfinger.net>

Larry