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

From: Arend van Spriel
Date: Mon Mar 04 2013 - 08:37:34 EST


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<akinobu.mita@xxxxxxxxx>
Acked-by: Thomas Sailer<t.sailer@xxxxxxxxxxxxxx>
Acked-by: Bing Zhao<bzhao@xxxxxxxxxxx> [mwifiex]
Cc: "David S. Miller"<davem@xxxxxxxxxxxxx>
Cc: Michael Chan<mchan@xxxxxxxxxxxx>
Cc: Thomas Sailer<t.sailer@xxxxxxxxxxxxxx>
Cc: Jean-Paul Roubelat<jpr@xxxxxxxxx>
Cc: Bing Zhao<bzhao@xxxxxxxxxxx>
Cc: Brett Rudley<brudley@xxxxxxxxxxxx>
Cc: Arend van Spriel<arend@xxxxxxxxxxxx>
Cc: "Franky (Zhenhui) Lin"<frankyl@xxxxxxxxxxxx>
Cc: Hante Meuleman<meuleman@xxxxxxxxxxxx>
Cc: brcm80211-dev-list@xxxxxxxxxxxx
Cc: netdev@xxxxxxxxxxxxxxx
Cc: linux-hams@xxxxxxxxxxxxxxx
Cc: linux-wireless@xxxxxxxxxxxxxxx
---
* 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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/