Arnd Bergmann wrote:
On Friday, June 24, 2016 6:46:48 PM CEST Timur Tabi wrote:
>+ /* The EMAC itself is capable of 64-bit DMA. If the SOC limits that
>+ * range, then we expect platform code to adjust the mask accordingly.
>+ */
>+ ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
>+ if (ret) {
>+ dev_err(&pdev->dev, "could not set DMA mask\n");
>+ return ret;
>+ }
>
The comment does not match the code: if the platform has no IOMMU
and the bus limit is smaller than the memory, dma_set_mask_and_coherent()
will fail, and the driver should instead ensure that the buffers are
allocated from the 32-bit area.
Alternatively, adjust the comment to explain that this is a limitation
in the driver that can be lifted if necessary.
I'm not sure I understand. The EMAC hardware is capable of 64-bit DMA.
This is true on every platform -- the hardware registers that take bus
addresses are 64-bit. The driver itself has no limitations.
And that's what the dma_set_mask_and_coherent() does. It tells the
kernel what the device is capable of.
However, on some SOCs, only a subset of those address lines are
connected to the memory bus. So for instance, some platforms only have
32 bits connected.
There's no way for the EMAC driver to know this, so it expects other
code in the kernel to adjust. I'm not exactly sure what this code is
supposed to be, because I get conflicting information. At one point, I
thought that the dma-ranges property would handle that. The kernel
would parse that property, see that the DMA range is limited to 32 bits,
and adjust the DMA mask accordingly. However, with dma-ranges in the
parent node, I don't see how that can work.
So my question is, how do I handle the situation where a subset of the
DMA address lines are masked off by the SOC? I've seen code like this:
ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
if (ret)
ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
But this has never made any sense to me. If DMA_BIT_MASK(64) fails,
then how can DMA_BIT_MASK(32) succeed?
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the
Code Aurora Forum, hosted by The Linux Foundation.
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
