diff mbox series

[06/12] tulip: Use the HP PCI vendor ID instead of number

Message ID 20231014195719.151397-7-deller@kernel.org
State New
Headers show
Series target/hppa: Add emulation of a C3700 HP-PARISC workstation | expand

Commit Message

Helge Deller Oct. 14, 2023, 7:57 p.m. UTC 
From: Helge Deller <deller@gmx.de>

Signed-off-by: Helge Deller <deller@gmx.de>
---
 hw/net/tulip.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

BALATON Zoltan Oct. 14, 2023, 8:37 p.m. UTC | #1 
On Sat, 14 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 +-
> 1 file changed, 1 insertion(+), 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;

Seems to be first use of value introduced in patch 2. I don't know if this 
is worrh two separate patches. I'd just squash patch 2 in this patch but 
not sure what others or maintainer prefers.

Regards,
BALATON Zoltan

>     k->subsystem_id = 0x104f;
>     k->class_id = PCI_CLASS_NETWORK_ETHERNET;
>     dc->vmsd = &vmstate_pci_tulip;
>
diff mbox series

Patch

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;