Re: [PATCH 00/22] staging: rtl87232bs: Fix errors and warnings detected by Smatch

2017-04-09 Thread Bastien Nocera
> > Signed-off-by: Larry Finger <larry.fin...@lwfinger.net> Feel free to add: Reviewed-by: Bastien Nocera <had...@hadess.net> for all the patches I didn't directly comment on. Cheers

Re: [PATCH 20/22] staging rtl8723bs: Fix indenting errors and an off-by-one mistake in core/rtw_mlme_ext.c

2017-04-09 Thread Bastien Nocera
On Sat, 2017-04-08 at 11:07 -0500, Larry Finger wrote: > } else { > -   for (pstat->aid = 1; pstat->aid <= NUM_STA; pstat->aid++) > +   for (pstat->aid = 1; pstat->aid < NUM_STA; pstat->aid++) > if (pstapriv->sta_aid[pstat->aid - 1] == NULL)

Re: [PATCH 16/22] staging: rtl8723bs: Fix some indenting problems and a potential data overrun

2017-04-09 Thread Bastien Nocera
On Sat, 2017-04-08 at 11:07 -0500, Larry Finger wrote: > +   if (cam_id >= 0 && cam_id < 32) Isn't there a constant we could use instead of hard-coding this?

Re: [PATCH 13/22] staging: rtl8723bs: Fix indenting mistake in core/rtw_ap.c

2017-04-09 Thread Bastien Nocera
On Sat, 2017-04-08 at 11:07 -0500, Larry Finger wrote: > Fixing this requires changing the indentatikon of a long for loop. Typo here.

Re: [PATCH 06/22] staging: rtl8723bs: Fix various errors in os_dep/ioctl_cfg80211.c

2017-04-09 Thread Bastien Nocera
On Sat, 2017-04-08 at 11:07 -0500, Larry Finger wrote: > Smatch lists the following: > >   CHECK   drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c:470 > rtw_cfg80211_ibss_indicate_connect() error: we previously assumed > 'scanned' could be

Re: [PATCH] rfkill-gpio: Add support for the Realtek 8723 BT

2015-07-15 Thread Bastien Nocera
On Tue, 2015-07-14 at 08:20 -0500, Larry Finger wrote: On 07/13/2015 05:11 PM, Bastien Nocera wrote: http://thread.gmane.org/gmane.linux.kernel.wireless.general/127706/ focu s=127896 Signed-off-by: Bastien Nocera had...@hadess.net Bastien, The subject of this patch needs a bit

Re: [PATCH] rfkill-gpio: Add support for the Realtek 8723 BT

2015-07-15 Thread Bastien Nocera
On Tue, 2015-07-14 at 17:29 +0200, Marcel Holtmann wrote: Hi Bastien, http://thread.gmane.org/gmane.linux.kernel.wireless.general/127706/ focu s=127896 Signed-off-by: Bastien Nocera had...@hadess.net --- net/rfkill/rfkill-gpio.c | 1 + 1 file changed, 1 insertion(+) diff

Re: [PATCH] rfkill-gpio: Add support for the Realtek 8723 BT

2015-07-15 Thread Bastien Nocera
On Tue, 2015-07-14 at 15:21 +0200, Johannes Berg wrote: On Tue, 2015-07-14 at 08:20 -0500, Larry Finger wrote: On 07/13/2015 05:11 PM, Bastien Nocera wrote: http://thread.gmane.org/gmane.linux.kernel.wireless.general/12770 6/ focu s=127896 Signed-off-by: Bastien Nocera had

[PATCH] rfkill-gpio: Add support for the Realtek 8723 BT

2015-07-13 Thread Bastien Nocera
http://thread.gmane.org/gmane.linux.kernel.wireless.general/127706/focu s=127896 Signed-off-by: Bastien Nocera had...@hadess.net --- net/rfkill/rfkill-gpio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/rfkill/rfkill-gpio.c b/net/rfkill/rfkill-gpio.c index d5d58d9..9471024 100644