RE: [PATCH v2 0/7] mwifiex: cleanups

2016-03-19 Thread Amitkumar Karwar
Hi Andreas,

> From: Andreas Fenkart [mailto:afenk...@gmail.com]
> Sent: Thursday, March 10, 2016 2:14 PM
> To: linux-wireless@vger.kernel.org
> Cc: Amitkumar Karwar; Nishant Sarmukadam; Julian Calaby; Kalle Valo;
> Andreas Fenkart
> Subject: [PATCH v2 0/7] mwifiex: cleanups
> 
> v2:
> - addressed issues from review
> - race-condition issue discussed in patch description
> 
> Andreas Fenkart (7):
>   mwifiex: scan: simplify dereference of bss_desc fields
>   mwifiex: scan: factor out has_ieee_hdr/has_vendor_hdr
>   mwifiex: scan: simplify ternary operators using gnu extension
>   mwifiex: scan: factor out dbg_security_flags
>   mwifiex: scan: replace pointer arithmetic with array access
>   mwifiex: factor out mwifiex_cancel_pending_scan_cmd
>   mwifiex: make mwifiex_insert_cmd_to_free_q local static
> 
>  drivers/net/wireless/marvell/mwifiex/cmdevt.c  | 125 +++---
>  drivers/net/wireless/marvell/mwifiex/main.h|   3 +-
>  drivers/net/wireless/marvell/mwifiex/scan.c| 185 --
> ---
>  drivers/net/wireless/marvell/mwifiex/sta_cmdresp.c |  13 +-
>  4 files changed, 128 insertions(+), 198 deletions(-)
> 

Thanks for the efforts for this cleanup work. Patches look fine to me.

Acked-by: Amitkumar Karwar <akar...@marvell.com>

Regards,
Amitkumar
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v2 0/7] mwifiex: cleanups

2016-03-10 Thread Andreas Fenkart
v2:
- addressed issues from review
- race-condition issue discussed in patch description

Andreas Fenkart (7):
  mwifiex: scan: simplify dereference of bss_desc fields
  mwifiex: scan: factor out has_ieee_hdr/has_vendor_hdr
  mwifiex: scan: simplify ternary operators using gnu extension
  mwifiex: scan: factor out dbg_security_flags
  mwifiex: scan: replace pointer arithmetic with array access
  mwifiex: factor out mwifiex_cancel_pending_scan_cmd
  mwifiex: make mwifiex_insert_cmd_to_free_q local static

 drivers/net/wireless/marvell/mwifiex/cmdevt.c  | 125 +++---
 drivers/net/wireless/marvell/mwifiex/main.h|   3 +-
 drivers/net/wireless/marvell/mwifiex/scan.c| 185 -
 drivers/net/wireless/marvell/mwifiex/sta_cmdresp.c |  13 +-
 4 files changed, 128 insertions(+), 198 deletions(-)

-- 
2.7.0

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html