On Tue, Oct 27, 2015 at 10:18:10PM +0700, Ivan Safonov wrote:
> min_t macro better conveys the meaning of the code.
>
> Signed-off-by: Ivan Safonov <insafo...@gmail.com>
> ---
<snip>
> diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> b/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> index 2d88c5a..5808763 100644
> --- a/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> +++ b/drivers/staging/rtl8188eu/hal/rtl8188e_cmd.c
> @@ -469,7 +469,7 @@ static int wpa_set_encryption(struct net_device *dev,
> struct ieee_param *param,
>
> rtw_setstakey_cmd(padapter, (unsigned
> char *)psta, true);
> } else { /* group key */
> -
> memcpy(padapter->securitypriv.dot118021XGrpKey[param->u.crypt.idx].skey,
> param->u.crypt.key, (param->u.crypt.key_len > 16 ? 16 :
> param->u.crypt.key_len));
> +
> memcpy(padapter->securitypriv.dot118021XGrpKey[param->u.crypt.idx].skey,
> param->u.crypt.key, min_t(u16, param->u.crypt.key_len, 16 ));
generates checkpatch error of space prohibited before close parenthesis.
regards
sudip
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
