staging: brcm80211: Convert ETHER_IS<FOO> to is_<foo>_ether_addr

Use the normal kernel calls and remove the #defines
for ETHER_IS_BCAST and ETHER_IS_NULLADDR.

Add #include for etherdevice.h where necessary.

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
Joe Perches 2010-12-13 16:57:00 -08:00 committed by Greg Kroah-Hartman
parent 47c6de7d37
commit ed9568687e
5 changed files with 10 additions and 21 deletions

View File

@ -772,7 +772,8 @@ wl_iw_set_wap(struct net_device *dev,
return -EINVAL; return -EINVAL;
} }
if (ETHER_ISBCAST(awrq->sa_data) || ETHER_ISNULLADDR(awrq->sa_data)) { if (is_broadcast_ether_addr(awrq->sa_data) ||
is_zero_ether_addr(awrq->sa_data)) {
scb_val_t scbval; scb_val_t scbval;
memset(&scbval, 0, sizeof(scb_val_t)); memset(&scbval, 0, sizeof(scb_val_t));
(void)dev_wlc_ioctl(dev, WLC_DISASSOC, &scbval, (void)dev_wlc_ioctl(dev, WLC_DISASSOC, &scbval,

View File

@ -66,19 +66,6 @@ BWL_PRE_PACKED_STRUCT struct ether_addr {
static const struct ether_addr ether_bcast = { {255, 255, 255, 255, 255, 255} }; static const struct ether_addr ether_bcast = { {255, 255, 255, 255, 255, 255} };
#define ETHER_ISBCAST(ea) ((((u8 *)(ea))[0] & \
((u8 *)(ea))[1] & \
((u8 *)(ea))[2] & \
((u8 *)(ea))[3] & \
((u8 *)(ea))[4] & \
((u8 *)(ea))[5]) == 0xff)
#define ETHER_ISNULLADDR(ea) ((((u8 *)(ea))[0] | \
((u8 *)(ea))[1] | \
((u8 *)(ea))[2] | \
((u8 *)(ea))[3] | \
((u8 *)(ea))[4] | \
((u8 *)(ea))[5]) == 0)
#define ETHER_MOVE_HDR(d, s) \ #define ETHER_MOVE_HDR(d, s) \
do { \ do { \
struct ether_header t; \ struct ether_header t; \

View File

@ -20,6 +20,7 @@
#include <linux/module.h> #include <linux/module.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/netdevice.h> #include <linux/netdevice.h>
#include <linux/etherdevice.h>
#include <bcmdefs.h> #include <bcmdefs.h>
#include <osl.h> #include <osl.h>
#include <proto/802.11.h> #include <proto/802.11.h>
@ -1022,8 +1023,8 @@ int wlc_bmac_attach(struct wlc_info *wlc, u16 vendor, u16 device, uint unit,
goto fail; goto fail;
} }
bcm_ether_atoe(macaddr, &wlc_hw->etheraddr); bcm_ether_atoe(macaddr, &wlc_hw->etheraddr);
if (ETHER_ISBCAST((char *)&wlc_hw->etheraddr) || if (is_broadcast_ether_addr(wlc_hw->etheraddr.octet) ||
ETHER_ISNULLADDR((char *)&wlc_hw->etheraddr)) { is_zero_ether_addr(wlc_hw->etheraddr.octet)) {
WL_ERROR(("wl%d: wlc_bmac_attach: bad macaddr %s\n", unit, WL_ERROR(("wl%d: wlc_bmac_attach: bad macaddr %s\n", unit,
macaddr)); macaddr));
err = 22; err = 22;

View File

@ -3633,8 +3633,8 @@ _wlc_ioctl(struct wlc_info *wlc, int cmd, void *arg, int len,
u16 lo; u16 lo;
u32 hi; u32 hi;
/* group keys in WPA-NONE (IBSS only, AES and TKIP) use a global TXIV */ /* group keys in WPA-NONE (IBSS only, AES and TKIP) use a global TXIV */
if ((bsscfg->WPA_auth & WPA_AUTH_NONE) if ((bsscfg->WPA_auth & WPA_AUTH_NONE) &&
&& ETHER_ISNULLADDR(&key->ea)) { is_zero_ether_addr(key->ea.octet)) {
lo = bsscfg->wpa_none_txiv.lo; lo = bsscfg->wpa_none_txiv.lo;
hi = bsscfg->wpa_none_txiv.hi; hi = bsscfg->wpa_none_txiv.hi;
} else { } else {
@ -7026,7 +7026,7 @@ void BCMFASTPATH wlc_recv(struct wlc_info *wlc, struct sk_buff *p)
if (!is_amsdu) { if (!is_amsdu) {
/* CTS and ACK CTL frames are w/o a2 */ /* CTS and ACK CTL frames are w/o a2 */
if (FC_TYPE(fc) == FC_TYPE_DATA || FC_TYPE(fc) == FC_TYPE_MNG) { if (FC_TYPE(fc) == FC_TYPE_DATA || FC_TYPE(fc) == FC_TYPE_MNG) {
if ((ETHER_ISNULLADDR(&h->a2) || if ((is_zero_ether_addr(h->a2.octet) ||
is_multicast_ether_addr(h->a2.octet))) { is_multicast_ether_addr(h->a2.octet))) {
WL_ERROR(("wl%d: %s: dropping a frame with " WL_ERROR(("wl%d: %s: dropping a frame with "
"invalid src mac address, a2: %pM\n", "invalid src mac address, a2: %pM\n",

View File

@ -502,7 +502,7 @@ int srom_parsecis(struct osl_info *osh, u8 *pcis[], uint ciscnt, char **vars,
/* set macaddr if HNBU_MACADDR not seen yet */ /* set macaddr if HNBU_MACADDR not seen yet */
if (eabuf[0] == '\0' && if (eabuf[0] == '\0' &&
cis[i] == LAN_NID && cis[i] == LAN_NID &&
!(ETHER_ISNULLADDR(&cis[i + 2])) && !is_zero_ether_addr(&cis[i + 2]) &&
!is_multicast_ether_addr(&cis[i + 2])) { !is_multicast_ether_addr(&cis[i + 2])) {
ASSERT(cis[i + 1] == ASSERT(cis[i + 1] ==
ETHER_ADDR_LEN); ETHER_ADDR_LEN);
@ -974,7 +974,7 @@ int srom_parsecis(struct osl_info *osh, u8 *pcis[], uint ciscnt, char **vars,
break; break;
case HNBU_MACADDR: case HNBU_MACADDR:
if (!(ETHER_ISNULLADDR(&cis[i + 1])) && if (!is_zero_ether_addr(&cis[i + 1]) &&
!is_multicast_ether_addr(&cis[i + 1])) { !is_multicast_ether_addr(&cis[i + 1])) {
snprintf(eabuf, sizeof(eabuf), snprintf(eabuf, sizeof(eabuf),
"%pM", &cis[i + 1]); "%pM", &cis[i + 1]);