Commit 6a878184 authored by Jan Blunck's avatar Jan Blunck Committed by Linus Torvalds
Browse files

[PATCH] Eliminate __attribute__ ((packed)) warnings for gcc-4.1 



Since version 4.1 the gcc is warning about ignored attributes. This patch is
using the equivalent attribute on the struct instead of on each of the
structure or union members.

GCC Manual:
  "Specifying Attributes of Types

   packed
    This attribute, attached to struct or union type definition, specifies
    that
    each member of the structure or union is placed to minimize the memory
    required. When attached to an enum definition, it indicates that the
    smallest integral type should be used.

    Specifying this attribute for struct and union types is equivalent to
    specifying the packed attribute on each of the structure or union
    members."

Signed-off-by: default avatarJan Blunck <jblunck@suse.de>
Cc: Dave Jones <davej@codemonkey.org.uk>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent d8a33496

drivers/isdn/hisax/hisax.h

+9 −9
Original line number Diff line number Diff line
@@ -396,17 +396,17 @@ struct isar_hw { 


struct hdlc_stat_reg {

#ifdef __BIG_ENDIAN

	u_char fill __attribute__((packed));

	u_char mode __attribute__((packed));

	u_char xml  __attribute__((packed));

	u_char cmd  __attribute__((packed));

	u_char fill;

	u_char mode;

	u_char xml;

	u_char cmd;

#else

	u_char cmd  __attribute__((packed));

	u_char xml  __attribute__((packed));

	u_char mode __attribute__((packed));

	u_char fill __attribute__((packed));

	u_char cmd;

	u_char xml;

	u_char mode;

	u_char fill;

#endif

};

} __attribute__((packed));



struct hdlc_hw {

	union {

drivers/isdn/hisax/hisax_fcpcipnp.h

+9 −9
Original line number Diff line number Diff line
@@ -12,17 +12,17 @@ enum { 


struct hdlc_stat_reg {

#ifdef __BIG_ENDIAN

	u_char fill __attribute__((packed));

	u_char mode __attribute__((packed));

	u_char xml  __attribute__((packed));

	u_char cmd  __attribute__((packed));

	u_char fill;

	u_char mode;

	u_char xml;

	u_char cmd;

#else

	u_char cmd  __attribute__((packed));

	u_char xml  __attribute__((packed));

	u_char mode __attribute__((packed));

	u_char fill __attribute__((packed));

	u_char cmd;

	u_char xml;

	u_char mode;

	u_char fill;

#endif

};

} __attribute__((packed));



struct fritz_bcs {

	struct hisax_b_if b_if;

drivers/net/3c527.h

+25 −25
Original line number Diff line number Diff line
@@ -32,43 +32,43 @@ 


struct mc32_mailbox

{

	u16	mbox __attribute((packed));

	u16	data[1] __attribute((packed));

};

 	u16 mbox;

 	u16 data[1];

} __attribute((packed));



struct skb_header

{

	u8	status __attribute((packed));

	u8	control __attribute((packed));

	u16	next __attribute((packed));	/* Do not change! */

	u16	length __attribute((packed));

	u32	data __attribute((packed));

};

	u8 status;

	u8 control;

	u16 next;	/* Do not change! */

	u16 length;

	u32 data;

} __attribute((packed));



struct mc32_stats

{

	/* RX Errors */

	u32     rx_crc_errors       __attribute((packed)); 	

	u32     rx_alignment_errors  __attribute((packed)); 	

	u32     rx_overrun_errors    __attribute((packed));

	u32     rx_tooshort_errors  __attribute((packed));

	u32     rx_toolong_errors   __attribute((packed));

	u32     rx_outofresource_errors  __attribute((packed)); 

	u32 rx_crc_errors;

	u32 rx_alignment_errors;

	u32 rx_overrun_errors;

	u32 rx_tooshort_errors;

	u32 rx_toolong_errors;

	u32 rx_outofresource_errors;



	u32     rx_discarded   __attribute((packed));  /* via card pattern match filter */ 

	u32 rx_discarded;  /* via card pattern match filter */



	/* TX Errors */

	u32     tx_max_collisions __attribute((packed)); 

	u32     tx_carrier_errors __attribute((packed)); 

	u32     tx_underrun_errors __attribute((packed)); 

	u32     tx_cts_errors     __attribute((packed)); 

	u32     tx_timeout_errors __attribute((packed)) ;

	u32 tx_max_collisions;

	u32 tx_carrier_errors;

	u32 tx_underrun_errors;

	u32 tx_cts_errors;

	u32 tx_timeout_errors;

	

	/* various cruft */

	u32     dataA[6] __attribute((packed));   

        u16	dataB[5] __attribute((packed));   

  	u32     dataC[14] __attribute((packed)); 	

};

	u32 dataA[6];

	u16 dataB[5];

	u32 dataC[14];

} __attribute((packed));



#define STATUS_MASK	0x0F

#define COMPLETED	(1<<7)

drivers/net/irda/vlsi_ir.h

+2 −2
Original line number Diff line number Diff line
@@ -577,8 +577,8 @@ struct ring_descr_hw { 
		struct {

			u8		addr_res[3];

			volatile u8	status;		/* descriptor status */

		} rd_s __attribute__((packed));

	} rd_u __attribute((packed));

		} __attribute__((packed)) rd_s;

	} __attribute((packed)) rd_u;

} __attribute__ ((packed));



#define rd_addr		rd_u.addr

drivers/net/wan/sdla.c

+3 −3
Original line number Diff line number Diff line
@@ -329,9 +329,9 @@ static int sdla_cpuspeed(struct net_device *dev, struct ifreq *ifr) 


struct _dlci_stat 

{

	short dlci		__attribute__((packed));

	char  flags		__attribute__((packed));

};

	short dlci;

	char  flags;

} __attribute__((packed));



struct _frad_stat 

{

CRA Git | Maintained and supported by SUSTech CRA and CCSE