Re: [PATCH 1/2] jsm: add support for additional Neo cards

[Greg KH]

On Sat, Aug 30, 2014 at 02:54:36PM -0400, wfp5p@xxxxxxxxxxxxxxx wrote:
> From: Bill Pemberton <wfp5p@xxxxxxxxxxxxxxx>
> 
> Add device ids for additional Neo cards.  The ids come from the dgnc
> driver.
> 
> Signed-off-by: Bill Pemberton <wfp5p@xxxxxxxxxxxxxxx>
> ---
>  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?

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/