Subject: Re: [PATCH -v3 16/23] drivers/net: rename random32() to prandom_u32()

From: "Arend van Spriel" <arend-dY08KVG/lbpWk0Htik3J/[email-masked]>
Date: Mon, 4 Mar 2013 14:37:01 +0100

On 03/04/13 13:58, Akinobu Mita wrote:
> Use more preferable function name which implies using a pseudo-random
> number generator.
>
> Signed-off-by: Akinobu Mita<[email-masked]>
> Acked-by: Thomas Sailer<[email-masked]>
> Acked-by: Bing Zhao<[email-masked]>  [mwifiex]
> Cc: "David S. Miller"<davem-fT/PcQaiUtIeIZ0/[email-masked]>
> Cc: Michael Chan<mchan-dY08KVG/lbpWk0Htik3J/[email-masked]>
> Cc: Thomas Sailer<[email-masked]>
> Cc: Jean-Paul Roubelat<[email-masked]>
> Cc: Bing Zhao<[email-masked]>
> Cc: Brett Rudley<brudley-dY08KVG/lbpWk0Htik3J/[email-masked]>
> Cc: Arend van Spriel<arend-dY08KVG/lbpWk0Htik3J/[email-masked]>
> Cc: "Franky (Zhenhui) Lin"<frankyl-dY08KVG/lbpWk0Htik3J/[email-masked]>
> Cc: Hante Meuleman<meuleman-dY08KVG/lbpWk0Htik3J/[email-masked]>
> Cc: brcm80211-dev-list-dY08KVG/lbpWk0Htik3J/[email-masked]
> Cc: [email-masked]
> Cc: [email-masked]
> Cc: [email-masked]
> ---
> * Changes from v2
> - add Acked-by: line
> - add rename for brcm80211
>
>   drivers/net/ethernet/broadcom/cnic.c          | 4 ++--
>   drivers/net/hamradio/baycom_epp.c             | 2 +-
>   drivers/net/hamradio/hdlcdrv.c                | 2 +-
>   drivers/net/hamradio/yam.c                    | 2 +-
>   drivers/net/wireless/brcm80211/brcmfmac/p2p.c | 2 +-
>   drivers/net/wireless/mwifiex/cfg80211.c       | 4 ++--
>   6 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/net/wireless/brcm80211/brcmfmac/p2p.c b/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
> index 4166e64..bca31a8 100644
> --- a/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
> +++ b/drivers/net/wireless/brcm80211/brcmfmac/p2p.c
> @@ -1118,7 +1118,7 @@ static void brcmf_p2p_afx_handler(struct work_struct *work)
>   	if (afx_hdl->is_listen&&  afx_hdl->my_listen_chan)
>   		/* 100ms ~ 300ms */
>   		err = brcmf_p2p_discover_listen(p2p, afx_hdl->my_listen_chan,
> -						100 * (1 + (random32() % 3)));
> +					100 * (1 + (prandom_u32() % 3)));

Seems like indentation is wrong now. Please retain the alignment.

Regards,
Arend

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