Search Linux Wireless

[PATCH 001/119] staging: brcm80211: fix checkpatch errors in types.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Arend van Spriel <arend@xxxxxxxxxxxx>

The file types.h contains several occurrences of the checkpatch
error "(foo*)" should be "(foo *)". This has been fixed in this
patch.

Signed-off-by: Arend van Spriel <arend@xxxxxxxxxxxx>
Reviewed-by: Roland Vossen <rvossen@xxxxxxxxxxxx>
---
 drivers/staging/brcm80211/brcmsmac/types.h |   30 ++++++++++++++--------------
 1 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/drivers/staging/brcm80211/brcmsmac/types.h b/drivers/staging/brcm80211/brcmsmac/types.h
index 163973d..af1514e 100644
--- a/drivers/staging/brcm80211/brcmsmac/types.h
+++ b/drivers/staging/brcm80211/brcmsmac/types.h
@@ -257,9 +257,9 @@ do {						\
 #define R_REG(r) \
 	({\
 		sizeof(*(r)) == sizeof(u8) ? \
-		readb((volatile u8*)(r)) : \
-		sizeof(*(r)) == sizeof(u16) ? readw((volatile u16*)(r)) : \
-		readl((volatile u32*)(r)); \
+		readb((u8 *)(r)) : \
+		sizeof(*(r)) == sizeof(u16) ? readw((u16 *)(r)) : \
+		readl((u32 *)(r)); \
 	})
 #else				/* __mips__ */
 #define R_REG(r) \
@@ -268,14 +268,14 @@ do {						\
 		__asm__ __volatile__("sync"); \
 		switch (sizeof(*(r))) { \
 		case sizeof(u8): \
-			__osl_v = readb((volatile u8*)(r)); \
+			__osl_v = readb((u8 *)(r)); \
 			break; \
 		case sizeof(u16): \
-			__osl_v = readw((volatile u16*)(r)); \
+			__osl_v = readw((u16 *)(r)); \
 			break; \
 		case sizeof(u32): \
 			__osl_v = \
-			readl((volatile u32*)(r)); \
+			readl((u32 *)(r)); \
 			break; \
 		} \
 		__asm__ __volatile__("sync"); \
@@ -286,11 +286,11 @@ do {						\
 #define W_REG(r, v) do { \
 		switch (sizeof(*(r))) { \
 		case sizeof(u8): \
-			writeb((u8)(v), (volatile u8*)(r)); break; \
+			writeb((u8)(v), (u8 *)(r)); break; \
 		case sizeof(u16): \
-			writew((u16)(v), (volatile u16*)(r)); break; \
+			writew((u16)(v), (u16 *)(r)); break; \
 		case sizeof(u32): \
-			writel((u32)(v), (volatile u32*)(r)); break; \
+			writel((u32)(v), (u32 *)(r)); break; \
 		}; \
 	} while (0)
 #else				/* __BIG_ENDIAN */
@@ -300,14 +300,14 @@ do {						\
 		switch (sizeof(*(r))) { \
 		case sizeof(u8): \
 			__osl_v = \
-			readb((volatile u8*)((r)^3)); \
+			readb((u8 *)((r)^3)); \
 			break; \
 		case sizeof(u16): \
 			__osl_v = \
-			readw((volatile u16*)((r)^2)); \
+			readw((u16 *)((r)^2)); \
 			break; \
 		case sizeof(u32): \
-			__osl_v = readl((volatile u32*)(r)); \
+			__osl_v = readl((u32 *)(r)); \
 			break; \
 		} \
 		__osl_v; \
@@ -317,13 +317,13 @@ do {						\
 		switch (sizeof(*(r))) { \
 		case sizeof(u8):	\
 			writeb((u8)(v), \
-			(volatile u8*)((r)^3)); break; \
+			(u8 *)((r)^3)); break; \
 		case sizeof(u16):	\
 			writew((u16)(v), \
-			(volatile u16*)((r)^2)); break; \
+			(u16 *)((r)^2)); break; \
 		case sizeof(u32):	\
 			writel((u32)(v), \
-			(volatile u32*)(r)); break; \
+			(u32 *)(r)); break; \
 		} \
 	} while (0)
 #endif				/* __BIG_ENDIAN */
-- 
1.7.1


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux