Re: [PATCH 1/1] vfio/pci: Remove the core dependency on driver data

[Danilo Krummrich]

On Tue Sep 29, 2026 at 10:42 PM CEST, Alex Williamson wrote:
> @@ -552,13 +560,16 @@ static int xe_vfio_pci_probe(struct pci_dev *pdev, const struct pci_device_id *i
>  
>  	dev_set_drvdata(&pdev->dev, &xe_vdev->core_device);
>  
> -	ret = vfio_pci_core_register_device(&xe_vdev->core_device);
> -	if (ret) {
> -		vfio_put_device(&xe_vdev->core_device.vdev);
> -		return ret;
> -	}
> +	ret = vfio_pci_core_register_device(&xe_vdev->core_device,
> +					    xe_vfio_vga_set_decode);
> +	if (ret)
> +		goto out_put_vdev;
>  
>  	return 0;
> +
> +out_put_vdev:
> +	vfio_put_device(&xe_vdev->core_device.vdev);
> +	return ret;
>  }

NIT: Except for the new function argument this change seems unrelated; I suppose
the clanker did this following the cleanup pattern used by most of the other
drivers.