On Mon, 2013-08-19 at 22:56 +0900, Akinobu Mita wrote:
> The device-tree probed device for ARM doesn't have dev->dma_mask.
> So dma_set_mask() for the device doesn't succeed.  The popular trick
> for this is - dev->dma_mask = &dev->coherent_dma_mask;
> 
> Currently there is no dma_set_mask() call in ufs-pltfrm, but the
> forthcoming fix needs proper DMA mask setting in ufs core driver.  So
> initializing dev->dma_mask as described above is required.
> 
> Signed-off-by: Akinobu Mita <[email protected]>
> Cc: Sujit Reddy Thumma <[email protected]>
> Cc: Vinayak Holikatti <[email protected]>
> Cc: Santosh Y <[email protected]>
> Cc: "James E.J. Bottomley" <[email protected]>
> Cc: [email protected]
> ---
>  drivers/scsi/ufs/ufshcd-pltfrm.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/scsi/ufs/ufshcd-pltfrm.c 
> b/drivers/scsi/ufs/ufshcd-pltfrm.c
> index 94ba40c..c780840 100644
> --- a/drivers/scsi/ufs/ufshcd-pltfrm.c
> +++ b/drivers/scsi/ufs/ufshcd-pltfrm.c
> @@ -122,6 +122,9 @@ static int ufshcd_pltfrm_probe(struct platform_device 
> *pdev)
>               goto out;
>       }
>  
> +     if (!dev->dma_mask)
> +             dev->dma_mask = &dev->coherent_dma_mask;
> +

If the DMA mask is NULL, it means there's buggy platform code somewhere;
I'm not sure we should be hacking a fix in a SCSI driver.

James


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to