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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <484D8CCC.1070701@embedromix.ro>
Date:	Mon, 09 Jun 2008 23:04:28 +0300
From:	"Catalin(ux) M BOIE" <catab@...edromix.ro>
To:	Russell King <rmk+lkml@....linux.org.uk>
CC:	Alan Cox <alan@...rguk.ukuu.org.uk>, linux-kernel@...r.kernel.org,
	linux-serial@...r.kernel.org, akpm@...ux-foundation.org,
	pazke@...ke.donpac.ru
Subject: Re: [PATCH] Add support for a no-name 4 ports multiserial card

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@...edromix.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.

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/

View attachment "0001-Add-support-for-a-no-name-4-ports-multiserial-card.patch" of type "text/plain" (3115 bytes)