Skip to content

Commit

Permalink
Staging: bcm: Fix warning: "__packed is preferred over __attribute__(…
Browse files Browse the repository at this point in the history
…(packed))" in Protocol.h

This patch fixes the following warning: "__packed
is preferred over __attribute__((packed))" as
reported by checkpatch.pl.

Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Kevin McKinney authored and Greg Kroah-Hartman committed Jan 7, 2013
1 parent 3df56e1 commit 5f8797b
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions drivers/staging/bcm/Protocol.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ struct TransportHeaderT {
struct udphdr uhdr;
struct tcphdr thdr;
};
} __attribute__((packed));
} __packed;
typedef struct TransportHeaderT xporthdr;

typedef enum _E_NWPKT_IPFRAME_TYPE {
Expand Down Expand Up @@ -50,14 +50,14 @@ typedef struct _ETH_CS_802_Q_FRAME {
USHORT CFI:1;
USHORT VLANID:12;
USHORT EthType;
} __attribute__((packed)) ETH_CS_802_Q_FRAME;
} __packed ETH_CS_802_Q_FRAME;

typedef struct _ETH_CS_802_LLC_FRAME {
struct bcm_eth_header EThHdr;
unsigned char DSAP;
unsigned char SSAP;
unsigned char Control;
} __attribute__((packed)) ETH_CS_802_LLC_FRAME;
} __packed ETH_CS_802_LLC_FRAME;

typedef struct _ETH_CS_802_LLC_SNAP_FRAME {
struct bcm_eth_header EThHdr;
Expand All @@ -66,11 +66,11 @@ typedef struct _ETH_CS_802_LLC_SNAP_FRAME {
unsigned char Control;
unsigned char OUI[3];
unsigned short usEtherType;
} __attribute__((packed)) ETH_CS_802_LLC_SNAP_FRAME;
} __packed ETH_CS_802_LLC_SNAP_FRAME;

typedef struct _ETH_CS_ETH2_FRAME {
struct bcm_eth_header EThHdr;
} __attribute__((packed)) ETH_CS_ETH2_FRAME;
} __packed ETH_CS_ETH2_FRAME;

#define ETHERNET_FRAMETYPE_IPV4 ntohs(0x0800)
#define ETHERNET_FRAMETYPE_IPV6 ntohs(0x86dd)
Expand Down

0 comments on commit 5f8797b

Please sign in to comment.