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;
Date:   Tue, 10 Aug 2021 23:50:34 +0100
From:   Phillip Potter <phil@philpotter.co.uk>
To:     Larry Finger <Larry.Finger@lwfinger.net>
Cc:     gregkh@linuxfoundation.org, linux-staging@lists.linux.dev,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] staging: r8188eu: Replace BITn with BIT(n)
Message-ID: <YRMCug2ULtA6JoLn@equinox>
References: <20210810180511.8986-1-Larry.Finger@lwfinger.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210810180511.8986-1-Larry.Finger@lwfinger.net>
Precedence: bulk

On Tue, Aug 10, 2021 at 01:05:11PM -0500, Larry Finger wrote:
> The driver uses BITn instead of BIT(n). All such usage is converted.
> 
> Note that this patch does not address any warnings that checkpatch
> will find. These include missing space around operators and lines
> that are too long. These problems will be addressed in a separate
> patch.
> 
> Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
> 
> ---
> v2 - Removed dependency on incorrect patch
>      Fixed typo n comment noted by Dan Carpenter
> ---
>  drivers/staging/r8188eu/core/rtw_ioctl_set.c  |   2 +-
>  drivers/staging/r8188eu/hal/HalPhyRf_8188e.c  |  40 +-
>  drivers/staging/r8188eu/hal/odm.c             |  68 +--
>  drivers/staging/r8188eu/hal/odm_HWConfig.c    |   4 +-
>  drivers/staging/r8188eu/hal/odm_RTL8188E.c    | 116 ++---
>  drivers/staging/r8188eu/hal/rtl8188e_cmd.c    |  14 +-
>  .../staging/r8188eu/hal/rtl8188e_hal_init.c   |  66 +--
>  drivers/staging/r8188eu/hal/rtl8188e_mp.c     |  34 +-
>  drivers/staging/r8188eu/hal/rtl8188e_phycfg.c |   8 +-
>  drivers/staging/r8188eu/hal/rtl8188eu_led.c   |  10 +-
>  drivers/staging/r8188eu/hal/usb_halinit.c     |  48 +-
>  .../staging/r8188eu/include/Hal8188EPhyCfg.h  |  14 +-
>  .../staging/r8188eu/include/Hal8188EPwrSeq.h  |  84 ++--
>  drivers/staging/r8188eu/include/hal_intf.h    |  16 +-
>  drivers/staging/r8188eu/include/odm.h         | 106 ++---
>  drivers/staging/r8188eu/include/odm_reg.h     |   2 +-
>  drivers/staging/r8188eu/include/odm_types.h   |   8 +-
>  .../staging/r8188eu/include/osdep_service.h   |  42 --
>  .../staging/r8188eu/include/rtl8188e_spec.h   | 415 ++++++++----------
>  drivers/staging/r8188eu/include/rtw_sreset.h  |  14 +-
>  drivers/staging/r8188eu/os_dep/ioctl_linux.c  |   4 +-
>  21 files changed, 518 insertions(+), 597 deletions(-)
>

Dear Larry,

Looks good and builds fine for me.

Acked-by: Phillip Potter <phil@philpotter.co.uk>

Regards,
Phil