On Wed, May 21, 2014 at 09:38:10AM +0200, jes.soren...@redhat.com wrote:
>  drivers/staging/rtl8723au/hal/rtl8723a_cmd.c | 23 +++++++++++++----------
>  1 file changed, 13 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c 
> b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> index eb747d6..5f188af 100644
> --- a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> +++ b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> @@ -215,7 +215,7 @@ static void ConstructBeacon(struct rtw_adapter *padapter, 
> u8 *pframe, u32 *pLeng
>  
>       pwlanhdr = (struct ieee80211_hdr *)pframe;
>  
> -     pwlanhdr->frame_control = 0;
> +     pwlanhdr->frame_control = IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_BEACON;

This looks like an endian bug.

It would be very easy to send a v2 version of this patch without
touching any of the others but it will also not hurt my feelings if you
fix this in a later patch.  It is up to you and Greg.

regards,
dan carpenter

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to