Re: [PATCH 3/3] nozomi: Tidy up the PCI table

[Greg KH]

On Mon, Feb 08, 2010 at 11:49:49AM +0100, Marcel Holtmann wrote:
> Hi Alan,
> 
> > Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
> > ---
> > 
> >  drivers/char/nozomi.c |    6 +-----
> >  1 files changed, 1 insertions(+), 5 deletions(-)
> > 
> > 
> > diff --git a/drivers/char/nozomi.c b/drivers/char/nozomi.c
> > index fac9157..86c9e64 100644
> > --- a/drivers/char/nozomi.c
> > +++ b/drivers/char/nozomi.c
> > @@ -136,10 +136,6 @@ static int debug;
> >  #define RECEIVE_BUF_MAX		4
> >  
> > 
> > -/* Define all types of vendors and devices to support */
> > -#define VENDOR1		0x1931	/* Vendor Option */
> > -#define DEVICE1		0x000c	/* HSDPA card */
> > -
> >  #define R_IIR		0x0000	/* Interrupt Identity Register */
> >  #define R_FCR		0x0000	/* Flow Control Register */
> >  #define R_IER		0x0004	/* Interrupt Enable Register */
> > @@ -407,7 +403,7 @@ struct buffer {
> >  
> >  /*    Global variables */
> >  static const struct pci_device_id nozomi_pci_tbl[] __devinitconst = {
> > -	{PCI_DEVICE(VENDOR1, DEVICE1)},
> > +	{PCI_DEVICE(0x1931, 0x000c)},	/* Nozomi HSPDA */
> >  	{},
> 
> that would be HSDPA actually.

I edited the patch, thanks.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/