[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAH9NwWeiaHcWEx9eUQuHZAFzsxz3BhHh4aKUKfRySoJOLxRtQA@mail.gmail.com>
Date: Tue, 18 Oct 2022 10:41:50 +0200
From: Christian Gmeiner <christian.gmeiner@...il.com>
To: Bjorn Helgaas <helgaas@...nel.org>
Cc: Bjorn Helgaas <bhelgaas@...gle.com>,
Christian GMEINER <Christian.GMEINER@...hmann.info>,
linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] PCI: Add Bachmann electronic GmbH vendor ID
Hi Bjorn
>
> On Mon, Oct 17, 2022 at 04:23:37PM +0200, Christian Gmeiner wrote:
> > From: Christian GMEINER <Christian.GMEINER@...hmann.info>
> >
> > Signed-off-by: Christian GMEINER <Christian.GMEINER@...hmann.info>
>
> I tweaked it to shorten the name in the style of other entries and
> sort it by numeric ID.
>
I am fine with that .. was not sure if there needs to be a match of
the used define and
the official "Member company" name:
https://pcisig.com/membership/member-companies?combine=%22Bachmann+electronic%22
> I assume there's a driver that will use this definition. If so, you
> might want to post this patch (including my ack) along with the driver
> so they get merged together. But let me know if you need me to take
> it directly.
>
I hope that one driver will see the public light sooner than later but
I have no official time frame.
>
> Also it will be helpful if you can add the item to the PCI ID database
> here: https://pci-ids.ucw.cz/read/PC?restrict=0, which will let lspci
> identify devices with this Vendor ID.
>
Done - thanks.
> Bjorn
>
>
> commit 2fa819fdbb2b ("PCI: Add Bachmann electronic GmbH vendor ID")
> Author: Christian GMEINER <Christian.GMEINER@...hmann.info>
> Date: Mon Oct 17 16:23:37 2022 +0200
>
> PCI: Add Bachmann electronic GmbH vendor ID
>
> Link: https://lore.kernel.org/r/20221017142338.1445199-1-christian.gmeiner@gmail.com
> Signed-off-by: Christian GMEINER <Christian.GMEINER@...hmann.info>
> Acked-by: Bjorn Helgaas <bhelgaas@...gle.com>
>
> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> index b362d90eb9b0..4cc0e9ecd398 100644
> --- a/include/linux/pci_ids.h
> +++ b/include/linux/pci_ids.h
> @@ -2,7 +2,7 @@
> /*
> * PCI Class, Vendor and Device IDs
> *
> - * Please keep sorted.
> + * Please keep sorted by numeric ID.
> *
> * Do not add new entries to this file unless the definitions
> * are shared between multiple drivers.
> @@ -153,7 +153,7 @@
>
> #define PCI_CLASS_OTHERS 0xff
>
> -/* Vendors and devices. Sort key: vendor first, device next. */
> +/* Vendors and devices. Numeric sort key: vendor first, device next. */
> #define PCI_VENDOR_ID_PCI_SIG 0x0001
>
> #define PCI_VENDOR_ID_LOONGSON 0x0014
> @@ -172,6 +172,8 @@
> #define PCI_DEVICE_ID_BERKOM_A4T 0xffa4
> #define PCI_DEVICE_ID_BERKOM_SCITEL_QUADRO 0xffa8
>
> +#define PCI_VENDOR_ID_BACHMANN 0x0bae
> +
> #define PCI_VENDOR_ID_COMPAQ 0x0e11
> #define PCI_DEVICE_ID_COMPAQ_TOKENRING 0x0508
> #define PCI_DEVICE_ID_COMPAQ_TACHYON 0xa0fc
>
> > ---
> > include/linux/pci_ids.h | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
> > index b362d90eb9b0..b93a52977d85 100644
> > --- a/include/linux/pci_ids.h
> > +++ b/include/linux/pci_ids.h
> > @@ -166,6 +166,8 @@
> >
> > #define PCI_VENDOR_ID_UBIQUITI 0x0777
> >
> > +#define PCI_VENDOR_ID_BACHMANN_ELECTRONIC 0x0bae
> > +
> > #define PCI_VENDOR_ID_BERKOM 0x0871
> > #define PCI_DEVICE_ID_BERKOM_A1T 0xffa1
> > #define PCI_DEVICE_ID_BERKOM_T_CONCEPT 0xffa2
> > --
> > 2.37.3
> >
--
greets
--
Christian Gmeiner, MSc
https://christian-gmeiner.info/privacypolicy
Powered by blists - more mailing lists