Re: [PATCH v2 02/11] pci_ids/tulip: Add PCI vendor ID for HP and use it in tulip

[BALATON Zoltan]

On Tue, 17 Oct 2023, deller@kernel.org wrote:
> From: Helge Deller <deller@gmx.de>
>
> Signed-off-by: Helge Deller <deller@gmx.de>
> ---
> hw/net/tulip.c           | 2 +-
> include/hw/pci/pci_ids.h | 2 ++
> 2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/hw/net/tulip.c b/hw/net/tulip.c
> index 915e5fb595..11d866e431 100644
> --- a/hw/net/tulip.c
> +++ b/hw/net/tulip.c
> @@ -1020,7 +1020,7 @@ static void tulip_class_init(ObjectClass *klass, void 
> *data)
>     k->exit = pci_tulip_exit;
>     k->vendor_id = PCI_VENDOR_ID_DEC;
>     k->device_id = PCI_DEVICE_ID_DEC_21143;
> -    k->subsystem_vendor_id = 0x103c;
> +    k->subsystem_vendor_id = PCI_VENDOR_ID_HP;
>     k->subsystem_id = 0x104f;
>     k->class_id = PCI_CLASS_NETWORK_ETHERNET;
>     dc->vmsd = &vmstate_pci_tulip;
> diff --git a/include/hw/pci/pci_ids.h b/include/hw/pci/pci_ids.h
> index 85469b9b53..3c0e72df0e 100644
> --- a/include/hw/pci/pci_ids.h
> +++ b/include/hw/pci/pci_ids.h
> @@ -171,6 +171,8 @@
> #define PCI_VENDOR_ID_DEC                0x1011
> #define PCI_DEVICE_ID_DEC_21143          0x0019
>
> +#define PCI_VENDOR_ID_HP                 0x103c
> +

Did not notice this in first round, sorry. These seems to be sorted 
(there's a comment further up about that) so this should be between AMD 
and TI a bit more down.

Regards,
BALATON Zoltan

> #define PCI_VENDOR_ID_CIRRUS             0x1013
>
> #define PCI_VENDOR_ID_IBM                0x1014