lkml.org 
[lkml]   [2008]   [Jun]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Add support for a no-name 4 ports multiserial card
On Fri, Jun 06, 2008 at 11:14:00AM +0100, Alan Cox wrote:
> On Fri, 6 Jun 2008 10:13:13 +0300
> "Catalin(ux) M BOIE" <catab@embedromix.ro> wrote:
>
> > It is a no-name PCI card. I found no reference to a producer so I
> > used the fake name "PDR".
>
> We don't really want to get fake names into pci_ids. I'd rather
>
>
> > .vendor = PCI_VENDOR_ID_PLX,
> > + .device = PCI_DEVICE_ID_PLX_9050,
> > + .subvendor = PCI_VENDOR_ID_PLX,
> > + .subdevice = PCI_SUBDEVICE_ID_PDR,
>
> Either the hex or a local and obviously unknown name like ID_UNKNOWN_1
>
>
> > + pbn_plx_pdr,
>
> _unknown1
>
> > pbn_oxsemi,
> > pbn_intel_i960,
> > pbn_sgi_ioc3,
> > @@ -1186,6 +1196,13 @@ static struct pciserial_board pci_boards[] __devinitdata = {
> > .base_baud = 115200,
> > .uart_offset = 8,
> > },
> > + /* PDR */
> > + [pbn_plx_pdr] = {
>
> Ditto
>
> > + /* PDR */
>
> Ditto
>
> > + { PCI_VENDOR_ID_PLX, PCI_DEVICE_ID_PLX_9050,
> > + PCI_VENDOR_ID_PLX,
> > + PCI_SUBDEVICE_ID_PDR, 0, 0,
>
> Ditto
>
>
> > +#define PCI_SUBDEVICE_ID_PDR 0x1584
> > +
>
> And not in here at all.

And there's no need for half the patch either - which I tried to explain
in a previous reply to Catalin but I just got repeated "My email address
has changed to ..." responses... so I gave up.

--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of:


\
 
 \ /
  Last update: 2008-06-06 21:43    [W:0.099 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site