Subject: Re: [PATCH 2.6.14-rc3 1/1] chrp_pegasos_eth: Added Marvell Discovery II SRAM support

From: Sven Luther <[email-masked]>
Date: Fri, 14 Oct 2005 17:18:30 +0200

On Fri, Oct 14, 2005 at 01:56:41PM -0000, Dale Farnsworth wrote:
> > -	static struct pci_device_id pci_marvell_mv64360[] = {
> > +	static struct pci_device_id pci_marvell_mv64360[] =
> > +	{
> 
> > +	if (pci_dev_present(pci_marvell_mv64360))
> > +	{
> 
> Open brace should not be on a line by itself except in function definitions.

Oh, neat, i knew i should not use same-line-braces for function but didn't
know i should use them for non-functions :)

Friendly,

Sven Luther