lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5440A9A6.6040507@gmail.com>
Date:	Fri, 17 Oct 2014 13:31:18 +0800
From:	hejianet <hejianet@...il.com>
To:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
CC:	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
	Joe Perches <joe@...ches.com>
Subject: Re: [PATCHv2] staging: rtl8188eu: Fix coding style space related
 ERROR  problems

 Hi Greg
Just ping it, coz this is a practising patch for Eudyptula Challenge task 10.
Any comment is welcome, thank you :)

On Tue, 14 Oct 2014 11:28:32 +0800 from hejianet@...il.com wrote:
> This fixes space related ERROR reports by checkpatch.pl
> Generated by $ git ls-files "drivers/staging/rtl8188eu/*.[ch]" | \
>   xargs ./scripts/checkpatch.pl -f --fix-inplace --strict --types=SPACING
> Already checked by text comparasion
> $git diff -w 
> and binary comparasion of r8188eu.ko
> $objdiff diff <old_commit> <new_commit>
>
> Signed-off-by: Jia He <hejianet@...il.com>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> ---
>  drivers/staging/rtl8188eu/core/rtw_cmd.c          |  2 +-
>  drivers/staging/rtl8188eu/core/rtw_ieee80211.c    | 16 +++++-----
>  drivers/staging/rtl8188eu/core/rtw_mlme.c         |  2 +-
>  drivers/staging/rtl8188eu/core/rtw_mlme_ext.c     | 22 +++++++-------
>  drivers/staging/rtl8188eu/core/rtw_pwrctrl.c      |  2 +-
>  drivers/staging/rtl8188eu/core/rtw_recv.c         | 12 ++++----
>  drivers/staging/rtl8188eu/core/rtw_security.c     | 20 ++++++-------
>  drivers/staging/rtl8188eu/core/rtw_sta_mgt.c      |  2 +-
>  drivers/staging/rtl8188eu/core/rtw_xmit.c         | 10 +++----
>  drivers/staging/rtl8188eu/hal/bb_cfg.c            |  4 +--
>  drivers/staging/rtl8188eu/hal/fw.c                |  8 ++---
>  drivers/staging/rtl8188eu/hal/mac_cfg.c           |  2 +-
>  drivers/staging/rtl8188eu/hal/odm.c               |  8 ++---
>  drivers/staging/rtl8188eu/hal/odm_HWConfig.c      |  2 +-
>  drivers/staging/rtl8188eu/hal/odm_RTL8188E.c      |  2 +-
>  drivers/staging/rtl8188eu/hal/phy.c               |  2 +-
>  drivers/staging/rtl8188eu/hal/rf.c                |  4 +--
>  drivers/staging/rtl8188eu/hal/rf_cfg.c            |  4 +--
>  drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c      |  2 +-
>  drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c    |  2 +-
>  drivers/staging/rtl8188eu/hal/usb_halinit.c       |  2 +-
>  drivers/staging/rtl8188eu/include/ieee80211_ext.h | 20 ++++++-------
>  drivers/staging/rtl8188eu/include/odm_debug.h     |  2 +-
>  drivers/staging/rtl8188eu/include/osdep_service.h |  4 +--
>  drivers/staging/rtl8188eu/include/rtw_debug.h     |  2 +-
>  drivers/staging/rtl8188eu/include/rtw_led.h       |  2 +-
>  drivers/staging/rtl8188eu/include/rtw_mlme_ext.h  | 26 ++++++++--------
>  drivers/staging/rtl8188eu/include/wifi.h          | 36 +++++++++++------------
>  drivers/staging/rtl8188eu/os_dep/ioctl_linux.c    |  2 +-
>  drivers/staging/rtl8188eu/os_dep/os_intfs.c       |  2 +-
>  drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c  |  4 +--
>  drivers/staging/rtl8188eu/os_dep/xmit_linux.c     |  2 +-
>  32 files changed, 116 insertions(+), 116 deletions(-)
>
> diff --git a/drivers/staging/rtl8188eu/core/rtw_cmd.c b/drivers/staging/rtl8188eu/core/rtw_cmd.c
> index 9935e66..7b59a10 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_cmd.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_cmd.c
> @@ -638,7 +638,7 @@ u8 rtw_setstakey_cmd(struct adapter *padapter, u8 *psta, u8 unicast_key)
>  	ether_addr_copy(psetstakey_para->addr, sta->hwaddr);
>  
>  	if (check_fwstate(pmlmepriv, WIFI_STATION_STATE))
> -		psetstakey_para->algorithm = (unsigned char) psecuritypriv->dot11PrivacyAlgrthm;
> +		psetstakey_para->algorithm = (unsigned char)psecuritypriv->dot11PrivacyAlgrthm;
>  	else
>  		GET_ENCRY_ALGO(psecuritypriv, sta, psetstakey_para->algorithm, false);
>  
> diff --git a/drivers/staging/rtl8188eu/core/rtw_ieee80211.c b/drivers/staging/rtl8188eu/core/rtw_ieee80211.c
> index 755d3ef..f2c3ca7 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_ieee80211.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_ieee80211.c
> @@ -159,7 +159,7 @@ u8 *rtw_set_ie
>  	return pbuf + len + 2;
>  }
>  
> -inline u8 *rtw_set_ie_ch_switch (u8 *buf, u32 *buf_len, u8 ch_switch_mode,
> +inline u8 *rtw_set_ie_ch_switch(u8 *buf, u32 *buf_len, u8 ch_switch_mode,
>  	u8 new_ch, u8 ch_switch_cnt)
>  {
>  	u8 ie_data[3];
> @@ -870,7 +870,7 @@ static int rtw_ieee802_11_parse_vendor_specific(u8 *pos, uint elen,
>  	if (elen < 4) {
>  		if (show_errors) {
>  			DBG_88E("short vendor specific information element ignored (len=%lu)\n",
> -				(unsigned long) elen);
> +				(unsigned long)elen);
>  		}
>  		return -1;
>  	}
> @@ -890,7 +890,7 @@ static int rtw_ieee802_11_parse_vendor_specific(u8 *pos, uint elen,
>  		case WME_OUI_TYPE: /* this is a Wi-Fi WME info. element */
>  			if (elen < 5) {
>  				DBG_88E("short WME information element ignored (len=%lu)\n",
> -					(unsigned long) elen);
> +					(unsigned long)elen);
>  				return -1;
>  			}
>  			switch (pos[4]) {
> @@ -905,7 +905,7 @@ static int rtw_ieee802_11_parse_vendor_specific(u8 *pos, uint elen,
>  				break;
>  			default:
>  				DBG_88E("unknown WME information element ignored (subtype=%d len=%lu)\n",
> -					pos[4], (unsigned long) elen);
> +					pos[4], (unsigned long)elen);
>  				return -1;
>  			}
>  			break;
> @@ -916,7 +916,7 @@ static int rtw_ieee802_11_parse_vendor_specific(u8 *pos, uint elen,
>  			break;
>  		default:
>  			DBG_88E("Unknown Microsoft information element ignored (type=%d len=%lu)\n",
> -				pos[3], (unsigned long) elen);
> +				pos[3], (unsigned long)elen);
>  			return -1;
>  		}
>  		break;
> @@ -929,13 +929,13 @@ static int rtw_ieee802_11_parse_vendor_specific(u8 *pos, uint elen,
>  			break;
>  		default:
>  			DBG_88E("Unknown Broadcom information element ignored (type=%d len=%lu)\n",
> -				pos[3], (unsigned long) elen);
> +				pos[3], (unsigned long)elen);
>  			return -1;
>  		}
>  		break;
>  	default:
>  		DBG_88E("unknown vendor specific information element ignored (vendor OUI %02x:%02x:%02x len=%lu)\n",
> -			pos[0], pos[1], pos[2], (unsigned long) elen);
> +			pos[0], pos[1], pos[2], (unsigned long)elen);
>  		return -1;
>  	}
>  	return 0;
> @@ -969,7 +969,7 @@ enum parse_res rtw_ieee802_11_parse_elems(u8 *start, uint len,
>  		if (elen > left) {
>  			if (show_errors) {
>  				DBG_88E("IEEE 802.11 element parse failed (id=%d elen=%d left=%lu)\n",
> -					id, elen, (unsigned long) left);
> +					id, elen, (unsigned long)left);
>  			}
>  			return ParseFailed;
>  		}
> diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme.c b/drivers/staging/rtl8188eu/core/rtw_mlme.c
> index 149c271..6c3c588 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_mlme.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_mlme.c
> @@ -2117,7 +2117,7 @@ void rtw_issue_addbareq_cmd(struct adapter *padapter, struct xmit_frame *pxmitfr
>  		if (0 == issued) {
>  			DBG_88E("rtw_issue_addbareq_cmd, p=%d\n", priority);
>  			psta->htpriv.candidate_tid_bitmap |= BIT((u8)priority);
> -			rtw_addbareq_cmd(padapter, (u8) priority, pattrib->ra);
> +			rtw_addbareq_cmd(padapter, (u8)priority, pattrib->ra);
>  		}
>  	}
>  }
> diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c b/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
> index 5ba5099..63d89bb 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
> @@ -227,7 +227,7 @@ static void init_mlme_ext_priv_value(struct adapter *padapter)
>  	pmlmeext->cur_channel = padapter->registrypriv.channel;
>  	pmlmeext->cur_bwmode = HT_CHANNEL_WIDTH_20;
>  	pmlmeext->cur_ch_offset = HAL_PRIME_CHNL_OFFSET_DONT_CARE;
> -	pmlmeext->oper_channel = pmlmeext->cur_channel ;
> +	pmlmeext->oper_channel = pmlmeext->cur_channel;
>  	pmlmeext->oper_bwmode = pmlmeext->cur_bwmode;
>  	pmlmeext->oper_ch_offset = pmlmeext->cur_ch_offset;
>  	pmlmeext->retry = 0;
> @@ -2000,7 +2000,7 @@ void issue_beacon(struct adapter *padapter, int timeout_ms)
>  		DBG_88E("%s, alloc mgnt frame fail\n", __func__);
>  		return;
>  	}
> -#if defined (CONFIG_88EU_AP_MODE)
> +#if defined(CONFIG_88EU_AP_MODE)
>  	spin_lock_bh(&pmlmepriv->bcn_update_lock);
>  #endif /* if defined (CONFIG_88EU_AP_MODE) */
>  
> @@ -2027,7 +2027,7 @@ void issue_beacon(struct adapter *padapter, int timeout_ms)
>  	SetFrameSubType(pframe, WIFI_BEACON);
>  
>  	pframe += sizeof(struct rtw_ieee80211_hdr_3addr);
> -	pattrib->pktlen = sizeof (struct rtw_ieee80211_hdr_3addr);
> +	pattrib->pktlen = sizeof(struct rtw_ieee80211_hdr_3addr);
>  
>  	if ((pmlmeinfo->state&0x03) == WIFI_FW_AP_STATE) {
>  		int len_diff;
> @@ -2042,8 +2042,8 @@ void issue_beacon(struct adapter *padapter, int timeout_ms)
>  			);
>  		pframe += (cur_network->IELength+len_diff);
>  		pattrib->pktlen += (cur_network->IELength+len_diff);
> -		wps_ie = rtw_get_wps_ie(pmgntframe->buf_addr+TXDESC_OFFSET+sizeof (struct rtw_ieee80211_hdr_3addr)+_BEACON_IE_OFFSET_,
> -			pattrib->pktlen-sizeof (struct rtw_ieee80211_hdr_3addr)-_BEACON_IE_OFFSET_, NULL, &wps_ielen);
> +		wps_ie = rtw_get_wps_ie(pmgntframe->buf_addr+TXDESC_OFFSET+sizeof(struct rtw_ieee80211_hdr_3addr)+_BEACON_IE_OFFSET_,
> +			pattrib->pktlen-sizeof(struct rtw_ieee80211_hdr_3addr)-_BEACON_IE_OFFSET_, NULL, &wps_ielen);
>  		if (wps_ie && wps_ielen > 0)
>  			rtw_get_wps_attr_content(wps_ie,  wps_ielen, WPS_ATTR_SELECTED_REGISTRAR, (u8 *)(&sr), NULL);
>  		if (sr != 0)
> @@ -2101,7 +2101,7 @@ void issue_beacon(struct adapter *padapter, int timeout_ms)
>  	/* todo:HT for adhoc */
>  _issue_bcn:
>  
> -#if defined (CONFIG_88EU_AP_MODE)
> +#if defined(CONFIG_88EU_AP_MODE)
>  	pmlmepriv->update_bcn = false;
>  
>  	spin_unlock_bh(&pmlmepriv->bcn_update_lock);
> @@ -2130,7 +2130,7 @@ void issue_probersp(struct adapter *padapter, unsigned char *da, u8 is_valid_p2p
>  	__le16 *fctrl;
>  	unsigned char					*mac, *bssid;
>  	struct xmit_priv	*pxmitpriv = &(padapter->xmitpriv);
> -#if defined (CONFIG_88EU_AP_MODE)
> +#if defined(CONFIG_88EU_AP_MODE)
>  	u8 *pwps_ie;
>  	uint wps_ielen;
>  	struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
> @@ -2323,8 +2323,8 @@ static int _issue_probereq(struct adapter *padapter, struct ndis_802_11_ssid *ps
>  	pmlmeext->mgnt_seq++;
>  	SetFrameSubType(pframe, WIFI_PROBEREQ);
>  
> -	pframe += sizeof (struct rtw_ieee80211_hdr_3addr);
> -	pattrib->pktlen = sizeof (struct rtw_ieee80211_hdr_3addr);
> +	pframe += sizeof(struct rtw_ieee80211_hdr_3addr);
> +	pattrib->pktlen = sizeof(struct rtw_ieee80211_hdr_3addr);
>  
>  	if (pssid)
>  		pframe = rtw_set_ie(pframe, _SSID_IE_, pssid->SsidLength, pssid->Ssid, &(pattrib->pktlen));
> @@ -3209,7 +3209,7 @@ exit:
>  	return ret;
>  }
>  
> -void issue_action_spct_ch_switch (struct adapter *padapter, u8 *ra, u8 new_ch, u8 ch_offset)
> +void issue_action_spct_ch_switch(struct adapter *padapter, u8 *ra, u8 new_ch, u8 ch_offset)
>  {
>  	struct xmit_frame			*pmgntframe;
>  	struct pkt_attrib			*pattrib;
> @@ -3260,7 +3260,7 @@ void issue_action_spct_ch_switch (struct adapter *padapter, u8 *ra, u8 new_ch, u
>  		pframe = rtw_set_fixed_ie(pframe, 1, &(action), &(pattrib->pktlen));
>  	}
>  
> -	pframe = rtw_set_ie_ch_switch (pframe, &(pattrib->pktlen), 0, new_ch, 0);
> +	pframe = rtw_set_ie_ch_switch(pframe, &(pattrib->pktlen), 0, new_ch, 0);
>  	pframe = rtw_set_ie_secondary_ch_offset(pframe, &(pattrib->pktlen),
>  		hal_ch_offset_to_secondary_ch_offset(ch_offset));
>  
> diff --git a/drivers/staging/rtl8188eu/core/rtw_pwrctrl.c b/drivers/staging/rtl8188eu/core/rtw_pwrctrl.c
> index 27ed83c..481c187 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_pwrctrl.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_pwrctrl.c
> @@ -527,7 +527,7 @@ void rtw_init_pwrctrl_priv(struct adapter *padapter)
>  
>  	pwrctrlpriv->LpsIdleCount = 0;
>  	if (padapter->registrypriv.mp_mode == 1)
> -		pwrctrlpriv->power_mgnt = PS_MODE_ACTIVE ;
> +		pwrctrlpriv->power_mgnt = PS_MODE_ACTIVE;
>  	else
>  		pwrctrlpriv->power_mgnt = padapter->registrypriv.power_mgnt;/*  PS_MODE_MIN; */
>  	pwrctrlpriv->bLeisurePs = (PS_MODE_ACTIVE != pwrctrlpriv->power_mgnt) ? true : false;
> diff --git a/drivers/staging/rtl8188eu/core/rtw_recv.c b/drivers/staging/rtl8188eu/core/rtw_recv.c
> index 4d56dba..273fbb1 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_recv.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_recv.c
> @@ -46,7 +46,7 @@ void rtw_signal_stat_timer_hdl(RTW_TIMER_HDL_ARGS);
>  void _rtw_init_sta_recv_priv(struct sta_recv_priv *psta_recvpriv)
>  {
>  
> -	memset((u8 *)psta_recvpriv, 0, sizeof (struct sta_recv_priv));
> +	memset((u8 *)psta_recvpriv, 0, sizeof(struct sta_recv_priv));
>  
>  	spin_lock_init(&psta_recvpriv->lock);
>  
> @@ -109,7 +109,7 @@ exit:
>  	return res;
>  }
>  
> -void _rtw_free_recv_priv (struct recv_priv *precvpriv)
> +void _rtw_free_recv_priv(struct recv_priv *precvpriv)
>  {
>  	struct adapter	*padapter = precvpriv->adapter;
>  
> @@ -124,7 +124,7 @@ void _rtw_free_recv_priv (struct recv_priv *precvpriv)
>  
>  }
>  
> -struct recv_frame *_rtw_alloc_recvframe (struct __queue *pfree_recv_queue)
> +struct recv_frame *_rtw_alloc_recvframe(struct __queue *pfree_recv_queue)
>  {
>  	struct recv_frame *hdr;
>  	struct list_head *plist, *phead;
> @@ -797,7 +797,7 @@ exit:
>  	return ret;
>  }
>  
> -static int ap2sta_data_frame (
> +static int ap2sta_data_frame(
>  	struct adapter *adapter,
>  	struct recv_frame *precv_frame,
>  	struct sta_info **psta)
> @@ -1266,7 +1266,7 @@ static int validate_recv_frame(struct adapter *adapter,
>  	u8 bDumpRxPkt;
>  	struct rx_pkt_attrib *pattrib = &precv_frame->attrib;
>  	u8 *ptr = precv_frame->rx_data;
> -	u8  ver = (unsigned char) (*ptr)&0x3;
> +	u8  ver = (unsigned char)(*ptr)&0x3;
>  	struct mlme_ext_priv *pmlmeext = &adapter->mlmeextpriv;
>  
>  
> @@ -1981,7 +1981,7 @@ static int process_recv_indicatepkts(struct adapter *padapter,
>  			}
>  		}
>  	} else { /* B/G mode */
> -		retval = wlanhdr_to_ethhdr (prframe);
> +		retval = wlanhdr_to_ethhdr(prframe);
>  		if (retval != _SUCCESS) {
>  			RT_TRACE(_module_rtl871x_recv_c_, _drv_err_, ("wlanhdr_to_ethhdr: drop pkt\n"));
>  			return retval;
> diff --git a/drivers/staging/rtl8188eu/core/rtw_security.c b/drivers/staging/rtl8188eu/core/rtw_security.c
> index f9096a5..9c0a8d1 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_security.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_security.c
> @@ -189,7 +189,7 @@ void rtw_wep_encrypt(struct adapter *padapter, u8 *pxmitframe)
>  				arcfour_encrypt(&mycontext, payload+length, crc, 4);
>  
>  				pframe += pxmitpriv->frag_len;
> -				pframe = (u8 *) round_up((size_t)(pframe), 4);
> +				pframe = (u8 *)round_up((size_t)(pframe), 4);
>  			}
>  		}
>  	}
> @@ -258,7 +258,7 @@ static void secmicputuint32(u8 *p, u32 val)
>  {
>  	long i;
>  	for (i = 0; i < 4; i++) {
> -		*p++ = (u8) (val & 0xff);
> +		*p++ = (u8)(val & 0xff);
>  		val >>= 8;
>  	}
>  }
> @@ -621,14 +621,14 @@ u32	rtw_tkip_encrypt(struct adapter *padapter, u8 *pxmitframe)
>  					arcfour_encrypt(&mycontext, payload, payload, length);
>  					arcfour_encrypt(&mycontext, payload+length, crc, 4);
>  				} else {
> -					length = pxmitpriv->frag_len-pattrib->hdrlen-pattrib->iv_len-pattrib->icv_len ;
> +					length = pxmitpriv->frag_len-pattrib->hdrlen-pattrib->iv_len-pattrib->icv_len;
>  					*((__le32 *)crc) = getcrc32(payload, length);/* modified by Amy*/
>  					arcfour_init(&mycontext, rc4key, 16);
>  					arcfour_encrypt(&mycontext, payload, payload, length);
>  					arcfour_encrypt(&mycontext, payload+length, crc, 4);
>  
>  					pframe += pxmitpriv->frag_len;
> -					pframe = (u8 *) round_up((size_t)(pframe), 4);
> +					pframe = (u8 *)round_up((size_t)(pframe), 4);
>  				}
>  			}
>  		} else {
> @@ -953,8 +953,8 @@ static void construct_mic_iv(u8 *mic_iv, int qc_exists, int a4_exists, u8 *mpdu,
>  		mic_iv[i] = mpdu[i + 8];	/* mic_iv[2:7] = A2[0:5] = mpdu[10:15] */
>  	for (i = 8; i < 14; i++)
>  		mic_iv[i] = pn_vector[13 - i];	/* mic_iv[8:13] = PN[5:0] */
> -	mic_iv[14] = (unsigned char) (payload_length / 256);
> -	mic_iv[15] = (unsigned char) (payload_length % 256);
> +	mic_iv[14] = (unsigned char)(payload_length / 256);
> +	mic_iv[15] = (unsigned char)(payload_length % 256);
>  }
>  
>  /************************************************/
> @@ -1045,8 +1045,8 @@ static void construct_ctr_preload(u8 *ctr_preload, int a4_exists, int qc_exists,
>  		ctr_preload[i] = mpdu[i + 8];		       /* ctr_preload[2:7] = A2[0:5] = mpdu[10:15] */
>  	for (i = 8; i < 14; i++)
>  		ctr_preload[i] =    pn_vector[13 - i];	  /* ctr_preload[8:13] = PN[5:0] */
> -	ctr_preload[14] =  (unsigned char) (c / 256); /* Ctr */
> -	ctr_preload[15] =  (unsigned char) (c % 256);
> +	ctr_preload[14] =  (unsigned char)(c / 256); /* Ctr */
> +	ctr_preload[15] =  (unsigned char)(c % 256);
>  }
>  
>  /************************************/
> @@ -1238,11 +1238,11 @@ u32	rtw_aes_encrypt(struct adapter *padapter, u8 *pxmitframe)
>  
>  					aes_cipher(prwskey, pattrib->hdrlen, pframe, length);
>  				} else{
> -					length = pxmitpriv->frag_len-pattrib->hdrlen-pattrib->iv_len-pattrib->icv_len ;
> +					length = pxmitpriv->frag_len-pattrib->hdrlen-pattrib->iv_len-pattrib->icv_len;
>  
>  					aes_cipher(prwskey, pattrib->hdrlen, pframe, length);
>  					pframe += pxmitpriv->frag_len;
> -					pframe = (u8 *) round_up((size_t)(pframe), 8);
> +					pframe = (u8 *)round_up((size_t)(pframe), 8);
>  				}
>  			}
>  		} else{
> diff --git a/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c b/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c
> index e1dc8fa..dc9d0dd 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c
> @@ -29,7 +29,7 @@
>  
>  static void _rtw_init_stainfo(struct sta_info *psta)
>  {
> -	memset((u8 *)psta, 0, sizeof (struct sta_info));
> +	memset((u8 *)psta, 0, sizeof(struct sta_info));
>  
>  	 spin_lock_init(&psta->lock);
>  	INIT_LIST_HEAD(&psta->list);
> diff --git a/drivers/staging/rtl8188eu/core/rtw_xmit.c b/drivers/staging/rtl8188eu/core/rtw_xmit.c
> index 639ace0..a0bbf90 100644
> --- a/drivers/staging/rtl8188eu/core/rtw_xmit.c
> +++ b/drivers/staging/rtl8188eu/core/rtw_xmit.c
> @@ -37,7 +37,7 @@ static void _init_txservq(struct tx_servq *ptxservq)
>  
>  void	_rtw_init_sta_xmit_priv(struct sta_xmit_priv *psta_xmitpriv)
>  {
> -	memset((unsigned char *)psta_xmitpriv, 0, sizeof (struct sta_xmit_priv));
> +	memset((unsigned char *)psta_xmitpriv, 0, sizeof(struct sta_xmit_priv));
>  	spin_lock_init(&psta_xmitpriv->lock);
>  	_init_txservq(&psta_xmitpriv->be_q);
>  	_init_txservq(&psta_xmitpriv->bk_q);
> @@ -223,7 +223,7 @@ exit:
>  	return res;
>  }
>  
> -void _rtw_free_xmit_priv (struct xmit_priv *pxmitpriv)
> +void _rtw_free_xmit_priv(struct xmit_priv *pxmitpriv)
>  {
>  	int i;
>  	struct adapter *padapter = pxmitpriv->adapter;
> @@ -691,7 +691,7 @@ static s32 xmitframe_addmic(struct adapter *padapter, struct xmit_frame *pxmitfr
>  			payload = pframe;
>  
>  			for (curfragnum = 0; curfragnum < pattrib->nr_frags; curfragnum++) {
> -				payload = (u8 *) round_up((size_t)(payload), 4);
> +				payload = (u8 *)round_up((size_t)(payload), 4);
>  				RT_TRACE(_module_rtl871x_xmit_c_, _drv_err_,
>  					 ("=== curfragnum=%d, pframe = 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x, 0x%.2x,!!!\n",
>  					 curfragnum, *payload, *(payload+1),
> @@ -772,7 +772,7 @@ static s32 xmitframe_swencrypt(struct adapter *padapter, struct xmit_frame *pxmi
>  	return _SUCCESS;
>  }
>  
> -s32 rtw_make_wlanhdr (struct adapter *padapter , u8 *hdr, struct pkt_attrib *pattrib)
> +s32 rtw_make_wlanhdr(struct adapter *padapter, u8 *hdr, struct pkt_attrib *pattrib)
>  {
>  	u16 *qc;
>  
> @@ -1098,7 +1098,7 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct
>  
>  		addr = (size_t)(pframe);
>  
> -		mem_start = (unsigned char *) round_up(addr, 4) + hw_hdr_offset;
> +		mem_start = (unsigned char *)round_up(addr, 4) + hw_hdr_offset;
>  		memcpy(mem_start, pbuf_start + hw_hdr_offset, pattrib->hdrlen);
>  	}
>  
> diff --git a/drivers/staging/rtl8188eu/hal/bb_cfg.c b/drivers/staging/rtl8188eu/hal/bb_cfg.c
> index 80e8cc9..1e963bf 100644
> --- a/drivers/staging/rtl8188eu/hal/bb_cfg.c
> +++ b/drivers/staging/rtl8188eu/hal/bb_cfg.c
> @@ -173,7 +173,7 @@ static bool set_baseband_agc_config(struct adapter *adapt)
>  		u32 v1 = array[i];
>  		u32 v2 = array[i+1];
>  
> -		if (v1 < 0xCDCDCDCD){
> +		if (v1 < 0xCDCDCDCD) {
>  			phy_set_bb_reg(adapt, v1, bMaskDWord, v2);
>  			udelay(1);
>  		}
> @@ -552,7 +552,7 @@ static void store_pwrindex_offset(struct adapter *Adapter, u32 regaddr, u32 bitm
>  	}
>  }
>  
> -static void rtl_addr_delay(struct adapter *adapt, u32 addr, u32 bit_mask ,u32 data)
> +static void rtl_addr_delay(struct adapter *adapt, u32 addr, u32 bit_mask, u32 data)
>  {
>  	if (addr == 0xfe) {
>  		msleep(50);
> diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c
> index 17b7f37..3b28754 100644
> --- a/drivers/staging/rtl8188eu/hal/fw.c
> +++ b/drivers/staging/rtl8188eu/hal/fw.c
> @@ -84,7 +84,7 @@ static void _rtl88e_fw_block_write(struct adapter *adapt,
>  static void _rtl88e_fill_dummy(u8 *pfwbuf, u32 *pfwlen)
>  {
>  	u32 fwlen = *pfwlen;
> -	u8 remain = (u8) (fwlen % 4);
> +	u8 remain = (u8)(fwlen % 4);
>  
>  	remain = (remain == 0) ? 0 : (4 - remain);
>  
> @@ -101,7 +101,7 @@ static void _rtl88e_fw_page_write(struct adapter *adapt,
>  				  u32 page, const u8 *buffer, u32 size)
>  {
>  	u8 value8;
> -	u8 u8page = (u8) (page & 0x07);
> +	u8 u8page = (u8)(page & 0x07);
>  
>  	value8 = (usb_read8(adapt, REG_MCUFWDL + 2) & 0xF8) | u8page;
>  
> @@ -193,13 +193,13 @@ int rtl88eu_download_fw(struct adapter *adapt)
>  	u32 fwsize;
>  	int err;
>  
> -	if (request_firmware(&fw, fw_name, device)){
> +	if (request_firmware(&fw, fw_name, device)) {
>  		dev_err(device, "Firmware %s not available\n", fw_name);
>  		return -ENOENT;
>  	}
>  
>  	if (fw->size > FW_8188E_SIZE) {
> -		dev_err(device,"Firmware size exceed 0x%X. Check it.\n",
> +		dev_err(device, "Firmware size exceed 0x%X. Check it.\n",
>  			 FW_8188E_SIZE);
>  		return -1;
>  	}
> diff --git a/drivers/staging/rtl8188eu/hal/mac_cfg.c b/drivers/staging/rtl8188eu/hal/mac_cfg.c
> index c0e7fa9..febc83a 100644
> --- a/drivers/staging/rtl8188eu/hal/mac_cfg.c
> +++ b/drivers/staging/rtl8188eu/hal/mac_cfg.c
> @@ -127,7 +127,7 @@ bool rtl88eu_phy_mac_config(struct adapter *adapt)
>  	ptrarray = array_MAC_REG_8188E;
>  
>  	for (i = 0; i < arraylength; i = i + 2)
> -		usb_write8(adapt, ptrarray[i], (u8) ptrarray[i + 1]);
> +		usb_write8(adapt, ptrarray[i], (u8)ptrarray[i + 1]);
>  
>  	usb_write8(adapt, REG_MAX_AGGR_NUM, MAX_AGGR_NUM);
>  	return true;
> diff --git a/drivers/staging/rtl8188eu/hal/odm.c b/drivers/staging/rtl8188eu/hal/odm.c
> index e4df837..9873998 100644
> --- a/drivers/staging/rtl8188eu/hal/odm.c
> +++ b/drivers/staging/rtl8188eu/hal/odm.c
> @@ -437,8 +437,8 @@ void odm_CommonInfoSelfInit(struct odm_dm_struct *pDM_Odm)
>  {
>  	struct adapter *adapter = pDM_Odm->Adapter;
>  
> -	pDM_Odm->bCckHighPower = (bool) phy_query_bb_reg(adapter, 0x824, BIT9);
> -	pDM_Odm->RFPathRxEnable = (u8) phy_query_bb_reg(adapter, 0xc04, 0x0F);
> +	pDM_Odm->bCckHighPower = (bool)phy_query_bb_reg(adapter, 0x824, BIT9);
> +	pDM_Odm->RFPathRxEnable = (u8)phy_query_bb_reg(adapter, 0xc04, 0x0F);
>  
>  	ODM_InitDebugSetting(pDM_Odm);
>  }
> @@ -529,7 +529,7 @@ void odm_DIGInit(struct odm_dm_struct *pDM_Odm)
>  	struct adapter *adapter = pDM_Odm->Adapter;
>  	struct rtw_dig *pDM_DigTable = &pDM_Odm->DM_DigTable;
>  
> -	pDM_DigTable->CurIGValue = (u8) phy_query_bb_reg(adapter, ODM_REG_IGI_A_11N, ODM_BIT_IGI_11N);
> +	pDM_DigTable->CurIGValue = (u8)phy_query_bb_reg(adapter, ODM_REG_IGI_A_11N, ODM_BIT_IGI_11N);
>  	pDM_DigTable->RssiLowThresh	= DM_DIG_THRESH_LOW;
>  	pDM_DigTable->RssiHighThresh	= DM_DIG_THRESH_HIGH;
>  	pDM_DigTable->FALowThresh	= DM_false_ALARM_THRESH_LOW;
> @@ -620,7 +620,7 @@ void odm_DIG(struct odm_dm_struct *pDM_Odm)
>  		} else if (pDM_Odm->SupportAbility & ODM_BB_ANT_DIV) {
>  			/* 1 Lower Bound for 88E AntDiv */
>  			if (pDM_Odm->AntDivType == CG_TRX_HW_ANTDIV) {
> -				DIG_Dynamic_MIN = (u8) pDM_DigTable->AntDiv_RSSI_max;
> +				DIG_Dynamic_MIN = (u8)pDM_DigTable->AntDiv_RSSI_max;
>  				ODM_RT_TRACE(pDM_Odm, ODM_COMP_ANT_DIV, ODM_DBG_LOUD,
>  					     ("odm_DIG(): pDM_DigTable->AntDiv_RSSI_max=%d\n",
>  					     pDM_DigTable->AntDiv_RSSI_max));
> diff --git a/drivers/staging/rtl8188eu/hal/odm_HWConfig.c b/drivers/staging/rtl8188eu/hal/odm_HWConfig.c
> index 4e4e219..29f87df 100644
> --- a/drivers/staging/rtl8188eu/hal/odm_HWConfig.c
> +++ b/drivers/staging/rtl8188eu/hal/odm_HWConfig.c
> @@ -118,7 +118,7 @@ static void odm_RxPhyStatus92CSeries_Parsing(struct odm_dm_struct *dm_odm,
>  
>  		cck_highpwr = dm_odm->bCckHighPower;
>  
> -		cck_agc_rpt =  pPhyStaRpt->cck_agc_rpt_ofdm_cfosho_a ;
> +		cck_agc_rpt =  pPhyStaRpt->cck_agc_rpt_ofdm_cfosho_a;
>  
>  		/* 2011.11.28 LukeLee: 88E use different LNA & VGA gain table */
>  		/* The RSSI formula should be modified according to the gain table */
> diff --git a/drivers/staging/rtl8188eu/hal/odm_RTL8188E.c b/drivers/staging/rtl8188eu/hal/odm_RTL8188E.c
> index 5342af7..d3c6873 100644
> --- a/drivers/staging/rtl8188eu/hal/odm_RTL8188E.c
> +++ b/drivers/staging/rtl8188eu/hal/odm_RTL8188E.c
> @@ -278,7 +278,7 @@ static void rtl88eu_dm_hw_ant_div(struct odm_dm_struct *dm_odm)
>  	struct rtw_dig *dig_table = &dm_odm->DM_DigTable;
>  	struct sta_info *entry;
>  	u32 i, min_rssi = 0xFF, ant_div_max_rssi = 0, max_rssi = 0;
> -	u32 local_min_rssi,local_max_rssi;
> +	u32 local_min_rssi, local_max_rssi;
>  	u32 main_rssi, aux_rssi;
>  	u8 RxIdleAnt = 0, target_ant = 7;
>  
> diff --git a/drivers/staging/rtl8188eu/hal/phy.c b/drivers/staging/rtl8188eu/hal/phy.c
> index c4f7f35..3f663fe 100644
> --- a/drivers/staging/rtl8188eu/hal/phy.c
> +++ b/drivers/staging/rtl8188eu/hal/phy.c
> @@ -478,7 +478,7 @@ void rtl88eu_dm_txpower_tracking_callback_thermalmeter(struct adapter *adapt)
>  		/* 2.4G, decrease power */
>  		{0, 0, 2, 3, 4, 4, 5, 6, 7, 7, 8, 9, 10, 10, 11},
>  		/* 2.4G, increase power */
> -		{0, 0, -1, -2, -3, -4,-4, -4, -4, -5, -7, -8,-9, -9, -10},
> +		{0, 0, -1, -2, -3, -4, -4, -4, -4, -5, -7, -8, -9, -9, -10},
>  	};
>  	u8 thermal_mapping[2][index_mapping_NUM_88E] = {
>  		/* 2.4G, decrease power */
> diff --git a/drivers/staging/rtl8188eu/hal/rf.c b/drivers/staging/rtl8188eu/hal/rf.c
> index c2fac34..8e7a0b1 100644
> --- a/drivers/staging/rtl8188eu/hal/rf.c
> +++ b/drivers/staging/rtl8188eu/hal/rf.c
> @@ -102,7 +102,7 @@ void rtl88eu_phy_rf6052_set_cck_txpower(struct adapter *adapt, u8 *powerlevel)
>  		}
>  	}
>  	rtl88eu_dm_txpower_track_adjust(&hal_data->odmpriv, 1, &direction,
> -				        &pwrtrac_value);
> +					&pwrtrac_value);
>  
>  	if (direction == 1) {
>  		/*  Increase TX power */
> @@ -131,7 +131,7 @@ void rtl88eu_phy_rf6052_set_cck_txpower(struct adapter *adapt, u8 *powerlevel)
>  /*  powerbase1 for HT MCS rates */
>  static void getpowerbase88e(struct adapter *adapt, u8 *pwr_level_ofdm,
>  			    u8 *pwr_level_bw20, u8 *pwr_level_bw40,
> -			    u8 channel,u32 *ofdmbase, u32 *mcs_base)
> +			    u8 channel, u32 *ofdmbase, u32 *mcs_base)
>  {
>  	struct hal_data_8188e *hal_data = GET_HAL_DATA(adapt);
>  	u32 powerbase0, powerbase1;
> diff --git a/drivers/staging/rtl8188eu/hal/rf_cfg.c b/drivers/staging/rtl8188eu/hal/rf_cfg.c
> index ddc2f55..5dc11ca 100644
> --- a/drivers/staging/rtl8188eu/hal/rf_cfg.c
> +++ b/drivers/staging/rtl8188eu/hal/rf_cfg.c
> @@ -164,7 +164,7 @@ do {								\
>  #define B3WIREDATALENGTH 0x800
>  #define BRFSI_RFENV 0x10
>  
> -static void rtl_rfreg_delay(struct adapter *adapt, enum rf_radio_path rfpath,u32 addr, u32 mask, u32 data)
> +static void rtl_rfreg_delay(struct adapter *adapt, enum rf_radio_path rfpath, u32 addr, u32 mask, u32 data)
>  {
>  	if (addr == 0xfe) {
>  		mdelay(50);
> @@ -190,7 +190,7 @@ static void rtl8188e_config_rf_reg(struct adapter *adapt,
>  	u32 content = 0x1000; /*RF Content: radio_a_txt*/
>  	u32 maskforphyset = (u32)(content & 0xE000);
>  
> -	rtl_rfreg_delay(adapt, RF90_PATH_A, addr| maskforphyset,
> +	rtl_rfreg_delay(adapt, RF90_PATH_A, addr | maskforphyset,
>  			RFREG_OFFSET_MASK,
>  			data);
>  }
> diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c b/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> index 023a3d8..ce78b3c 100644
> --- a/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> +++ b/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> @@ -254,7 +254,7 @@ void rtl8188e_set_FwMediaStatus_cmd(struct adapter *adapt, __le16 mstatus_rpt)
>  {
>  	u8 opmode, macid;
>  	u16 mst_rpt = le16_to_cpu(mstatus_rpt);
> -	opmode = (u8) mst_rpt;
> +	opmode = (u8)mst_rpt;
>  	macid = (u8)(mst_rpt >> 8);
>  
>  	DBG_88E("### %s: MStatus=%x MACID=%d\n", __func__, opmode, macid);
> diff --git a/drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c b/drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c
> index be9eede..594c1da 100644
> --- a/drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c
> +++ b/drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c
> @@ -399,7 +399,7 @@ static s32 rtw_dump_xframe(struct adapter *adapt, struct xmit_frame *pxmitframe)
>  
>  		mem_addr += w_sz;
>  
> -		mem_addr = (u8 *) round_up((size_t)mem_addr, 4);
> +		mem_addr = (u8 *)round_up((size_t)mem_addr, 4);
>  	}
>  
>  	rtw_free_xmitframe(pxmitpriv, pxmitframe);
> diff --git a/drivers/staging/rtl8188eu/hal/usb_halinit.c b/drivers/staging/rtl8188eu/hal/usb_halinit.c
> index caf2ca3..6e46f4e 100644
> --- a/drivers/staging/rtl8188eu/hal/usb_halinit.c
> +++ b/drivers/staging/rtl8188eu/hal/usb_halinit.c
> @@ -2012,7 +2012,7 @@ static u8 SetHalDefVar8188EUsb(struct adapter *Adapter, enum hal_def_variable eV
>  			u8 bRSSIDump = *((u8 *)pValue);
>  			struct odm_dm_struct *dm_ocm = &(haldata->odmpriv);
>  			if (bRSSIDump)
> -				dm_ocm->DebugComponents	=	ODM_COMP_DIG|ODM_COMP_FA_CNT	;
> +				dm_ocm->DebugComponents	=	ODM_COMP_DIG|ODM_COMP_FA_CNT;
>  			else
>  				dm_ocm->DebugComponents	= 0;
>  		}
> diff --git a/drivers/staging/rtl8188eu/include/ieee80211_ext.h b/drivers/staging/rtl8188eu/include/ieee80211_ext.h
> index 1052d18..15e53d3 100644
> --- a/drivers/staging/rtl8188eu/include/ieee80211_ext.h
> +++ b/drivers/staging/rtl8188eu/include/ieee80211_ext.h
> @@ -103,24 +103,24 @@ struct wme_parameter_element {
>  
>  #define WPA_PUT_LE16(a, val)			\
>  	do {					\
> -		(a)[1] = ((u16) (val)) >> 8;	\
> -		(a)[0] = ((u16) (val)) & 0xff;	\
> +		(a)[1] = ((u16)(val)) >> 8;	\
> +		(a)[0] = ((u16)(val)) & 0xff;	\
>  	} while (0)
>  
>  #define WPA_PUT_BE32(a, val)					\
>  	do {							\
> -		(a)[0] = (u8) ((((u32) (val)) >> 24) & 0xff);	\
> -		(a)[1] = (u8) ((((u32) (val)) >> 16) & 0xff);	\
> -		(a)[2] = (u8) ((((u32) (val)) >> 8) & 0xff);	\
> -		(a)[3] = (u8) (((u32) (val)) & 0xff);		\
> +		(a)[0] = (u8)((((u32) (val)) >> 24) & 0xff);	\
> +		(a)[1] = (u8)((((u32) (val)) >> 16) & 0xff);	\
> +		(a)[2] = (u8)((((u32) (val)) >> 8) & 0xff);	\
> +		(a)[3] = (u8)(((u32) (val)) & 0xff);		\
>  	} while (0)
>  
>  #define WPA_PUT_LE32(a, val)					\
>  	do {							\
> -		(a)[3] = (u8) ((((u32) (val)) >> 24) & 0xff);	\
> -		(a)[2] = (u8) ((((u32) (val)) >> 16) & 0xff);	\
> -		(a)[1] = (u8) ((((u32) (val)) >> 8) & 0xff);	\
> -		(a)[0] = (u8) (((u32) (val)) & 0xff);		\
> +		(a)[3] = (u8)((((u32) (val)) >> 24) & 0xff);	\
> +		(a)[2] = (u8)((((u32) (val)) >> 16) & 0xff);	\
> +		(a)[1] = (u8)((((u32) (val)) >> 8) & 0xff);	\
> +		(a)[0] = (u8)(((u32) (val)) & 0xff);		\
>  	} while (0)
>  
>  #define RSN_SELECTOR_PUT(a, val) WPA_PUT_BE32((u8 *)(a), (val))
> diff --git a/drivers/staging/rtl8188eu/include/odm_debug.h b/drivers/staging/rtl8188eu/include/odm_debug.h
> index db7b44e..7ca7358a 100644
> --- a/drivers/staging/rtl8188eu/include/odm_debug.h
> +++ b/drivers/staging/rtl8188eu/include/odm_debug.h
> @@ -125,7 +125,7 @@
>  		DbgPrint(title_str);					\
>  		DbgPrint(" ");						\
>  		for (__i = 0; __i < 6; __i++)				\
> -			DbgPrint("%02X%s", __ptr[__i], (__i == 5)?"":"-");\
> +			DbgPrint("%02X%s", __ptr[__i], (__i == 5) ? "" : "-");\
>  		DbgPrint("\n");						\
>  	}
>  
> diff --git a/drivers/staging/rtl8188eu/include/osdep_service.h b/drivers/staging/rtl8188eu/include/osdep_service.h
> index fed9c86..82f58f8 100644
> --- a/drivers/staging/rtl8188eu/include/osdep_service.h
> +++ b/drivers/staging/rtl8188eu/include/osdep_service.h
> @@ -182,8 +182,8 @@ u64 rtw_modular64(u64 x, u64 y);
>  
>  /* Macros for handling unaligned memory accesses */
>  
> -#define RTW_GET_BE24(a) ((((u32) (a)[0]) << 16) | (((u32) (a)[1]) << 8) | \
> -			 ((u32) (a)[2]))
> +#define RTW_GET_BE24(a) ((((u32)(a)[0]) << 16) | (((u32) (a)[1]) << 8) | \
> +			 ((u32)(a)[2]))
>  
>  void rtw_buf_free(u8 **buf, u32 *buf_len);
>  void rtw_buf_update(u8 **buf, u32 *buf_len, u8 *src, u32 src_len);
> diff --git a/drivers/staging/rtl8188eu/include/rtw_debug.h b/drivers/staging/rtl8188eu/include/rtw_debug.h
> index a38616e..971bf45 100644
> --- a/drivers/staging/rtl8188eu/include/rtw_debug.h
> +++ b/drivers/staging/rtl8188eu/include/rtw_debug.h
> @@ -106,7 +106,7 @@ extern u32 GlobalDebugLevel;
>  			u8	*ptr = (u8 *)_hexdata;			\
>  			pr_info("%s", DRIVER_PREFIX);			\
>  			pr_info(_titlestring);				\
> -			for (__i = 0; __i < (int)_hexdatalen; __i++ ) {	\
> +			for (__i = 0; __i < (int)_hexdatalen; __i++) {	\
>  				pr_info("%02X%s", ptr[__i],		\
>  					 (((__i + 1) % 4) == 0) ?	\
>  					 "  " : " ");	\
> diff --git a/drivers/staging/rtl8188eu/include/rtw_led.h b/drivers/staging/rtl8188eu/include/rtw_led.h
> index c5194b6..dec8dbc 100644
> --- a/drivers/staging/rtl8188eu/include/rtw_led.h
> +++ b/drivers/staging/rtl8188eu/include/rtw_led.h
> @@ -92,7 +92,7 @@ struct LED_871x {
>  
>  void LedControl8188eu(struct adapter *padapter, enum LED_CTL_MODE	LedAction);
>  
> -struct led_priv{
> +struct led_priv {
>  	/* add for led control */
>  	struct LED_871x			SwLed0;
>  	u8	bRegUseLed;
> diff --git a/drivers/staging/rtl8188eu/include/rtw_mlme_ext.h b/drivers/staging/rtl8188eu/include/rtw_mlme_ext.h
> index d699ca1..8d72ccf 100644
> --- a/drivers/staging/rtl8188eu/include/rtw_mlme_ext.h
> +++ b/drivers/staging/rtl8188eu/include/rtw_mlme_ext.h
> @@ -448,7 +448,7 @@ struct mlme_ext_priv {
>  
>  int init_mlme_ext_priv(struct adapter *adapter);
>  int init_hw_mlme_ext(struct adapter *padapter);
> -void free_mlme_ext_priv (struct mlme_ext_priv *pmlmeext);
> +void free_mlme_ext_priv(struct mlme_ext_priv *pmlmeext);
>  extern void init_mlme_ext_timer(struct adapter *padapter);
>  extern void init_addba_retry_timer(struct adapter *adapt, struct sta_info *sta);
>  extern struct xmit_frame *alloc_mgtxmitframe(struct xmit_priv *pxmitpriv);
> @@ -646,8 +646,8 @@ void mlmeext_sta_add_event_callback(struct adapter *padapter,
>  
>  void linked_status_chk(struct adapter *padapter);
>  
> -void survey_timer_hdl (void *function_context);
> -void link_timer_hdl (void *funtion_context);
> +void survey_timer_hdl(void *function_context);
> +void link_timer_hdl(void *funtion_context);
>  void addba_timer_hdl(void *function_context);
>  
>  #define set_survey_timer(mlmeext, ms) \
> @@ -708,15 +708,15 @@ u8 tdls_hdl(struct adapter *padapter, unsigned char *pbuf);
>  #ifdef _RTW_CMD_C_
>  
>  static struct cmd_hdl wlancmds[] = {
> -	GEN_MLME_EXT_HANDLER(sizeof (struct wlan_bssid_ex), join_cmd_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct disconnect_parm), disconnect_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct wlan_bssid_ex), createbss_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct setopmode_parm), setopmode_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct sitesurvey_parm), sitesurvey_cmd_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct setauth_parm), setauth_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct setkey_parm), setkey_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct set_stakey_parm), set_stakey_hdl)
> -	GEN_MLME_EXT_HANDLER(sizeof (struct set_assocsta_parm), NULL)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct wlan_bssid_ex), join_cmd_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct disconnect_parm), disconnect_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct wlan_bssid_ex), createbss_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct setopmode_parm), setopmode_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct sitesurvey_parm), sitesurvey_cmd_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct setauth_parm), setauth_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct setkey_parm), setkey_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct set_stakey_parm), set_stakey_hdl)
> +	GEN_MLME_EXT_HANDLER(sizeof(struct set_assocsta_parm), NULL)
>  	GEN_MLME_EXT_HANDLER(sizeof(struct addBaReq_parm), add_ba_hdl)
>  	GEN_MLME_EXT_HANDLER(sizeof(struct set_ch_parm), set_ch_hdl)
>  	GEN_MLME_EXT_HANDLER(sizeof(struct wlan_bssid_ex), tx_beacon_hdl)
> @@ -787,7 +787,7 @@ static struct fwevent wlanevents[] = {
>  	{0, NULL},
>  	{0, NULL},
>  	{0, &rtw_survey_event_callback},		/*8*/
> -	{sizeof (struct surveydone_event), &rtw_surveydone_event_callback},/*9*/
> +	{sizeof(struct surveydone_event), &rtw_surveydone_event_callback},/*9*/
>  	{0, &rtw_joinbss_event_callback},		/*10*/
>  	{sizeof(struct stassoc_event), &rtw_stassoc_event_callback},
>  	{sizeof(struct stadel_event), &rtw_stadel_event_callback},
> diff --git a/drivers/staging/rtl8188eu/include/wifi.h b/drivers/staging/rtl8188eu/include/wifi.h
> index a88ebf4..8dbdfaf 100644
> --- a/drivers/staging/rtl8188eu/include/wifi.h
> +++ b/drivers/staging/rtl8188eu/include/wifi.h
> @@ -766,27 +766,27 @@ enum ht_cap_ampdu_factor {
>  #define OP_MODE_20MHZ_HT_STA_ASSOCED    2
>  #define OP_MODE_MIXED                   3
>  
> -#define HT_INFO_HT_PARAM_SECONDARY_CHNL_OFF_MASK	((u8) BIT(0) | BIT(1))
> -#define HT_INFO_HT_PARAM_SECONDARY_CHNL_ABOVE		((u8) BIT(0))
> -#define HT_INFO_HT_PARAM_SECONDARY_CHNL_BELOW		((u8) BIT(0) | BIT(1))
> -#define HT_INFO_HT_PARAM_REC_TRANS_CHNL_WIDTH		((u8) BIT(2))
> -#define HT_INFO_HT_PARAM_RIFS_MODE			((u8) BIT(3))
> -#define HT_INFO_HT_PARAM_CTRL_ACCESS_ONLY		((u8) BIT(4))
> -#define HT_INFO_HT_PARAM_SRV_INTERVAL_GRANULARITY	((u8) BIT(5))
> +#define HT_INFO_HT_PARAM_SECONDARY_CHNL_OFF_MASK	((u8)BIT(0) | BIT(1))
> +#define HT_INFO_HT_PARAM_SECONDARY_CHNL_ABOVE		((u8)BIT(0))
> +#define HT_INFO_HT_PARAM_SECONDARY_CHNL_BELOW		((u8)BIT(0) | BIT(1))
> +#define HT_INFO_HT_PARAM_REC_TRANS_CHNL_WIDTH		((u8)BIT(2))
> +#define HT_INFO_HT_PARAM_RIFS_MODE			((u8)BIT(3))
> +#define HT_INFO_HT_PARAM_CTRL_ACCESS_ONLY		((u8)BIT(4))
> +#define HT_INFO_HT_PARAM_SRV_INTERVAL_GRANULARITY	((u8)BIT(5))
>  
>  #define HT_INFO_OPERATION_MODE_OP_MODE_MASK	\
> -		((u16) (0x0001 | 0x0002))
> +		((u16)(0x0001 | 0x0002))
>  #define HT_INFO_OPERATION_MODE_OP_MODE_OFFSET		0
> -#define HT_INFO_OPERATION_MODE_NON_GF_DEVS_PRESENT	((u8) BIT(2))
> -#define HT_INFO_OPERATION_MODE_TRANSMIT_BURST_LIMIT	((u8) BIT(3))
> -#define HT_INFO_OPERATION_MODE_NON_HT_STA_PRESENT	((u8) BIT(4))
> -
> -#define HT_INFO_STBC_PARAM_DUAL_BEACON		((u16) BIT(6))
> -#define HT_INFO_STBC_PARAM_DUAL_STBC_PROTECT	((u16) BIT(7))
> -#define HT_INFO_STBC_PARAM_SECONDARY_BC		((u16) BIT(8))
> -#define HT_INFO_STBC_PARAM_LSIG_TXOP_PROTECT_ALLOWED	((u16) BIT(9))
> -#define HT_INFO_STBC_PARAM_PCO_ACTIVE		((u16) BIT(10))
> -#define HT_INFO_STBC_PARAM_PCO_PHASE		((u16) BIT(11))
> +#define HT_INFO_OPERATION_MODE_NON_GF_DEVS_PRESENT	((u8)BIT(2))
> +#define HT_INFO_OPERATION_MODE_TRANSMIT_BURST_LIMIT	((u8)BIT(3))
> +#define HT_INFO_OPERATION_MODE_NON_HT_STA_PRESENT	((u8)BIT(4))
> +
> +#define HT_INFO_STBC_PARAM_DUAL_BEACON		((u16)BIT(6))
> +#define HT_INFO_STBC_PARAM_DUAL_STBC_PROTECT	((u16)BIT(7))
> +#define HT_INFO_STBC_PARAM_SECONDARY_BC		((u16)BIT(8))
> +#define HT_INFO_STBC_PARAM_LSIG_TXOP_PROTECT_ALLOWED	((u16)BIT(9))
> +#define HT_INFO_STBC_PARAM_PCO_ACTIVE		((u16)BIT(10))
> +#define HT_INFO_STBC_PARAM_PCO_PHASE		((u16)BIT(11))
>  
>  /*	===============WPS Section=============== */
>  /*	For WPSv1.0 */
> diff --git a/drivers/staging/rtl8188eu/os_dep/ioctl_linux.c b/drivers/staging/rtl8188eu/os_dep/ioctl_linux.c
> index d598fec..678dea1 100644
> --- a/drivers/staging/rtl8188eu/os_dep/ioctl_linux.c
> +++ b/drivers/staging/rtl8188eu/os_dep/ioctl_linux.c
> @@ -361,7 +361,7 @@ static int wpa_set_encryption(struct net_device *dev, struct ieee_param *param,
>  	param->u.crypt.err = 0;
>  	param->u.crypt.alg[IEEE_CRYPT_ALG_NAME_LEN - 1] = '\0';
>  
> -	if (param_len < (u32) ((u8 *)param->u.crypt.key - (u8 *)param) + param->u.crypt.key_len) {
> +	if (param_len < (u32)((u8 *)param->u.crypt.key - (u8 *)param) + param->u.crypt.key_len) {
>  		ret =  -EINVAL;
>  		goto exit;
>  	}
> diff --git a/drivers/staging/rtl8188eu/os_dep/os_intfs.c b/drivers/staging/rtl8188eu/os_dep/os_intfs.c
> index 08a80f7..0b488d2 100644
> --- a/drivers/staging/rtl8188eu/os_dep/os_intfs.c
> +++ b/drivers/staging/rtl8188eu/os_dep/os_intfs.c
> @@ -527,7 +527,7 @@ static uint loadparam(struct adapter *padapter,  struct  net_device *pnetdev)
>  
>  	registry_par->channel = (u8)rtw_channel;
>  	registry_par->wireless_mode = (u8)rtw_wireless_mode;
> -	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense ;
> +	registry_par->vrtl_carrier_sense = (u8)rtw_vrtl_carrier_sense;
>  	registry_par->vcs_type = (u8)rtw_vcs_type;
>  	registry_par->rts_thresh = (u16)rtw_rts_thresh;
>  	registry_par->frag_thresh = (u16)rtw_frag_thresh;
> diff --git a/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c b/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c
> index ba1e178..d459a81 100644
> --- a/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c
> +++ b/drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c
> @@ -160,10 +160,10 @@ static int recvbuf2recvframe(struct adapter *adapt, struct sk_buff *pskb)
>  		switch (haldata->UsbRxAggMode) {
>  		case USB_RX_AGG_DMA:
>  		case USB_RX_AGG_MIX:
> -			pkt_offset = (u16) round_up(pkt_offset, 128);
> +			pkt_offset = (u16)round_up(pkt_offset, 128);
>  			break;
>  		case USB_RX_AGG_USB:
> -			pkt_offset = (u16) round_up(pkt_offset, 4);
> +			pkt_offset = (u16)round_up(pkt_offset, 4);
>  			break;
>  		case USB_RX_AGG_DISABLE:
>  		default:
> diff --git a/drivers/staging/rtl8188eu/os_dep/xmit_linux.c b/drivers/staging/rtl8188eu/os_dep/xmit_linux.c
> index 0ce47b0..d5e41a5 100644
> --- a/drivers/staging/rtl8188eu/os_dep/xmit_linux.c
> +++ b/drivers/staging/rtl8188eu/os_dep/xmit_linux.c
> @@ -46,7 +46,7 @@ void _rtw_open_pktfile(struct sk_buff *pktptr, struct pkt_file *pfile)
>  
>  }
>  
> -uint _rtw_pktfile_read (struct pkt_file *pfile, u8 *rmem, uint rlen)
> +uint _rtw_pktfile_read(struct pkt_file *pfile, u8 *rmem, uint rlen)
>  {
>  	uint	len = 0;
>  

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ