Re: [PATCH] Add support for a no-name 4 ports multiserial card

[Catalin(ux) M BOIE]

This is a multi-part message in MIME format.Russell King wrote:
> 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@xxxxxxxxxxxxx> 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.

Rusell, I am very sorry about: sending e-mail without patch,
sending from an old mail address and calling you "Rusty".

The last version of the patch is attached. Please Ack it if it is OK
for you. If not, please let me know.

Andrew, please hold on a little bit to get the Ack from Russel.

Thank you all for your support and patience!

--
Catalin(ux) M BOIE
http://kernel.embedromix.ro/