| To: | Juan Quintela <quintela@mandrakesoft.com> |
|---|---|
| Subject: | Re: [patch] TANBAC TB0226(NEC VR4131) for v2.5 |
| From: | Ralf Baechle <ralf@linux-mips.org> |
| Date: | Tue, 22 Apr 2003 13:36:42 +0200 |
| Cc: | Yoichi Yuasa <yuasa@hh.iij4u.or.jp>, linux-mips@linux-mips.org |
| In-reply-to: | <86lly2ixxa.fsf@trasno.mitica>; from quintela@mandrakesoft.com on Tue, Apr 22, 2003 at 01:17:21PM +0200 |
| Original-recipient: | rfc822;linux-mips@linux-mips.org |
| References: | <20030422191137.4ac88897.yuasa@hh.iij4u.or.jp> <86lly2ixxa.fsf@trasno.mitica> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Mutt/1.2.5.1i |
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.
Ralf
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [patch] TANBAC TB0226(NEC VR4131) for v2.5, Juan Quintela |
|---|---|
| Next by Date: | Re: [patch] TANBAC TB0226(NEC VR4131) for v2.5, Geert Uytterhoeven |
| Previous by Thread: | Re: [patch] TANBAC TB0226(NEC VR4131) for v2.5, Juan Quintela |
| Next by Thread: | Re: [patch] TANBAC TB0226(NEC VR4131) for v2.5, Geert Uytterhoeven |
| Indexes: | [Date] [Thread] [Top] [All Lists] |