From mboxrd@z Thu Jan 1 00:00:00 1970 From: Charles =?iso-8859-1?Q?Cl=E9ment?= Date: Tue, 20 Mar 2007 14:14:58 +0000 Subject: [KJ] [PATCH] wireless: gcc attribute short form for hermes Message-Id: <20070320141458.GA9391@tux> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: kernel-janitors@vger.kernel.org Hello kernel-janitors, Changed calls in hermes driver to use the short defined gcc attribute macro. Signed-off-by: Charles Cl=E9ment --- I would be glad to be assigned that task on the TODO list, if someone is not currently working on it. Thanks. Index: linux-2.6.21-rc4/drivers/net/wireless/hermes.h =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D--- linux-2.6.21-rc4.orig/drivers/net/wireless/herm= es.h +++ linux-2.6.21-rc4/drivers/net/wireless/hermes.h @@ -198,7 +198,7 @@ struct hermes_tx_descriptor { u8 retry_count; u8 tx_rate; __le16 tx_control;=09 -} __attribute__ ((packed)); +} __packed; =20 #define HERMES_TXSTAT_RETRYERR (0x0001) #define HERMES_TXSTAT_AGEDERR (0x0002) @@ -245,7 +245,7 @@ struct hermes_tallies_frame { /* Those last are probably not available in very old firmwares */ __le16 RxDiscards_WEPICVError; __le16 RxDiscards_WEPExcluded; -} __attribute__ ((packed)); +} __packed; =20 /* Grabbed from wlan-ng - Thanks Mark... - Jean II * This is the result of a scan inquiry command */ @@ -262,7 +262,7 @@ struct prism2_scan_apinfo { u8 rates[10]; /* Bit rate supported */ __le16 proberesp_rate; /* Data rate of the response frame */ __le16 atim; /* ATIM window time, Kus (hostscan only) */ -} __attribute__ ((packed)); +} __packed; =20 /* Same stuff for the Lucent/Agere card. * Thanks to h1kari - Jean II */ @@ -276,7 +276,7 @@ struct agere_scan_apinfo { /* bits: 0-ess, 1-ibss, 4-privacy [wep] */ __le16 essid_len; /* ESSID length */ u8 essid[32]; /* ESSID of the network */ -} __attribute__ ((packed)); +} __packed; =20 /* Moustafa: Scan structure for Symbol cards */ struct symbol_scan_apinfo { @@ -294,7 +294,7 @@ struct symbol_scan_apinfo { __le16 basic_rates; /* Basic rates bitmask */ u8 unknown2[6]; /* Always FF:FF:FF:FF:00:00 */ u8 unknown3[8]; /* Always 0, appeared in f/w 3.91-68 */ -} __attribute__ ((packed)); +} __packed; =20 union hermes_scan_info { struct agere_scan_apinfo a; @@ -312,7 +312,7 @@ union hermes_scan_info { =20 struct hermes_linkstatus { __le16 linkstatus; /* Link status */ -} __attribute__ ((packed)); +} __packed; =20 struct hermes_response { u16 status, resp0, resp1, resp2; @@ -322,11 +322,11 @@ struct hermes_response { struct hermes_idstring { __le16 len; __le16 val[16]; -} __attribute__ ((packed)); +} __packed; =20 struct hermes_multicast { u8 addr[HERMES_MAX_MULTICAST][ETH_ALEN]; -} __attribute__ ((packed)); +} __packed; =20 /* Timeouts */ #define HERMES_BAP_BUSY_TIMEOUT (10000) /* In iterations of ~1us */ _______________________________________________ Kernel-janitors mailing list Kernel-janitors@lists.linux-foundation.org https://lists.linux-foundation.org/mailman/listinfo/kernel-janitors