From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stanislav Yakovlev Subject: Re: [PATCH] ipw2200: use is_zero_ether_addr() and is_broadcast_ether_addr() Date: Fri, 24 Aug 2012 01:13:07 +0400 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Cc: linville@tuxdriver.com, yongjun_wei@trendmicro.com.cn, linux-wireless@vger.kernel.org, netdev@vger.kernel.org To: Wei Yongjun Return-path: Received: from mail-vb0-f46.google.com ([209.85.212.46]:36486 "EHLO mail-vb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934412Ab2HWVNI (ORCPT ); Thu, 23 Aug 2012 17:13:08 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Hi, On 23 August 2012 10:54, Wei Yongjun wrote: > From: Wei Yongjun > > Using is_zero_ether_addr() and is_broadcast_ether_addr() instead of > directly use memcmp() to determine if the ethernet address is all zeros. > > spatch with a semantic match is used to found this problem. > (http://coccinelle.lip6.fr/) > > Signed-off-by: Wei Yongjun > --- > drivers/net/wireless/ipw2x00/ipw2200.c | 11 ++--------- > 1 file changed, 2 insertions(+), 9 deletions(-) Looks fine, thanks. Stanislav.