scsi: qedf: fully convert to the generic DMA API
authorChristoph Hellwig <hch@lst.de>
Wed, 10 Oct 2018 18:04:15 +0000 (20:04 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 18 Oct 2018 01:58:53 +0000 (21:58 -0400)
The driver is currently using an odd mix of legacy PCI DMA API and
generic DMA API calls, switch it over to the generic API entirely.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Acked-by: Chad Dupuis <chad.dupuis@cavium.com>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qedf/qedf_main.c

index 0a5dd5595dd3c42179543d8453e9d3afc98ba3f7..d5a4f17fce5128e515b5c7492bebaecfefc4d332 100644 (file)
@@ -2855,12 +2855,12 @@ static int qedf_set_fcoe_pf_param(struct qedf_ctx *qedf)
        QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC, "Number of CQs is %d.\n",
                   qedf->num_queues);
 
-       qedf->p_cpuq = pci_alloc_consistent(qedf->pdev,
+       qedf->p_cpuq = dma_alloc_coherent(&qedf->pdev->dev,
            qedf->num_queues * sizeof(struct qedf_glbl_q_params),
-           &qedf->hw_p_cpuq);
+           &qedf->hw_p_cpuq, GFP_KERNEL);
 
        if (!qedf->p_cpuq) {
-               QEDF_ERR(&(qedf->dbg_ctx), "pci_alloc_consistent failed.\n");
+               QEDF_ERR(&(qedf->dbg_ctx), "dma_alloc_coherent failed.\n");
                return 1;
        }
 
@@ -2929,7 +2929,7 @@ static void qedf_free_fcoe_pf_param(struct qedf_ctx *qedf)
 
        if (qedf->p_cpuq) {
                size = qedf->num_queues * sizeof(struct qedf_glbl_q_params);
-               pci_free_consistent(qedf->pdev, size, qedf->p_cpuq,
+               dma_free_coherent(&qedf->pdev->dev, size, qedf->p_cpuq,
                    qedf->hw_p_cpuq);
        }