On Tue, 22 Apr 2003, Ralf Baechle wrote: > On Tue, Apr 22, 2003 at 01:17:21PM +0200, Juan Quintela wrote: > > yoichi> +static struct resource vr41xx_pci_mem_resource = { > > yoichi> + "PCI memory space", > > yoichi> + VR41XX_PCI_MEM_START, > > yoichi> + VR41XX_PCI_MEM_END, > > yoichi> + IORESOURCE_MEM > > yoichi> +}; > > > > Please, use C99 named initializers in the whole file. > > I don't think there's much point in using ISO style initializers everywhere. > So far the convention is only to replace the GNU-style inializer. > We unfortunately have a few places where the code got inflated by at least > the factor of 3 because now some code uses the ISO initializers for > everything - for no good reason. What if someone will change struct resource in the future? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds