Drivers: hv: vmbus: Use PCI_VENDOR_ID_MICROSOFT for better discoverability

easwarh · liuw · commit a99aaf2e3b33 · 2022-09-23T10:30:37.000Z
pci_ids.h already defines PCI_VENDOR_ID_MICROSOFT, and is included via linux/pci.h. Use the define instead of the magic number. Signed-off-by: Easwar Hariharan <easwar.hariharan@microsoft.com> Reviewed-by: Michael Kelley <mikelley@microsoft.com> Link: https://lore.kernel.org/r/1663625084-2518-2-git-send-email-eahariha@linux.microsoft.com Signed-off-by: Wei Liu <wei.liu@kernel.org>
diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c
@@ -2052,7 +2052,7 @@ struct hv_device *vmbus_device_create(const guid_t *type,
 	child_device_obj->channel = channel;
 	guid_copy(&child_device_obj->dev_type, type);
 	guid_copy(&child_device_obj->dev_instance, instance);
-	child_device_obj->vendor_id = 0x1414; /* MSFT vendor ID */
+	child_device_obj->vendor_id = PCI_VENDOR_ID_MICROSOFT;
 
 	return child_device_obj;
 }

Original file line number	Diff line number	Diff line change
`@@ -2052,7 +2052,7 @@ struct hv_device vmbus_device_create(const guid_t type,`
`2052`	`2052`	`child_device_obj->channel = channel;`
`2053`	`2053`	`guid_copy(&child_device_obj->dev_type, type);`
`2054`	`2054`	`guid_copy(&child_device_obj->dev_instance, instance);`
`2055`		`- child_device_obj->vendor_id = 0x1414; /* MSFT vendor ID */`
	`2055`	`+ child_device_obj->vendor_id = PCI_VENDOR_ID_MICROSOFT;`
`2056`	`2056`
`2057`	`2057`	`return child_device_obj;`
`2058`	`2058`	`}`