diff options
author | Joe Perches <joe@perches.com> | 2012-06-04 08:44:17 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-06-06 12:31:33 -0400 |
commit | 2c208890c6d4e16076c6664137703ec813e8fa6c (patch) | |
tree | dd25049d7fdaf305679acc08f4b36fbcdbdb0213 /drivers/net/wireless/airo.c | |
parent | 6469933605a3ecdfa66b98160cde98ecd256cb3f (diff) |
wireless: Remove casts to same type
Adding casts of objects to the same type is unnecessary
and confusing for a human reader.
For example, this cast:
int y;
int *p = (int *)&y;
I used the coccinelle script below to find and remove these
unnecessary casts. I manually removed the conversions this
script produces of casts with __force, __iomem and __user.
@@
type T;
T *p;
@@
- (T *)p
+ p
Neatened the mwifiex_deauthenticate_infra function which
was doing odd things with array pointers and not using
is_zero_ether_addr.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/airo.c')
-rw-r--r-- | drivers/net/wireless/airo.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/airo.c b/drivers/net/wireless/airo.c index 520a4b2eb9cc..252c2c2d76c1 100644 --- a/drivers/net/wireless/airo.c +++ b/drivers/net/wireless/airo.c | |||
@@ -1997,7 +1997,7 @@ static int mpi_send_packet (struct net_device *dev) | |||
1997 | * ------------------------------------------------ | 1997 | * ------------------------------------------------ |
1998 | */ | 1998 | */ |
1999 | 1999 | ||
2000 | memcpy((char *)ai->txfids[0].virtual_host_addr, | 2000 | memcpy(ai->txfids[0].virtual_host_addr, |
2001 | (char *)&wifictlhdr8023, sizeof(wifictlhdr8023)); | 2001 | (char *)&wifictlhdr8023, sizeof(wifictlhdr8023)); |
2002 | 2002 | ||
2003 | payloadLen = (__le16 *)(ai->txfids[0].virtual_host_addr + | 2003 | payloadLen = (__le16 *)(ai->txfids[0].virtual_host_addr + |
@@ -4212,7 +4212,7 @@ static int PC4500_writerid(struct airo_info *ai, u16 rid, | |||
4212 | airo_print_err(ai->dev->name, "%s: len=%d", __func__, len); | 4212 | airo_print_err(ai->dev->name, "%s: len=%d", __func__, len); |
4213 | rc = -1; | 4213 | rc = -1; |
4214 | } else { | 4214 | } else { |
4215 | memcpy((char *)ai->config_desc.virtual_host_addr, | 4215 | memcpy(ai->config_desc.virtual_host_addr, |
4216 | pBuf, len); | 4216 | pBuf, len); |
4217 | 4217 | ||
4218 | rc = issuecommand(ai, &cmd, &rsp); | 4218 | rc = issuecommand(ai, &cmd, &rsp); |