vfio/pci: Use pci_is_display()

superm1 · bjorn-helgaas · commit a7feca7c8818 · 2025-07-17T15:30:13.000-05:00
The inline pci_is_display() helper does the same thing. Use it. Suggested-by: Bjorn Helgaas <bhelgaas@google.com> Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> Reviewed-by: Daniel Dadap <ddadap@nvidia.com> Reviewed-by: Simona Vetter <simona.vetter@ffwll.ch> Acked-by: Alex Williamson <alex.williamson@redhat.com> Link: https://patch.msgid.link/20250717173812.3633478-3-superm1@kernel.org
diff --git a/drivers/vfio/pci/vfio_pci_igd.c b/drivers/vfio/pci/vfio_pci_igd.c
@@ -437,8 +437,7 @@ static int vfio_pci_igd_cfg_init(struct vfio_pci_core_device *vdev)
 
 bool vfio_pci_is_intel_display(struct pci_dev *pdev)
 {
-	return (pdev->vendor == PCI_VENDOR_ID_INTEL) &&
-	       ((pdev->class >> 16) == PCI_BASE_CLASS_DISPLAY);
+	return (pdev->vendor == PCI_VENDOR_ID_INTEL) && pci_is_display(pdev);
 }
 
 int vfio_pci_igd_init(struct vfio_pci_core_device *vdev)

Original file line number	Diff line number	Diff line change
`@@ -437,8 +437,7 @@ static int vfio_pci_igd_cfg_init(struct vfio_pci_core_device *vdev)`
`437`	`437`
`438`	`438`	`bool vfio_pci_is_intel_display(struct pci_dev *pdev)`
`439`	`439`	`{`
`440`		`- return (pdev->vendor == PCI_VENDOR_ID_INTEL) &&`
`441`		`- ((pdev->class >> 16) == PCI_BASE_CLASS_DISPLAY);`
	`440`	`+ return (pdev->vendor == PCI_VENDOR_ID_INTEL) && pci_is_display(pdev);`
`442`	`441`	`}`
`443`	`442`
`444`	`443`	`int vfio_pci_igd_init(struct vfio_pci_core_device *vdev)`