On Friday 02 March 2007 13:45, Helge Deller wrote: > -static unsigned int tx_ring_size = 16; > -static unsigned int rx_ring_size = 16; > -static int debug = 1; > +static unsigned int tx_ring_size __read_mostly = 16; > +static unsigned int rx_ring_size __read_mostly = 16; > +static int debug __read_mostly = 1; > Hm, sure why not.. > -const u8 rtl8225_agc[] = { > +const const u8 rtl8225_agc[] = { What does const const do? > -static const char *rfs[] = { > +static const char * const rfs[] = { ? BTW, you should have an updated summary for this patch. Rest of the patch looks okay to me. Thanks, -Michael Wu
Attachment:
pgpGelmkxNJn5.pgp
Description: PGP signature