diff options
author | Joe Perches <joe@perches.com> | 2012-06-04 08:44:18 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-06-06 12:31:33 -0400 |
commit | c2fd03a0115a244c5f622453b2b1f038ed5700a6 (patch) | |
tree | 55c6a15e0ab898f15b08dfe4dcba4db55dc2af9b /drivers/net/hamradio/mkiss.c | |
parent | 2c208890c6d4e16076c6664137703ec813e8fa6c (diff) |
drivers: net: 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
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/hamradio/mkiss.c')
-rw-r--r-- | drivers/net/hamradio/mkiss.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/hamradio/mkiss.c b/drivers/net/hamradio/mkiss.c index aed1a6105b24..2c0894a92abd 100644 --- a/drivers/net/hamradio/mkiss.c +++ b/drivers/net/hamradio/mkiss.c | |||
@@ -485,7 +485,7 @@ static void ax_encaps(struct net_device *dev, unsigned char *icp, int len) | |||
485 | 485 | ||
486 | return; | 486 | return; |
487 | default: | 487 | default: |
488 | count = kiss_esc(p, (unsigned char *)ax->xbuff, len); | 488 | count = kiss_esc(p, ax->xbuff, len); |
489 | } | 489 | } |
490 | } else { | 490 | } else { |
491 | unsigned short crc; | 491 | unsigned short crc; |
@@ -497,7 +497,7 @@ static void ax_encaps(struct net_device *dev, unsigned char *icp, int len) | |||
497 | case CRC_MODE_SMACK: | 497 | case CRC_MODE_SMACK: |
498 | *p |= 0x80; | 498 | *p |= 0x80; |
499 | crc = swab16(crc16(0, p, len)); | 499 | crc = swab16(crc16(0, p, len)); |
500 | count = kiss_esc_crc(p, (unsigned char *)ax->xbuff, crc, len+2); | 500 | count = kiss_esc_crc(p, ax->xbuff, crc, len+2); |
501 | break; | 501 | break; |
502 | case CRC_MODE_FLEX_TEST: | 502 | case CRC_MODE_FLEX_TEST: |
503 | ax->crcmode = CRC_MODE_NONE; | 503 | ax->crcmode = CRC_MODE_NONE; |
@@ -506,11 +506,11 @@ static void ax_encaps(struct net_device *dev, unsigned char *icp, int len) | |||
506 | case CRC_MODE_FLEX: | 506 | case CRC_MODE_FLEX: |
507 | *p |= 0x20; | 507 | *p |= 0x20; |
508 | crc = calc_crc_flex(p, len); | 508 | crc = calc_crc_flex(p, len); |
509 | count = kiss_esc_crc(p, (unsigned char *)ax->xbuff, crc, len+2); | 509 | count = kiss_esc_crc(p, ax->xbuff, crc, len+2); |
510 | break; | 510 | break; |
511 | 511 | ||
512 | default: | 512 | default: |
513 | count = kiss_esc(p, (unsigned char *)ax->xbuff, len); | 513 | count = kiss_esc(p, ax->xbuff, len); |
514 | } | 514 | } |
515 | } | 515 | } |
516 | spin_unlock_bh(&ax->buflock); | 516 | spin_unlock_bh(&ax->buflock); |