Commit 4d10367f authored by Christophe JAILLET's avatar Christophe JAILLET Committed by Michael S. Tsirkin
Browse files

vdpa: ifcvf: Use dma_set_mask_and_coherent to simplify code 



'pci_set_dma_mask()' + 'pci_set_consistent_dma_mask()' can be replaced by
an equivalent 'dma_set_mask_and_coherent()' which is much less verbose.

While at it, fix a typo (s/confiugration/configuration)

Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/20201129125434.1462638-1-christophe.jaillet@wanadoo.fr


Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
Acked-by: default avatarJason Wang <jasowang@redhat.com>
parent 0ab4b890

drivers/vdpa/ifcvf/ifcvf_main.c

+2 −9
Original line number Diff line number Diff line
@@ -417,16 +417,9 @@ static int ifcvf_probe(struct pci_dev *pdev, const struct pci_device_id *id) 
		return ret;

	}



	ret = pci_set_dma_mask(pdev, DMA_BIT_MASK(64));

	ret = dma_set_mask_and_coherent(dev, DMA_BIT_MASK(64));

	if (ret) {

		IFCVF_ERR(pdev, "No usable DMA confiugration\n");

		return ret;

	}



	ret = pci_set_consistent_dma_mask(pdev, DMA_BIT_MASK(64));

	if (ret) {

		IFCVF_ERR(pdev,

			  "No usable coherent DMA confiugration\n");

		IFCVF_ERR(pdev, "No usable DMA configuration\n");

		return ret;

	}