lkml.org 
[lkml]   [2014]   [Aug]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] jsm: add support for additional Neo cards
On 08/30/2014 02:59 PM, Greg KH wrote:
> On Sat, Aug 30, 2014 at 02:54:36PM -0400, wfp5p@worldbroken.com wrote:
>> From: Bill Pemberton <wfp5p@worldbroken.com>
>>
>> Add device ids for additional Neo cards. The ids come from the dgnc
>> driver.
>>
>> Signed-off-by: Bill Pemberton <wfp5p@worldbroken.com>
>> ---
>> drivers/tty/serial/jsm/jsm_driver.c | 38 +++++++++++++++++++++++++++++++++----
>> include/linux/pci_ids.h | 8 ++++++++
>> 2 files changed, 42 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/tty/serial/jsm/jsm_driver.c b/drivers/tty/serial/jsm/jsm_driver.c
>> index a47d882d6743..d2885a7bb090 100644
>> --- a/drivers/tty/serial/jsm/jsm_driver.c
>> +++ b/drivers/tty/serial/jsm/jsm_driver.c
>> @@ -93,12 +93,34 @@ static int jsm_probe_one(struct pci_dev *pdev, const struct pci_device_id *ent)
>> /* store the info for the board we've found */
>> brd->boardnum = adapter_count++;
>> brd->pci_dev = pdev;
>> - if (pdev->device == PCIE_DEVICE_ID_NEO_4_IBM)
>> +
>> + switch (pdev->device) {
>> +
>> + case PCI_DEVICE_ID_NEO_2DB9:
>> + case PCI_DEVICE_ID_NEO_2DB9PRI:
>> + case PCI_DEVICE_ID_NEO_2RJ45:
>> + case PCI_DEVICE_ID_NEO_2RJ45PRI:
>> + case PCI_DEVICE_ID_NEO_2_422_485:
>> + brd->maxports = 2;
>> + break;
>> +
>> + case PCI_DEVICE_ID_NEO_4:
>> + case PCIE_DEVICE_ID_NEO_4:
>> + case PCIE_DEVICE_ID_NEO_4RJ45:
>> + case PCIE_DEVICE_ID_NEO_4_IBM:
>> brd->maxports = 4;
>> - else if (pdev->device == PCI_DEVICE_ID_DIGI_NEO_8)
>> + break;
>> +
>> + case PCI_DEVICE_ID_DIGI_NEO_8:
>> + case PCIE_DEVICE_ID_NEO_8:
>> + case PCIE_DEVICE_ID_NEO_8RJ45:
>> brd->maxports = 8;
>> - else
>> - brd->maxports = 2;
>> + break;
>> +
>> + default:
>> + brd->maxports = 1;
>> + break;
>> + }
>>
>> spin_lock_init(&brd->bd_intr_lock);
>>
>> @@ -209,6 +231,14 @@ static struct pci_device_id jsm_pci_tbl[] = {
>> { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCI_DEVICE_ID_NEO_2RJ45PRI), 0, 0, 3 },
>> { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCIE_DEVICE_ID_NEO_4_IBM), 0, 0, 4 },
>> { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCI_DEVICE_ID_DIGI_NEO_8), 0, 0, 5 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCI_DEVICE_ID_NEO_4), 0, 0, 6 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCI_DEVICE_ID_NEO_1_422), 0, 0, 7 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCI_DEVICE_ID_NEO_1_422_485), 0, 0, 8 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCI_DEVICE_ID_NEO_2_422_485), 0, 0, 9 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCIE_DEVICE_ID_NEO_8), 0, 0, 10 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCIE_DEVICE_ID_NEO_4), 0, 0, 11 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCIE_DEVICE_ID_NEO_4RJ45), 0, 0, 12 },
>> + { PCI_DEVICE(PCI_VENDOR_ID_DIGI, PCIE_DEVICE_ID_NEO_8RJ45), 0, 0, 13 },
>> { 0, }
>> };
>> MODULE_DEVICE_TABLE(pci, jsm_pci_tbl);
>> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
>> index 6ed0bb73a864..f0f610c94bb2 100644
>> --- a/include/linux/pci_ids.h
>> +++ b/include/linux/pci_ids.h
>> @@ -1516,11 +1516,19 @@
>> #define PCI_DEVICE_ID_DIGI_DF_M_E 0x0071
>> #define PCI_DEVICE_ID_DIGI_DF_M_IOM2_A 0x0072
>> #define PCI_DEVICE_ID_DIGI_DF_M_A 0x0073
>> +#define PCI_DEVICE_ID_NEO_4 0x00B0
>> #define PCI_DEVICE_ID_DIGI_NEO_8 0x00B1
>> #define PCI_DEVICE_ID_NEO_2DB9 0x00C8
>> #define PCI_DEVICE_ID_NEO_2DB9PRI 0x00C9
>> #define PCI_DEVICE_ID_NEO_2RJ45 0x00CA
>> #define PCI_DEVICE_ID_NEO_2RJ45PRI 0x00CB
>> +#define PCI_DEVICE_ID_NEO_1_422 0x00CC
>> +#define PCI_DEVICE_ID_NEO_1_422_485 0x00CD
>> +#define PCI_DEVICE_ID_NEO_2_422_485 0x00CE
>> +#define PCIE_DEVICE_ID_NEO_8 0x00F0
>> +#define PCIE_DEVICE_ID_NEO_4 0x00F1
>> +#define PCIE_DEVICE_ID_NEO_4RJ45 0x00F2
>> +#define PCIE_DEVICE_ID_NEO_8RJ45 0x00F3
>> #define PCIE_DEVICE_ID_NEO_4_IBM 0x00F4
>
> As per the information in the top of this file, we only add new ids when
> they cross multiple files. Right now these values you are adding are
> only for a single driver, so they don't need to be added here.
>
> Can you redo this series without touching the pci_ids.h file?
>

Ah, my apologies, obviously I didn't read the top of the pci_ids.h.
I'll redo these.

--
Bill



\
 
 \ /
  Last update: 2014-08-30 22:21    [W:0.036 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site