From: Elad Nachman <[email protected]>
AC5 is a 64-bit platform so extend the dma mask accordingly.
Checked this mask on armv7 a38x SoC (which has this
USB controller) platform with simple fs ops on the storage device
but on older 4.14 Linux version.
Signed-off-by: Elad Nachman <[email protected]>
Signed-off-by: Vadym Kochan <[email protected]>
---
v2:
Add missing description.
drivers/usb/host/ehci-orion.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c
index a3454a3ea4e0..c6205abebbdf 100644
--- a/drivers/usb/host/ehci-orion.c
+++ b/drivers/usb/host/ehci-orion.c
@@ -230,7 +230,7 @@ static int ehci_orion_drv_probe(struct platform_device *pdev)
* set. Since shared usb code relies on it, set it here for
* now. Once we have dma capability bindings this can go away.
*/
- err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
+ err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
if (err)
goto err;
--
2.17.1
On Tue, Oct 18, 2022 at 02:34:00PM +0300, Vadym Kochan wrote:
> From: Elad Nachman <[email protected]>
>
> AC5 is a 64-bit platform so extend the dma mask accordingly.
>
> Checked this mask on armv7 a38x SoC (which has this
> USB controller) platform with simple fs ops on the storage device
> but on older 4.14 Linux version.
Odd line-wrapping.
That aside, I have no idea what this changelog text means at all. What
is 4.14 here for? Does this fix a specific commit id? What is being
changed? Why is it being changed? What is happening?
confused,
greg k-h
On Tue, Oct 18, 2022 at 02:34:00PM +0300, Vadym Kochan wrote:
> From: Elad Nachman <[email protected]>
>
> AC5 is a 64-bit platform so extend the dma mask accordingly.
>
> Checked this mask on armv7 a38x SoC (which has this
> USB controller) platform with simple fs ops on the storage device
> but on older 4.14 Linux version.
>
> Signed-off-by: Elad Nachman <[email protected]>
> Signed-off-by: Vadym Kochan <[email protected]>
> ---
> v2:
> Add missing description.
>
> drivers/usb/host/ehci-orion.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c
> index a3454a3ea4e0..c6205abebbdf 100644
> --- a/drivers/usb/host/ehci-orion.c
> +++ b/drivers/usb/host/ehci-orion.c
> @@ -230,7 +230,7 @@ static int ehci_orion_drv_probe(struct platform_device *pdev)
> * set. Since shared usb code relies on it, set it here for
> * now. Once we have dma capability bindings this can go away.
> */
> - err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
> + err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
> if (err)
> goto err;
>
NAK. The description says this changes the DMA mask on AC5 platforms to
64 bits, but that isn't what the patch actually does. It changes the
DMA mask on _all_ platforms using the orion hardware.
Alan Stern