Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c8e1ea58 authored by Akinobu Mita's avatar Akinobu Mita Committed by Sujit Reddy Thumma
Browse files

ufs: fix DMA mask setting 



If the controller doesn't support 64-bit addressing mode, it must not
set the DMA mask to 64-bit.  But it's unconditionally trying to set to
64-bit without checking 64-bit addressing support in the controller
capabilities.

It was correctly checked before commit 3b1d0580
("[SCSI] ufs: Segregate PCI Specific Code"), this aims to restores
the correct behaviour.

To achieve this in a generic way, firstly we should push down the DMA
mask setting routine ufshcd_set_dma_mask() from PCI glue driver to core
driver in order to do it for both PCI glue driver and Platform glue
driver.  Secondly, we should change pci_ DMA mapping API to dma_ DMA
mapping API because core driver is independent of glue drivers.

Change-Id: I7fd0208b56b82f8a844491bffc9a523cc655c74f
Signed-off-by: default avatarAkinobu Mita <mita@fixstars.com>
Cc: Sujit Reddy Thumma <sthumma@codeaurora.org>
Cc: Vinayak Holikatti <vinholikatti@gmail.com>
Cc: Santosh Y <santoshsy@gmail.com>
Cc: "James E.J. Bottomley" <JBottomley@parallels.com>
Cc: linux-scsi@vger.kernel.org
Patch-mainline: linux-scsi@vger.kernel.org @ 2013/08/19 19:26
Signed-off-by: default avatarSujit Reddy Thumma <sthumma@codeaurora.org>
parent 1b5d6faf

drivers/scsi/ufs/ufshcd-pci.c

+0 −26
Original line number Diff line number Diff line
@@ -135,26 +135,6 @@ static void ufshcd_pci_remove(struct pci_dev *pdev) 
	pci_set_drvdata(pdev, NULL);

}



/**

 * ufshcd_set_dma_mask - Set dma mask based on the controller

 *			 addressing capability

 * @pdev: PCI device structure

 *

 * Returns 0 for success, non-zero for failure

 */

static int ufshcd_set_dma_mask(struct pci_dev *pdev)

{

	int err;



	if (!pci_set_dma_mask(pdev, DMA_BIT_MASK(64))

		&& !pci_set_consistent_dma_mask(pdev, DMA_BIT_MASK(64)))

		return 0;

	err = pci_set_dma_mask(pdev, DMA_BIT_MASK(32));

	if (!err)

		err = pci_set_consistent_dma_mask(pdev, DMA_BIT_MASK(32));

	return err;

}



/**

 * ufshcd_pci_probe - probe routine of the driver

 * @pdev: pointer to PCI device handle
@@ -185,12 +165,6 @@ ufshcd_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) 


	mmio_base = pcim_iomap_table(pdev)[0];



	err = ufshcd_set_dma_mask(pdev);

	if (err) {

		dev_err(&pdev->dev, "set dma mask failed\n");

		return err;

	}



	err = ufshcd_alloc_host(&pdev->dev, &hba);

	if (err) {

		dev_err(&pdev->dev, "Allocation failed\n");

drivers/scsi/ufs/ufshcd.c

+30 −0
Original line number Diff line number Diff line
@@ -3685,6 +3685,30 @@ out_error: 
}

EXPORT_SYMBOL(ufshcd_alloc_host);



/**

 * ufshcd_set_dma_mask - Set dma mask based on the controller

 *			 addressing capability

 * @hba: per adapter instance

 *

 * Returns 0 for success, non-zero for failure

 */

static int ufshcd_set_dma_mask(struct ufs_hba *hba)

{

	int err;



	if (hba->capabilities & MASK_64_ADDRESSING_SUPPORT) {

		if (!dma_set_mask(hba->dev, DMA_BIT_MASK(64))) {

			dma_set_coherent_mask(hba->dev, DMA_BIT_MASK(64));

			return 0;

		}

	}

	err = dma_set_mask(hba->dev, DMA_BIT_MASK(32));

	if (!err)

		dma_set_coherent_mask(hba->dev, DMA_BIT_MASK(32));



	return err;

}



/**

 * ufshcd_init - Driver initialization routine

 * @hba: per-adapter instance
@@ -3724,6 +3748,12 @@ int ufshcd_init(struct ufs_hba *hba, void __iomem *mmio_base, unsigned int irq) 
	/* Get Interrupt bit mask per version */

	hba->intr_mask = ufshcd_get_intr_mask(hba);



	err = ufshcd_set_dma_mask(hba);

	if (err) {

		dev_err(hba->dev, "set dma mask failed\n");

		goto out_disable;

	}



	/* Allocate memory for host memory space */

	err = ufshcd_memory_alloc(hba);

	if (err) {