Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 20 Oct 2022 14:48:01 +0530
From:   Deepak R Varma <drv@mailo.com>
To:     Julia Lawall <julia.lawall@inria.fr>
Cc:     Dan Carpenter <dan.carpenter@oracle.com>,
        outreachy@lists.linux.dev, Larry.Finger@lwfinger.net,
        phil@philpotter.co.uk, paskripkin@gmail.com,
        gregkh@linuxfoundation.org, linux-staging@lists.linux.dev,
        linux-kernel@vger.kernel.org, kumarpraveen@linux.microsoft.com,
        saurabh.truth@gmail.com
Subject: Re: [PATCH v3 05/10] staging: r8188eu: correct misspelled words in
 comments
Message-ID: <Y1ESSfUc/wW+cc6F@debian-BULLSEYE-live-builder-AMD64>
References: <cover.1666249715.git.drv@mailo.com>
 <3235c0f681d817f7f89dd80537f36e1f9686fa24.1666249716.git.drv@mailo.com>
 <Y1EFMt6fjLdRirnA@kadam>
 <72b89e31-20b8-ab7c-a447-ff2d216da8de@inria.fr>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <72b89e31-20b8-ab7c-a447-ff2d216da8de@inria.fr>
Precedence: bulk

On Thu, Oct 20, 2022 at 11:12:54AM +0200, Julia Lawall wrote:
>
>
> On Thu, 20 Oct 2022, Dan Carpenter wrote:
>
> > On Thu, Oct 20, 2022 at 01:25:16PM +0530, Deepak R Varma wrote:
> > > diff --git a/drivers/staging/r8188eu/core/rtw_ioctl_set.c b/drivers/staging/r8188eu/core/rtw_ioctl_set.c
> > > index 55e6b0f41dc3..1de808832ed8 100644
> > > --- a/drivers/staging/r8188eu/core/rtw_ioctl_set.c
> > > +++ b/drivers/staging/r8188eu/core/rtw_ioctl_set.c
> > > @@ -287,7 +287,7 @@ u8 rtw_set_802_11_infrastructure_mode(struct adapter *padapter,
> > >
> > >  		if ((*pold_state == Ndis802_11Infrastructure) || (*pold_state == Ndis802_11IBSS)) {
> > >  			if (check_fwstate(pmlmepriv, _FW_LINKED))
> > > -				rtw_indicate_disconnect(padapter); /* will clr Linked_state; before this function, we must have chked whether  issue dis-assoc_cmd or not */
> > > +				rtw_indicate_disconnect(padapter); /* will clr Linked_state; before this function, we must have cheked whether  issue dis-assoc_cmd or not */
> >
> >
> > s/cheked/checked/
>
> There is also a strange double space before issue.

Thanks. "While in there..." opportunity on revision :)

>
> julia
>
> >
> > >  	       }
> > >
> > >  		*pold_state = networktype;
> >
> > regards,
> > dan carpenter
> >
> >
>