* [PATCH] Staging: wlan-ng: fix most of the style issues in hfa384x.h
@ 2010-02-14 19:38 Alessandro Ghedini
2010-02-14 19:55 ` Gábor Stefanik
0 siblings, 1 reply; 3+ messages in thread
From: Alessandro Ghedini @ 2010-02-14 19:38 UTC (permalink / raw)
To: gregkh, jmm, mithlesh, karllinuxtest.relton
Cc: devel, linux-kernel, Alessandro Ghedini
This patch fixes all the errors and the majority of the warnings found with
checkpatch.pl script in hfa384x.h
Signed-off-by: Alessandro Ghedini <al3xbio@gmail.com>
---
drivers/staging/wlan-ng/hfa384x.h | 173 ++++++++++++++++++++++--------------
1 files changed, 106 insertions(+), 67 deletions(-)
diff --git a/drivers/staging/wlan-ng/hfa384x.h b/drivers/staging/wlan-ng/hfa384x.h
index 46cce81..8e39f76 100644
--- a/drivers/staging/wlan-ng/hfa384x.h
+++ b/drivers/staging/wlan-ng/hfa384x.h
@@ -63,13 +63,20 @@
/*--- Mins & Maxs -----------------------------------*/
#define HFA384x_PORTID_MAX ((u16)7)
#define HFA384x_NUMPORTS_MAX ((u16)(HFA384x_PORTID_MAX+1))
-#define HFA384x_PDR_LEN_MAX ((u16)512) /* in bytes, from EK */
+#define HFA384x_PDR_LEN_MAX ((u16)512) /*
+ * in bytes,
+ * from EK
+ */
#define HFA384x_PDA_RECS_MAX ((u16)200) /* a guess */
-#define HFA384x_PDA_LEN_MAX ((u16)1024) /* in bytes, from EK */
+#define HFA384x_PDA_LEN_MAX ((u16)1024) /*
+ * in bytes,
+ * from EK
+ */
#define HFA384x_SCANRESULT_MAX ((u16)31)
#define HFA384x_HSCANRESULT_MAX ((u16)31)
#define HFA384x_CHINFORESULT_MAX ((u16)16)
-#define HFA384x_RID_GUESSING_MAXLEN 2048 /* I'm not really sure */
+#define HFA384x_RID_GUESSING_MAXLEN 2048 /* I'm not
+ * really sure */
#define HFA384x_RIDDATA_MAXLEN HFA384x_RID_GUESSING_MAXLEN
#define HFA384x_USB_RWMEM_MAXLEN 2048
@@ -114,9 +121,9 @@
#define HFA384x_ADDR_AUX_OFF_MASK (0x007f)
/* Make a 32-bit flat address from AUX format 16-bit page and offset */
-#define HFA384x_ADDR_AUX_MKFLAT(p, o) \
- (((u32)(((u16)(p))&HFA384x_ADDR_AUX_PAGE_MASK)) << 7) | \
- ((u32)(((u16)(o))&HFA384x_ADDR_AUX_OFF_MASK))
+#define HFA384x_ADDR_AUX_MKFLAT(p, o) \
+ ((((u32)(((u16)(p))&HFA384x_ADDR_AUX_PAGE_MASK)) << 7) | \
+ ((u32)(((u16)(o))&HFA384x_ADDR_AUX_OFF_MASK)))
/* Make CMD format offset and page from a 32-bit flat address */
#define HFA384x_ADDR_CMD_MKPAGE(f) \
@@ -135,12 +142,21 @@
#define HFA384x_DLSTATE_FLASHENABLED 2
/*--- Register Field Masks --------------------------*/
-#define HFA384x_CMD_AINFO ((u16)(BIT(14) | BIT(13) | BIT(12) | BIT(11) | BIT(10) | BIT(9) | BIT(8)))
-#define HFA384x_CMD_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_CMD_AINFO ((u16)(BIT(14) | BIT(13) \
+ | BIT(12) | BIT(11) \
+ | BIT(10) | BIT(9) \
+ | BIT(8)))
+#define HFA384x_CMD_MACPORT ((u16)(BIT(10) | BIT(9) | \
+ BIT(8)))
#define HFA384x_CMD_PROGMODE ((u16)(BIT(9) | BIT(8)))
-#define HFA384x_CMD_CMDCODE ((u16)(BIT(5) | BIT(4) | BIT(3) | BIT(2) | BIT(1) | BIT(0)))
+#define HFA384x_CMD_CMDCODE ((u16)(BIT(5) | BIT(4) | \
+ BIT(3) | BIT(2) | \
+ BIT(1) | BIT(0)))
-#define HFA384x_STATUS_RESULT ((u16)(BIT(14) | BIT(13) | BIT(12) | BIT(11) | BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_STATUS_RESULT ((u16)(BIT(14) | BIT(13) \
+ | BIT(12) | BIT(11) \
+ | BIT(10) | BIT(9) \
+ | BIT(8)))
/*--- Command Code Constants --------------------------*/
/*--- Controller Commands --------------------------*/
@@ -244,8 +260,10 @@ Information RID Lengths: MAC Information
This is the length of JUST the DATA part of the RID (does not
include the len or code fields)
--------------------------------------------------------------------*/
-#define HFA384x_RID_DBMCOMMSQUALITY_LEN ((u16)sizeof(hfa384x_dbmcommsquality_t))
-#define HFA384x_RID_JOINREQUEST_LEN ((u16)sizeof(hfa384x_JoinRequest_data_t))
+#define HFA384x_RID_DBMCOMMSQUALITY_LEN \
+ ((u16) sizeof(hfa384x_dbmcommsquality_t))
+#define HFA384x_RID_JOINREQUEST_LEN \
+ ((u16)sizeof(hfa384x_JoinRequest_data_t))
/*--------------------------------------------------------------------
Information RIDs: Modem Information
@@ -322,9 +340,11 @@ PD Record codes
/*--- Register Test/Get/Set Field macros ------------------------*/
-#define HFA384x_CMD_AINFO_SET(value) ((u16)((u16)(value) << 8))
-#define HFA384x_CMD_MACPORT_SET(value) ((u16)HFA384x_CMD_AINFO_SET(value))
-#define HFA384x_CMD_PROGMODE_SET(value) ((u16)HFA384x_CMD_AINFO_SET((u16)value))
+#define HFA384x_CMD_AINFO_SET(value) ((u16)((u16)(value) << 8))
+#define HFA384x_CMD_MACPORT_SET(value) \
+ ((u16)HFA384x_CMD_AINFO_SET(value))
+#define HFA384x_CMD_PROGMODE_SET(value) \
+ ((u16)HFA384x_CMD_AINFO_SET((u16)value))
#define HFA384x_CMD_CMDCODE_SET(value) ((u16)(value))
#define HFA384x_STATUS_RESULT_SET(value) (((u16)(value)) << 8)
@@ -402,7 +422,7 @@ typedef struct hfa384x_authenticateStation_data {
/*-- Configuration Record: WPAData (data portion only) --*/
typedef struct hfa384x_WPAData {
u16 datalen;
- u8 data[0]; // max 80
+ u8 data[0]; /* max 80 */
} __attribute__ ((packed)) hfa384x_WPAData_t;
/*--------------------------------------------------------------------
@@ -479,7 +499,8 @@ Communication Frames: Field Masks for Transmit Frames
#define HFA384x_TXSTATUS_AGEDERR ((u16)BIT(1))
#define HFA384x_TXSTATUS_RETRYERR ((u16)BIT(0))
/*-- Transmit Control Field --*/
-#define HFA384x_TX_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_TX_MACPORT ((u16)(BIT(10) | \
+ BIT(9) | BIT(8)))
#define HFA384x_TX_STRUCTYPE ((u16)(BIT(4) | BIT(3)))
#define HFA384x_TX_TXEX ((u16)BIT(2))
#define HFA384x_TX_TXOK ((u16)BIT(1))
@@ -496,7 +517,8 @@ Communication Frames: Test/Get/Set Field Values for Transmit Frames
#define HFA384x_TX_SET(v, m, s) ((((u16)(v))<<((u16)(s)))&((u16)(m)))
#define HFA384x_TX_MACPORT_SET(v) HFA384x_TX_SET(v, HFA384x_TX_MACPORT, 8)
-#define HFA384x_TX_STRUCTYPE_SET(v) HFA384x_TX_SET(v, HFA384x_TX_STRUCTYPE, 3)
+#define HFA384x_TX_STRUCTYPE_SET(v) HFA384x_TX_SET(v, \
+ HFA384x_TX_STRUCTYPE, 3)
#define HFA384x_TX_TXEX_SET(v) HFA384x_TX_SET(v, HFA384x_TX_TXEX, 2)
#define HFA384x_TX_TXOK_SET(v) HFA384x_TX_SET(v, HFA384x_TX_TXOK, 1)
/*--------------------------------------------------------------------
@@ -534,13 +556,17 @@ Communication Frames: Field Masks for Receive Frames
--------------------------------------------------------------------*/
/*-- Status Fields --*/
-#define HFA384x_RXSTATUS_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_RXSTATUS_MACPORT ((u16)(BIT(10) | \
+ BIT(9) | \
+ BIT(8)))
#define HFA384x_RXSTATUS_FCSERR ((u16)BIT(0))
/*--------------------------------------------------------------------
Communication Frames: Test/Get/Set Field Values for Receive Frames
--------------------------------------------------------------------*/
-#define HFA384x_RXSTATUS_MACPORT_GET(value) ((u16)((((u16)(value)) & HFA384x_RXSTATUS_MACPORT) >> 8))
-#define HFA384x_RXSTATUS_ISFCSERR(value) ((u16)(((u16)(value)) & HFA384x_RXSTATUS_FCSERR))
+#define HFA384x_RXSTATUS_MACPORT_GET(value) ((u16)((((u16)(value)) \
+ & HFA384x_RXSTATUS_MACPORT) >> 8))
+#define HFA384x_RXSTATUS_ISFCSERR(value) ((u16)(((u16)(value)) \
+ & HFA384x_RXSTATUS_FCSERR))
/*--------------------------------------------------------------------
FRAME STRUCTURES: Information Types and Information Frame Structures
----------------------------------------------------------------------
@@ -1133,7 +1159,7 @@ struct hfa384x;
typedef void (*ctlx_cmdcb_t) (struct hfa384x *, const struct hfa384x_usbctlx *);
-typedef void (*ctlx_usercb_t) (struct hfa384x * hw,
+typedef void (*ctlx_usercb_t) (struct hfa384x *hw,
void *ctlxresult, void *usercb_data);
typedef struct hfa384x_usbctlx {
@@ -1174,14 +1200,14 @@ typedef struct hfa484x_metacmd {
} hfa384x_metacmd_t;
#define MAX_GRP_ADDR 32
-#define WLAN_COMMENT_MAX 80 /* Max. length of user comment string. */
+#define WLAN_COMMENT_MAX 80 /* Max. length of user comment string. */
-#define WLAN_AUTH_MAX 60 /* Max. # of authenticated stations. */
-#define WLAN_ACCESS_MAX 60 /* Max. # of stations in an access list. */
-#define WLAN_ACCESS_NONE 0 /* No stations may be authenticated. */
-#define WLAN_ACCESS_ALL 1 /* All stations may be authenticated. */
-#define WLAN_ACCESS_ALLOW 2 /* Authenticate only "allowed" stations. */
-#define WLAN_ACCESS_DENY 3 /* Do not authenticate "denied" stations. */
+#define WLAN_AUTH_MAX 60 /* Max. # of authenticated stations. */
+#define WLAN_ACCESS_MAX 60 /* Max. # of stations in an access list. */
+#define WLAN_ACCESS_NONE 0 /* No stations may be authenticated. */
+#define WLAN_ACCESS_ALL 1 /* All stations may be authenticated. */
+#define WLAN_ACCESS_ALLOW 2 /* Authenticate only "allowed" stations. */
+#define WLAN_ACCESS_DENY 3 /* Do not authenticate "denied" stations. */
/* XXX These are going away ASAP */
typedef struct prism2sta_authlist {
@@ -1294,10 +1320,23 @@ typedef struct hfa384x {
hfa384x_caplevel_t cap_sup_ap;
/* Actor compatibility ranges */
- hfa384x_caplevel_t cap_act_pri_cfi; /* pri f/w to controller interface */
- hfa384x_caplevel_t cap_act_sta_cfi; /* sta f/w to controller interface */
+ hfa384x_caplevel_t cap_act_pri_cfi; /*
+ * pri f/w to controller
+ * interface
+ */
+
+ hfa384x_caplevel_t cap_act_sta_cfi; /*
+ * sta f/w to controller
+ * interface
+ */
+
hfa384x_caplevel_t cap_act_sta_mfi; /* sta f/w to modem interface */
- hfa384x_caplevel_t cap_act_ap_cfi; /* ap f/w to controller interface */
+
+ hfa384x_caplevel_t cap_act_ap_cfi; /*
+ * ap f/w to controller
+ * interface
+ */
+
hfa384x_caplevel_t cap_act_ap_mfi; /* ap f/w to modem interface */
u32 psusercount; /* Power save user count. */
@@ -1320,25 +1359,25 @@ typedef struct hfa384x {
} hfa384x_t;
-void hfa384x_create(hfa384x_t * hw, struct usb_device *usb);
-void hfa384x_destroy(hfa384x_t * hw);
+void hfa384x_create(hfa384x_t *hw, struct usb_device *usb);
+void hfa384x_destroy(hfa384x_t *hw);
int
-hfa384x_corereset(hfa384x_t * hw, int holdtime, int settletime, int genesis);
-int hfa384x_drvr_commtallies(hfa384x_t * hw);
-int hfa384x_drvr_disable(hfa384x_t * hw, u16 macport);
-int hfa384x_drvr_enable(hfa384x_t * hw, u16 macport);
-int hfa384x_drvr_flashdl_enable(hfa384x_t * hw);
-int hfa384x_drvr_flashdl_disable(hfa384x_t * hw);
-int hfa384x_drvr_flashdl_write(hfa384x_t * hw, u32 daddr, void *buf, u32 len);
-int hfa384x_drvr_getconfig(hfa384x_t * hw, u16 rid, void *buf, u16 len);
-int hfa384x_drvr_ramdl_enable(hfa384x_t * hw, u32 exeaddr);
-int hfa384x_drvr_ramdl_disable(hfa384x_t * hw);
-int hfa384x_drvr_ramdl_write(hfa384x_t * hw, u32 daddr, void *buf, u32 len);
-int hfa384x_drvr_readpda(hfa384x_t * hw, void *buf, unsigned int len);
-int hfa384x_drvr_setconfig(hfa384x_t * hw, u16 rid, void *buf, u16 len);
-
-static inline int hfa384x_drvr_getconfig16(hfa384x_t * hw, u16 rid, void *val)
+hfa384x_corereset(hfa384x_t *hw, int holdtime, int settletime, int genesis);
+int hfa384x_drvr_commtallies(hfa384x_t *hw);
+int hfa384x_drvr_disable(hfa384x_t *hw, u16 macport);
+int hfa384x_drvr_enable(hfa384x_t *hw, u16 macport);
+int hfa384x_drvr_flashdl_enable(hfa384x_t *hw);
+int hfa384x_drvr_flashdl_disable(hfa384x_t *hw);
+int hfa384x_drvr_flashdl_write(hfa384x_t *hw, u32 daddr, void *buf, u32 len);
+int hfa384x_drvr_getconfig(hfa384x_t *hw, u16 rid, void *buf, u16 len);
+int hfa384x_drvr_ramdl_enable(hfa384x_t *hw, u32 exeaddr);
+int hfa384x_drvr_ramdl_disable(hfa384x_t *hw);
+int hfa384x_drvr_ramdl_write(hfa384x_t *hw, u32 daddr, void *buf, u32 len);
+int hfa384x_drvr_readpda(hfa384x_t *hw, void *buf, unsigned int len);
+int hfa384x_drvr_setconfig(hfa384x_t *hw, u16 rid, void *buf, u16 len);
+
+static inline int hfa384x_drvr_getconfig16(hfa384x_t *hw, u16 rid, void *val)
{
int result = 0;
result = hfa384x_drvr_getconfig(hw, rid, val, sizeof(u16));
@@ -1347,46 +1386,46 @@ static inline int hfa384x_drvr_getconfig16(hfa384x_t * hw, u16 rid, void *val)
return result;
}
-static inline int hfa384x_drvr_setconfig16(hfa384x_t * hw, u16 rid, u16 val)
+static inline int hfa384x_drvr_setconfig16(hfa384x_t *hw, u16 rid, u16 val)
{
u16 value = cpu_to_le16(val);
return hfa384x_drvr_setconfig(hw, rid, &value, sizeof(value));
}
int
-hfa384x_drvr_getconfig_async(hfa384x_t * hw,
+hfa384x_drvr_getconfig_async(hfa384x_t *hw,
u16 rid, ctlx_usercb_t usercb, void *usercb_data);
int
-hfa384x_drvr_setconfig_async(hfa384x_t * hw,
+hfa384x_drvr_setconfig_async(hfa384x_t *hw,
u16 rid,
void *buf,
u16 len, ctlx_usercb_t usercb, void *usercb_data);
static inline int
-hfa384x_drvr_setconfig16_async(hfa384x_t * hw, u16 rid, u16 val)
+hfa384x_drvr_setconfig16_async(hfa384x_t *hw, u16 rid, u16 val)
{
u16 value = cpu_to_le16(val);
return hfa384x_drvr_setconfig_async(hw, rid, &value, sizeof(value),
NULL, NULL);
}
-int hfa384x_drvr_start(hfa384x_t * hw);
-int hfa384x_drvr_stop(hfa384x_t * hw);
+int hfa384x_drvr_start(hfa384x_t *hw);
+int hfa384x_drvr_stop(hfa384x_t *hw);
int
-hfa384x_drvr_txframe(hfa384x_t * hw, struct sk_buff *skb,
- p80211_hdr_t * p80211_hdr, p80211_metawep_t * p80211_wep);
-void hfa384x_tx_timeout(wlandevice_t * wlandev);
-
-int hfa384x_cmd_initialize(hfa384x_t * hw);
-int hfa384x_cmd_enable(hfa384x_t * hw, u16 macport);
-int hfa384x_cmd_disable(hfa384x_t * hw, u16 macport);
-int hfa384x_cmd_allocate(hfa384x_t * hw, u16 len);
-int hfa384x_cmd_monitor(hfa384x_t * hw, u16 enable);
+hfa384x_drvr_txframe(hfa384x_t *hw, struct sk_buff *skb,
+ p80211_hdr_t *p80211_hdr, p80211_metawep_t *p80211_wep);
+void hfa384x_tx_timeout(wlandevice_t *wlandev);
+
+int hfa384x_cmd_initialize(hfa384x_t *hw);
+int hfa384x_cmd_enable(hfa384x_t *hw, u16 macport);
+int hfa384x_cmd_disable(hfa384x_t *hw, u16 macport);
+int hfa384x_cmd_allocate(hfa384x_t *hw, u16 len);
+int hfa384x_cmd_monitor(hfa384x_t *hw, u16 enable);
int
-hfa384x_cmd_download(hfa384x_t * hw,
+hfa384x_cmd_download(hfa384x_t *hw,
u16 mode, u16 lowaddr, u16 highaddr, u16 codelen);
-#endif /* __KERNEL__ */
+#endif /*__KERNEL__ */
-#endif /* _HFA384x_H */
+#endif /*_HFA384x_H */
--
1.6.6.2
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] Staging: wlan-ng: fix most of the style issues in hfa384x.h
2010-02-14 19:38 [PATCH] Staging: wlan-ng: fix most of the style issues in hfa384x.h Alessandro Ghedini
@ 2010-02-14 19:55 ` Gábor Stefanik
0 siblings, 0 replies; 3+ messages in thread
From: Gábor Stefanik @ 2010-02-14 19:55 UTC (permalink / raw)
To: Alessandro Ghedini
Cc: gregkh, jmm, mithlesh, karllinuxtest.relton, devel, linux-kernel
On Sun, Feb 14, 2010 at 8:38 PM, Alessandro Ghedini <al3xbio@gmail.com> wrote:
> This patch fixes all the errors and the majority of the warnings found with
> checkpatch.pl script in hfa384x.h
>
> Signed-off-by: Alessandro Ghedini <al3xbio@gmail.com>
> ---
> drivers/staging/wlan-ng/hfa384x.h | 173 ++++++++++++++++++++++--------------
> 1 files changed, 106 insertions(+), 67 deletions(-)
>
> diff --git a/drivers/staging/wlan-ng/hfa384x.h b/drivers/staging/wlan-ng/hfa384x.h
> index 46cce81..8e39f76 100644
> --- a/drivers/staging/wlan-ng/hfa384x.h
> +++ b/drivers/staging/wlan-ng/hfa384x.h
> @@ -63,13 +63,20 @@
> /*--- Mins & Maxs -----------------------------------*/
> #define HFA384x_PORTID_MAX ((u16)7)
> #define HFA384x_NUMPORTS_MAX ((u16)(HFA384x_PORTID_MAX+1))
> -#define HFA384x_PDR_LEN_MAX ((u16)512) /* in bytes, from EK */
> +#define HFA384x_PDR_LEN_MAX ((u16)512) /*
> + * in bytes,
> + * from EK
> + */
This is probably too much. Don't sacrifice readability for 80-char
limiting. Maybe reduce the spacing after "#define" and/or before the
comment instead.
> #define HFA384x_PDA_RECS_MAX ((u16)200) /* a guess */
> -#define HFA384x_PDA_LEN_MAX ((u16)1024) /* in bytes, from EK */
> +#define HFA384x_PDA_LEN_MAX ((u16)1024) /*
> + * in bytes,
> + * from EK
> + */
> #define HFA384x_SCANRESULT_MAX ((u16)31)
> #define HFA384x_HSCANRESULT_MAX ((u16)31)
> #define HFA384x_CHINFORESULT_MAX ((u16)16)
> -#define HFA384x_RID_GUESSING_MAXLEN 2048 /* I'm not really sure */
> +#define HFA384x_RID_GUESSING_MAXLEN 2048 /* I'm not
> + * really sure */
> #define HFA384x_RIDDATA_MAXLEN HFA384x_RID_GUESSING_MAXLEN
> #define HFA384x_USB_RWMEM_MAXLEN 2048
>
> @@ -114,9 +121,9 @@
> #define HFA384x_ADDR_AUX_OFF_MASK (0x007f)
>
> /* Make a 32-bit flat address from AUX format 16-bit page and offset */
> -#define HFA384x_ADDR_AUX_MKFLAT(p, o) \
> - (((u32)(((u16)(p))&HFA384x_ADDR_AUX_PAGE_MASK)) << 7) | \
> - ((u32)(((u16)(o))&HFA384x_ADDR_AUX_OFF_MASK))
> +#define HFA384x_ADDR_AUX_MKFLAT(p, o) \
> + ((((u32)(((u16)(p))&HFA384x_ADDR_AUX_PAGE_MASK)) << 7) | \
> + ((u32)(((u16)(o))&HFA384x_ADDR_AUX_OFF_MASK)))
>
> /* Make CMD format offset and page from a 32-bit flat address */
> #define HFA384x_ADDR_CMD_MKPAGE(f) \
> @@ -135,12 +142,21 @@
> #define HFA384x_DLSTATE_FLASHENABLED 2
>
> /*--- Register Field Masks --------------------------*/
> -#define HFA384x_CMD_AINFO ((u16)(BIT(14) | BIT(13) | BIT(12) | BIT(11) | BIT(10) | BIT(9) | BIT(8)))
> -#define HFA384x_CMD_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
> +#define HFA384x_CMD_AINFO ((u16)(BIT(14) | BIT(13) \
> + | BIT(12) | BIT(11) \
> + | BIT(10) | BIT(9) \
> + | BIT(8)))
> +#define HFA384x_CMD_MACPORT ((u16)(BIT(10) | BIT(9) | \
> + BIT(8)))
> #define HFA384x_CMD_PROGMODE ((u16)(BIT(9) | BIT(8)))
> -#define HFA384x_CMD_CMDCODE ((u16)(BIT(5) | BIT(4) | BIT(3) | BIT(2) | BIT(1) | BIT(0)))
> +#define HFA384x_CMD_CMDCODE ((u16)(BIT(5) | BIT(4) | \
> + BIT(3) | BIT(2) | \
> + BIT(1) | BIT(0)))
>
> -#define HFA384x_STATUS_RESULT ((u16)(BIT(14) | BIT(13) | BIT(12) | BIT(11) | BIT(10) | BIT(9) | BIT(8)))
> +#define HFA384x_STATUS_RESULT ((u16)(BIT(14) | BIT(13) \
> + | BIT(12) | BIT(11) \
> + | BIT(10) | BIT(9) \
> + | BIT(8)))
>
> /*--- Command Code Constants --------------------------*/
> /*--- Controller Commands --------------------------*/
> @@ -244,8 +260,10 @@ Information RID Lengths: MAC Information
> This is the length of JUST the DATA part of the RID (does not
> include the len or code fields)
> --------------------------------------------------------------------*/
> -#define HFA384x_RID_DBMCOMMSQUALITY_LEN ((u16)sizeof(hfa384x_dbmcommsquality_t))
> -#define HFA384x_RID_JOINREQUEST_LEN ((u16)sizeof(hfa384x_JoinRequest_data_t))
> +#define HFA384x_RID_DBMCOMMSQUALITY_LEN \
> + ((u16) sizeof(hfa384x_dbmcommsquality_t))
> +#define HFA384x_RID_JOINREQUEST_LEN \
> + ((u16)sizeof(hfa384x_JoinRequest_data_t))
>
> /*--------------------------------------------------------------------
> Information RIDs: Modem Information
> @@ -322,9 +340,11 @@ PD Record codes
>
> /*--- Register Test/Get/Set Field macros ------------------------*/
>
> -#define HFA384x_CMD_AINFO_SET(value) ((u16)((u16)(value) << 8))
> -#define HFA384x_CMD_MACPORT_SET(value) ((u16)HFA384x_CMD_AINFO_SET(value))
> -#define HFA384x_CMD_PROGMODE_SET(value) ((u16)HFA384x_CMD_AINFO_SET((u16)value))
> +#define HFA384x_CMD_AINFO_SET(value) ((u16)((u16)(value) << 8))
> +#define HFA384x_CMD_MACPORT_SET(value) \
> + ((u16)HFA384x_CMD_AINFO_SET(value))
> +#define HFA384x_CMD_PROGMODE_SET(value) \
> + ((u16)HFA384x_CMD_AINFO_SET((u16)value))
> #define HFA384x_CMD_CMDCODE_SET(value) ((u16)(value))
>
> #define HFA384x_STATUS_RESULT_SET(value) (((u16)(value)) << 8)
> @@ -402,7 +422,7 @@ typedef struct hfa384x_authenticateStation_data {
> /*-- Configuration Record: WPAData (data portion only) --*/
> typedef struct hfa384x_WPAData {
> u16 datalen;
> - u8 data[0]; // max 80
> + u8 data[0]; /* max 80 */
> } __attribute__ ((packed)) hfa384x_WPAData_t;
>
> /*--------------------------------------------------------------------
> @@ -479,7 +499,8 @@ Communication Frames: Field Masks for Transmit Frames
> #define HFA384x_TXSTATUS_AGEDERR ((u16)BIT(1))
> #define HFA384x_TXSTATUS_RETRYERR ((u16)BIT(0))
> /*-- Transmit Control Field --*/
> -#define HFA384x_TX_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
> +#define HFA384x_TX_MACPORT ((u16)(BIT(10) | \
> + BIT(9) | BIT(8)))
> #define HFA384x_TX_STRUCTYPE ((u16)(BIT(4) | BIT(3)))
> #define HFA384x_TX_TXEX ((u16)BIT(2))
> #define HFA384x_TX_TXOK ((u16)BIT(1))
> @@ -496,7 +517,8 @@ Communication Frames: Test/Get/Set Field Values for Transmit Frames
> #define HFA384x_TX_SET(v, m, s) ((((u16)(v))<<((u16)(s)))&((u16)(m)))
>
> #define HFA384x_TX_MACPORT_SET(v) HFA384x_TX_SET(v, HFA384x_TX_MACPORT, 8)
> -#define HFA384x_TX_STRUCTYPE_SET(v) HFA384x_TX_SET(v, HFA384x_TX_STRUCTYPE, 3)
> +#define HFA384x_TX_STRUCTYPE_SET(v) HFA384x_TX_SET(v, \
> + HFA384x_TX_STRUCTYPE, 3)
> #define HFA384x_TX_TXEX_SET(v) HFA384x_TX_SET(v, HFA384x_TX_TXEX, 2)
> #define HFA384x_TX_TXOK_SET(v) HFA384x_TX_SET(v, HFA384x_TX_TXOK, 1)
> /*--------------------------------------------------------------------
> @@ -534,13 +556,17 @@ Communication Frames: Field Masks for Receive Frames
> --------------------------------------------------------------------*/
>
> /*-- Status Fields --*/
> -#define HFA384x_RXSTATUS_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
> +#define HFA384x_RXSTATUS_MACPORT ((u16)(BIT(10) | \
> + BIT(9) | \
> + BIT(8)))
> #define HFA384x_RXSTATUS_FCSERR ((u16)BIT(0))
> /*--------------------------------------------------------------------
> Communication Frames: Test/Get/Set Field Values for Receive Frames
> --------------------------------------------------------------------*/
> -#define HFA384x_RXSTATUS_MACPORT_GET(value) ((u16)((((u16)(value)) & HFA384x_RXSTATUS_MACPORT) >> 8))
> -#define HFA384x_RXSTATUS_ISFCSERR(value) ((u16)(((u16)(value)) & HFA384x_RXSTATUS_FCSERR))
> +#define HFA384x_RXSTATUS_MACPORT_GET(value) ((u16)((((u16)(value)) \
> + & HFA384x_RXSTATUS_MACPORT) >> 8))
> +#define HFA384x_RXSTATUS_ISFCSERR(value) ((u16)(((u16)(value)) \
> + & HFA384x_RXSTATUS_FCSERR))
> /*--------------------------------------------------------------------
> FRAME STRUCTURES: Information Types and Information Frame Structures
> ----------------------------------------------------------------------
> @@ -1133,7 +1159,7 @@ struct hfa384x;
>
> typedef void (*ctlx_cmdcb_t) (struct hfa384x *, const struct hfa384x_usbctlx *);
>
> -typedef void (*ctlx_usercb_t) (struct hfa384x * hw,
> +typedef void (*ctlx_usercb_t) (struct hfa384x *hw,
> void *ctlxresult, void *usercb_data);
>
> typedef struct hfa384x_usbctlx {
> @@ -1174,14 +1200,14 @@ typedef struct hfa484x_metacmd {
> } hfa384x_metacmd_t;
>
> #define MAX_GRP_ADDR 32
> -#define WLAN_COMMENT_MAX 80 /* Max. length of user comment string. */
> +#define WLAN_COMMENT_MAX 80 /* Max. length of user comment string. */
>
> -#define WLAN_AUTH_MAX 60 /* Max. # of authenticated stations. */
> -#define WLAN_ACCESS_MAX 60 /* Max. # of stations in an access list. */
> -#define WLAN_ACCESS_NONE 0 /* No stations may be authenticated. */
> -#define WLAN_ACCESS_ALL 1 /* All stations may be authenticated. */
> -#define WLAN_ACCESS_ALLOW 2 /* Authenticate only "allowed" stations. */
> -#define WLAN_ACCESS_DENY 3 /* Do not authenticate "denied" stations. */
> +#define WLAN_AUTH_MAX 60 /* Max. # of authenticated stations. */
> +#define WLAN_ACCESS_MAX 60 /* Max. # of stations in an access list. */
> +#define WLAN_ACCESS_NONE 0 /* No stations may be authenticated. */
> +#define WLAN_ACCESS_ALL 1 /* All stations may be authenticated. */
> +#define WLAN_ACCESS_ALLOW 2 /* Authenticate only "allowed" stations. */
> +#define WLAN_ACCESS_DENY 3 /* Do not authenticate "denied" stations. */
>
> /* XXX These are going away ASAP */
> typedef struct prism2sta_authlist {
> @@ -1294,10 +1320,23 @@ typedef struct hfa384x {
> hfa384x_caplevel_t cap_sup_ap;
>
> /* Actor compatibility ranges */
> - hfa384x_caplevel_t cap_act_pri_cfi; /* pri f/w to controller interface */
> - hfa384x_caplevel_t cap_act_sta_cfi; /* sta f/w to controller interface */
> + hfa384x_caplevel_t cap_act_pri_cfi; /*
> + * pri f/w to controller
> + * interface
> + */
> +
> + hfa384x_caplevel_t cap_act_sta_cfi; /*
> + * sta f/w to controller
> + * interface
> + */
> +
> hfa384x_caplevel_t cap_act_sta_mfi; /* sta f/w to modem interface */
> - hfa384x_caplevel_t cap_act_ap_cfi; /* ap f/w to controller interface */
> +
> + hfa384x_caplevel_t cap_act_ap_cfi; /*
> + * ap f/w to controller
> + * interface
> + */
> +
> hfa384x_caplevel_t cap_act_ap_mfi; /* ap f/w to modem interface */
>
> u32 psusercount; /* Power save user count. */
> @@ -1320,25 +1359,25 @@ typedef struct hfa384x {
>
> } hfa384x_t;
>
> -void hfa384x_create(hfa384x_t * hw, struct usb_device *usb);
> -void hfa384x_destroy(hfa384x_t * hw);
> +void hfa384x_create(hfa384x_t *hw, struct usb_device *usb);
> +void hfa384x_destroy(hfa384x_t *hw);
>
> int
> -hfa384x_corereset(hfa384x_t * hw, int holdtime, int settletime, int genesis);
> -int hfa384x_drvr_commtallies(hfa384x_t * hw);
> -int hfa384x_drvr_disable(hfa384x_t * hw, u16 macport);
> -int hfa384x_drvr_enable(hfa384x_t * hw, u16 macport);
> -int hfa384x_drvr_flashdl_enable(hfa384x_t * hw);
> -int hfa384x_drvr_flashdl_disable(hfa384x_t * hw);
> -int hfa384x_drvr_flashdl_write(hfa384x_t * hw, u32 daddr, void *buf, u32 len);
> -int hfa384x_drvr_getconfig(hfa384x_t * hw, u16 rid, void *buf, u16 len);
> -int hfa384x_drvr_ramdl_enable(hfa384x_t * hw, u32 exeaddr);
> -int hfa384x_drvr_ramdl_disable(hfa384x_t * hw);
> -int hfa384x_drvr_ramdl_write(hfa384x_t * hw, u32 daddr, void *buf, u32 len);
> -int hfa384x_drvr_readpda(hfa384x_t * hw, void *buf, unsigned int len);
> -int hfa384x_drvr_setconfig(hfa384x_t * hw, u16 rid, void *buf, u16 len);
> -
> -static inline int hfa384x_drvr_getconfig16(hfa384x_t * hw, u16 rid, void *val)
> +hfa384x_corereset(hfa384x_t *hw, int holdtime, int settletime, int genesis);
> +int hfa384x_drvr_commtallies(hfa384x_t *hw);
> +int hfa384x_drvr_disable(hfa384x_t *hw, u16 macport);
> +int hfa384x_drvr_enable(hfa384x_t *hw, u16 macport);
> +int hfa384x_drvr_flashdl_enable(hfa384x_t *hw);
> +int hfa384x_drvr_flashdl_disable(hfa384x_t *hw);
> +int hfa384x_drvr_flashdl_write(hfa384x_t *hw, u32 daddr, void *buf, u32 len);
> +int hfa384x_drvr_getconfig(hfa384x_t *hw, u16 rid, void *buf, u16 len);
> +int hfa384x_drvr_ramdl_enable(hfa384x_t *hw, u32 exeaddr);
> +int hfa384x_drvr_ramdl_disable(hfa384x_t *hw);
> +int hfa384x_drvr_ramdl_write(hfa384x_t *hw, u32 daddr, void *buf, u32 len);
> +int hfa384x_drvr_readpda(hfa384x_t *hw, void *buf, unsigned int len);
> +int hfa384x_drvr_setconfig(hfa384x_t *hw, u16 rid, void *buf, u16 len);
> +
> +static inline int hfa384x_drvr_getconfig16(hfa384x_t *hw, u16 rid, void *val)
> {
> int result = 0;
> result = hfa384x_drvr_getconfig(hw, rid, val, sizeof(u16));
> @@ -1347,46 +1386,46 @@ static inline int hfa384x_drvr_getconfig16(hfa384x_t * hw, u16 rid, void *val)
> return result;
> }
>
> -static inline int hfa384x_drvr_setconfig16(hfa384x_t * hw, u16 rid, u16 val)
> +static inline int hfa384x_drvr_setconfig16(hfa384x_t *hw, u16 rid, u16 val)
> {
> u16 value = cpu_to_le16(val);
> return hfa384x_drvr_setconfig(hw, rid, &value, sizeof(value));
> }
>
> int
> -hfa384x_drvr_getconfig_async(hfa384x_t * hw,
> +hfa384x_drvr_getconfig_async(hfa384x_t *hw,
> u16 rid, ctlx_usercb_t usercb, void *usercb_data);
>
> int
> -hfa384x_drvr_setconfig_async(hfa384x_t * hw,
> +hfa384x_drvr_setconfig_async(hfa384x_t *hw,
> u16 rid,
> void *buf,
> u16 len, ctlx_usercb_t usercb, void *usercb_data);
>
> static inline int
> -hfa384x_drvr_setconfig16_async(hfa384x_t * hw, u16 rid, u16 val)
> +hfa384x_drvr_setconfig16_async(hfa384x_t *hw, u16 rid, u16 val)
> {
> u16 value = cpu_to_le16(val);
> return hfa384x_drvr_setconfig_async(hw, rid, &value, sizeof(value),
> NULL, NULL);
> }
>
> -int hfa384x_drvr_start(hfa384x_t * hw);
> -int hfa384x_drvr_stop(hfa384x_t * hw);
> +int hfa384x_drvr_start(hfa384x_t *hw);
> +int hfa384x_drvr_stop(hfa384x_t *hw);
> int
> -hfa384x_drvr_txframe(hfa384x_t * hw, struct sk_buff *skb,
> - p80211_hdr_t * p80211_hdr, p80211_metawep_t * p80211_wep);
> -void hfa384x_tx_timeout(wlandevice_t * wlandev);
> -
> -int hfa384x_cmd_initialize(hfa384x_t * hw);
> -int hfa384x_cmd_enable(hfa384x_t * hw, u16 macport);
> -int hfa384x_cmd_disable(hfa384x_t * hw, u16 macport);
> -int hfa384x_cmd_allocate(hfa384x_t * hw, u16 len);
> -int hfa384x_cmd_monitor(hfa384x_t * hw, u16 enable);
> +hfa384x_drvr_txframe(hfa384x_t *hw, struct sk_buff *skb,
> + p80211_hdr_t *p80211_hdr, p80211_metawep_t *p80211_wep);
> +void hfa384x_tx_timeout(wlandevice_t *wlandev);
> +
> +int hfa384x_cmd_initialize(hfa384x_t *hw);
> +int hfa384x_cmd_enable(hfa384x_t *hw, u16 macport);
> +int hfa384x_cmd_disable(hfa384x_t *hw, u16 macport);
> +int hfa384x_cmd_allocate(hfa384x_t *hw, u16 len);
> +int hfa384x_cmd_monitor(hfa384x_t *hw, u16 enable);
> int
> -hfa384x_cmd_download(hfa384x_t * hw,
> +hfa384x_cmd_download(hfa384x_t *hw,
> u16 mode, u16 lowaddr, u16 highaddr, u16 codelen);
>
> -#endif /* __KERNEL__ */
> +#endif /*__KERNEL__ */
>
> -#endif /* _HFA384x_H */
> +#endif /*_HFA384x_H */
> --
> 1.6.6.2
>
> _______________________________________________
> devel mailing list
> devel@linuxdriverproject.org
> http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
>
--
Vista: [V]iruses, [I]ntruders, [S]pyware, [T]rojans and [A]dware. :-)
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] Staging: wlan-ng: fix most of the style issues in hfa384x.h
@ 2010-02-15 11:25 Alessandro Ghedini
0 siblings, 0 replies; 3+ messages in thread
From: Alessandro Ghedini @ 2010-02-15 11:25 UTC (permalink / raw)
To: gregkh, jmm, mithlesh, karllinuxtest.relton
Cc: devel, linux-kernel, Alessandro Ghedini
This patch fixes all the errors and the majority of the warnings found with
checkpatch.pl script in hfa384x.h, following Gábor Stefanik hints
Signed-off-by: Alessandro Ghedini <al3xbio@gmail.com>
---
drivers/staging/wlan-ng/hfa384x.h | 182 ++++++++++++++++++++++---------------
1 files changed, 107 insertions(+), 75 deletions(-)
diff --git a/drivers/staging/wlan-ng/hfa384x.h b/drivers/staging/wlan-ng/hfa384x.h
index 46cce81..5984d74 100644
--- a/drivers/staging/wlan-ng/hfa384x.h
+++ b/drivers/staging/wlan-ng/hfa384x.h
@@ -61,17 +61,17 @@
#include <linux/if_ether.h>
/*--- Mins & Maxs -----------------------------------*/
-#define HFA384x_PORTID_MAX ((u16)7)
-#define HFA384x_NUMPORTS_MAX ((u16)(HFA384x_PORTID_MAX+1))
-#define HFA384x_PDR_LEN_MAX ((u16)512) /* in bytes, from EK */
-#define HFA384x_PDA_RECS_MAX ((u16)200) /* a guess */
-#define HFA384x_PDA_LEN_MAX ((u16)1024) /* in bytes, from EK */
-#define HFA384x_SCANRESULT_MAX ((u16)31)
-#define HFA384x_HSCANRESULT_MAX ((u16)31)
-#define HFA384x_CHINFORESULT_MAX ((u16)16)
-#define HFA384x_RID_GUESSING_MAXLEN 2048 /* I'm not really sure */
-#define HFA384x_RIDDATA_MAXLEN HFA384x_RID_GUESSING_MAXLEN
-#define HFA384x_USB_RWMEM_MAXLEN 2048
+#define HFA384x_PORTID_MAX ((u16)7)
+#define HFA384x_NUMPORTS_MAX ((u16)(HFA384x_PORTID_MAX+1))
+#define HFA384x_PDR_LEN_MAX ((u16)512) /* in bytes, from EK */
+#define HFA384x_PDA_RECS_MAX ((u16)200) /* a guess */
+#define HFA384x_PDA_LEN_MAX ((u16)1024) /* in bytes, from EK*/
+#define HFA384x_SCANRESULT_MAX ((u16)31)
+#define HFA384x_HSCANRESULT_MAX ((u16)31)
+#define HFA384x_CHINFORESULT_MAX ((u16)16)
+#define HFA384x_RID_GUESSING_MAXLEN 2048 /* I'm not really sure */
+#define HFA384x_RIDDATA_MAXLEN HFA384x_RID_GUESSING_MAXLEN
+#define HFA384x_USB_RWMEM_MAXLEN 2048
/*--- Support Constants -----------------------------*/
#define HFA384x_PORTTYPE_IBSS ((u16)0)
@@ -114,9 +114,9 @@
#define HFA384x_ADDR_AUX_OFF_MASK (0x007f)
/* Make a 32-bit flat address from AUX format 16-bit page and offset */
-#define HFA384x_ADDR_AUX_MKFLAT(p, o) \
- (((u32)(((u16)(p))&HFA384x_ADDR_AUX_PAGE_MASK)) << 7) | \
- ((u32)(((u16)(o))&HFA384x_ADDR_AUX_OFF_MASK))
+#define HFA384x_ADDR_AUX_MKFLAT(p, o) \
+ ((((u32)(((u16)(p))&HFA384x_ADDR_AUX_PAGE_MASK)) << 7) | \
+ ((u32)(((u16)(o))&HFA384x_ADDR_AUX_OFF_MASK)))
/* Make CMD format offset and page from a 32-bit flat address */
#define HFA384x_ADDR_CMD_MKPAGE(f) \
@@ -135,12 +135,21 @@
#define HFA384x_DLSTATE_FLASHENABLED 2
/*--- Register Field Masks --------------------------*/
-#define HFA384x_CMD_AINFO ((u16)(BIT(14) | BIT(13) | BIT(12) | BIT(11) | BIT(10) | BIT(9) | BIT(8)))
-#define HFA384x_CMD_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_CMD_AINFO ((u16)(BIT(14) | BIT(13) \
+ | BIT(12) | BIT(11) \
+ | BIT(10) | BIT(9) \
+ | BIT(8)))
+#define HFA384x_CMD_MACPORT ((u16)(BIT(10) | BIT(9) | \
+ BIT(8)))
#define HFA384x_CMD_PROGMODE ((u16)(BIT(9) | BIT(8)))
-#define HFA384x_CMD_CMDCODE ((u16)(BIT(5) | BIT(4) | BIT(3) | BIT(2) | BIT(1) | BIT(0)))
+#define HFA384x_CMD_CMDCODE ((u16)(BIT(5) | BIT(4) | \
+ BIT(3) | BIT(2) | \
+ BIT(1) | BIT(0)))
-#define HFA384x_STATUS_RESULT ((u16)(BIT(14) | BIT(13) | BIT(12) | BIT(11) | BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_STATUS_RESULT ((u16)(BIT(14) | BIT(13) \
+ | BIT(12) | BIT(11) \
+ | BIT(10) | BIT(9) \
+ | BIT(8)))
/*--- Command Code Constants --------------------------*/
/*--- Controller Commands --------------------------*/
@@ -244,8 +253,10 @@ Information RID Lengths: MAC Information
This is the length of JUST the DATA part of the RID (does not
include the len or code fields)
--------------------------------------------------------------------*/
-#define HFA384x_RID_DBMCOMMSQUALITY_LEN ((u16)sizeof(hfa384x_dbmcommsquality_t))
-#define HFA384x_RID_JOINREQUEST_LEN ((u16)sizeof(hfa384x_JoinRequest_data_t))
+#define HFA384x_RID_DBMCOMMSQUALITY_LEN \
+ ((u16) sizeof(hfa384x_dbmcommsquality_t))
+#define HFA384x_RID_JOINREQUEST_LEN \
+ ((u16)sizeof(hfa384x_JoinRequest_data_t))
/*--------------------------------------------------------------------
Information RIDs: Modem Information
@@ -322,9 +333,11 @@ PD Record codes
/*--- Register Test/Get/Set Field macros ------------------------*/
-#define HFA384x_CMD_AINFO_SET(value) ((u16)((u16)(value) << 8))
-#define HFA384x_CMD_MACPORT_SET(value) ((u16)HFA384x_CMD_AINFO_SET(value))
-#define HFA384x_CMD_PROGMODE_SET(value) ((u16)HFA384x_CMD_AINFO_SET((u16)value))
+#define HFA384x_CMD_AINFO_SET(value) ((u16)((u16)(value) << 8))
+#define HFA384x_CMD_MACPORT_SET(value) \
+ ((u16)HFA384x_CMD_AINFO_SET(value))
+#define HFA384x_CMD_PROGMODE_SET(value) \
+ ((u16)HFA384x_CMD_AINFO_SET((u16)value))
#define HFA384x_CMD_CMDCODE_SET(value) ((u16)(value))
#define HFA384x_STATUS_RESULT_SET(value) (((u16)(value)) << 8)
@@ -402,7 +415,7 @@ typedef struct hfa384x_authenticateStation_data {
/*-- Configuration Record: WPAData (data portion only) --*/
typedef struct hfa384x_WPAData {
u16 datalen;
- u8 data[0]; // max 80
+ u8 data[0]; /* max 80 */
} __attribute__ ((packed)) hfa384x_WPAData_t;
/*--------------------------------------------------------------------
@@ -479,7 +492,8 @@ Communication Frames: Field Masks for Transmit Frames
#define HFA384x_TXSTATUS_AGEDERR ((u16)BIT(1))
#define HFA384x_TXSTATUS_RETRYERR ((u16)BIT(0))
/*-- Transmit Control Field --*/
-#define HFA384x_TX_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_TX_MACPORT ((u16)(BIT(10) | \
+ BIT(9) | BIT(8)))
#define HFA384x_TX_STRUCTYPE ((u16)(BIT(4) | BIT(3)))
#define HFA384x_TX_TXEX ((u16)BIT(2))
#define HFA384x_TX_TXOK ((u16)BIT(1))
@@ -496,7 +510,8 @@ Communication Frames: Test/Get/Set Field Values for Transmit Frames
#define HFA384x_TX_SET(v, m, s) ((((u16)(v))<<((u16)(s)))&((u16)(m)))
#define HFA384x_TX_MACPORT_SET(v) HFA384x_TX_SET(v, HFA384x_TX_MACPORT, 8)
-#define HFA384x_TX_STRUCTYPE_SET(v) HFA384x_TX_SET(v, HFA384x_TX_STRUCTYPE, 3)
+#define HFA384x_TX_STRUCTYPE_SET(v) HFA384x_TX_SET(v, \
+ HFA384x_TX_STRUCTYPE, 3)
#define HFA384x_TX_TXEX_SET(v) HFA384x_TX_SET(v, HFA384x_TX_TXEX, 2)
#define HFA384x_TX_TXOK_SET(v) HFA384x_TX_SET(v, HFA384x_TX_TXOK, 1)
/*--------------------------------------------------------------------
@@ -534,13 +549,17 @@ Communication Frames: Field Masks for Receive Frames
--------------------------------------------------------------------*/
/*-- Status Fields --*/
-#define HFA384x_RXSTATUS_MACPORT ((u16)(BIT(10) | BIT(9) | BIT(8)))
+#define HFA384x_RXSTATUS_MACPORT ((u16)(BIT(10) | \
+ BIT(9) | \
+ BIT(8)))
#define HFA384x_RXSTATUS_FCSERR ((u16)BIT(0))
/*--------------------------------------------------------------------
Communication Frames: Test/Get/Set Field Values for Receive Frames
--------------------------------------------------------------------*/
-#define HFA384x_RXSTATUS_MACPORT_GET(value) ((u16)((((u16)(value)) & HFA384x_RXSTATUS_MACPORT) >> 8))
-#define HFA384x_RXSTATUS_ISFCSERR(value) ((u16)(((u16)(value)) & HFA384x_RXSTATUS_FCSERR))
+#define HFA384x_RXSTATUS_MACPORT_GET(value) ((u16)((((u16)(value)) \
+ & HFA384x_RXSTATUS_MACPORT) >> 8))
+#define HFA384x_RXSTATUS_ISFCSERR(value) ((u16)(((u16)(value)) \
+ & HFA384x_RXSTATUS_FCSERR))
/*--------------------------------------------------------------------
FRAME STRUCTURES: Information Types and Information Frame Structures
----------------------------------------------------------------------
@@ -1133,7 +1152,7 @@ struct hfa384x;
typedef void (*ctlx_cmdcb_t) (struct hfa384x *, const struct hfa384x_usbctlx *);
-typedef void (*ctlx_usercb_t) (struct hfa384x * hw,
+typedef void (*ctlx_usercb_t) (struct hfa384x *hw,
void *ctlxresult, void *usercb_data);
typedef struct hfa384x_usbctlx {
@@ -1174,14 +1193,14 @@ typedef struct hfa484x_metacmd {
} hfa384x_metacmd_t;
#define MAX_GRP_ADDR 32
-#define WLAN_COMMENT_MAX 80 /* Max. length of user comment string. */
+#define WLAN_COMMENT_MAX 80 /* Max. length of user comment string. */
-#define WLAN_AUTH_MAX 60 /* Max. # of authenticated stations. */
-#define WLAN_ACCESS_MAX 60 /* Max. # of stations in an access list. */
-#define WLAN_ACCESS_NONE 0 /* No stations may be authenticated. */
-#define WLAN_ACCESS_ALL 1 /* All stations may be authenticated. */
-#define WLAN_ACCESS_ALLOW 2 /* Authenticate only "allowed" stations. */
-#define WLAN_ACCESS_DENY 3 /* Do not authenticate "denied" stations. */
+#define WLAN_AUTH_MAX 60 /* Max. # of authenticated stations. */
+#define WLAN_ACCESS_MAX 60 /* Max. # of stations in an access list. */
+#define WLAN_ACCESS_NONE 0 /* No stations may be authenticated. */
+#define WLAN_ACCESS_ALL 1 /* All stations may be authenticated. */
+#define WLAN_ACCESS_ALLOW 2 /* Authenticate only "allowed" stations. */
+#define WLAN_ACCESS_DENY 3 /* Do not authenticate "denied" stations. */
/* XXX These are going away ASAP */
typedef struct prism2sta_authlist {
@@ -1294,10 +1313,23 @@ typedef struct hfa384x {
hfa384x_caplevel_t cap_sup_ap;
/* Actor compatibility ranges */
- hfa384x_caplevel_t cap_act_pri_cfi; /* pri f/w to controller interface */
- hfa384x_caplevel_t cap_act_sta_cfi; /* sta f/w to controller interface */
+ hfa384x_caplevel_t cap_act_pri_cfi; /*
+ * pri f/w to controller
+ * interface
+ */
+
+ hfa384x_caplevel_t cap_act_sta_cfi; /*
+ * sta f/w to controller
+ * interface
+ */
+
hfa384x_caplevel_t cap_act_sta_mfi; /* sta f/w to modem interface */
- hfa384x_caplevel_t cap_act_ap_cfi; /* ap f/w to controller interface */
+
+ hfa384x_caplevel_t cap_act_ap_cfi; /*
+ * ap f/w to controller
+ * interface
+ */
+
hfa384x_caplevel_t cap_act_ap_mfi; /* ap f/w to modem interface */
u32 psusercount; /* Power save user count. */
@@ -1320,25 +1352,25 @@ typedef struct hfa384x {
} hfa384x_t;
-void hfa384x_create(hfa384x_t * hw, struct usb_device *usb);
-void hfa384x_destroy(hfa384x_t * hw);
+void hfa384x_create(hfa384x_t *hw, struct usb_device *usb);
+void hfa384x_destroy(hfa384x_t *hw);
int
-hfa384x_corereset(hfa384x_t * hw, int holdtime, int settletime, int genesis);
-int hfa384x_drvr_commtallies(hfa384x_t * hw);
-int hfa384x_drvr_disable(hfa384x_t * hw, u16 macport);
-int hfa384x_drvr_enable(hfa384x_t * hw, u16 macport);
-int hfa384x_drvr_flashdl_enable(hfa384x_t * hw);
-int hfa384x_drvr_flashdl_disable(hfa384x_t * hw);
-int hfa384x_drvr_flashdl_write(hfa384x_t * hw, u32 daddr, void *buf, u32 len);
-int hfa384x_drvr_getconfig(hfa384x_t * hw, u16 rid, void *buf, u16 len);
-int hfa384x_drvr_ramdl_enable(hfa384x_t * hw, u32 exeaddr);
-int hfa384x_drvr_ramdl_disable(hfa384x_t * hw);
-int hfa384x_drvr_ramdl_write(hfa384x_t * hw, u32 daddr, void *buf, u32 len);
-int hfa384x_drvr_readpda(hfa384x_t * hw, void *buf, unsigned int len);
-int hfa384x_drvr_setconfig(hfa384x_t * hw, u16 rid, void *buf, u16 len);
-
-static inline int hfa384x_drvr_getconfig16(hfa384x_t * hw, u16 rid, void *val)
+hfa384x_corereset(hfa384x_t *hw, int holdtime, int settletime, int genesis);
+int hfa384x_drvr_commtallies(hfa384x_t *hw);
+int hfa384x_drvr_disable(hfa384x_t *hw, u16 macport);
+int hfa384x_drvr_enable(hfa384x_t *hw, u16 macport);
+int hfa384x_drvr_flashdl_enable(hfa384x_t *hw);
+int hfa384x_drvr_flashdl_disable(hfa384x_t *hw);
+int hfa384x_drvr_flashdl_write(hfa384x_t *hw, u32 daddr, void *buf, u32 len);
+int hfa384x_drvr_getconfig(hfa384x_t *hw, u16 rid, void *buf, u16 len);
+int hfa384x_drvr_ramdl_enable(hfa384x_t *hw, u32 exeaddr);
+int hfa384x_drvr_ramdl_disable(hfa384x_t *hw);
+int hfa384x_drvr_ramdl_write(hfa384x_t *hw, u32 daddr, void *buf, u32 len);
+int hfa384x_drvr_readpda(hfa384x_t *hw, void *buf, unsigned int len);
+int hfa384x_drvr_setconfig(hfa384x_t *hw, u16 rid, void *buf, u16 len);
+
+static inline int hfa384x_drvr_getconfig16(hfa384x_t *hw, u16 rid, void *val)
{
int result = 0;
result = hfa384x_drvr_getconfig(hw, rid, val, sizeof(u16));
@@ -1347,46 +1379,46 @@ static inline int hfa384x_drvr_getconfig16(hfa384x_t * hw, u16 rid, void *val)
return result;
}
-static inline int hfa384x_drvr_setconfig16(hfa384x_t * hw, u16 rid, u16 val)
+static inline int hfa384x_drvr_setconfig16(hfa384x_t *hw, u16 rid, u16 val)
{
u16 value = cpu_to_le16(val);
return hfa384x_drvr_setconfig(hw, rid, &value, sizeof(value));
}
int
-hfa384x_drvr_getconfig_async(hfa384x_t * hw,
+hfa384x_drvr_getconfig_async(hfa384x_t *hw,
u16 rid, ctlx_usercb_t usercb, void *usercb_data);
int
-hfa384x_drvr_setconfig_async(hfa384x_t * hw,
+hfa384x_drvr_setconfig_async(hfa384x_t *hw,
u16 rid,
void *buf,
u16 len, ctlx_usercb_t usercb, void *usercb_data);
static inline int
-hfa384x_drvr_setconfig16_async(hfa384x_t * hw, u16 rid, u16 val)
+hfa384x_drvr_setconfig16_async(hfa384x_t *hw, u16 rid, u16 val)
{
u16 value = cpu_to_le16(val);
return hfa384x_drvr_setconfig_async(hw, rid, &value, sizeof(value),
NULL, NULL);
}
-int hfa384x_drvr_start(hfa384x_t * hw);
-int hfa384x_drvr_stop(hfa384x_t * hw);
+int hfa384x_drvr_start(hfa384x_t *hw);
+int hfa384x_drvr_stop(hfa384x_t *hw);
int
-hfa384x_drvr_txframe(hfa384x_t * hw, struct sk_buff *skb,
- p80211_hdr_t * p80211_hdr, p80211_metawep_t * p80211_wep);
-void hfa384x_tx_timeout(wlandevice_t * wlandev);
-
-int hfa384x_cmd_initialize(hfa384x_t * hw);
-int hfa384x_cmd_enable(hfa384x_t * hw, u16 macport);
-int hfa384x_cmd_disable(hfa384x_t * hw, u16 macport);
-int hfa384x_cmd_allocate(hfa384x_t * hw, u16 len);
-int hfa384x_cmd_monitor(hfa384x_t * hw, u16 enable);
+hfa384x_drvr_txframe(hfa384x_t *hw, struct sk_buff *skb,
+ p80211_hdr_t *p80211_hdr, p80211_metawep_t *p80211_wep);
+void hfa384x_tx_timeout(wlandevice_t *wlandev);
+
+int hfa384x_cmd_initialize(hfa384x_t *hw);
+int hfa384x_cmd_enable(hfa384x_t *hw, u16 macport);
+int hfa384x_cmd_disable(hfa384x_t *hw, u16 macport);
+int hfa384x_cmd_allocate(hfa384x_t *hw, u16 len);
+int hfa384x_cmd_monitor(hfa384x_t *hw, u16 enable);
int
-hfa384x_cmd_download(hfa384x_t * hw,
+hfa384x_cmd_download(hfa384x_t *hw,
u16 mode, u16 lowaddr, u16 highaddr, u16 codelen);
-#endif /* __KERNEL__ */
+#endif /*__KERNEL__ */
-#endif /* _HFA384x_H */
+#endif /*_HFA384x_H */
--
1.6.6.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-02-15 11:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-02-14 19:38 [PATCH] Staging: wlan-ng: fix most of the style issues in hfa384x.h Alessandro Ghedini
2010-02-14 19:55 ` Gábor Stefanik
-- strict thread matches above, loose matches on Subject: below --
2010-02-15 11:25 Alessandro Ghedini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.