Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753560Ab3DKTwY (ORCPT ); Thu, 11 Apr 2013 15:52:24 -0400 Received: from mail-oa0-f48.google.com ([209.85.219.48]:52259 "EHLO mail-oa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752762Ab3DKTwW (ORCPT ); Thu, 11 Apr 2013 15:52:22 -0400 MIME-Version: 1.0 In-Reply-To: <1362579278-6534-1-git-send-email-abbotti@mev.co.uk> References: <1362579278-6534-1-git-send-email-abbotti@mev.co.uk> From: Bjorn Helgaas Date: Thu, 11 Apr 2013 13:52:01 -0600 Message-ID: Subject: Re: [PATCH] pci_ids.h: add PCI_VENDOR_ID_AMCC To: Ian Abbott Cc: "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2413 Lines: 59 On Wed, Mar 6, 2013 at 7:14 AM, Ian Abbott wrote: > Vendor ID 0x10e8 is assigned to Applied Micro Circuits Corporation > (recently AppliedMicro, but AMCC on NASDAQ). The ID currently appears > as PCI_VENDOR_ID_ADDIDATA_OLD in pci_ids.h, which is wrong. Some older > PCI boards made by ADDI-DATA used PCI interface chips made by AMCC and > used device IDs assigned by AMCC. > > Add PCI_VENDOR_ID_AMCC and mark PCI_VENDOR_ID_ADDIDATA_OLD as a > duplicate. > > Note that PCI_VENDOR_ID_AMCC is defined identically in > "drivers/staging/comedi/comedidev.h" for use by the adl_pci9118 comedi > driver. Various other comedi drivers currently use > PCI_VENDOR_ID_ADDIDATA_OLD, but these can be changed to use > PCI_VENDOR_ID_AMCC easily enough. The only other user of > PCI_VENDOR_ID_ADDIDATA_OLD is "drivers/tty/serial/8250_pci.c" which > could also be changed to use AMCC easily enough. Once all that is done, > PCI_VENDOR_ID_ADDIDATA_OLD can be removed. > > Signed-off-by: Ian Abbott > --- > include/linux/pci_ids.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) This should be posted along with the patches that actually use PCI_VENDOR_ID_AMCC. I'll drop this for now and watch for those. Bjorn > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index f11c1c2..902e41a 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -1309,6 +1309,8 @@ > #define PCI_DEVICE_ID_IMS_TT128 0x9128 > #define PCI_DEVICE_ID_IMS_TT3D 0x9135 > > +#define PCI_VENDOR_ID_AMCC 0x10e8 > + > #define PCI_VENDOR_ID_INTERG 0x10ea > #define PCI_DEVICE_ID_INTERG_1682 0x1682 > #define PCI_DEVICE_ID_INTERG_2000 0x2000 > @@ -2249,7 +2251,7 @@ > /* > * ADDI-DATA GmbH communication cards > */ > -#define PCI_VENDOR_ID_ADDIDATA_OLD 0x10E8 > +#define PCI_VENDOR_ID_ADDIDATA_OLD 0x10E8 /* actually AMCC */ > #define PCI_VENDOR_ID_ADDIDATA 0x15B8 > #define PCI_DEVICE_ID_ADDIDATA_APCI7500 0x7000 > #define PCI_DEVICE_ID_ADDIDATA_APCI7420 0x7001 > -- > 1.8.1.2 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/