Re: [PATCH] drivers: staging: rtl8723au: Fix "(foo*)" should be "(foo *)" errors

[Joe Perches <joe@perches.com>]

On Fri, 2014-08-22 at 12:20 -0500, Greg Donald wrote:
> Fix checkpatch.pl "(foo*)" should be "(foo *)" errors

Hello Greg.

> diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging/rtl8723au/core/rtw_ap.c
[]
> @@ -599,7 +599,7 @@ void update_sta_info23a_apmode23a(struct rtw_adapter *padapter, struct sta_info
>  
>  	/* todo: init other variables */
>  
> -	memset((void*)&psta->sta_stats, 0, sizeof(struct stainfo_stats));
> +	memset((void *)&psta->sta_stats, 0, sizeof(struct stainfo_stats));

It'd be better to remove the (void *) cast instead.

[]
> diff --git a/drivers/staging/rtl8723au/core/rtw_cmd.c b/drivers/staging/rtl8723au/core/rtw_cmd.c
[]
> @@ -1493,7 +1493,7 @@ void rtw_setstaKey_cmdrsp_callback23a(struct rtw_adapter *padapter,
>  	struct sta_info *psta;
>  
>  	pstapriv = &padapter->stapriv;
> -	psetstakey_rsp = (struct set_stakey_rsp*) (pcmd->rsp);
> +	psetstakey_rsp = (struct set_stakey_rsp *) (pcmd->rsp);

Probably better to remove the space and the
unnecessary parentheses:

	psetstakey_rsp = (struct set_stakey_rsp *)pcmd->rsp;

> @@ -1518,7 +1518,7 @@ void rtw_setassocsta_cmdrsp_callback23a(struct rtw_adapter *padapter,
>  	struct sta_info *psta;
>  
>  	passocsta_parm = (struct set_assocsta_parm *)(pcmd->parmbuf);
> -	passocsta_rsp = (struct set_assocsta_rsp*) (pcmd->rsp);
> +	passocsta_rsp = (struct set_assocsta_rsp *) (pcmd->rsp);

same unnecessary parentheses, maybe
submit separate patches for those.

> diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
[]
> @@ -4763,7 +4763,7 @@ void report_survey_event23a(struct rtw_adapter *padapter,
>  	pcmd_obj->rsp = NULL;
>  	pcmd_obj->rspsz  = 0;
>  
> -	pc2h_evt_hdr = (struct C2HEvent_Header*)(pevtcmd);
> +	pc2h_evt_hdr = (struct C2HEvent_Header *)(pevtcmd);

more unnecessary parentheses, etc...